Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38786 )
Change subject: mb/pcengines/apu2: Remove unnecessary variable ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
https://review.coreboot.org/c/coreboot/+/38786/1/src/mainboard/pcengines/apu... File src/mainboard/pcengines/apu2/mainboard.c:
https://review.coreboot.org/c/coreboot/+/38786/1/src/mainboard/pcengines/apu... PS1, Line 246: return mainboard_smbios_type16(agesa_dmi, handle, current) + handle incrementation by two must be done after the tables are assembled, not before, otherwise, the handle numbers in the tables will not conform to SMBIOS specification