diff options
| author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-08-07 08:43:22 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-08-09 08:33:44 +0000 |
| commit | 2fdbe0caf7ea324c6b18bdbc5f75272f85bcd480 (patch) | |
| tree | ba3aa5eec8cb88540df9d1df8e16dbbe741dd6e4 /src/lib | |
| parent | ea45ecfb89add18bb4ed1a8f6f3d4337307c124c (diff) | |
| download | coreboot-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>
Diffstat (limited to 'src/lib')
| -rw-r--r-- | src/lib/edid.c | 3 |
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; } } |