Re: [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120

[Alex Williamson <alex.williamson@redhat.com>]

On Thu, 2015-05-21 at 12:54 +0300, Sakari Ailus wrote:
> Ping.
> 
> (Cc Alex.)

You can add my ack, but the linux-pci list is the proper list to get
this upstream.  I suggest resending it there.  Thanks,

Ale

> Sakari Ailus wrote:
> > Marvell 9120 SATA controller has the same issue as a number of others, use
> > the same quirk for this one. The other quirks were added by patch
> > "PCI: Add function 1 DMA alias quirk for Marvell devices"
> > (commit id cc346a4714a59d08c118e8f33fd86692d3563133).
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>


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


> > ---
> > Hi,
> > 
> > I just booted the v4.0.1 kernel with intel-iommu enabled and had similar
> > issues as in the original bug report with Marvell 88SE9120 (rev 12). The
> > same quirk appears to work for that one as well.
> > 
> >  drivers/pci/quirks.c |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> > index 85f247e..d4af5a8 100644
> > --- a/drivers/pci/quirks.c
> > +++ b/drivers/pci/quirks.c
> > @@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
> >   * SKUs this function is not present, making this a ghost requester.
> >   * https://bugzilla.kernel.org/show_bug.cgi?id=42679
> >   */
> > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
> > +			 quirk_dma_func1_alias);
> >  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
> >  			 quirk_dma_func1_alias);
> >  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
> > 
> 
>