Change in flashrom[master]: [v4, 4/6] ENE EDI - print debug info like others while probing for ENE...

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/23261 )

Change subject: [v4,4/6] ENE EDI - print debug info like others while probing for ENE chips
......................................................................

Patch Set 1:

(4 comments)

https://review.coreboot.org/#/c/23261/1/edi.c
File edi.c:

https://review.coreboot.org/#/c/23261/1/edi.c@154
PS1, Line 154: 0x%x 
It's a signed integer, why print it in hex?

https://review.coreboot.org/#/c/23261/1/edi.c@164
PS1, Line 164: 	msg_cdbg("%s: rc1 0x%x, rc2 0x%x; ", __func__, rc1, rc2);
No need to print them, they are 0 on this path.

https://review.coreboot.org/#/c/23261/1/edi.c@169
PS1, Line 169: 	} else
Please add braces here too (they are not required by syntax but help
to maintain visual balance).

https://review.coreboot.org/#/c/23261/1/edi.c@171
PS1, Line 171: 			 hwversion, chip->hwversion, ediid, chip->ediid);
Actually, I'm not sure if we want this message at all. This function
is supposed to be called multiple times with different ids. So it's
expected that they don't match.

If you want to keep it, please use a higher debug level, e.g. msg_cdbg2().