From 4e4c2816e2239299bc02e3a57fb18056db62b56c Mon Sep 17 00:00:00 2001 From: Douglas Anderson Date: Wed, 28 Oct 2015 10:19:52 -0700 Subject: [PATCH] edid: Don't set standard timings as supported if they're not The set to say that a standard timing was supported was not properly in the "if" test. That meant that even when standard timings weren't supported, we thought that they were. That had the side effect of never using the detailed mode. BRANCH=none BUG=chrome-os-partner:46998 TEST=Adafruit panel works now Change-Id: Ib67735219fd28516857d9b63f1ba156573f1bea3 Signed-off-by: Douglas Anderson Reviewed-on: https://chromium-review.googlesource.com/309521 --- src/lib/edid.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/lib/edid.c b/src/lib/edid.c index 66996e19f0..f550d92a56 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -1258,13 +1258,13 @@ int decode_edid(unsigned char *edid, int size, struct edid *out) if (edid[0x23 + i / 8] & (1 << (7 - i % 8))) { printk(BIOS_SPEW, " %dx%d@%dHz\n", established_timings[i].x, established_timings[i].y, established_timings[i].refresh); - } - for (j = 0; j < NUM_KNOWN_MODES; j++) { - if (known_modes[j].ha == established_timings[i].x && - known_modes[j].va == established_timings[i].y && - known_modes[j].refresh == established_timings[i].refresh) + for (j = 0; j < NUM_KNOWN_MODES; j++) { + if (known_modes[j].ha == established_timings[i].x && + known_modes[j].va == established_timings[i].y && + known_modes[j].refresh == established_timings[i].refresh) out->mode_is_supported[j] = 1; + } } }