[coreboot-gerrit] Change in coreboot[master]: intel/i440bx: Use smbus_read_byte() for raminit debug

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38381 )

Change subject: intel/i440bx: Use smbus_read_byte() for raminit debug
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38381/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/38381/1//COMMIT_MSG@9 
PS1, Line 9: Build broke with CONFIG_DEBUG_RAM_SETUP
> You might want to put a config file in /configs so that it gets build-tested.

+1 on adding a config file that exercises this option (can, and probably should be in a separate commit, but please do so before marking this comment "resolved")

https://review.coreboot.org/c/coreboot/+/38381/1//COMMIT_MSG@9 
PS1, Line 9: Build broke with CONFIG_DEBUG_RAM_SETUP enabled after commit 3f882faf.
> Please use […]

Done