diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-09 16:21:34 -0800 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-03-13 15:46:02 +0100 |
| commit | e20a3191f58ffb37da3ed6414491d61dbfc7e583 (patch) | |
| tree | 2ab3d87fb88eb80f4f9238567feb2ac3e1198ffc /src/lib/edid.c | |
| parent | cdd7686a9dcf5200cabeab806648e87993662619 (diff) | |
| download | coreboot-e20a3191f58ffb37da3ed6414491d61dbfc7e583.tar.xz | |
src/lib: Use tabs instead of spaces
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: code indent should use tabs where possible
ERROR: switch and case should be at the same indent
WARNING: Statements should start on a tabstop
WARNING: please, no spaces at the start of a line
WARNING: please, no space before tabs
WARNING: suspect code indent for conditional statements
WARNING: labels should not be indented
TEST=Build and run on Galileo Gen2
Change-Id: Iebcff26ad41ab6eb0027b871a1c06f3b52dd207c
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18732
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/lib/edid.c')
| -rw-r--r-- | src/lib/edid.c | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/src/lib/edid.c b/src/lib/edid.c index 50a023819f..50bb2c282a 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -875,11 +875,9 @@ parse_cea(struct edid *out, unsigned char *x, struct edid_context *c) if (offset < 4) break; - if (version < 3) { + if (version < 3) printk(BIOS_SPEW, "%d 8-byte timing descriptors\n", (offset - 4) / 8); - if (offset - 4 > 0) - /* do stuff */ ; - } else if (version == 3) { + 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) @@ -1188,7 +1186,7 @@ int decode_edid(unsigned char *edid, int size, struct edid *out) break; } extra_info.type = edid[0x14] & 0x0f; - } else if (c.claims_one_point_two) { + } else if (c.claims_one_point_two) { conformance_mask = 0x7E; if (edid[0x14] & 0x01) printk(BIOS_SPEW, "DFP 1.x compatible TMDS\n"); @@ -1273,18 +1271,18 @@ int decode_edid(unsigned char *edid, int size, struct edid *out) /* FIXME: this is from 1.4 spec, check earlier */ if (analog) { switch (edid[0x18] & 0x18) { - case 0x00: - printk(BIOS_SPEW, "Monochrome or grayscale display\n"); - break; - case 0x08: - printk(BIOS_SPEW, "RGB color display\n"); - break; - case 0x10: - printk(BIOS_SPEW, "Non-RGB color display\n"); - break; - case 0x18: - printk(BIOS_SPEW, "Undefined display color type\n"); - break; + case 0x00: + printk(BIOS_SPEW, "Monochrome or grayscale display\n"); + break; + case 0x08: + printk(BIOS_SPEW, "RGB color display\n"); + break; + case 0x10: + printk(BIOS_SPEW, "Non-RGB color display\n"); + break; + case 0x18: + printk(BIOS_SPEW, "Undefined display color type\n"); + break; } } else { printk(BIOS_SPEW, "Supported color formats: RGB 4:4:4"); |