[PATCH] drivers/pci: disable extension-tags on 3w-9xxx controller

[PATCH] drivers/pci: disable extension-tags on 3w-9xxx controller

[Jörg Wedekind]

Signed-off-by: Jörg Wedekind <joerg@wedekind.de>
---
 drivers/pci/quirks.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index d797df6e5f3e..2ebbe51a7efe 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -5527,6 +5527,7 @@ static void quirk_no_ext_tags(struct pci_dev *pdev)
 
 	pci_walk_bus(bridge->bus, pci_configure_extended_tags, NULL);
 }
+DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_3WARE, 0x1004, quirk_no_ext_tags);
 DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0132, quirk_no_ext_tags);
 DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0140, quirk_no_ext_tags);
 DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0141, quirk_no_ext_tags);
-- 
2.35.3

Re: [PATCH] drivers/pci: disable extension-tags on 3w-9xxx controller

[Ilpo Järvinen]

[-- Attachment #1: Type: text/plain, Size: 931 bytes --]

On Fri, 16 Feb 2024, Jörg Wedekind wrote:

Hi,

You need to provide a description here too, the shortlog in subject is not 
enough.

-- 
 i.

> Signed-off-by: Jörg Wedekind <joerg@wedekind.de>
> ---
>  drivers/pci/quirks.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index d797df6e5f3e..2ebbe51a7efe 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -5527,6 +5527,7 @@ static void quirk_no_ext_tags(struct pci_dev *pdev)
>  
>  	pci_walk_bus(bridge->bus, pci_configure_extended_tags, NULL);
>  }
> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_3WARE, 0x1004, quirk_no_ext_tags);
>  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0132, quirk_no_ext_tags);
>  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0140, quirk_no_ext_tags);
>  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0141, quirk_no_ext_tags);
> 

Re: [PATCH] drivers/pci: disable extension-tags on 3w-9xxx controller

[Bjorn Helgaas]

On Fri, Feb 16, 2024 at 04:59:57PM +0200, Ilpo Järvinen wrote:
> On Fri, 16 Feb 2024, Jörg Wedekind wrote:
> You need to provide a description here too, the shortlog in subject is not 
> enough.

Yes, we would like a little background, e.g., a URL for a bug report
if it exists, a description of how a user might recognize this
problem, etc.  See https://git.kernel.org/linus/1b30dfd376e2 ("PCI:
Mark Broadcom HT1100 and HT2000 Root Port Extended Tags as broken")
for a sample.

Also update the subject line to use the same pattern as 1b30dfd376e2.

> > Signed-off-by: Jörg Wedekind <joerg@wedekind.de>
> > ---
> >  drivers/pci/quirks.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> > index d797df6e5f3e..2ebbe51a7efe 100644
> > --- a/drivers/pci/quirks.c
> > +++ b/drivers/pci/quirks.c
> > @@ -5527,6 +5527,7 @@ static void quirk_no_ext_tags(struct pci_dev *pdev)
> >  
> >  	pci_walk_bus(bridge->bus, pci_configure_extended_tags, NULL);
> >  }
> > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_3WARE, 0x1004, quirk_no_ext_tags);
> >  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0132, quirk_no_ext_tags);
> >  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0140, quirk_no_ext_tags);
> >  DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_SERVERWORKS, 0x0141, quirk_no_ext_tags);
> >