Re: [PATCH 2/4] USB: MUSB: Enable MUSB for OMAP4 in Kconfig

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 25, 2010 at 10:48:34AM -0800, Tony Lindgren wrote:
> * Maulik Mankad <x0082077@xxxxxx> [100224 22:33]:
> > This patch adds the OMAP4 Kconfig options for MUSB.
> > 
> > Signed-off-by: Maulik Mankad <x0082077@xxxxxx>
> > Cc: Felipe Balbi <felipe.balbi@xxxxxxxxx>
> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxx>
> > Cc: David Brownell <david-b@xxxxxxxxxxx
> > ---
> > Index: mainline/drivers/usb/musb/Kconfig
> > ===================================================================
> > --- mainline.orig/drivers/usb/musb/Kconfig
> > +++ mainline/drivers/usb/musb/Kconfig
> > @@ -38,6 +38,7 @@ config USB_MUSB_SOC
> >  	default y if ARCH_DAVINCI
> >  	default y if ARCH_OMAP2430
> >  	default y if ARCH_OMAP34XX
> > +	default y if ARCH_OMAP4
> >  	default y if (BF54x && !BF544)
> >  	default y if (BF52x && !BF522 && !BF523)
> >  
> > @@ -50,6 +51,9 @@ comment "OMAP 243x high speed USB suppor
> >  comment "OMAP 343x high speed USB support"
> >  	depends on USB_MUSB_HDRC && ARCH_OMAP34XX
> >  
> > +comment "OMAP 44xx high speed USB support"
> > +	depends on USB_MUSB_HDRC && ARCH_OMAP4
> > +
> >  comment "Blackfin high speed USB Support"
> >  	depends on USB_MUSB_HDRC && ((BF54x && !BF544) || (BF52x && !BF522 && !BF523))
> >  
> > @@ -153,7 +157,7 @@ config MUSB_PIO_ONLY
> >  config USB_INVENTRA_DMA
> >  	bool
> >  	depends on USB_MUSB_HDRC && !MUSB_PIO_ONLY
> > -	default ARCH_OMAP2430 || ARCH_OMAP34XX || BLACKFIN
> > +	default ARCH_OMAP2430 || ARCH_OMAP34XX || BLACKFIN || ARCH_OMAP4
> >  	help
> >  	  Enable DMA transfers using Mentor's engine.
> 
> FYI, this needs to be updated as I have patches queued up
> to replace ARCH_OMAP34XX and with ARCH_OMAP3. You might
> want to check all these patches for that and make sure
> they build and work with the omap-for-linus branch.

Adding more and more depends also doesn't scale well. It's time for
someone to switch the test around, add a ARCH_HAS_MUSB, select it on
the platforms that has it and make the config option depend on that,
similar to how (some) platforms to EHCI today.

(i.e. if there's a janitor out there reading this, feel free to go ahead
and do it. :-)


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux