[PATCH V8 00/10] Add-Synopsys-DesignWare-HS-USB-OTG-driver

[Tirumala Marri]

-----Original Message-----
From: Greg KH [mailto:greg at kroah.com]
Sent: Saturday, January 22, 2011 7:02 PM
To: tmarri at apm.com
Cc: linux-usb at vger.kernel.org; linuxppc-dev at lists.ozlabs.org
Subject: Re: [PATCH V8 00/10] Add-Synopsys-DesignWare-HS-USB-OTG-driver

On Wed, Jan 19, 2011 at 02:57:16PM -0800, tmarri at apm.com wrote:
> From: Tirumala Marri <tmarri at apm.com>
>
> v8:
>  1. Add set_wedge to usb_ep_ops.
>
> v7:
>  1. Fix sparse tool warnings.
>  2. Fix checkpatch errors and warnings.
>  3. Rename USB_OTG config variable to USB_DWC_CONFIG
>
> Tirumala Marri (10):
>   USB/ppc4xx: Add Synopsys DWC OTG Register definitions
>   USB/ppc4xx: Add Synopsys DWC OTG driver framework
>   USB/ppc4xx: Add Synopsys DWC OTG Core Interface Layer
>   USB/ppc4xx: Add Synopsys DWC OTG HCD function
>   USB/ppc4xx: Add Synopsys DWC OTG HCD interrupt function
>   USB/ppc4xx: Add Synopsys DWC OTG HCD queue function
>   USB/ppc4xx: Add Synopsys DWC OTG PCD function
>   USB ppc4xx: Add Synopsys DWC OTG PCD interrupt function
>   USB/ppc4xx:Synopsys DWC OTG driver enable gadget support
>   USB ppc4xx: Add Synopsys DWC OTG driver kernel configuration and
>     Makefile
>
>  drivers/Makefile                        |    2 +
>  drivers/usb/Kconfig                     |    3 +-
>  drivers/usb/dwc_otg/Kconfig             |   96 ++

Why are you creating a new subdirectory here?  Shouldn't it be in
drivers/usb/otg/dwc/ instead?

>  drivers/usb/dwc_otg/Makefile            |   19 +
>  drivers/usb/dwc_otg/dwc_otg_apmppc.c    |  414 ++++++
>  drivers/usb/dwc_otg/dwc_otg_cil.c       |  972 ++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_cil.h       | 1220 +++++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_cil_intr.c  |  616 ++++++++
>  drivers/usb/dwc_otg/dwc_otg_driver.h    |   76 +
>  drivers/usb/dwc_otg/dwc_otg_hcd.c       | 2466
+++++++++++++++++++++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_hcd.h       |  416 ++++++
>  drivers/usb/dwc_otg/dwc_otg_hcd_intr.c  | 1477 ++++++++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_hcd_queue.c |  696 +++++++++
>  drivers/usb/dwc_otg/dwc_otg_param.c     |  180 +++
>  drivers/usb/dwc_otg/dwc_otg_pcd.c       | 1766 ++++++++++++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_pcd.h       |  139 ++
>  drivers/usb/dwc_otg/dwc_otg_pcd_intr.c  | 2311
+++++++++++++++++++++++++++++
>  drivers/usb/dwc_otg/dwc_otg_regs.h      | 1325 +++++++++++++++++

As you are in your own subdir, no need to put "dwc_otg_" at the start of
every file name, right?

Care to make these changes and resend?

[Marri] Sure I will make changes and resend.
-Marri