linux-kernel - RE: [RFC PATCH 0/7] usb: musb: add driver for control module

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: <6C6B28D4DC342643927BEAFCE8707BF63EA5ACAB@DBDE01.ent.ti.com>
Date:	Tue, 15 Jan 2013 16:06:44 +0000
From:	"B, Ravi" <ravibabu@...com>
To:	"Balbi, Felipe" <balbi@...com>,
	"ABRAHAM, KISHON VIJAY" <kishon@...com>
CC:	Arnd Bergmann <arnd@...db.de>,
	"grant.likely@...retlab.ca" <grant.likely@...retlab.ca>,
	"Cousson, Benoit" <b-cousson@...com>,
	"rob.herring@...xeda.com" <rob.herring@...xeda.com>,
	"rob@...dley.net" <rob@...dley.net>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>,
	"tony@...mide.com" <tony@...mide.com>,
	"devicetree-discuss@...ts.ozlabs.org" 
	<devicetree-discuss@...ts.ozlabs.org>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>
Subject: RE: [RFC PATCH 0/7] usb: musb: add driver for control module

> Hi,
> 
> On Tue, Jan 15, 2013 at 08:09:22PM +0530, kishon wrote:
> > Hi Arnd,
> > 
> > On Tuesday 15 January 2013 07:11 PM, Arnd Bergmann wrote:
> > >On Tuesday 15 January 2013, Kishon Vijay Abraham I wrote:
> > >>Added a new driver for the usb part of control module. 
> This has an 
> > >>API to power on the USB2 phy and an API to write to the mailbox 
> > >>depending on whether MUSB has to act in host mode or in 
> device mode.
> > >>
> > >>Writing to control module registers for doing the above 
> task which 
> > >>was previously done in omap glue and in omap-usb2 phy is removed.
> > >>
> > >>Also added the dt data to get MUSB working in OMAP platforms.
> > >>This series has patches for both drivers and ARCH 
> folders, so If it 
> > >>has to be split I'll do it.
> > >>
> > >
> > >The series looks good to me, I just had a minor comment on 
> one patch.
> > >
> > >One a somewhat related topic, I wonder whether there are 
> any plans on 
> > >your side to change this driver to support multiple bus 
> glues to be 
> > >built for one kernel image. With a multiplatform kernel, 
> we may need 
> > >all of TUSB6010/OMAP2PLUS/DSPS/UX500 for instance.
> > 
> > We don't have plans as of now. I actually don't expect any 
> changes in 
> > the driver other than the Kconfig changes. Anyways the 
> probe of glue's 
> > other than the platform it's running won't get called. right Felipe?

If understand correctly the control module driver used to configure the respective usb phy of SoC to respective usb modes using the common set of control module APIs. What if, if control module interface (register defintions) varies b/w different revision or spin of same type of SoCs, if usbphy type is changed. In this case whether the single instance of control module driver is good enough to cater of all cpu types of same SoC series ? 
Whether cpu_is_xxx() can be used to differentiate b/w different cpu types in CM driver?

> 
> AFAICT there's nothing preventing those from being built 
> together as long as you don't use DMA (yeah, that's a touchy 
> subject still with MUSB).
> 
> If there are any build breaks, please report them so bus glue 
> owners can fix. I see that at least the davinci folks need to 
> work a bit
> 
> $ git grep -e "mach\/" drivers/usb/musb/ 
> drivers/usb/musb/da8xx.c:#include <mach/da8xx.h> 
> drivers/usb/musb/davinci.c:#include <mach/cputype.h> 
> drivers/usb/musb/davinci.c:#include <mach/hardware.h>
> 
> I'm adding Ravi B to the loop here for those.
> 
> --
> balbi
> --
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/