[U-Boot] [PATCH 2/3] usb: gadget: at91_udc: add support for at91_udc

Heiko Schocher hs at denx.de
Mon Aug 10 07:16:02 CEST 2015


Hello Luaksz,

Am 21.07.2015 um 08:59 schrieb Lukasz Majewski:
> Hi Heiko,
>
>> ported from linux:
>>
>> b2ba27a5c56ff: usb: gadget: at91_udc: move prepare clk into process
>> context
>>
>
> IMHO, presented above description is not enough. I'd prefer to see one
> patch which adds the code from linux - including the exact commit
> message.

Ok, I change this.

> Then, u-boot specific adjustments should be applied in a separate patch.
>
> In that way you would:
> 1. Avoid #ifdef __UBOOT__

Hmm.. I change this, but I do not hate this define, if I debug in the
Code I immediately see, if it is linux code, or U-Boot specific ...
we had such a discussion also on other places ... maybe we need
here a decision, if we remove this define complete?

> 2. Make the process of updating code easier

I soon post a v2, just did a rebase and found some compile errors
for the smartweb board ...

bye,
Heiko
>
>
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>> ---
>> checkpatch detects a lot of errors, but as this code is copied
>> from linux, I tend to not fix them, so later updates with
>> linux code is easier.
>>
>>   drivers/usb/gadget/Makefile   |    1 +
>>   drivers/usb/gadget/at91_udc.c | 2203
>> +++++++++++++++++++++++++++++++++++++++++
>> drivers/usb/gadget/at91_udc.h |  171 ++++
>> include/linux/usb/at91_udc.h  |   20 + 4 files changed, 2395
>> insertions(+) create mode 100644 drivers/usb/gadget/at91_udc.c
>>   create mode 100644 drivers/usb/gadget/at91_udc.h
>>   create mode 100644 include/linux/usb/at91_udc.h
>>
>> diff --git a/drivers/usb/gadget/Makefile b/drivers/usb/gadget/Makefile
>> index 70bb550..22323fa 100644
>> --- a/drivers/usb/gadget/Makefile
>> +++ b/drivers/usb/gadget/Makefile
>> @@ -10,6 +10,7 @@ obj-$(CONFIG_USB_ETHER) += epautoconf.o config.o
>> usbstring.o
>>   # new USB gadget layer dependencies
>>   ifdef CONFIG_USB_GADGET
>> +obj-$(CONFIG_USB_GADGET_AT91) += at91_udc.o
>>   obj-$(CONFIG_USB_GADGET_ATMEL_USBA) += atmel_usba_udc.o
>>   obj-$(CONFIG_USB_GADGET_S3C_UDC_OTG) += s3c_udc_otg.o
>>   obj-$(CONFIG_USB_GADGET_S3C_UDC_OTG_PHY) += s3c_udc_otg_phy.o
>> diff --git a/drivers/usb/gadget/at91_udc.c
>> b/drivers/usb/gadget/at91_udc.c new file mode 100644
>> index 0000000..f4ae13b
>> --- /dev/null
>> +++ b/drivers/usb/gadget/at91_udc.c
>> @@
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list