linux-kernel - Re: [PATCH v3 1/2] usb: musb: add support for JZ4740 usb device controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140104110622.GA7525@tao>
Date:	Sat, 4 Jan 2014 12:06:22 +0100
From:	Apelete Seketeli <apelete@...eteli.net>
To:	Richard Weinberger <richard.weinberger@...il.com>
Cc:	linux-usb@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
	Felipe Balbi <balbi@...com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Lars-Peter Clausen <lars@...afoo.de>
Subject: Re: [PATCH v3 1/2] usb: musb: add support for JZ4740 usb device
 controller

On 04-Jan-14, Richard Weinberger wrote:
> On Thu, Dec 19, 2013 at 9:42 PM, Apelete Seketeli <apelete@...eteli.net> wrote:
> > Add support for Ingenic JZ4740 USB Device Controller through a
> > specific musb glue layer.
> >
> > 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@...eteli.net>
> > Signed-off-by: Lars-Peter Clausen <lars@...afoo.de>
> > ---
> >  drivers/usb/musb/Kconfig  |    8 +-
> >  drivers/usb/musb/Makefile |    1 +
> >  drivers/usb/musb/jz4740.c |  201 +++++++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 209 insertions(+), 1 deletion(-)
> >  create mode 100644 drivers/usb/musb/jz4740.c
> >
> > diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig
> > index 57dfc0c..14d7e72 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 || COMPILE_TEST
> > +       depends on USB_MUSB_GADGET
> > +       depends on USB_OTG_BLACKLIST_HUB
> > +
> >  endchoice
> >
> >  config USB_MUSB_AM335X_CHILD
> > @@ -100,7 +106,7 @@ config USB_MUSB_AM335X_CHILD
> >
> >  choice
> >         prompt 'MUSB DMA mode'
> > -       default MUSB_PIO_ONLY if ARCH_MULTIPLATFORM
> > +       default MUSB_PIO_ONLY if ARCH_MULTIPLATFORM || USB_MUSB_JZ4740
> 
> Just out of curiosity, why can't we use DMA?

We wrote the musb glue layer for the Ben Nanonote handheld computer
only to provide ethernet-over-usb functionality.

I was thinking about adding DMA support later, but Lars-Peter Clausen
did experiment with DMA, and found out that all ethernet packets were
unaligned, while the DMA only works if they are aligned.
As a consequence, the DMA was bypassed all the time, making it useless
at least for the ethernet gadget we are interested in.

That's why DMA didn't make it into the glue layer.

Cheers.
-- 
        Apelete
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/