build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32022 )
Change subject: northbridge/amd/pi: Fix null pointer check ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32022/1/src/northbridge/amd/pi/00730F01/dimm... File src/northbridge/amd/pi/00730F01/dimmSpd.c:
https://review.coreboot.org/#/c/32022/1/src/northbridge/amd/pi/00730F01/dimm... PS1, Line 27: AGESA_STATUS AmdMemoryReadSPD(UINT32 unused1, UINTN unused2, AGESA_READ_SPD_PARAMS *info) line over 80 characters
https://review.coreboot.org/#/c/32022/1/src/northbridge/amd/pi/00730F01/dimm... PS1, Line 47: [info->SocketId][info->MemChannelId][info->DimmId]; line over 80 characters