On 2021/10/11 20:48, István Pongrácz wrote: > Signed-off-by: István Pongrácz <pongracz.istvan@xxxxxxxxx> Getting there ! But you forgot to add a short commit message. You do not need much I think: simply describe the hardware being supported. I think this is a SATA expander card with port multiplier, right ? So something like: Add support for SATA expander card based on ASMedia blah controller chip. Will be fine. > --- > drivers/ata/ahci.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c > index 186cbf90c8ea..97fd96b67731 100644 > --- a/drivers/ata/ahci.c > +++ b/drivers/ata/ahci.c > @@ -592,6 +592,7 @@ static const struct pci_device_id ahci_pci_tbl[] = { > { PCI_VDEVICE(ASMEDIA, 0x0612), board_ahci }, /* ASM1062 */ > { PCI_VDEVICE(ASMEDIA, 0x0621), board_ahci }, /* ASM1061R */ > { PCI_VDEVICE(ASMEDIA, 0x0622), board_ahci }, /* ASM1062R */ > + { PCI_VDEVICE(ASMEDIA, 0x0624), board_ahci }, /* ASM1062+JMB575 */ > > /* > * Samsung SSDs found on some macbooks. NCQ times out if MSI is > -- Damien Le Moal Western Digital Research