[U-Boot] [PATCH 2/3] armv8/fsl-layerscape: add dwc3 gadget driver support

[Felipe Balbi <felipe.balbi@linux.intel.com>]

Hi,

Rajesh Bhagat <rajesh.bhagat@nxp.com> writes:
>> Marek Vasut <marex@denx.de> writes:
>> >> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index
>> >> 85cc96a..5eeb71d 100644
>> >> --- a/drivers/usb/dwc3/core.c
>> >> +++ b/drivers/usb/dwc3/core.c
>> >> @@ -690,6 +690,18 @@ int dwc3_uboot_init(struct dwc3_device *dwc3_dev)
>> >>  		return -ENOMEM;
>> >>  	}
>> >>
>> >> +#if defined(CONFIG_LS1043A)
>> >
>> > How was this added in Linux ? I doubt such horrid ifdef was allowed.
>> > I presume it went through either DT or struct dwc3_device, right ?
>> > So fix this here the same way.
>> 
>> dwc3 doesn't have any of that. I wouldn't take it :-p
>> 
>
> Hello Felipe/Marek,
>
> It is not currently added in Linux, These registers setting is required for 
> Setting the burst beat transactions and enabling the snooping for DWC3 USB IP. 

that's fine, but you don't need to ifdef around that. Pass these values
from your platform code.

> Can it be added using dwc3 uboot glue layer e.g. dwc3-fsl.c ?

keep in core, but pass them from plat code instead of hardcoding under ifdef.

Also, get rid of the magic constant ;-)

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160601/53441d1c/attachment.sig>