[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/i82801gx: Detect if the southbridge supports AHCI

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30822 )

Change subject: sb/intel/i82801gx: Detect if the southbridge supports AHCI
......................................................................

Patch Set 11:

(1 comment)

https://review.coreboot.org/#/c/30822/11/src/southbridge/intel/i82801gx/sata... 
File src/southbridge/intel/i82801gx/sata.c:

https://review.coreboot.org/#/c/30822/11/src/southbridge/intel/i82801gx/sata... 
PS11, Line 64: 			return;
> The print message needs updating, but if the first function on the PCI device cannot be found it mak […]

Yeah, technically we could die() here. Though, even with that information
in mind, wouldn't it be nicer if one can read (and follow) the code without
having to think about it?