Re: [PATCH v3 01/13] USB: s3c-hsotg: Add platform data callbacks for phy control

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Felipe,


> On Fri, Mar 23, 2012 at 12:48:47PM +0100, Lukasz Majewski wrote:
> > From: Joonyoung Shim <jy0922.shim@xxxxxxxxxxx>
> > 
> > The s3c-hsotg driver controls S3C64XX specific registers directly
> > but this driver can be to EXYNOS also. This removes arch specific
> > parts from driver using platform data callbacks.
> > 
> > Signed-off-by: Joonyoung Shim <jy0922.shim@xxxxxxxxxxx>
> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> > ---
> >  drivers/usb/gadget/s3c-hsotg.c |   94
> > ++++++---------------------------------- 1 files changed, 14
> > insertions(+), 80 deletions(-)
> > 
> > diff --git a/drivers/usb/gadget/s3c-hsotg.c
> > b/drivers/usb/gadget/s3c-hsotg.c index 105b206..0e8e2f4 100644
> > --- a/drivers/usb/gadget/s3c-hsotg.c
> > +++ b/drivers/usb/gadget/s3c-hsotg.c
> > @@ -35,9 +35,9 @@
> >  
> >  #include <plat/regs-usb-hsotg-phy.h>
> >  #include <plat/regs-usb-hsotg.h>
> > -#include <mach/regs-sys.h>
> >  #include <plat/udc-hs.h>
> >  #include <plat/cpu.h>
> > +#include <plat/usb-phy.h>
> 
> please put platform_data under <linux/platform_data/>

But this code is phy related and used by other Samsung's devices (e.g.
exynos4, host controllers, etc.).
 
Following files are using it:

./arch/arm/mach-exynos/setup-usb-phy.c:20:#include <plat/usb-phy.h>
./arch/arm/mach-exynos/dev-ohci.c:21:#include <plat/usb-phy.h>
./arch/arm/mach-s5pv210/setup-usb-phy.c:19:#include <plat/usb-phy.h>
./arch/arm/mach-s5pv210/setup-usb-phy.c~:20:#include <plat/usb-phy.h>
./arch/arm/mach-s3c64xx/setup-usb-phy.c:21:#include <plat/usb-phy.h>
./arch/arm/plat-samsung/devs.c:62:#include <plat/usb-phy.h>
./drivers/usb/gadget/s3c-hsotg.c.orig:41:#include <plat/usb-phy.h>
./drivers/usb/gadget/s3c-hsotg.c:42:#include <plat/usb-phy.h>
./drivers/usb/host/ehci-s5p.c:18:#include <plat/usb-phy.h>
./drivers/usb/host/ohci-exynos.c:17:#include <plat/usb-phy.h>


> 
> >  #define DMA_ADDR_INVALID (~((dma_addr_t)0))
> 
> you should move to the generic map/unmap routines and get rid of this
> define.
> 

The DMA mode is still not complete (if it was ever), so I'd like to
leave it as it is. Moreover this driver uses PIO instead of DMA, so
for this fixes patch set it would be better to leave the DMA alone. 

Will you prepare the USB for-next branch, so I could resend the
s3c-hsotg patches?

-- 
Best regards,

Lukasz Majewski

Samsung Poland R&D Center
Platform Group
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux