summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-07-16 00:56:27 +0200
committerAngel Pons <th3fanbus@gmail.com>2020-07-20 17:13:22 +0000
commit51e138c25b2e91fcaa09ed701b82a5a20540edde (patch)
tree288713eff108a6a996ac2e5a37f321081cdf9edd
parentb21bffae0ce5dee5d316ad544ccc6dedbc4475a1 (diff)
drivers/intel/gma/Kconfig: Avoid dependency hell when ignoring straps
Unconditionally selecting `GFX_GMA_IGNORE_PRESENCE_STRAPS` creates a hard dependency on `MAINBOARD_USE_LIBGFXINIT`, which is undesired. Move it out of the `if GFX_GMA` block to break this unwanted dependency. TEST=Build for Librem 13v4 with no graphics init successfully. Change-Id: I53e132c209c065068f20959fa1a6f5195f5fe766 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/43491 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Nico Huber <nico.h@gmx.de>
-rw-r--r--src/drivers/intel/gma/Kconfig15
1 files changed, 8 insertions, 7 deletions
diff --git a/src/drivers/intel/gma/Kconfig b/src/drivers/intel/gma/Kconfig
index 635de1c87a..c515888a1b 100644
--- a/src/drivers/intel/gma/Kconfig
+++ b/src/drivers/intel/gma/Kconfig
@@ -68,6 +68,14 @@ config GFX_GMA_ANALOG_I2C_HDMI_C
config GFX_GMA_ANALOG_I2C_HDMI_D
bool
+config GFX_GMA_IGNORE_PRESENCE_STRAPS
+ def_bool n
+ depends on MAINBOARD_HAS_LIBGFXINIT
+ help
+ libgfxinit uses the GPU presence straps to determine if a display port
+ is present/enabled. Select this option if a board doesn't correctly implement
+ these straps, causing libgfxinit to fail to detect an attached panel.
+
config GFX_GMA
def_bool y
depends on NORTHBRIDGE_INTEL_GM45 || NORTHBRIDGE_INTEL_X4X \
@@ -127,11 +135,4 @@ config GFX_GMA_ANALOG_I2C_PORT
digital displays. In that case, the EDID for a VGA display has to be
read over the I2C interface of the coupled digital port.
-config GFX_GMA_IGNORE_PRESENCE_STRAPS
- def_bool n
- help
- libgfxinit uses the GPU presence straps to determine if a display port
- is present/enabled. Select this option if a board doesn't correctly implement
- these straps, causing libgfxinit to fail to detect an attached panel.
-
endif