build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47962 )
Change subject: mb/system76/oryp5: Enable TAS5825M smart amp ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47962/1/src/mainboard/system76/oryp... File src/mainboard/system76/oryp5/tas5825m.c:
https://review.coreboot.org/c/coreboot/+/47962/1/src/mainboard/system76/oryp... PS1, Line 3: #define R(F) { \ Macros with flow control statements should be avoided
https://review.coreboot.org/c/coreboot/+/47962/1/src/mainboard/system76/oryp... PS1, Line 5: if (res < 0) return res; \ trailing statements should be on next line