[U-Boot] [PATCH 0/5] mx5: USB host support for mx51evk and mx53loco

[Wolfgang Grandegger]

On 11/14/2011 11:46 AM, Marek Vasut wrote:
>> On 11/11/2011 10:44 PM, Marek Vasut wrote:
>>>> This patches series provides USB host support for the mx51evk and
>>>> mx53loco. The generic ehci-mx5 interface is derived from ehci-mxc
>>>> and uses the header files of the ehci-fsl interface. The callback
>>>> board_ehci_hcd_init() allows to do board-specific setup when USB is
>>>> started.
>>>>
>>>> Wolfgang.
>>>>
>>>> Wolfgang Grandegger (5):
>>>>   mx5: add helper functions to enable USB clocks
>>>>   mx5: add helper function to retrieve the GPIO number
>>>>   mx5: add generic USB EHCI support for mx51 and mx53
>>>>   mx53loco: add end enable USB host support on port 1
>>>>   mx51evk: add end enable USB host support on port 1
>>>>  
>>>>  arch/arm/cpu/armv7/mx5/clock.c            |   72 ++++++++++++
>>>>  arch/arm/include/asm/arch-mx5/clock.h     |    5 +
>>>>  arch/arm/include/asm/arch-mx5/crm_regs.h  |    3 +
>>>>  arch/arm/include/asm/arch-mx5/mx5x_pins.h |    4 +
>>>>  board/freescale/mx51evk/mx51evk.c         |   92 +++++++++++++++
>>>>  board/freescale/mx53loco/mx53loco.c       |   10 ++
>>>>  drivers/usb/host/Makefile                 |    1 +
>>>>  drivers/usb/host/ehci-mx5.c               |  174
>>>>
>>>> +++++++++++++++++++++++++++++ include/configs/mx51evk.h                
>>>> |
>>>>
>>>>  13 ++
>>>>  include/configs/mx53loco.h                |   13 ++
>>>>  include/usb/ehci-fsl.h                    |   10 ++
>>>>  11 files changed, 397 insertions(+), 0 deletions(-)
>>>>  create mode 100644 drivers/usb/host/ehci-mx5.c
>>>
>>> Hi,
>>>
>>> isn't Jana already doing this kind of USB support for mx5 based on my old
>>> patches? You didn't notice?
>>
>> Well, I already responded on Jana's patches and presented my ideas here:
>>
>>   http://marc.info/?l=u-boot&m=132067806107887&w=2
>>
>> Unfortunately, only the mails with subject "[PATCH] ..." get enough
>> attention ;-).
> 
> Well aparently she isn't even signed to the ML ... so you need to CC her ... doh 
> :-/

What do you mean? I replied to *all*!

>> The big difference to Jana's patch is that he uses a board-specific
>> implementation (ehci_hcd_init() is in board specific code). I think we
>> should share some code.
> 
> Please do.

See my patches ;-). Well, maybe it's more straight forward if we try to
get my patches for the MX51/53 boards accepted first and then Jana could
then use the negotiated interface. But also vise versa is fine for me.

Wolfgang.