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]

 



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

>  #define DMA_ADDR_INVALID (~((dma_addr_t)0))

you should move to the generic map/unmap routines and get rid of this
define.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


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

  Powered by Linux