Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Felipe Balbi
Hi, On Thu, Jul 25, 2013 at 10:28:37AM -0500, Bin Liu wrote: > Sebastian, > > On Thu, Jul 25, 2013 at 10:16 AM, Sebastian Andrzej Siewior > wrote: > > On 07/25/2013 05:12 PM, Bin Liu wrote: > >> Sebastian, > > > > Hi Bin, > > > >>> Is it really there or was it never there and it has been added t

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Felipe Balbi
Hi, On Thu, Jul 25, 2013 at 05:16:53PM +0200, Sebastian Andrzej Siewior wrote: > On 07/25/2013 05:12 PM, Bin Liu wrote: > > Sebastian, > > Hi Bin, > > >> Is it really there or was it never there and it has been added to TRM by > >> accident? > > The EOI register IS in the USB subsystem of AM33xx

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Bin Liu
Sebastian, On Thu, Jul 25, 2013 at 10:16 AM, Sebastian Andrzej Siewior wrote: > On 07/25/2013 05:12 PM, Bin Liu wrote: >> Sebastian, > > Hi Bin, > >>> Is it really there or was it never there and it has been added to TRM by >>> accident? >> The EOI register IS in the USB subsystem of AM33xx, but

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Sebastian Andrzej Siewior
On 07/25/2013 05:12 PM, Bin Liu wrote: > Sebastian, Hi Bin, >> Is it really there or was it never there and it has been added to TRM by >> accident? > The EOI register IS in the USB subsystem of AM33xx, but the SoC does > not use it because it uses level triggering for USB interrupt. I see. >>>

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Bin Liu
Sebastian, On Thu, Jul 25, 2013 at 9:41 AM, Sebastian Andrzej Siewior wrote: > * Bin Liu | 2013-07-23 13:23:57 [-0500]: > >>Hi Sebastian, > Hi Liu, > >>On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior < >>bige...@linutronix.de> wrote: >> >>> This patch renames the type struct from ti81x

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Sebastian Andrzej Siewior
* Bin Liu | 2013-07-23 13:23:57 [-0500]: >Hi Sebastian, Hi Liu, >On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior < >bige...@linutronix.de> wrote: > >> This patch renames the type struct from ti81xx_driver_data to >> am33xx_driver_data since it is not used for ti81xx anymore. The EOI >>

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-23 Thread Bin Liu
(re-send, due to last delivery failure to the mailing list...) On Tue, Jul 23, 2013 at 1:23 PM, Bin Liu wrote: > Hi Sebastian, > > On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior > wrote: >> >> This patch renames the type struct from ti81xx_driver_data to >> am33xx_driver_data since i

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-23 Thread George Cherian
On 7/23/2013 2:39 PM, Sebastian Andrzej Siewior wrote: On 07/23/2013 08:04 AM, George Cherian wrote: Hi Sebastian, On 7/22/2013 11:39 PM, Sebastian Andrzej Siewior wrote: This patch renames the type struct from ti81xx_driver_data to am33xx_driver_data since it is not used for ti81xx anymo

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-23 Thread Sebastian Andrzej Siewior
On 07/23/2013 08:04 AM, George Cherian wrote: > Hi Sebastian, > > > On 7/22/2013 11:39 PM, Sebastian Andrzej Siewior wrote: > >> This patch renames the type struct from ti81xx_driver_data to >> am33xx_driver_data since it is not used for ti81xx anymore. The EOI >> member is also removed since t

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-22 Thread George Cherian
Hi Sebastian, On 7/22/2013 11:39 PM, Sebastian Andrzej Siewior wrote: This patch renames the type struct from ti81xx_driver_data to am33xx_driver_data since it is not used for ti81xx anymore. The EOI member is also removed since the am33xx SoC does not have such register. The interrupt is ac