Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
Hello. On 02/03/2015 07:08 PM, Andy Shevchenko wrote: This is a follow up to the previously submitted commit cefa9a31a5f0 (ehci-pci: disable for Intel MID platforms). It includes the following changes: - table and function are renamed to reflect this is not only about ChipIdea - ChipIdea PCI driver (ci_hdrc_pci.c) gets the comment about the table in ehci-pci.c - MIPS IDs removed from the list since it was discovered and tested on Intel MID platforms Reviewed-by: Alexander Shishkin Signed-off-by: Andy Shevchenko [...] diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c index 9652021..2a5d2fd 100644 --- a/drivers/usb/host/ehci-pci.c +++ b/drivers/usb/host/ehci-pci.c @@ -42,18 +42,22 @@ static inline bool is_intel_quark_x1000(struct pci_dev *pdev) pdev->device == PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC; } -static const struct pci_device_id ci_hdrc_pci_id_table[] = { - { PCI_DEVICE(0x153F, 0x1004), }, - { PCI_DEVICE(0x153F, 0x1006), }, +/* + * This is the list of PCI IDs for the devices that have EHCI USB class and + * specific drivers for that. One of the example is a ChipIdea device installed + * on some Intel MID platforms. + */ +static const struct pci_device_id bypass_pci_id_table[] = { + /* ChipIdea on Intel MID platform */ { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0811), }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0829), }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0xe006), }, I'd suggest using PCI_VDEVICE() here instead. [...] WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
On Tue, Feb 03, 2015 at 11:15:34PM +0300, Sergei Shtylyov wrote: > Hello. > > On 02/03/2015 07:08 PM, Andy Shevchenko wrote: > > >This is a follow up to the previously submitted commit cefa9a31a5f0 > >(ehci-pci: > >disable for Intel MID platforms). > > >It includes the following changes: > >- table and function are renamed to reflect this is not only about ChipIdea > >- ChipIdea PCI driver (ci_hdrc_pci.c) gets the comment about the table in > > ehci-pci.c > >- MIPS IDs removed from the list since it was discovered and tested on Intel > > MID platforms > > >Reviewed-by: Alexander Shishkin > >Signed-off-by: Andy Shevchenko > > [...] > >diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c > >index 9652021..2a5d2fd 100644 > >--- a/drivers/usb/host/ehci-pci.c > >+++ b/drivers/usb/host/ehci-pci.c > >@@ -42,18 +42,22 @@ static inline bool is_intel_quark_x1000(struct pci_dev > >*pdev) > > pdev->device == PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC; > > } > > > >-static const struct pci_device_id ci_hdrc_pci_id_table[] = { > >-{ PCI_DEVICE(0x153F, 0x1004), }, > >-{ PCI_DEVICE(0x153F, 0x1006), }, > >+/* > >+ * This is the list of PCI IDs for the devices that have EHCI USB class and > >+ * specific drivers for that. One of the example is a ChipIdea device > >installed > >+ * on some Intel MID platforms. > >+ */ > >+static const struct pci_device_id bypass_pci_id_table[] = { > >+/* ChipIdea on Intel MID platform */ > > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0811), }, > > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0829), }, > > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0xe006), }, > >I'd suggest using PCI_VDEVICE() here instead. Not a big deal. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
On Tue, Feb 03, 2015 at 06:08:39PM +0200, Andy Shevchenko wrote: > This is a follow up to the previously submitted commit cefa9a31a5f0 (ehci-pci: > disable for Intel MID platforms). > > It includes the following changes: > - table and function are renamed to reflect this is not only about ChipIdea > - ChipIdea PCI driver (ci_hdrc_pci.c) gets the comment about the table in > ehci-pci.c > - MIPS IDs removed from the list since it was discovered and tested on Intel > MID platforms > > Reviewed-by: Alexander Shishkin > Signed-off-by: Andy Shevchenko > --- > drivers/usb/chipidea/ci_hdrc_pci.c | 3 +++ > drivers/usb/host/ehci-pci.c| 16 ++-- > 2 files changed, 13 insertions(+), 6 deletions(-) > > diff --git a/drivers/usb/chipidea/ci_hdrc_pci.c > b/drivers/usb/chipidea/ci_hdrc_pci.c > index 241ae34..4df6694 100644 > --- a/drivers/usb/chipidea/ci_hdrc_pci.c > +++ b/drivers/usb/chipidea/ci_hdrc_pci.c > @@ -111,6 +111,9 @@ static void ci_hdrc_pci_remove(struct pci_dev *pdev) > * PCI device structure > * > * Check "pci.h" for details > + * > + * Note: ehci-pci driver may try to probe the device first. You have to add > an > + * ID to the bypass_pci_id_table in ehci-pci driver to prevent this. > */ This one is the same with your v3. > static const struct pci_device_id ci_hdrc_pci_id_table[] = { > { > diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c > index 9652021..2a5d2fd 100644 > --- a/drivers/usb/host/ehci-pci.c > +++ b/drivers/usb/host/ehci-pci.c > @@ -42,18 +42,22 @@ static inline bool is_intel_quark_x1000(struct pci_dev > *pdev) > pdev->device == PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC; > } > > -static const struct pci_device_id ci_hdrc_pci_id_table[] = { > - { PCI_DEVICE(0x153F, 0x1004), }, > - { PCI_DEVICE(0x153F, 0x1006), }, > +/* > + * This is the list of PCI IDs for the devices that have EHCI USB class and > + * specific drivers for that. One of the example is a ChipIdea device > installed > + * on some Intel MID platforms. > + */ > +static const struct pci_device_id bypass_pci_id_table[] = { > + /* ChipIdea on Intel MID platform */ > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0811), }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0829), }, > { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0xe006), }, > {} > }; > > -static inline bool is_ci_hdrc_pci(struct pci_dev *pdev) > +static inline bool is_bypassed_id(struct pci_dev *pdev) > { > - return !!pci_match_id(ci_hdrc_pci_id_table, pdev); > + return !!pci_match_id(bypass_pci_id_table, pdev); > } > > /* > @@ -368,7 +372,7 @@ static const struct ehci_driver_overrides pci_overrides > __initconst = { > > static int ehci_pci_probe(struct pci_dev *pdev, const struct pci_device_id > *id) > { > - if (is_ci_hdrc_pci(pdev)) > + if (is_bypassed_id(pdev)) > return -ENODEV; > return usb_hcd_pci_probe(pdev, id); > } > -- I have no idea for this change. If Greg has still not queued your pci patch, you can squash all your versions for pci and chipidea to one patch set, in this set, one patch for pci, and another for chipidea. -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
On Wed, 2015-02-04 at 08:04 +0800, Peter Chen wrote: > On Tue, Feb 03, 2015 at 06:08:39PM +0200, Andy Shevchenko wrote: > > + * > > + * Note: ehci-pci driver may try to probe the device first. You have to > > add an > > + * ID to the bypass_pci_id_table in ehci-pci driver to prevent this. > > */ > > This one is the same with your v3. Correct. > > -- > > I have no idea for this change. > > If Greg has still not queued your pci patch, you can squash all your > versions for pci and chipidea to one patch set, in this set, one patch > for pci, and another for chipidea. We discussed with him and agreed on one patch to update what was applied to what is in v3. -- Andy Shevchenko Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
Hello. On 02/04/2015 02:24 AM, Greg Kroah-Hartman wrote: This is a follow up to the previously submitted commit cefa9a31a5f0 (ehci-pci: disable for Intel MID platforms). It includes the following changes: - table and function are renamed to reflect this is not only about ChipIdea - ChipIdea PCI driver (ci_hdrc_pci.c) gets the comment about the table in ehci-pci.c - MIPS IDs removed from the list since it was discovered and tested on Intel MID platforms Reviewed-by: Alexander Shishkin Signed-off-by: Andy Shevchenko [...] diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c index 9652021..2a5d2fd 100644 --- a/drivers/usb/host/ehci-pci.c +++ b/drivers/usb/host/ehci-pci.c @@ -42,18 +42,22 @@ static inline bool is_intel_quark_x1000(struct pci_dev *pdev) pdev->device == PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC; } -static const struct pci_device_id ci_hdrc_pci_id_table[] = { - { PCI_DEVICE(0x153F, 0x1004), }, - { PCI_DEVICE(0x153F, 0x1006), }, +/* + * This is the list of PCI IDs for the devices that have EHCI USB class and + * specific drivers for that. One of the example is a ChipIdea device installed + * on some Intel MID platforms. + */ +static const struct pci_device_id bypass_pci_id_table[] = { + /* ChipIdea on Intel MID platform */ { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0811), }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x0829), }, { PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0xe006), }, I'd suggest using PCI_VDEVICE() here instead. Not a big deal. Just an idea for a future patch. I might venture into doing such one in my copious free time... :-) WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
> > On Wed, 2015-02-04 at 08:04 +0800, Peter Chen wrote: > > On Tue, Feb 03, 2015 at 06:08:39PM +0200, Andy Shevchenko wrote: > > > > + * > > > + * Note: ehci-pci driver may try to probe the device first. You > > > + have to add an > > > + * ID to the bypass_pci_id_table in ehci-pci driver to prevent this. > > > */ > > > > This one is the same with your v3. > > Correct. > > > > -- > > > > I have no idea for this change. > > > > If Greg has still not queued your pci patch, you can squash all your > > versions for pci and chipidea to one patch set, in this set, one patch > > for pci, and another for chipidea. > > We discussed with him and agreed on one patch to update what was applied to > what is in v3. > So no patch I need to queue in chipidea tree, right? For this patch: Acked-by: Peter Chen Peter
Re: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
On Thu, 2015-02-05 at 01:21 +, Peter Chen wrote: > So no patch I need to queue in chipidea tree, right? In this series no, but 'chipidea: pci: register nop PHY' from the other patch if not queued yet. Thanks! > > For this patch: > > Acked-by: Peter Chen > > Peter > -- Andy Shevchenko Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v4] ehci-pci: disable for Intel MID platforms (update)
> > In this series no, but 'chipidea: pci: register nop PHY' from the other patch > if not > queued yet. Thanks! > > > It is already at my next tree, will send Greg it soon. Peter