[PATCH] ahci: add PCI bindings for Marvell 88SE6121/45 SATA controllers

Add AHCI PCI bindings for Marvell 88SE6121/45 SATA controllers.
The 88SE6121 controller is used, for example, in the Seagate Blackarmor NAS440 or the Iomega ix4-200d NAS.
As Pali Rohár explained [1], these controllers do not match the standard AHCI class code and therefore require an explizit PCI binding. The Linux kernel also uses this approach [2].
[1] https://lists.denx.de/pipermail/u-boot/2022-March/479197.html [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/driv...
Signed-off-by: Hajo Noerenberg hajo-uboot@noerenberg.de --- drivers/ata/ahci-pci.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/ata/ahci-pci.c b/drivers/ata/ahci-pci.c index b1d231e0f9..797e0d570e 100644 --- a/drivers/ata/ahci-pci.c +++ b/drivers/ata/ahci-pci.c @@ -38,6 +38,8 @@ U_BOOT_DRIVER(ahci_pci) = { static struct pci_device_id ahci_pci_supported[] = { { PCI_DEVICE_CLASS(PCI_CLASS_STORAGE_SATA_AHCI, ~0) }, { PCI_DEVICE(0x1b21, 0x0611) }, + { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x6121) }, + { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x6145) }, {}, };

On Monday 28 March 2022 15:49:08 Hajo Noerenberg wrote:
Add AHCI PCI bindings for Marvell 88SE6121/45 SATA controllers.
The 88SE6121 controller is used, for example, in the Seagate Blackarmor NAS440 or the Iomega ix4-200d NAS.
As Pali Rohár explained [1], these controllers do not match the standard AHCI class code and therefore require an explizit PCI binding. The Linux kernel also uses this approach [2].
[1] https://lists.denx.de/pipermail/u-boot/2022-March/479197.html [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/driv...
Signed-off-by: Hajo Noerenberg hajo-uboot@noerenberg.de
Reviewed-by: Pali Rohár pali@kernel.org
drivers/ata/ahci-pci.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/ata/ahci-pci.c b/drivers/ata/ahci-pci.c index b1d231e0f9..797e0d570e 100644 --- a/drivers/ata/ahci-pci.c +++ b/drivers/ata/ahci-pci.c @@ -38,6 +38,8 @@ U_BOOT_DRIVER(ahci_pci) = { static struct pci_device_id ahci_pci_supported[] = { { PCI_DEVICE_CLASS(PCI_CLASS_STORAGE_SATA_AHCI, ~0) }, { PCI_DEVICE(0x1b21, 0x0611) },
- { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x6121) },
- { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x6145) }, {},
};
-- 2.20.1

On Mon, Mar 28, 2022 at 03:49:08PM +0200, Hajo Noerenberg wrote:
Add AHCI PCI bindings for Marvell 88SE6121/45 SATA controllers.
The 88SE6121 controller is used, for example, in the Seagate Blackarmor NAS440 or the Iomega ix4-200d NAS.
As Pali Rohár explained [1], these controllers do not match the standard AHCI class code and therefore require an explizit PCI binding. The Linux kernel also uses this approach [2].
[1] https://lists.denx.de/pipermail/u-boot/2022-March/479197.html [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/driv...
Signed-off-by: Hajo Noerenberg hajo-uboot@noerenberg.de Reviewed-by: Pali Rohár pali@kernel.org
Applied to u-boot/master, thanks!
participants (3)
-
Hajo Noerenberg
-
Pali Rohár
-
Tom Rini