Re: [PATCH v2 2/3] usb: musb: add support for JZ4740 usb devicecontroller

From: Felipe Balbi
Date: Thu Dec 19 2013 - 10:36:54 EST


On Wed, Dec 18, 2013 at 12:56:00AM +0100, Apelete Seketeli wrote:
> Add support for Ingenic JZ4740 USB Device Controller through a
> specific musb glue layer.
>
> The platform data already available in tree for that USB Device
> Controller was previously used by an out-of-tree USB gadget driver
> which was not relying on the musb driver and was written by Ingenic
> and the Qi-Hardware community.
>
> JZ4740 UDC not being OTG compatible and missing some hardware
> registers, this musb glue layer is written from scratch to be used in
> gadget mode only and take silicon design specifics into account.
>
> Signed-off-by: Apelete Seketeli <apelete@xxxxxxxxxxxx>
> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
> ---
> arch/mips/include/asm/mach-jz4740/platform.h | 1 +
> arch/mips/jz4740/board-qi_lb60.c | 1 +
> arch/mips/jz4740/platform.c | 40 +++--
> drivers/usb/musb/Kconfig | 8 +-
> drivers/usb/musb/Makefile | 1 +
> drivers/usb/musb/jz4740.c | 201 ++++++++++++++++++++++++++

you can split this patch. drivers part has no dependency on arch part.

> diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig
> index 57dfc0c..a45ed15 100644
> --- a/drivers/usb/musb/Kconfig
> +++ b/drivers/usb/musb/Kconfig
> @@ -93,6 +93,12 @@ config USB_MUSB_BLACKFIN
> config USB_MUSB_UX500
> tristate "Ux500 platforms"
>
> +config USB_MUSB_JZ4740
> + tristate "JZ4740"
> + depends on MACH_JZ4740

add || COMPILE_TEST

--
balbi

Attachment: signature.asc
Description: Digital signature