[PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-24 Thread Chen, Alvin
From: Bryan O'Donoghue This patch is to enable USB host controller for Intel Quark X1000. Add pci quirks to adjust the packet buffer in/out threshold value, and ensure EHCI packet buffer i/o threshold value is reconfigured to half. Signed-off-by: Bryan O'Donoghue Signed-off-by: Alvin (Weike)

[PATCH] USB: ehci-pci: Add support for Intel Quark X1000 USB host controller

2014-06-24 Thread Chen, Alvin
From: "Alvin (Weike) Chen" Hi, Intel Quark X1000 consists of one USB host controller which can be PCI enumerated. But the exsiting EHCI-PCI framework doesn't support it. Thus, we enable it to support Intel Quark X1000 USB host controller by adding pci quirks to configure buffer i/o threshold

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
See my comments below: > > + /* Reset the threshold limit */ > > + if(unlikely(usb_is_intel_qrk(pdev))) > > Please run your patch thru scripts/checkpatch.pl -- space needed after > *if*. OK, I will improve it in the PATCH v2. > > > + usb_set_qrk_bulk_thresh(pdev); > > + > >

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
See my comments below: > -Original Message- > From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, June 24, 2014 9:25 PM > To: Chen, Alvin > Cc: Alan Stern; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; Ong, > Boon Leong > Subject:

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
> -Original Message- > From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, June 24, 2014 9:25 PM > To: Chen, Alvin > Cc: Alan Stern; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; Ong, > Boon Leong > Subject: Re: [PATCH] US

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
> > This patch is to enable USB host controller for Intel Quark X1000. Add > > pci quirks to adjust the packet buffer in/out threshold value, and > > ensure EHCI packet buffer i/o threshold value is reconfigured to half. > > Please add more detailed description. For example, why is it necessary to

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
> > > > OK, I will change ' usb_is_intel_qrk ' to ' usb_is_intel_quark'. > > Or even usb_is_intel_quark_x1000() ? > OK, I will change the function name as your suggestion to make it more specific. > David > > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the

RE: [PATCH] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-25 Thread Chen, Alvin
> > This patch is to enable USB host controller for Intel Quark X1000. Add >> pci quirks to adjust the packet buffer in/out threshold value, and > >ensure EHCI packet buffer i/o threshold value is reconfigured to half > > > What is the packet buffer in/out threshold value and why does it need to

[PATCH v2] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-26 Thread Chen, Alvin
From: Bryan O'Donoghue The EHCI packet buffer in/out threshold is programmable for Intel Quark X1000 USB host controller, and the default value is 0x20 dwords. The in/out threshold can be programmed to 0x80 dwords, but only when isochronous/interrupt transactions are not initiated by the USB h

[PATCH v2] USB: ehci-pci: Add support for Intel Quark X1000 USB host controller

2014-06-26 Thread Chen, Alvin
From: "Alvin (Weike) Chen" Hi, Intel Quark X1000 consists of one USB host controller which can be PCI enumerated. And the exsiting EHCI-PCI framework supports it with the default packet buffer in/out threshold. We reconfigure the in/out threshold as maximal as possible. Bryan O'Donoghue (1):

RE: [PATCH v2] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-27 Thread Chen, Alvin
> > > The EHCI packet buffer in/out threshold is programmable for Intel > > > Quark X1000 USB host controller, and the default value is 0x20 > > > dwords. The in/out threshold can be programmed to 0x80 dwords, but > > > only when isochronous/interrupt transactions are not initiated by > > > the US

RE: [PATCH v2] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-27 Thread Chen, Alvin
> -Original Message- > From: David Laight [mailto:david.lai...@aculab.com] > Sent: Friday, June 27, 2014 4:08 PM > ... > > /* The maximal threshold value is 0x80, which means 512 bytes */ > > #define EHCI_THRESHOLD_512BYTES 0x80 > > #define EHCI_THRESHOLD_508BYTES

RE: [PATCH v2] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-30 Thread Chen, Alvin
> > The EHCI packet buffer in/out threshold is programmable for Intel > > Quark X1000 USB host controller, and the default value is 0x20 dwords. > > The in/out threshold can be programmed to 0x80 dwords, but only when > > isochronous/interrupt transactions are not initiated by the USB host > > cont

[PATCH v3] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-30 Thread Chen, Alvin
From: Bryan O'Donoghue The EHCI packet buffer in/out threshold is programmable for Intel Quark X1000 USB host controller, and the default value is 0x20 dwords. The in/out threshold can be programmed to 0x80 dwords (512 Bytes) to maximize the perfomrance, but only when isochronous/interrupt transa

[PATCH v3] USB: ehci-pci: Add support for Intel Quark X1000 USB

2014-06-30 Thread Chen, Alvin
From: "Alvin (Weike) Chen" Hi, Intel Quark X1000 consists of one USB host controller which can be PCI enumerated. And the exsiting EHCI-PCI framework supports it with the default packet buffer in/out threshold. We reconfigure the in/out threshold as maximal as possible to maximize the performanc

RE: [PATCH v3] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-30 Thread Chen, Alvin
> > /* > > -*/ > > +#define PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC0x0939 > > +static inline bool is_intel_quark_x1000(struct pci_dev *pdev) { > > + return pdev->vendor == PCI_VENDOR_ID_INTEL && > > + pd

RE: [PATCH v3] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-06-30 Thread Chen, Alvin
> > > > /* > > -*/ > > +#define PCI_DEVICE_ID_INTEL_QUARK_X1000_SOC0x0939 > > +static inline bool is_intel_quark_x1000(struct pci_dev *pdev) { > > + return pdev->vendor == PCI_VENDOR_ID_INTEL && > > +

[PATCH v4] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-07-01 Thread Chen, Alvin
From: Bryan O'Donoghue The EHCI packet buffer in/out threshold is programmable for Intel Quark X1000 USB host controller, and the default value is 0x20 dwords. The in/out threshold can be programmed to 0x80 dwords (512 Bytes) to maximize the perfomrance, but only when isochronous/interrupt transa

[PATCH v4] USB: ehci-pci: Add support for Intel Quark X1000 USB

2014-07-01 Thread Chen, Alvin
From: "Alvin (Weike) Chen" Hi, Intel Quark X1000 consists of one USB host controller which can be PCI enumerated. And the exsiting EHCI-PCI framework supports it with the default packet buffer in/out threshold. We reconfigure the in/out threshold as maximal as possible to maximize the performanc

[PATCH v5] USB: ehci-pci: USB host controller support for Intel Quark X1000

2014-07-01 Thread Chen, Alvin
From: Bryan O'Donoghue The EHCI packet buffer in/out threshold is programmable for Intel Quark X1000 USB host controller, and the default value is 0x20 dwords. The in/out threshold can be programmed to 0x80 dwords (512 Bytes) to maximize the perfomrance, but only when isochronous/interrupt transa

[PATCH] USB: pch_udc: USB gadget device support for Intel Quark X1000

2014-08-04 Thread Chen, Alvin
From: Bryan O'Donoghue This patch is to enable the USB gadget device for Intel Quark X1000 Signed-off-by: Bryan O'Donoghue Signed-off-by: Bing Niu Signed-off-by: Alvin (Weike) Chen --- drivers/usb/gadget/udc/Kconfig |3 ++- drivers/usb/gadget/udc/pch_udc.c | 22 +++---

[PATCH] USB: pch_udc: Add support for Intel Quark X1000 USB gadget device

2014-08-04 Thread Chen, Alvin
From: "Alvin (Weike) Chen" Hi, Intel Quark X1000 consists of one USB gadget device which can be PCI enumerated. pch_udc layer doesn't support it. Thus, we add support for Intel Quark X1000 USB gadget device as well. Bryan O'Donoghue (1): USB: pch_udc: USB gadget device support for Intel Quar

Re: [PATCH] USB: pch_udc: USB gadget device support for Intel Quark X1000

2014-08-10 Thread Chen, Alvin
Hi Felipe, Any update for this patch? Just want to follow-up. > -Original Message- > From: Chen, Alvin > Sent: Tuesday, August 05, 2014 1:23 AM > To: Felipe Balbi > Cc: linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; Ong, Boon > Leong > Subject: [PAT

RE: [PATCH] USB: pch_udc: USB gadget device support for Intel Quark X1000

2014-08-19 Thread Chen, Alvin
> Hi, > > On Mon, Aug 04, 2014 at 10:22:54AM -0700, Chen, Alvin wrote: > > From: Bryan O'Donoghue > > > > This patch is to enable the USB gadget device for Intel Quark X1000 > > > > Signed-off-by: Bryan O'Donoghue > > Signed-off-by: Bi