diff mbox

OMAP3+: Kconfig: has EHCI only if USB present

Message ID 1307718873-27652-1-git-send-email-nm@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Nishanth Menon June 10, 2011, 3:14 p.m. UTC
Attempting to disable USB generates the following warning at the moment:
warning: (ARCH_OMAP3 && ARCH_OMAP4) selects USB_ARCH_HAS_EHCI which has unmet direct dependencies (USB_SUPPORT)

Signed-off-by: Nishanth Menon <nm@ti.com>
---
based on 3.0-rc2
 arch/arm/mach-omap2/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Sanjeev Premi June 10, 2011, 4:08 p.m. UTC | #1
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org 
> [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Menon, Nishanth
> Sent: Friday, June 10, 2011 8:45 PM
> To: linux-omap
> Cc: Tony; linux-arm; Balbi, Felipe; Menon, Nishanth
> Subject: [PATCH] OMAP3+: Kconfig: has EHCI only if USB present
> 
> Attempting to disable USB generates the following warning at 
> the moment:
> warning: (ARCH_OMAP3 && ARCH_OMAP4) selects USB_ARCH_HAS_EHCI 
> which has unmet direct dependencies (USB_SUPPORT)
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> based on 3.0-rc2
>  arch/arm/mach-omap2/Kconfig |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index ef7e12f..d5cf1a8 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -33,7 +33,7 @@ config ARCH_OMAP3
>  	depends on ARCH_OMAP2PLUS
>  	default y
>  	select CPU_V7
> -	select USB_ARCH_HAS_EHCI
> +	select USB_ARCH_HAS_EHCI if USB_SUPPORT
>  	select ARM_L1_CACHE_SHIFT_6 if !ARCH_OMAP4
>  	select ARCH_HAS_OPP
>  	select PM_OPP if PM
> @@ -50,7 +50,7 @@ config ARCH_OMAP4
>  	select ARM_ERRATA_720789
>  	select ARCH_HAS_OPP
>  	select PM_OPP if PM
> -	select USB_ARCH_HAS_EHCI
> +	select USB_ARCH_HAS_EHCI if USB_SUPPORT
>  
>  comment "OMAP Core Type"
>  	depends on ARCH_OMAP2


Here is earlier discussion on the same subject:
http://www.mail-archive.com/linux-omap@vger.kernel.org/msg42064.html

Exactly same patch was proposed.

~sanjeev

> -- 
> 1.7.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe 
> linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
Nishanth Menon June 10, 2011, 4:20 p.m. UTC | #2
On Fri, Jun 10, 2011 at 11:08, Premi, Sanjeev <premi@ti.com> wrote:
>> -----Original Message-----
>> From: linux-omap-owner@vger.kernel.org
>> [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Menon, Nishanth
>> Sent: Friday, June 10, 2011 8:45 PM
>> To: linux-omap
>> Cc: Tony; linux-arm; Balbi, Felipe; Menon, Nishanth
>> Subject: [PATCH] OMAP3+: Kconfig: has EHCI only if USB present
>>
>> Attempting to disable USB generates the following warning at
>> the moment:
>> warning: (ARCH_OMAP3 && ARCH_OMAP4) selects USB_ARCH_HAS_EHCI
>> which has unmet direct dependencies (USB_SUPPORT)
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> ---
>> based on 3.0-rc2
>>  arch/arm/mach-omap2/Kconfig |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
>> index ef7e12f..d5cf1a8 100644
>> --- a/arch/arm/mach-omap2/Kconfig
>> +++ b/arch/arm/mach-omap2/Kconfig
>> @@ -33,7 +33,7 @@ config ARCH_OMAP3
>>       depends on ARCH_OMAP2PLUS
>>       default y
>>       select CPU_V7
>> -     select USB_ARCH_HAS_EHCI
>> +     select USB_ARCH_HAS_EHCI if USB_SUPPORT
>>       select ARM_L1_CACHE_SHIFT_6 if !ARCH_OMAP4
>>       select ARCH_HAS_OPP
>>       select PM_OPP if PM
>> @@ -50,7 +50,7 @@ config ARCH_OMAP4
>>       select ARM_ERRATA_720789
>>       select ARCH_HAS_OPP
>>       select PM_OPP if PM
>> -     select USB_ARCH_HAS_EHCI
>> +     select USB_ARCH_HAS_EHCI if USB_SUPPORT
>>
>>  comment "OMAP Core Type"
>>       depends on ARCH_OMAP2
>
>
> Here is earlier discussion on the same subject:
> http://www.mail-archive.com/linux-omap@vger.kernel.org/msg42064.html
>
> Exactly same patch was proposed.

You may want to do that for the full Kconfig file(not just for OMAP3)
and post out..

Regards,
Nishanth Menon
diff mbox

Patch

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index ef7e12f..d5cf1a8 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -33,7 +33,7 @@  config ARCH_OMAP3
 	depends on ARCH_OMAP2PLUS
 	default y
 	select CPU_V7
-	select USB_ARCH_HAS_EHCI
+	select USB_ARCH_HAS_EHCI if USB_SUPPORT
 	select ARM_L1_CACHE_SHIFT_6 if !ARCH_OMAP4
 	select ARCH_HAS_OPP
 	select PM_OPP if PM
@@ -50,7 +50,7 @@  config ARCH_OMAP4
 	select ARM_ERRATA_720789
 	select ARCH_HAS_OPP
 	select PM_OPP if PM
-	select USB_ARCH_HAS_EHCI
+	select USB_ARCH_HAS_EHCI if USB_SUPPORT
 
 comment "OMAP Core Type"
 	depends on ARCH_OMAP2