Re: kirkwood devicetree respin

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

 



On Tue, Mar 20, 2012 at 10:15:09PM +0000, Arnd Bergmann wrote:
> On Tuesday 20 March 2012, Jason Cooper wrote:
> > > If you want to add a dependency, it should be
> > > 
> > >       depends on PLAT_PXA
> > > 
> > > Most other platform drivers have a dependency on the platform
> > > they are for, but USB_EHCI_MV was only recently added, and nobody
> > > has bothered to fix this yet.
> >
> > It's my understanding that will make the driver visible in menuconfig
> > for PLAT_PXA.  That may be useful, but what I'm trying to fix is a new
> > user (me) from selecting USB_EHCI_MV on PLAT_ORION.  It breaks when
> > you do that (at runtime).
> 
> Are those two things not the same? The dependency makes the driver
> visible only on PLAT_PXA, which means it is invisible on PLAT_ORION
> and you can no longer select it. PLAT_PXA and PLAT_ORION are mutually
> exclusive.

Ok, that's what I missed.

...
> > > > Maybe also -Werror for that one file to catch other similar cases?
> > > 
> > > No, we are actually trying to make sure that any configuration you pick
> > > results in a kernel that builds, so that would be counterproductive.
> > 
> > I would argue it should build and work.  Otherwise, there's no point
> > in having a successful compile.  So, maybe the answer is not to have
> > it as a configuration option.  Or, at least, invisible in menuconfig.
> 
> Making USB_EHCI_MV invisible everywhere would also be possible, in that
> case the right logic would be
> 
> config USB_EHCI_MV
> 	def_bool y
> 	depends on USB_EHCI_HCD && PLAT_PXA
> 	select USB_EHCI_ROOT_HUB_TT
> 
> This would unconditionally enable the pxa ehci driver whenever the
> common ehci code is enabled, which is a reasonable choice, and matches
> the behavior of the orion driver.

Ok, I'll gin up a patch for this and CC: stable / linux-usb.

thx,

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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux