Change in coreboot[master]: util/inteltool: ahci: rework AHCI

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39504 )

Change subject: util/inteltool: ahci: rework AHCI
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39504/4/util/inteltool/ahci.c 
File util/inteltool/ahci.c:

https://review.coreboot.org/c/coreboot/+/39504/4/util/inteltool/ahci.c@71 
PS4, Line 71: 	case PCI_DEVICE_ID_INTEL_SUNRISEPOINT_SATA:
> iirc coverity and maybe even gcc want a comment here that the fall-through is intended. […]

I don't think they consider fallthrough when there's no code between case statements.

https://review.coreboot.org/c/coreboot/+/39504/4/util/inteltool/ahci.c@82 
PS4, Line 82: ahci_registers_size
Why is this renamed, but "mmio" is not?