Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/19722 )
Change subject: soc: Add AMD StoneyRidge southbridge code ......................................................................
Patch Set 2:
(1 comment)
Other files could likely benefit from similar include file cleanup, but we'd missed the path for dimmSpd.h.
https://review.coreboot.org/#/c/19722/2/src/soc/amd/stoneyridge/smbus_spd.c File src/soc/amd/stoneyridge/smbus_spd.c:
PS2, Line 20: #include "Porting.h" : #include "AGESA.h" : #include "amdlib.h" : : #include <northbridge/amd/pi/dimmSpd.h> #include <Porting.h> #include <AGESA.h> #include <amdlib.h>
#include <dimmSpd.h>
maybe remove blank line