[U-Boot] [PATCH 3/5] usb: s3c-otg: Split out PHY control

[Lukasz Majewski]

Hi Marek,

> On Thursday, November 06, 2014 at 10:29:03 PM, Lukasz Majewski wrote:
> > On Thu, 6 Nov 2014 21:23:46 +0100
> 
> Hi!
> 
> > Marek Vasut <marex at denx.de> wrote:
> > > On Thursday, November 06, 2014 at 10:59:17 AM, Lukasz Majewski
> > > wrote:
> > > > Hi Marek,
> > > > 
> > > > > Split the Samsung specific PHY control into a separate file
> > > > > and compile this into the S3C OTG driver only if used on a
> > > > > Samsung system.
> > > > > 
> > > > > 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>
> > > > > Cc: Pavel Machek <pavel at denx.de>
> > > > > Cc: Stefan Roese <sr at denx.de>
> > > > > Cc: Lukasz Majewski <l.majewski at samsung.com>
> > > 
> > > [...]
> > > 
> > > > Build HW: trats2 (Exynos4412)
> > > > 
> > > > CROSS_COMPILE=/home/lukma/work/ptxdist/toolchains/arm/OSELAS.Toolchain-
> > > > 2013
> > > > .12.0/arm-v7a-linux-gnueabi/gcc-4.8.2-glibc-2.18-binutils-2.24-kernel-
> > > > 3.12- sanitized/bin/arm-v7a-linux-gnueabi-
> > > > 
> > > > Patches applied on top of v2014.10.
> > > 
> > > Try this trivial patch (add missing include):
> > > 
> > > diff --git a/drivers/usb/gadget/s3c_udc_otg_phy.c
> > > b/drivers/usb/gadget/s3c_udc_otg_phy.c
> > > index 055fe86..95bd8cf 100644
> > > --- a/drivers/usb/gadget/s3c_udc_otg_phy.c
> > > +++ b/drivers/usb/gadget/s3c_udc_otg_phy.c
> > > @@ -35,6 +35,8 @@
> > > 
> > >  #include "regs-otg.h"
> > >  #include <usb/lin_gadget_compat.h>
> > > 
> > > +#include <usb/s3c_udc.h>
> > > +
> > 
> > Ok, I will apply it and let you know about the result.
> > 
> > BTW: I assume that your work is to reuse the USB 2.0 UDC IP block
> > (probably in Altera).
> 
> Yep, I got the UDC working on my SoCFPGA board. We should probably
> rename the s3c-otg driver to something more fitting (dwc2-udc ?)
> and strive to somehow merge the dwc2-host and dwc2-udc drivers in
> the long run.
> 
> Speaking of the rename, I'd like to leave that for the next MW, to
> make sure it doesn't break any tools which might for example depend
> on the name of the driver.

+1

IMHO, we should devote one MW only for rename (as I've written
previously).

> 
> > I'd be elated if this is true :-).
> 
> It is :) ... it must have been hard to tell from patch 5/5 ;-)
> 
> Best regards,
> Marek Vasut



-- 
Best regards,

Lukasz Majewski

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