diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-08 17:37:06 -0800 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-09 17:29:33 +0100 |
| commit | 2f919ec4765e4484d79b866332fa19ec338db5c0 (patch) | |
| tree | 1ddb3a7eba06bdd40657c5f63bc8dce322c530a6 /src/lib/edid.c | |
| parent | b2d834a93afe129851f9aa7400f3fb2f42be20a4 (diff) | |
| download | coreboot-2f919ec4765e4484d79b866332fa19ec338db5c0.tar.xz | |
src/lib: Remove braces for single statements
Fix the following warning detected by checkpatch.pl:
WARNING: braces {} are not necessary for single statement blocks
TEST=Build and run on Galileo Gen2
Change-Id: Ie4b41f6fb75142ddd75103a55e0347ed85e7e873
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18697
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/lib/edid.c')
| -rw-r--r-- | src/lib/edid.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/src/lib/edid.c b/src/lib/edid.c index d1b9c9fc47..4fb361aa0f 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -296,15 +296,13 @@ detailed_block(struct edid *result_edid, unsigned char *x, int in_extension, if (c->claims_one_point_four) { if (x[4] & 0x02) { v_max_offset = 255; - if (x[4] & 0x01) { + if (x[4] & 0x01) v_min_offset = 255; - } } if (x[4] & 0x04) { h_max_offset = 255; - if (x[4] & 0x03) { + if (x[4] & 0x03) h_min_offset = 255; - } } } else if (x[4]) { c->has_valid_range_descriptor = 0; @@ -446,9 +444,8 @@ detailed_block(struct edid *result_edid, unsigned char *x, int in_extension, } } - if (c->seen_non_detailed_descriptor && !in_extension) { + if (c->seen_non_detailed_descriptor && !in_extension) c->has_valid_descriptor_ordering = 0; - } /* Edid contains pixel clock in terms of 10KHz */ out->mode.pixel_clock = (x[0] + (x[1] << 8)) * 10; @@ -871,9 +868,8 @@ parse_cea(struct edid *out, unsigned char *x, struct edid_context *c) } else if (version == 3) { int i; printk(BIOS_SPEW, "%d bytes of CEA data\n", offset - 4); - for (i = 4; i < offset; i += (x[i] & 0x1f) + 1) { + for (i = 4; i < offset; i += (x[i] & 0x1f) + 1) cea_block(out, x + i); - } } if (version >= 2) { @@ -1156,9 +1152,8 @@ int decode_edid(unsigned char *edid, int size, struct edid *out) extra_info.type = edid[0x14] & 0x0f; } else if (c.claims_one_point_two) { conformance_mask = 0x7E; - if (edid[0x14] & 0x01) { + if (edid[0x14] & 0x01) printk(BIOS_SPEW, "DFP 1.x compatible TMDS\n"); - } } else conformance_mask = 0x7F; |