Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35306 )
Change subject: soc/intel/cannonlake: Add config for sata devslp pad reset configuration ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35306/4/src/soc/intel/cannonlake/ch... File src/soc/intel/cannonlake/chip.h:
https://review.coreboot.org/c/coreboot/+/35306/4/src/soc/intel/cannonlake/ch... PS4, Line 145: 7
do we need a comma here ?
not required.