* [PATCH] usb: ehci: add pci device support for Aspeed platforms
@ 2022-02-08 6:29 Neal Liu
2022-02-08 7:37 ` Greg Kroah-Hartman
0 siblings, 1 reply; 5+ messages in thread
From: Neal Liu @ 2022-02-08 6:29 UTC (permalink / raw)
To: Alan Stern, Greg Kroah-Hartman, Bjorn Helgaas, linux-usb,
linux-kernel, linux-pci
Cc: Neal Liu, Tao Ren, BMC-SW
Enable Aspeed quirks in commit 7f2d73788d90 ("usb: ehci:
handshake CMD_RUN instead of STS_HALT") to support Aspeed
ehci-pci device.
Signed-off-by: Neal Liu <neal_liu@aspeedtech.com>
---
drivers/usb/host/ehci-pci.c | 6 ++++++
include/linux/pci_ids.h | 3 +++
2 files changed, 9 insertions(+)
diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c
index e87cf3a00fa4..a91b25d216ae 100644
--- a/drivers/usb/host/ehci-pci.c
+++ b/drivers/usb/host/ehci-pci.c
@@ -222,6 +222,12 @@ static int ehci_pci_setup(struct usb_hcd *hcd)
ehci->has_synopsys_hc_bug = 1;
}
break;
+ case PCI_VENDOR_ID_ASPEED:
+ if (pdev->device == PCI_DEVICE_ID_ASPEED_EHCI) {
+ ehci_info(ehci, "applying Aspeed HC workaround\n");
+ ehci->is_aspeed = 1;
+ }
+ break;
}
/* optional debug port, normally in the first BAR */
diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
index aad54c666407..410b395fe56c 100644
--- a/include/linux/pci_ids.h
+++ b/include/linux/pci_ids.h
@@ -3096,4 +3096,7 @@
#define PCI_VENDOR_ID_NCUBE 0x10ff
+#define PCI_VENDOR_ID_ASPEED 0x1a03
+#define PCI_DEVICE_ID_ASPEED_EHCI 0x2603
+
#endif /* _LINUX_PCI_IDS_H */
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] usb: ehci: add pci device support for Aspeed platforms 2022-02-08 6:29 [PATCH] usb: ehci: add pci device support for Aspeed platforms Neal Liu @ 2022-02-08 7:37 ` Greg Kroah-Hartman 2022-02-08 8:20 ` Neal Liu 0 siblings, 1 reply; 5+ messages in thread From: Greg Kroah-Hartman @ 2022-02-08 7:37 UTC (permalink / raw) To: Neal Liu Cc: Alan Stern, Bjorn Helgaas, linux-usb, linux-kernel, linux-pci, Tao Ren, BMC-SW On Tue, Feb 08, 2022 at 02:29:27PM +0800, Neal Liu wrote: > Enable Aspeed quirks in commit 7f2d73788d90 ("usb: ehci: > handshake CMD_RUN instead of STS_HALT") to support Aspeed > ehci-pci device. > > Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> > --- > drivers/usb/host/ehci-pci.c | 6 ++++++ > include/linux/pci_ids.h | 3 +++ > 2 files changed, 9 insertions(+) > > diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c > index e87cf3a00fa4..a91b25d216ae 100644 > --- a/drivers/usb/host/ehci-pci.c > +++ b/drivers/usb/host/ehci-pci.c > @@ -222,6 +222,12 @@ static int ehci_pci_setup(struct usb_hcd *hcd) > ehci->has_synopsys_hc_bug = 1; > } > break; > + case PCI_VENDOR_ID_ASPEED: > + if (pdev->device == PCI_DEVICE_ID_ASPEED_EHCI) { > + ehci_info(ehci, "applying Aspeed HC workaround\n"); > + ehci->is_aspeed = 1; > + } > + break; > } > > /* optional debug port, normally in the first BAR */ > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index aad54c666407..410b395fe56c 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -3096,4 +3096,7 @@ > > #define PCI_VENDOR_ID_NCUBE 0x10ff > > +#define PCI_VENDOR_ID_ASPEED 0x1a03 > +#define PCI_DEVICE_ID_ASPEED_EHCI 0x2603 Please read the top of this file, this does not need to be added here. thanks, greg k-h ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] usb: ehci: add pci device support for Aspeed platforms 2022-02-08 7:37 ` Greg Kroah-Hartman @ 2022-02-08 8:20 ` Neal Liu 2022-02-08 8:41 ` Greg Kroah-Hartman 0 siblings, 1 reply; 5+ messages in thread From: Neal Liu @ 2022-02-08 8:20 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Alan Stern, Bjorn Helgaas, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, Tao Ren, BMC-SW > -----Original Message----- > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Sent: Tuesday, February 8, 2022 3:38 PM > To: Neal Liu <neal_liu@aspeedtech.com> > Cc: Alan Stern <stern@rowland.harvard.edu>; Bjorn Helgaas > <bhelgaas@google.com>; linux-usb@vger.kernel.org; > linux-kernel@vger.kernel.org; linux-pci@vger.kernel.org; Tao Ren > <rentao.bupt@gmail.com>; BMC-SW <BMC-SW@aspeedtech.com> > Subject: Re: [PATCH] usb: ehci: add pci device support for Aspeed platforms > > On Tue, Feb 08, 2022 at 02:29:27PM +0800, Neal Liu wrote: > > Enable Aspeed quirks in commit 7f2d73788d90 ("usb: ehci: > > handshake CMD_RUN instead of STS_HALT") to support Aspeed ehci-pci > > device. > > > > Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> > > --- > > drivers/usb/host/ehci-pci.c | 6 ++++++ > > include/linux/pci_ids.h | 3 +++ > > 2 files changed, 9 insertions(+) > > > > diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c > > index e87cf3a00fa4..a91b25d216ae 100644 > > --- a/drivers/usb/host/ehci-pci.c > > +++ b/drivers/usb/host/ehci-pci.c > > @@ -222,6 +222,12 @@ static int ehci_pci_setup(struct usb_hcd *hcd) > > ehci->has_synopsys_hc_bug = 1; > > } > > break; > > + case PCI_VENDOR_ID_ASPEED: > > + if (pdev->device == PCI_DEVICE_ID_ASPEED_EHCI) { > > + ehci_info(ehci, "applying Aspeed HC workaround\n"); > > + ehci->is_aspeed = 1; > > + } > > + break; > > } > > > > /* optional debug port, normally in the first BAR */ diff --git > > a/include/linux/pci_ids.h b/include/linux/pci_ids.h index > > aad54c666407..410b395fe56c 100644 > > --- a/include/linux/pci_ids.h > > +++ b/include/linux/pci_ids.h > > @@ -3096,4 +3096,7 @@ > > > > #define PCI_VENDOR_ID_NCUBE 0x10ff > > > > +#define PCI_VENDOR_ID_ASPEED 0x1a03 > > +#define PCI_DEVICE_ID_ASPEED_EHCI 0x2603 > > Please read the top of this file, this does not need to be added here. > > thanks, > > greg k-h Are you suggesting to hard code vendor/device ID in driver instead? Thanks -Neal ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: ehci: add pci device support for Aspeed platforms 2022-02-08 8:20 ` Neal Liu @ 2022-02-08 8:41 ` Greg Kroah-Hartman 2022-02-08 8:53 ` Neal Liu 0 siblings, 1 reply; 5+ messages in thread From: Greg Kroah-Hartman @ 2022-02-08 8:41 UTC (permalink / raw) To: Neal Liu Cc: Alan Stern, Bjorn Helgaas, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, Tao Ren, BMC-SW On Tue, Feb 08, 2022 at 08:20:07AM +0000, Neal Liu wrote: > > -----Original Message----- > > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Sent: Tuesday, February 8, 2022 3:38 PM > > To: Neal Liu <neal_liu@aspeedtech.com> > > Cc: Alan Stern <stern@rowland.harvard.edu>; Bjorn Helgaas > > <bhelgaas@google.com>; linux-usb@vger.kernel.org; > > linux-kernel@vger.kernel.org; linux-pci@vger.kernel.org; Tao Ren > > <rentao.bupt@gmail.com>; BMC-SW <BMC-SW@aspeedtech.com> > > Subject: Re: [PATCH] usb: ehci: add pci device support for Aspeed platforms > > > > On Tue, Feb 08, 2022 at 02:29:27PM +0800, Neal Liu wrote: > > > Enable Aspeed quirks in commit 7f2d73788d90 ("usb: ehci: > > > handshake CMD_RUN instead of STS_HALT") to support Aspeed ehci-pci > > > device. > > > > > > Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> > > > --- > > > drivers/usb/host/ehci-pci.c | 6 ++++++ > > > include/linux/pci_ids.h | 3 +++ > > > 2 files changed, 9 insertions(+) > > > > > > diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c > > > index e87cf3a00fa4..a91b25d216ae 100644 > > > --- a/drivers/usb/host/ehci-pci.c > > > +++ b/drivers/usb/host/ehci-pci.c > > > @@ -222,6 +222,12 @@ static int ehci_pci_setup(struct usb_hcd *hcd) > > > ehci->has_synopsys_hc_bug = 1; > > > } > > > break; > > > + case PCI_VENDOR_ID_ASPEED: > > > + if (pdev->device == PCI_DEVICE_ID_ASPEED_EHCI) { > > > + ehci_info(ehci, "applying Aspeed HC workaround\n"); > > > + ehci->is_aspeed = 1; > > > + } > > > + break; > > > } > > > > > > /* optional debug port, normally in the first BAR */ diff --git > > > a/include/linux/pci_ids.h b/include/linux/pci_ids.h index > > > aad54c666407..410b395fe56c 100644 > > > --- a/include/linux/pci_ids.h > > > +++ b/include/linux/pci_ids.h > > > @@ -3096,4 +3096,7 @@ > > > > > > #define PCI_VENDOR_ID_NCUBE 0x10ff > > > > > > +#define PCI_VENDOR_ID_ASPEED 0x1a03 > > > +#define PCI_DEVICE_ID_ASPEED_EHCI 0x2603 > > > > Please read the top of this file, this does not need to be added here. > > > > thanks, > > > > greg k-h > > Are you suggesting to hard code vendor/device ID in driver instead? Or put the define in the ehci-pci.c file like other drivers do. Only add to the pci_ids.h file if your define is needed in multiple drivers, otherwise it is a merge nightmare. thanks, greg k-h ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] usb: ehci: add pci device support for Aspeed platforms 2022-02-08 8:41 ` Greg Kroah-Hartman @ 2022-02-08 8:53 ` Neal Liu 0 siblings, 0 replies; 5+ messages in thread From: Neal Liu @ 2022-02-08 8:53 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Alan Stern, Bjorn Helgaas, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, Tao Ren, BMC-SW > -----Original Message----- > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Sent: Tuesday, February 8, 2022 4:42 PM > To: Neal Liu <neal_liu@aspeedtech.com> > Cc: Alan Stern <stern@rowland.harvard.edu>; Bjorn Helgaas > <bhelgaas@google.com>; linux-usb@vger.kernel.org; > linux-kernel@vger.kernel.org; linux-pci@vger.kernel.org; Tao Ren > <rentao.bupt@gmail.com>; BMC-SW <BMC-SW@aspeedtech.com> > Subject: Re: [PATCH] usb: ehci: add pci device support for Aspeed platforms > > On Tue, Feb 08, 2022 at 08:20:07AM +0000, Neal Liu wrote: > > > -----Original Message----- > > > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > Sent: Tuesday, February 8, 2022 3:38 PM > > > To: Neal Liu <neal_liu@aspeedtech.com> > > > Cc: Alan Stern <stern@rowland.harvard.edu>; Bjorn Helgaas > > > <bhelgaas@google.com>; linux-usb@vger.kernel.org; > > > linux-kernel@vger.kernel.org; linux-pci@vger.kernel.org; Tao Ren > > > <rentao.bupt@gmail.com>; BMC-SW <BMC-SW@aspeedtech.com> > > > Subject: Re: [PATCH] usb: ehci: add pci device support for Aspeed > > > platforms > > > > > > On Tue, Feb 08, 2022 at 02:29:27PM +0800, Neal Liu wrote: > > > > Enable Aspeed quirks in commit 7f2d73788d90 ("usb: ehci: > > > > handshake CMD_RUN instead of STS_HALT") to support Aspeed ehci-pci > > > > device. > > > > > > > > Signed-off-by: Neal Liu <neal_liu@aspeedtech.com> > > > > --- > > > > drivers/usb/host/ehci-pci.c | 6 ++++++ > > > > include/linux/pci_ids.h | 3 +++ > > > > 2 files changed, 9 insertions(+) > > > > > > > > diff --git a/drivers/usb/host/ehci-pci.c > > > > b/drivers/usb/host/ehci-pci.c index e87cf3a00fa4..a91b25d216ae > > > > 100644 > > > > --- a/drivers/usb/host/ehci-pci.c > > > > +++ b/drivers/usb/host/ehci-pci.c > > > > @@ -222,6 +222,12 @@ static int ehci_pci_setup(struct usb_hcd *hcd) > > > > ehci->has_synopsys_hc_bug = 1; > > > > } > > > > break; > > > > + case PCI_VENDOR_ID_ASPEED: > > > > + if (pdev->device == PCI_DEVICE_ID_ASPEED_EHCI) { > > > > + ehci_info(ehci, "applying Aspeed HC workaround\n"); > > > > + ehci->is_aspeed = 1; > > > > + } > > > > + break; > > > > } > > > > > > > > /* optional debug port, normally in the first BAR */ diff --git > > > > a/include/linux/pci_ids.h b/include/linux/pci_ids.h index > > > > aad54c666407..410b395fe56c 100644 > > > > --- a/include/linux/pci_ids.h > > > > +++ b/include/linux/pci_ids.h > > > > @@ -3096,4 +3096,7 @@ > > > > > > > > #define PCI_VENDOR_ID_NCUBE 0x10ff > > > > > > > > +#define PCI_VENDOR_ID_ASPEED 0x1a03 > > > > +#define PCI_DEVICE_ID_ASPEED_EHCI 0x2603 > > > > > > Please read the top of this file, this does not need to be added here. > > > > > > thanks, > > > > > > greg k-h > > > > Are you suggesting to hard code vendor/device ID in driver instead? > > Or put the define in the ehci-pci.c file like other drivers do. Only add to the > pci_ids.h file if your define is needed in multiple drivers, otherwise it is a > merge nightmare. > > thanks, > > greg k-h Okay, I'll revise it. Thanks for your suggestion. -Neal ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-02-08 8:53 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-02-08 6:29 [PATCH] usb: ehci: add pci device support for Aspeed platforms Neal Liu 2022-02-08 7:37 ` Greg Kroah-Hartman 2022-02-08 8:20 ` Neal Liu 2022-02-08 8:41 ` Greg Kroah-Hartman 2022-02-08 8:53 ` Neal Liu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox