diff mbox

PCI: quirk dma_alias_devfn for HighPoint RocketRaid 642L (Marvell 9235)

Message ID 20140603154156.4f1b46c4@Vantage (mailing list archive)
State New, archived
Delegated to: Bjorn Helgaas
Headers show

Commit Message

Jérôme Carretero June 3, 2014, 7:41 p.m. UTC
This vendor has similar boards based on the same Marvell 88SE9235 chipset,
but this patch was only tested with the 642L.

Tested on ASUS Sabertooth 990FX (AMD).

If you listen to the vendor support stating:

  Dear Customer,
  Sorry about that.
  Currently all our products' software package don't support IOMMU
  and we don't have plan to support that.

... you'll think you also wasted your money, but it turns out that you
just lost a few hours of your valuable time talking to their support,
because the good people from the Internet made their board work,
with the regular AHCI driver, and even with IOMMU. Nice!


Signed-off-by: Jérôme Carretero <cJ-ko@zougloub.eu>

---
 drivers/pci/quirks.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Alex Williamson June 11, 2014, 7:25 p.m. UTC | #1
On Tue, 2014-06-03 at 15:41 -0400, Jérôme Carretero wrote:
> This vendor has similar boards based on the same Marvell 88SE9235 chipset,
> but this patch was only tested with the 642L.
> 
> Tested on ASUS Sabertooth 990FX (AMD).
> 
> If you listen to the vendor support stating:
> 
>   Dear Customer,
>   Sorry about that.
>   Currently all our products' software package don't support IOMMU
>   and we don't have plan to support that.
> 
> ... you'll think you also wasted your money, but it turns out that you
> just lost a few hours of your valuable time talking to their support,
> because the good people from the Internet made their board work,
> with the regular AHCI driver, and even with IOMMU. Nice!
> 
> 
> Signed-off-by: Jérôme Carretero <cJ-ko@zougloub.eu>

Acked-by: Alex Williamson <alex.williamson@redhat.com>

> ---
>  drivers/pci/quirks.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index f6a42bc..43c0ea0 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3380,6 +3380,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
>  			 quirk_dma_func1_alias);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
> +			 quirk_dma_func1_alias);
>  /* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
>  			 PCI_DEVICE_ID_JMICRON_JMB388_ESD,



--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Bjorn Helgaas June 11, 2014, 7:54 p.m. UTC | #2
On Tue, Jun 03, 2014 at 03:41:56PM -0400, Jérôme Carretero wrote:
> This vendor has similar boards based on the same Marvell 88SE9235 chipset,
> but this patch was only tested with the 642L.
> 
> Tested on ASUS Sabertooth 990FX (AMD).
> 
> If you listen to the vendor support stating:
> 
>   Dear Customer,
>   Sorry about that.
>   Currently all our products' software package don't support IOMMU
>   and we don't have plan to support that.
> 
> ... you'll think you also wasted your money, but it turns out that you
> just lost a few hours of your valuable time talking to their support,
> because the good people from the Internet made their board work,
> with the regular AHCI driver, and even with IOMMU. Nice!
> 
> 
> Signed-off-by: Jérôme Carretero <cJ-ko@zougloub.eu>

Applied with Alex's ack to pci/iommu for v3.16, thanks!

Note that I only have the PCI parts here.  There are several IOMMU patches
required in addition, and I haven't heard the fate of those.

> ---
>  drivers/pci/quirks.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index f6a42bc..43c0ea0 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3380,6 +3380,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
>  			 quirk_dma_func1_alias);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
> +			 quirk_dma_func1_alias);
>  /* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
>  			 PCI_DEVICE_ID_JMICRON_JMB388_ESD,
> -- 
> 2.0.0
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index f6a42bc..43c0ea0 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3380,6 +3380,8 @@  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
 			 quirk_dma_func1_alias);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
+			 quirk_dma_func1_alias);
 /* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
 			 PCI_DEVICE_ID_JMICRON_JMB388_ESD,