diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2017-08-06 16:00:18 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2017-08-11 19:49:39 +0000 |
| commit | c3cbe9433c22b5ac3bd616c858fb87b0de3ebab8 (patch) | |
| tree | bdfcfaf54461dabaac9e2abbeed1ba0a25319c6f /src | |
| parent | ccdef19cfbdfe19acf993872d646ba04ccfb6932 (diff) | |
| download | coreboot-c3cbe9433c22b5ac3bd616c858fb87b0de3ebab8.tar.xz | |
nb/intel/x4x/gma.c: Probe VGA EDID on DVI-I ports
This allows the use of the native VGA init on boards featuring DVI-I
ports. Digital output is not supported.
Change-Id: I11a4dd68746e06c7e27ecf3e765bdd0d8cf40515
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/20890
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src')
| -rw-r--r-- | src/northbridge/intel/x4x/gma.c | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/src/northbridge/intel/x4x/gma.c b/src/northbridge/intel/x4x/gma.c index e91d778774..db96facbef 100644 --- a/src/northbridge/intel/x4x/gma.c +++ b/src/northbridge/intel/x4x/gma.c @@ -74,6 +74,15 @@ static void intel_gma_init(const struct northbridge_intel_x4x_config *info, u32 pixel_m1 = 1; u32 pixel_m2 = 1; + u8 vga_gmbus = GMBUS_PORT_VGADDC; + + if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_B)) + vga_gmbus = GMBUS_PORT_DPB; + else if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_C)) + vga_gmbus = GMBUS_PORT_DPC; + else if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_D)) + vga_gmbus = GMBUS_PORT_DPD; + vga_gr_write(0x18, 0); /* Set up GTT */ @@ -113,7 +122,11 @@ static void intel_gma_init(const struct northbridge_intel_x4x_config *info, udelay(1); - intel_gmbus_read_edid(mmio + GMBUS0, GMBUS_PORT_VGADDC, 0x50, edid_data, + /* + * TODO: check if it is actually an analog display. + * No harm is done but the console output could be confusing. + */ + intel_gmbus_read_edid(mmio + GMBUS0, vga_gmbus, 0x50, edid_data, sizeof(edid_data)); intel_gmbus_stop(mmio + GMBUS0); decode_edid(edid_data, |