summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@amd.corp-partner.google.com>2018-08-07 08:43:22 -0700
committerPatrick Georgi <pgeorgi@google.com>2018-08-09 08:33:44 +0000
commit2fdbe0caf7ea324c6b18bdbc5f75272f85bcd480 (patch)
treeba3aa5eec8cb88540df9d1df8e16dbbe741dd6e4
parentea45ecfb89add18bb4ed1a8f6f3d4337307c124c (diff)
downloadcoreboot-2fdbe0caf7ea324c6b18bdbc5f75272f85bcd480.tar.xz
lib/edid.c: Avoid static analysis error for unused value
Within procedure cea_hdmi_block, the variable "b" is used as an index into a buffer of EDID bytes. At the end, it's incremented but not used, thus causing a static analysis error. Add a "(void)b;" at the end to avoid the static analysis error. BUG=b:112253891 TEST=Build and boot grunt. Change-Id: Ibd0b4a21bf82fcc46a627bc75564a850b7374989 Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/27929 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Julius Werner <jwerner@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/lib/edid.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lib/edid.c b/src/lib/edid.c
index 7ad4136814..9cf9b8a1ed 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -817,8 +817,9 @@ cea_hdmi_block(struct edid *out, unsigned char *x)
* (optionally: 3D_Detail_X and reserved)
*/
}
-
}
+ /* Tell static analysis we know index b is left unused. */
+ (void)b;
}
}