On Sat, Aug 10, 2013 at 01:14:15PM +0530, Manjunath Goudar wrote:
> Separate the OHCI NXP host controller driver from ohci-hcd
> host code so that it can be built as a separate driver module.
> This work is part of enabling multi-platform kernels on ARM.
> 
> Many place function name and struct name started with usb,
> current scenario replaced usb with ohci for proper naming.
> 
> Signed-off-by: Manjunath Goudar <[email protected]>
> Signed-off-by: Deepak Saxena <[email protected]>
> Acked-by: Alan Stern <[email protected]>
> Cc: Arnd Bergmann <[email protected]>
> Cc: Greg KH <[email protected]>
> Cc: [email protected]
> 
> V2:
>   - Directive check NXP_PLATFORM_DRIVER block has been removed.
> ---
>  drivers/usb/host/Kconfig    |    8 +++
>  drivers/usb/host/Makefile   |    1 +
>  drivers/usb/host/ohci-hcd.c |   18 -------
>  drivers/usb/host/ohci-nxp.c |  124 
> +++++++++++++++++--------------------------
>  4 files changed, 58 insertions(+), 93 deletions(-)

Also failed to apply:

checking file drivers/usb/host/Kconfig
Hunk #1 FAILED at 398.
1 out of 1 hunk FAILED
checking file drivers/usb/host/Makefile
Hunk #1 FAILED at 50.
1 out of 1 hunk FAILED
checking file drivers/usb/host/ohci-hcd.c
Hunk #1 succeeded at 1212 with fuzz 1 (offset 18 lines).
Hunk #2 succeeded at 1349 with fuzz 1 (offset 53 lines).
Hunk #3 FAILED at 1309.
Hunk #4 FAILED at 1357.
2 out of 4 hunks FAILED
checking file drivers/usb/host/ohci-nxp.c

{sigh}

That's why I required a senior Linaro developer to approve these
patches.  You just wasted a bunch of my time, and it was obvious that no
one else actually tried these patches out, as they can't even be
applied!

Deepak, come on, you know better :(

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to