Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/21033 )
Change subject: amd/pi/hudson: Convert 48Mhz en to read/write32 ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/21033/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/21033/1//COMMIT_MSG@7 PS1, Line 7: 48Mhz en It looks like this got modified. I'd change it to MHz and enabled.
https://review.coreboot.org/#/c/21033/1/src/southbridge/amd/pi/hudson/early_... File src/southbridge/amd/pi/hudson/early_setup.c:
https://review.coreboot.org/#/c/21033/1/src/southbridge/amd/pi/hudson/early_... PS1, Line 272: (u32) This cast should come out.