Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-30 Thread Pavel Machek
Hi! + case (USB_REQ_GET_DESCRIPTOR 8) | USB_DIR_IN | USB_TYPE_CLASS: + { + __u32 temp = 0x0001; temp is never writtenn to. Can you just write 1 instead of it? I added a comment. This is the root port config, so it should be tweakable in case there is some mutated

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-26 Thread Marek Vasut
On Tuesday, September 23, 2014 at 11:59:28 PM, Dinh Nguyen wrote: [...] +static int wait_for_bit(void *reg, const uint32_t mask, bool set) +{ + unsigned int timeout = 100; + uint32_t val; + + while (--timeout) { + val = readl(reg); + if (!set) +

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-26 Thread Dinh Nguyen
On 09/26/2014 02:29 AM, Marek Vasut wrote: On Tuesday, September 23, 2014 at 11:59:28 PM, Dinh Nguyen wrote: btw. please try to trim down the content of the patch when replying only to the relevant part, so others don't have to look up the relevant bits among billions of lines of

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-26 Thread Marek Vasut
On Friday, September 26, 2014 at 05:01:22 PM, Dinh Nguyen wrote: [...] +#define DWC2_GUSBCFG_TX_END_DELAY(1 28) +#define DWC2_GUSBCFG_TX_END_DELAY_OFFSET 28 bits 29 and 30 of GUSBCFG are to Force Host and Device mode, respectively. These bits may get

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-24 Thread Marek Vasut
On Wednesday, September 24, 2014 at 05:31:26 AM, Stephen Warren wrote: On 09/21/2014 07:13 AM, Marek Vasut wrote: From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-24 Thread Stephen Warren
On 09/23/2014 09:31 PM, Stephen Warren wrote: On 09/21/2014 07:13 AM, Marek Vasut wrote: From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to make sure it stays at least

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-24 Thread Marek Vasut
On Wednesday, September 24, 2014 at 05:37:16 PM, Stephen Warren wrote: On 09/23/2014 09:31 PM, Stephen Warren wrote: On 09/21/2014 07:13 AM, Marek Vasut wrote: From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi.

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-23 Thread Dinh Nguyen
On Sun, 21 Sep 2014, Marek Vasut wrote: From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to make sure it stays at least readable and clear, these are not fixed. These bugs

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-23 Thread Stephen Warren
On 09/21/2014 07:13 AM, Marek Vasut wrote: From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to make sure it stays at least readable and clear, these are not fixed. These

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-22 Thread Pavel Machek
Hi! This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to make sure it stays at least readable and clear, these are not fixed. These bugs are in the USB request handling combinatorial logic, so any abstracting of those is

Re: [U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-22 Thread Marek Vasut
On Monday, September 22, 2014 at 11:40:36 AM, Pavel Machek wrote: [...] index c4f5157..c9d2ed5 100644 --- a/drivers/usb/host/Makefile +++ b/drivers/usb/host/Makefile @@ -45,3 +45,6 @@ obj-$(CONFIG_USB_EHCI_ZYNQ) += ehci-zynq.o obj-$(CONFIG_USB_XHCI) += xhci.o xhci-mem.o xhci-ring.o

[U-Boot] [PATCH 1/3] usb: dwc2: Add driver for Synopsis DWC2 USB IP block

2014-09-21 Thread Marek Vasut
From: Oleksandr Tymoshenko go...@bluezbox.com This is the USB host controller used on the Altera SoCFPGA and Raspbery Pi. This code has three checkpatch warnings, but to make sure it stays at least readable and clear, these are not fixed. These bugs are in the USB request handling combinatorial