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

Lukasz Majewski
2014-11-07T10:22:26Z

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@denx.de>
> > > > > Cc: Chin Liang See <clsee@altera.com>
> > > > > Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
> > > > > Cc: Vince Bridgers <vbridger@altera.com>
> > > >
> > > > Acked-by: Pavel Machek <pavel@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.

I'd prefer to spent one MW only for s3c-hsotg rename. In other words we
do the "fixes" and "clean ups" now and only when the code gets stable
we perform the rename (at the MW).

Does it work for you?

>
> > > 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.

Ok.

>
> Best regards,
> Marek Vasut

--
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group