| Message ID | 20201106100526.17726-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/2] PCI: Disable MSI for Pericom PCIe-USB adapter | expand |
On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > Pericom PCIe-USB adapter advertises MSI, but documentation says > "The MSI Function is not implemented on this device." in the chapters > 7.3.27, 7.3.29-7.3.31. > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > Reported-by: alberto.vignani@fastwebnet.it Is there a URL to a problem report we can include here? > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > v2: dropped confusing word (David), added ifdeffery (Ben) > drivers/pci/quirks.c | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index f70692ac79c5..e66e0cc8f99b 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -5567,17 +5567,26 @@ static void pci_fixup_no_d0_pme(struct pci_dev *dev) > DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASMEDIA, 0x2142, pci_fixup_no_d0_pme); > > /* > - * Device [12d8:0x400e] and [12d8:0x400f] > + * Device 12d8:0x400e [OHCI] and 12d8:0x400f [EHCI] > + * > * These devices advertise PME# support in all power states but don't > * reliably assert it. > + * > + * These devices advertise MSI, but documentation (PI7C9X440SL.pdf) says > + * "The MSI Function is not implemented on this device." in the chapters > + * 7.3.27, 7.3.29-7.3.31. > */ > -static void pci_fixup_no_pme(struct pci_dev *dev) > +static void pci_fixup_no_msi_no_pme(struct pci_dev *dev) > { > +#ifdef CONFIG_PCI_MSI > + pci_info(dev, "The MSI is not implemented on this device, disabling it\n"); > + dev->no_msi = 1; > +#endif > pci_info(dev, "PME# is unreliable, disabling it\n"); > dev->pme_support = 0; > } > -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_pme); > -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_pme); > +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_msi_no_pme); > +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_msi_no_pme); > > static void apex_pci_fixup_class(struct pci_dev *pdev) > { > -- > 2.28.0 >
On Fri, Nov 13, 2020 at 05:21:38PM -0600, Bjorn Helgaas wrote: > On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > > Pericom PCIe-USB adapter advertises MSI, but documentation says > > "The MSI Function is not implemented on this device." in the chapters > > 7.3.27, 7.3.29-7.3.31. > > > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > > Reported-by: alberto.vignani@fastwebnet.it > > Is there a URL to a problem report we can include here? You mean URL to email archives or something else?
On Mon, Nov 16, 2020 at 11:46:44AM +0200, Andy Shevchenko wrote: > On Fri, Nov 13, 2020 at 05:21:38PM -0600, Bjorn Helgaas wrote: > > On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > > > Pericom PCIe-USB adapter advertises MSI, but documentation says > > > "The MSI Function is not implemented on this device." in the chapters > > > 7.3.27, 7.3.29-7.3.31. > > > > > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > > > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > > > Reported-by: alberto.vignani@fastwebnet.it > > > > Is there a URL to a problem report we can include here? > > You mean URL to email archives or something else? Yes. URL to lore email archive, bugzilla, or any other report of the problem this caused, i.e., how Alberto noticed something wrong.
On Mon, Nov 16, 2020 at 2:42 PM Bjorn Helgaas <helgaas@kernel.org> wrote: > On Mon, Nov 16, 2020 at 11:46:44AM +0200, Andy Shevchenko wrote: > > On Fri, Nov 13, 2020 at 05:21:38PM -0600, Bjorn Helgaas wrote: > > > On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > > > > Pericom PCIe-USB adapter advertises MSI, but documentation says > > > > "The MSI Function is not implemented on this device." in the chapters > > > > 7.3.27, 7.3.29-7.3.31. > > > > > > > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > > > > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > > > > Reported-by: alberto.vignani@fastwebnet.it > > > > > > Is there a URL to a problem report we can include here? > > > > You mean URL to email archives or something else? > > Yes. URL to lore email archive, bugzilla, or any other report of the > problem this caused, i.e., how Alberto noticed something wrong. Okay, I'll include in v3.
On Mon, Nov 16, 2020 at 03:46:44PM +0200, Andy Shevchenko wrote: > On Mon, Nov 16, 2020 at 2:42 PM Bjorn Helgaas <helgaas@kernel.org> wrote: > > On Mon, Nov 16, 2020 at 11:46:44AM +0200, Andy Shevchenko wrote: > > > On Fri, Nov 13, 2020 at 05:21:38PM -0600, Bjorn Helgaas wrote: > > > > On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > > > > > Pericom PCIe-USB adapter advertises MSI, but documentation says > > > > > "The MSI Function is not implemented on this device." in the chapters > > > > > 7.3.27, 7.3.29-7.3.31. > > > > > > > > > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > > > > > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > > > > > Reported-by: alberto.vignani@fastwebnet.it > > > > > > > > Is there a URL to a problem report we can include here? > > > > > > You mean URL to email archives or something else? > > > > Yes. URL to lore email archive, bugzilla, or any other report of the > > problem this caused, i.e., how Alberto noticed something wrong. > > Okay, I'll include in v3. I may not include that. It misses archives probably due to some footer in the message. Closest what I have found is [1]. Do you still want me to include that link? [1]: https://lore.kernel.org/linux-usb/20201030134826.GP4077@smile.fi.intel.com/
On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: > Pericom PCIe-USB adapter advertises MSI, but documentation says > "The MSI Function is not implemented on this device." in the chapters > 7.3.27, 7.3.29-7.3.31. > > Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") > Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf > Reported-by: alberto.vignani@fastwebnet.it > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> I added the URL you sent and applied both of these, thanks! > --- > v2: dropped confusing word (David), added ifdeffery (Ben) > drivers/pci/quirks.c | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index f70692ac79c5..e66e0cc8f99b 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -5567,17 +5567,26 @@ static void pci_fixup_no_d0_pme(struct pci_dev *dev) > DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASMEDIA, 0x2142, pci_fixup_no_d0_pme); > > /* > - * Device [12d8:0x400e] and [12d8:0x400f] > + * Device 12d8:0x400e [OHCI] and 12d8:0x400f [EHCI] > + * > * These devices advertise PME# support in all power states but don't > * reliably assert it. > + * > + * These devices advertise MSI, but documentation (PI7C9X440SL.pdf) says > + * "The MSI Function is not implemented on this device." in the chapters > + * 7.3.27, 7.3.29-7.3.31. > */ > -static void pci_fixup_no_pme(struct pci_dev *dev) > +static void pci_fixup_no_msi_no_pme(struct pci_dev *dev) > { > +#ifdef CONFIG_PCI_MSI > + pci_info(dev, "The MSI is not implemented on this device, disabling it\n"); > + dev->no_msi = 1; > +#endif > pci_info(dev, "PME# is unreliable, disabling it\n"); > dev->pme_support = 0; > } > -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_pme); > -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_pme); > +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_msi_no_pme); > +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_msi_no_pme); > > static void apex_pci_fixup_class(struct pci_dev *pdev) > { > -- > 2.28.0 >
On Tue, Nov 17, 2020 at 05:32:52PM -0600, Bjorn Helgaas wrote: > On Fri, Nov 06, 2020 at 12:05:25PM +0200, Andy Shevchenko wrote: ... > I added the URL you sent and applied both of these, thanks! Thank you!
diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index f70692ac79c5..e66e0cc8f99b 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -5567,17 +5567,26 @@ static void pci_fixup_no_d0_pme(struct pci_dev *dev) DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASMEDIA, 0x2142, pci_fixup_no_d0_pme); /* - * Device [12d8:0x400e] and [12d8:0x400f] + * Device 12d8:0x400e [OHCI] and 12d8:0x400f [EHCI] + * * These devices advertise PME# support in all power states but don't * reliably assert it. + * + * These devices advertise MSI, but documentation (PI7C9X440SL.pdf) says + * "The MSI Function is not implemented on this device." in the chapters + * 7.3.27, 7.3.29-7.3.31. */ -static void pci_fixup_no_pme(struct pci_dev *dev) +static void pci_fixup_no_msi_no_pme(struct pci_dev *dev) { +#ifdef CONFIG_PCI_MSI + pci_info(dev, "The MSI is not implemented on this device, disabling it\n"); + dev->no_msi = 1; +#endif pci_info(dev, "PME# is unreliable, disabling it\n"); dev->pme_support = 0; } -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_pme); -DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_pme); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400e, pci_fixup_no_msi_no_pme); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_PERICOM, 0x400f, pci_fixup_no_msi_no_pme); static void apex_pci_fixup_class(struct pci_dev *pdev) {
Pericom PCIe-USB adapter advertises MSI, but documentation says "The MSI Function is not implemented on this device." in the chapters 7.3.27, 7.3.29-7.3.31. Fixes: 306c54d0edb6 ("usb: hcd: Try MSI interrupts on PCI devices") Datasheet: https://www.diodes.com/assets/Datasheets/PI7C9X440SL.pdf Reported-by: alberto.vignani@fastwebnet.it Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- v2: dropped confusing word (David), added ifdeffery (Ben) drivers/pci/quirks.c | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-)