[U-Boot] [PATCH 4/5] usb: s3c-otg: Allow custom gusbcfg

[Marek Vasut]

On Friday, November 07, 2014 at 03:33:10 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> > On Friday, November 07, 2014 at 10:11:35 AM, Lukasz Majewski wrote:
> > > Hi Marek,
> > > 
> > > > On Tuesday, November 04, 2014 at 08:36:57 PM, Pavel Machek wrote:
> > > > > On Tue 2014-11-04 06:07:33, Marek Vasut wrote:
> > > > > > Allow passing in a custom configuration of the gusbcfg
> > > > > > register via platform data.
> > > > > > 
> > > > > > Signed-off-by: Marek Vasut <marex at denx.de>
> > > > > > Cc: Chin Liang See <clsee at altera.com>
> > > > > > Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> > > > > > Cc: Vince Bridgers <vbridger at altera.com>
> > > > > 
> > > > > Acked-by: Pavel Machek <pavel at denx.de>
> > > > > 
> > > > > >  	debug("Reseting OTG controller\n");
> > > > > > 
> > > > > > -	writel(0<<15		/* PHY Low Power Clock
> > > > > > sel*/
> > > > > > +	dflt_gusbcfg =
> > > > > > +		0<<15		/* PHY Low Power Clock
> > > > > > sel*/
> > > > > > 
> > > > > >  		|1<<14		/* Non-Periodic TxFIFO
> > > > > > 
> > > > > > Rewind Enable*/ |0x5<<10	/* Turnaround time*/
> > > > > > 
> > > > > >  		|0<<9 | 0<<8	/* [0:HNP disable,1:HNP
> > > > > > 
> > > > > > enable][ 0:SRP disable*/
> > > > > > 
> > > > > > @@ -413,8 +415,12 @@ static void reconfig_usbd(void)
> > > > > > 
> > > > > >  		|0<<6		/* 0: high speed utmi+,
> > > > > > 
> > > > > > 1: full speed serial*/ |0<<4		/* 0: utmi+,
> > > > > > 1:ulpi*/
> > > > > > 
> > > > > >  		|1<<3		/* phy i/f  0:8bit,
> > > > > > 
> > > > > > 1:16bit*/
> > > > > > 
> > > > > > -		|0x7<<0,	/* HS/FS Timeout**/
> > > > > > -		&reg->gusbcfg);
> > > > > > +		|0x7<<0;	/* HS/FS Timeout**/
> > > > > > +
> > > > > 
> > > > > "Interesting" comment style. And I'd remove 0x prefixes from
> > > > > constants <8...
> > > > 
> > > > Lukasz, what do you say to just convert those four boards to use
> > > > the gusbcfg platform data afterward ?
> > > 
> > > +1
> > > 
> > > It is a very good idea since we are using the same dwc2 IP block for
> > > host (./usb/host/dwc2.c) and device (s3c_hsotg).
> > 
> > Yep ... I'm just wondering if we should do that rename in the next MW.
> > If you're fine with this MW though, then I will do just that.
> > 
> > > > btw. you can find the latest version in u-boot-usb:topic/s3c-otg
> > > 
> > > I've read comments from Pavel and tested the code up till first
> > > build error :-).
> > 
> > I rewrote that branch, so feel free to use that as a base for your
> > testing. I will repost the corrected patches once I get feedback from
> > you.
> > 
> > Best regards,
> > Marek Vasut
> 
> Reviewed-by: Lukasz Majewski <l.majewski at samsung.com>
> 
> Tested-by: Lukasz Majewski <l.majewski at samsung.com>
> 
> Test HW: Trats2 (Exynos4412)

Queued for master, thanks!