Re: [PATCH 00/10] usb: phy: cleanups to Kconfig and directories

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

 



On 03/08/2013 04:08 PM, Arnd Bergmann wrote:
> On Friday 08 March 2013, Stephen Warren wrote:
>> Yes, I think it should instead work like:
>>
>> ARCH_TEGRA* selects nothing in particular related to USB.
>>
>> The Tegra EHCI controller Kconfig depends on ARCH_TEGRA so it doesn't
>> show up for other builds.
> 
> Yes, that's fine.
> 
>> I hope it's OK for the EHCI controller to select USB_ARCH_HAS_EHCI?
> 
> I think that would create a circular dependency, which Kconfig will
> refuse. We talked about the USB_ARCH_HAS_* Kconfig symbols recently
> and Alan Stern agreed to my suggestion of removing all of them,
> reworking the logic so we can always enable USB and EHCI but even
> when there is no bus glue enabled.
> 
> I'll have to do a proper patch one of these days, or find someone in
> my team to do it right for all the corner cases.
> 
>> The Tegra PHY Kconfig probably shouldn't be user-visible (relying on
>> being selected by the Tegra EHCI controller) and itself selects
>> anything it relies on.
>>
>> Does that sound reasonable?
> 
> It is often safer to express the logic using "depends on" than using
> "select", e.g. doing
> 
> config USB_EHCI_TEGRA
> 	bool "EHCI support for NVIDIA Tegra"
> 	depends on USB_EHCI_HCD
> 	depends on ARCH_TEGRA
> 
> config USB_ULPI_VIEWPORT
> 	def_bool y
> 	depends on USB_EHCI_TEGRA
> 
> If USB_ULPI_VIEWPORT has any other dependencies, the best solution
> in the above scenario is to make USB_EHCI_TEGRA depend on those.

USB_ULPI_VIEWPORT is, AFAIK, a generic piece of infra-structure that
USB_EHCI_TEGRA depends upon.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux