Hi, On 6/15/21 9:08 PM, Charles Rose wrote: > This patch enables support for Dell S140 and later controllers > that use Intel's PCHs configured as PCI_CLASS_STORAGE_RAID. > > Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxx> > Signed-off-by: Charles Rose <charles.rose@xxxxxxxx> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> Regards, Hans > --- > drivers/ata/ahci.c | 4 ++++ > include/linux/pci_ids.h | 2 ++ > 2 files changed, 6 insertions(+) > > diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c > index 33192a8f687d..186cbf90c8ea 100644 > --- a/drivers/ata/ahci.c > +++ b/drivers/ata/ahci.c > @@ -446,6 +446,10 @@ static const struct pci_device_id ahci_pci_tbl[] = { > { PCI_VENDOR_ID_AMD, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, > PCI_CLASS_STORAGE_RAID << 8, 0xffffff, board_ahci }, > > + /* Dell S140/S150 */ > + { PCI_VENDOR_ID_INTEL, PCI_ANY_ID, PCI_SUBVENDOR_ID_DELL, PCI_ANY_ID, > + PCI_CLASS_STORAGE_RAID << 8, 0xffffff, board_ahci }, > + > /* VIA */ > { PCI_VDEVICE(VIA, 0x3349), board_ahci_vt8251 }, /* VIA VT8251 */ > { PCI_VDEVICE(VIA, 0x6287), board_ahci_vt8251 }, /* VIA VT8251 */ > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index 4c3fa5293d76..803ec446a729 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -631,6 +631,8 @@ > #define PCI_DEVICE_ID_DELL_RAC4 0x0012 > #define PCI_DEVICE_ID_DELL_PERC5 0x0015 > > +#define PCI_SUBVENDOR_ID_DELL 0x1028 > + > #define PCI_VENDOR_ID_MATROX 0x102B > #define PCI_DEVICE_ID_MATROX_MGA_2 0x0518 > #define PCI_DEVICE_ID_MATROX_MIL 0x0519 >