src/lib/edid.c: Replace #if 1 with something useful
Since `#if 1` is rather useless, and the code inside it is just several `printk(BIOS_SPEW)`, using `if (console_log_level(BIOS_SPEW))` instead seems more reasonable. Change-Id: I93dcab3db958480626fea6d99ab5289ebff04e8f Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/28872 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Julius Werner <jwerner@chromium.org>
This commit is contained in:
@ -204,12 +204,13 @@ detailed_block(struct edid *result_edid, unsigned char *x, int in_extension,
|
|||||||
{
|
{
|
||||||
struct edid *out = &tmp_edid;
|
struct edid *out = &tmp_edid;
|
||||||
int i;
|
int i;
|
||||||
#if 1
|
|
||||||
printk(BIOS_SPEW, "Hex of detail: ");
|
if (console_log_level(BIOS_SPEW)) {
|
||||||
for (i = 0; i < 18; i++)
|
printk(BIOS_SPEW, "Hex of detail: ");
|
||||||
printk(BIOS_SPEW, "%02x", x[i]);
|
for (i = 0; i < 18; i++)
|
||||||
printk(BIOS_SPEW, "\n");
|
printk(BIOS_SPEW, "%02x", x[i]);
|
||||||
#endif
|
printk(BIOS_SPEW, "\n");
|
||||||
|
}
|
||||||
|
|
||||||
/* Result might already have some valid fields like mode_is_supported */
|
/* Result might already have some valid fields like mode_is_supported */
|
||||||
*out = *result_edid;
|
*out = *result_edid;
|
||||||
|
Reference in New Issue
Block a user