Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44299 )
Change subject: soc/intel/common/block/sata: Add common sata driver ......................................................................
Patch Set 6:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44299/1/src/soc/intel/common/block/... File src/soc/intel/common/block/sata/Kconfig:
https://review.coreboot.org/c/coreboot/+/44299/1/src/soc/intel/common/block/... PS1, Line 4: Intel Processor
Or `Intel PCH` (not everything is a SoC)
Ack
https://review.coreboot.org/c/coreboot/+/44299/1/src/soc/intel/common/block/... File src/soc/intel/common/block/sata/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/44299/1/src/soc/intel/common/block/... PS1, Line 2:
nit: extra line?
https://qa.coreboot.org/job/coreboot-gerrit/137939/testReport/junit/(root)/l...