linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] PCI: Add function 1 DMA alias quirk for Marvell 9170 SATA controller
@ 2019-04-05 15:20 Andre Przywara
  2019-04-05 21:13 ` Bjorn Helgaas
  0 siblings, 1 reply; 2+ messages in thread
From: Andre Przywara @ 2019-04-05 15:20 UTC (permalink / raw)
  To: Bjorn Helgaas; +Cc: linux-pci, linux-kernel

There is a Marvell 88SE9170 PCIe SATA controller I found on a board
here. Some quick testing with the ARM SMMU enabled reveals that it
suffers from the same requester ID mixup problems as the other Marvell
chips listed already.

Add the PCI vendor/device ID to the list of chips which need the
workaround.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
---
 drivers/pci/quirks.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index a59ad09ce911..a077f67fe1da 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3877,6 +3877,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9128,
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9130,
 			 quirk_dma_func1_alias);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9170,
+			 quirk_dma_func1_alias);
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c47 + c57 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9172,
 			 quirk_dma_func1_alias);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] PCI: Add function 1 DMA alias quirk for Marvell 9170 SATA controller
  2019-04-05 15:20 [PATCH] PCI: Add function 1 DMA alias quirk for Marvell 9170 SATA controller Andre Przywara
@ 2019-04-05 21:13 ` Bjorn Helgaas
  0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2019-04-05 21:13 UTC (permalink / raw)
  To: Andre Przywara; +Cc: linux-pci, linux-kernel

On Fri, Apr 05, 2019 at 04:20:47PM +0100, Andre Przywara wrote:
> There is a Marvell 88SE9170 PCIe SATA controller I found on a board
> here. Some quick testing with the ARM SMMU enabled reveals that it
> suffers from the same requester ID mixup problems as the other Marvell
> chips listed already.
> 
> Add the PCI vendor/device ID to the list of chips which need the
> workaround.
> 
> Signed-off-by: Andre Przywara <andre.przywara@arm.com>

Applied with stable tag to for-linus for v5.1, thanks!

> ---
>  drivers/pci/quirks.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index a59ad09ce911..a077f67fe1da 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3877,6 +3877,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9128,
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9130,
>  			 quirk_dma_func1_alias);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9170,
> +			 quirk_dma_func1_alias);
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c47 + c57 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9172,
>  			 quirk_dma_func1_alias);
> -- 
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-04-05 21:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-05 15:20 [PATCH] PCI: Add function 1 DMA alias quirk for Marvell 9170 SATA controller Andre Przywara
2019-04-05 21:13 ` Bjorn Helgaas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).