[OpenPower-Firmware] [RFC] Drop untested graphic support from the build

Joel Stanley joel at jms.id.au
Mon Aug 20 16:54:10 AEST 2018


Back in the day Tuletta (P8) machines would use Radeon GPU drivers. No
one has tested this in many moons. To save all systems are paying for
the kernel driver, disable it in the kernel configuration and drop the
firmware from the firenze, pseries and zz configurations.

A similar story applies for Matrox cards.

If someone comes hunting for support in the future they can revert this
commit.

Signed-off-by: Joel Stanley <joel at jms.id.au>
---
I will submit this for inclusion into op-build master after a brief
period. Ideally anyone wanting to keep it in the build will be able to
commit to regular testing on the relevant hardware.

 openpower/configs/firenze_defconfig       | 1 -
 openpower/configs/linux/skiroot_defconfig | 9 ---------
 openpower/configs/pseries_defconfig       | 1 -
 openpower/configs/zz_defconfig            | 1 -
 openpower/scripts/firmware-whitelist      | 7 +------
 5 files changed, 1 insertion(+), 18 deletions(-)

diff --git a/openpower/configs/firenze_defconfig b/openpower/configs/firenze_defconfig
index 0628affe419e..1a8b7073f82d 100644
--- a/openpower/configs/firenze_defconfig
+++ b/openpower/configs/firenze_defconfig
@@ -22,7 +22,6 @@ BR2_LINUX_KERNEL_XZ=y
 BR2_PACKAGE_BUSYBOX_CONFIG_FRAGMENT_FILES="$(BR2_EXTERNAL_OP_BUILD_PATH)/configs/busybox.fragment"
 BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
 BR2_PACKAGE_LINUX_FIRMWARE=y
-BR2_PACKAGE_LINUX_FIRMWARE_RADEON=y
 BR2_PACKAGE_LINUX_FIRMWARE_BNX2X=y
 BR2_PACKAGE_LINUX_FIRMWARE_CXGB4_T4=y
 BR2_PACKAGE_IPMITOOL=y
diff --git a/openpower/configs/linux/skiroot_defconfig b/openpower/configs/linux/skiroot_defconfig
index 2bdc0780c0c2..751a4fd3cb43 100644
--- a/openpower/configs/linux/skiroot_defconfig
+++ b/openpower/configs/linux/skiroot_defconfig
@@ -215,19 +215,10 @@ CONFIG_I2C_OPAL=m
 CONFIG_PPS=y
 CONFIG_SENSORS_IBMPOWERNV=m
 CONFIG_DRM=m
-CONFIG_DRM_RADEON=m
 CONFIG_DRM_AST=m
 CONFIG_FB=y
 CONFIG_FIRMWARE_EDID=y
-CONFIG_FB_MODE_HELPERS=y
 CONFIG_FB_OF=y
-CONFIG_FB_MATROX=y
-CONFIG_FB_MATROX_MILLENIUM=y
-CONFIG_FB_MATROX_MYSTIQUE=y
-CONFIG_FB_MATROX_G=y
-# CONFIG_LCD_CLASS_DEVICE is not set
-CONFIG_BACKLIGHT_CLASS_DEVICE=y
-# CONFIG_BACKLIGHT_GENERIC is not set
 # CONFIG_VGA_CONSOLE is not set
 CONFIG_FRAMEBUFFER_CONSOLE=y
 CONFIG_LOGO=y
diff --git a/openpower/configs/pseries_defconfig b/openpower/configs/pseries_defconfig
index f86735e0dc0f..2c936d03b3ca 100644
--- a/openpower/configs/pseries_defconfig
+++ b/openpower/configs/pseries_defconfig
@@ -22,7 +22,6 @@ BR2_LINUX_KERNEL_XZ=y
 BR2_PACKAGE_BUSYBOX_CONFIG_FRAGMENT_FILES="$(BR2_EXTERNAL_OP_BUILD_PATH)/configs/busybox.fragment"
 BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
 BR2_PACKAGE_LINUX_FIRMWARE=y
-BR2_PACKAGE_LINUX_FIRMWARE_RADEON=y
 BR2_PACKAGE_LINUX_FIRMWARE_BNX2X=y
 BR2_PACKAGE_LINUX_FIRMWARE_CXGB4_T4=y
 BR2_PACKAGE_I2C_TOOLS=y
diff --git a/openpower/configs/zz_defconfig b/openpower/configs/zz_defconfig
index 7577d0096954..7d2604fe64a0 100644
--- a/openpower/configs/zz_defconfig
+++ b/openpower/configs/zz_defconfig
@@ -22,7 +22,6 @@ BR2_LINUX_KERNEL_XZ=y
 BR2_PACKAGE_BUSYBOX_CONFIG_FRAGMENT_FILES="$(BR2_EXTERNAL_OP_BUILD_PATH)/configs/busybox.fragment"
 BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
 BR2_PACKAGE_LINUX_FIRMWARE=y
-BR2_PACKAGE_LINUX_FIRMWARE_RADEON=y
 BR2_PACKAGE_LINUX_FIRMWARE_BNX2X=y
 BR2_PACKAGE_LINUX_FIRMWARE_CXGB4_T4=y
 BR2_PACKAGE_IPMITOOL=y
diff --git a/openpower/scripts/firmware-whitelist b/openpower/scripts/firmware-whitelist
index f61f284b09c3..cd217b965ed7 100755
--- a/openpower/scripts/firmware-whitelist
+++ b/openpower/scripts/firmware-whitelist
@@ -11,12 +11,7 @@ whitelist=(     'acenic/'
                 'cxgb4/t4fw.bin'
                 'cxgb3/'
                 'e100/'
-                'ql2500_fw.bin'
-                'radeon/CEDAR_rlc.bin'
-                'radeon/CEDAR_pfp.bin'
-                'radeon/CEDAR_smc.bin'
-                'radeon/CEDAR_me.bin'
-                'radeon/CYPRESS_uvd.bin')
+                'ql2500_fw.bin')
 
 if [ -z "${TARGET_DIR}" ] ; then
         echo "TARGET_DIR not defined, setting to $1"
-- 
2.17.1



More information about the OpenPower-Firmware mailing list