Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42009 )
Change subject: sb/intel/bd82x6x: Align some ME functions ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42009/2/src/southbridge/intel/bd82x... File src/southbridge/intel/bd82x6x/me_8.x.c:
https://review.coreboot.org/c/coreboot/+/42009/2/src/southbridge/intel/bd82x... PS2, Line 568: #if CONFIG(CHROMEOS) : #include <vendorcode/google/chromeos/chromeos.h> : #endif
please, why this is moved here ?
I moved this here to make the diff simpler. We can remove all unneeded headers in a follow-up (I'm pretty sure there will be lots of unused stuff). Does it sound good?