[PATCH 00/10] kconfigs cleanups and fixes

[PATCH 00/10] kconfigs cleanups and fixes

[Felipe Contreras]

Hi,

Here are some cleanups for kconfigs, mostly in ARM and USB stuff.

Some of these have already been sent in the past, but never splitted nor
applied, now I'm doing that. Others are new issues, and some actual warnings
are fixed.

Cheers.

Felipe Contreras (10):
  arm: omap4: fix kconfig warning
  arm: vexpress: fix kconfig warning
  arm: omap2: fix omap3 touchbook kconfig warning
  arm: omap2: fix regulator warnings
  usb: musb: trivial Kconfig cleanups
  usb: core: trivial Kconfig cleanups
  usb: gadget: remove useless depends on Kconfig
  usb: gadget: remove useless dependency in config
  usb: otg: trivial cleanups
  musb: cleanup kconfig

 arch/arm/mach-omap2/Kconfig    |   13 ++++++-------
 arch/arm/mach-vexpress/Kconfig |    4 ++--
 drivers/usb/core/Kconfig       |    4 ----
 drivers/usb/gadget/Kconfig     |    5 -----
 drivers/usb/musb/Kconfig       |   15 ++++++++-------
 drivers/usb/otg/Kconfig        |   20 ++++++++++----------
 6 files changed, 26 insertions(+), 35 deletions(-)

-- 
1.7.8.rc1.14.g248db

[PATCH 01/10] arm: omap4: fix kconfig warning

[Felipe Contreras]

warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index e1293aa..eb7c3ac 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -46,7 +46,7 @@ config ARCH_OMAP4
 	select LOCAL_TIMERS if SMP
 	select PL310_ERRATA_588369
 	select PL310_ERRATA_727915
-	select ARM_ERRATA_720789
+	select ARM_ERRATA_720789 if SMP
 	select ARCH_HAS_OPP
 	select PM_OPP if PM
 	select USB_ARCH_HAS_EHCI
-- 
1.7.8.rc1.14.g248db

[PATCH 01/10] arm: omap4: fix kconfig warning

[Felipe Contreras]

warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index e1293aa..eb7c3ac 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -46,7 +46,7 @@ config ARCH_OMAP4
 	select LOCAL_TIMERS if SMP
 	select PL310_ERRATA_588369
 	select PL310_ERRATA_727915
-	select ARM_ERRATA_720789
+	select ARM_ERRATA_720789 if SMP
 	select ARCH_HAS_OPP
 	select PM_OPP if PM
 	select USB_ARCH_HAS_EHCI
-- 
1.7.8.rc1.14.g248db

[PATCH 01/10] arm: omap4: fix kconfig warning

[Felipe Contreras]

warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index e1293aa..eb7c3ac 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -46,7 +46,7 @@ config ARCH_OMAP4
 	select LOCAL_TIMERS if SMP
 	select PL310_ERRATA_588369
 	select PL310_ERRATA_727915
-	select ARM_ERRATA_720789
+	select ARM_ERRATA_720789 if SMP
 	select ARCH_HAS_OPP
 	select PM_OPP if PM
 	select USB_ARCH_HAS_EHCI
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 01/10] arm: omap4: fix kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)

Thanks applying into fixes-non-critical-part3.

Tony

[PATCH 01/10] arm: omap4: fix kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)

Thanks applying into fixes-non-critical-part3.

Tony

Re: [PATCH 01/10] arm: omap4: fix kconfig warning

[Felipe Contreras]

On Thu, Dec 22, 2011 at 8:22 PM, Tony Lindgren <tony@atomide.com> wrote:
> * Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
>> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
>
> Thanks applying into fixes-non-critical-part3.

Actually, please disregard this one, it should be handled by this:
http://article.gmane.org/gmane.linux.ports.arm.kernel/141942

Cheers.

-- 
Felipe Contreras

[PATCH 01/10] arm: omap4: fix kconfig warning

[Felipe Contreras]

On Thu, Dec 22, 2011 at 8:22 PM, Tony Lindgren <tony@atomide.com> wrote:
> * Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
>> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
>
> Thanks applying into fixes-non-critical-part3.

Actually, please disregard this one, it should be handled by this:
http://article.gmane.org/gmane.linux.ports.arm.kernel/141942

Cheers.

-- 
Felipe Contreras

Re: [PATCH 01/10] arm: omap4: fix kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111222 11:37]:
> On Thu, Dec 22, 2011 at 8:22 PM, Tony Lindgren <tony@atomide.com> wrote:
> > * Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> >> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
> >
> > Thanks applying into fixes-non-critical-part3.
> 
> Actually, please disregard this one, it should be handled by this:
> http://article.gmane.org/gmane.linux.ports.arm.kernel/141942

OK dropping.

Tony

[PATCH 01/10] arm: omap4: fix kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111222 11:37]:
> On Thu, Dec 22, 2011 at 8:22 PM, Tony Lindgren <tony@atomide.com> wrote:
> > * Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> >> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
> >
> > Thanks applying into fixes-non-critical-part3.
> 
> Actually, please disregard this one, it should be handled by this:
> http://article.gmane.org/gmane.linux.ports.arm.kernel/141942

OK dropping.

Tony

[PATCH 02/10] arm: vexpress: fix kconfig warning

warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
warning: (ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_751472 which has unmet direct dependencies (CPU_V7 && SMP)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-vexpress/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
index 9311484..3fbcb8c 100644
--- a/arch/arm/mach-vexpress/Kconfig
+++ b/arch/arm/mach-vexpress/Kconfig
@@ -5,8 +5,8 @@ config ARCH_VEXPRESS_CA9X4
 	bool "Versatile Express Cortex-A9x4 tile"
 	select CPU_V7
 	select ARM_GIC
-	select ARM_ERRATA_720789
-	select ARM_ERRATA_751472
+	select ARM_ERRATA_720789 if SMP
+	select ARM_ERRATA_751472 if SMP
 	select ARM_ERRATA_753970
 
 endmenu
-- 
1.7.8.rc1.14.g248db

[PATCH 02/10] arm: vexpress: fix kconfig warning

[Felipe Contreras]

warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
warning: (ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_751472 which has unmet direct dependencies (CPU_V7 && SMP)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-vexpress/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
index 9311484..3fbcb8c 100644
--- a/arch/arm/mach-vexpress/Kconfig
+++ b/arch/arm/mach-vexpress/Kconfig
@@ -5,8 +5,8 @@ config ARCH_VEXPRESS_CA9X4
 	bool "Versatile Express Cortex-A9x4 tile"
 	select CPU_V7
 	select ARM_GIC
-	select ARM_ERRATA_720789
-	select ARM_ERRATA_751472
+	select ARM_ERRATA_720789 if SMP
+	select ARM_ERRATA_751472 if SMP
 	select ARM_ERRATA_753970
 
 endmenu
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 02/10] arm: vexpress: fix kconfig warning

[Dave Martin]

On Tue, Dec 20, 2011 at 02:42:19AM +0200, Felipe Contreras wrote:
> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
> warning: (ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_751472 which has unmet direct dependencies (CPU_V7 && SMP)
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  arch/arm/mach-vexpress/Kconfig |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index 9311484..3fbcb8c 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -5,8 +5,8 @@ config ARCH_VEXPRESS_CA9X4
>  	bool "Versatile Express Cortex-A9x4 tile"
>  	select CPU_V7
>  	select ARM_GIC
> -	select ARM_ERRATA_720789
> -	select ARM_ERRATA_751472
> +	select ARM_ERRATA_720789 if SMP
> +	select ARM_ERRATA_751472 if SMP
>  	select ARM_ERRATA_753970

NAK -- I hit this problem too, but in fact the errata configs should not
really be dependent on SMP; these are fixed in a couple of patches
currently queued in Russell's patch system:

7196/1 - errata: remove SMP dependency for erratum 720789
7197/1 - errata: remove SMP dependency for erratum 751472

...so we should avoid duplicating knowledge about exactly what these
errata workarounds do, where it can be avoided.

In the meantime, the warning is harmless.

Cheers
---Dave

[PATCH 02/10] arm: vexpress: fix kconfig warning

[Dave Martin]

On Tue, Dec 20, 2011 at 02:42:19AM +0200, Felipe Contreras wrote:
> warning: (ARCH_OMAP4 && ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_720789 which has unmet direct dependencies (CPU_V7 && SMP)
> warning: (ARCH_VEXPRESS_CA9X4) selects ARM_ERRATA_751472 which has unmet direct dependencies (CPU_V7 && SMP)
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  arch/arm/mach-vexpress/Kconfig |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index 9311484..3fbcb8c 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -5,8 +5,8 @@ config ARCH_VEXPRESS_CA9X4
>  	bool "Versatile Express Cortex-A9x4 tile"
>  	select CPU_V7
>  	select ARM_GIC
> -	select ARM_ERRATA_720789
> -	select ARM_ERRATA_751472
> +	select ARM_ERRATA_720789 if SMP
> +	select ARM_ERRATA_751472 if SMP
>  	select ARM_ERRATA_753970

NAK -- I hit this problem too, but in fact the errata configs should not
really be dependent on SMP; these are fixed in a couple of patches
currently queued in Russell's patch system:

7196/1 - errata: remove SMP dependency for erratum 720789
7197/1 - errata: remove SMP dependency for erratum 751472

...so we should avoid duplicating knowledge about exactly what these
errata workarounds do, where it can be avoided.

In the meantime, the warning is harmless.

Cheers
---Dave

Re: [PATCH 02/10] arm: vexpress: fix kconfig warning

[Felipe Contreras]

On Thu, Dec 22, 2011 at 1:39 PM, Dave Martin <dave.martin@linaro.org> wrote:
> NAK -- I hit this problem too, but in fact the errata configs should not
> really be dependent on SMP; these are fixed in a couple of patches
> currently queued in Russell's patch system:
>
> 7196/1 - errata: remove SMP dependency for erratum 720789
> 7197/1 - errata: remove SMP dependency for erratum 751472
>
> ...so we should avoid duplicating knowledge about exactly what these
> errata workarounds do, where it can be avoided.
>
> In the meantime, the warning is harmless.

All right.

-- 
Felipe Contreras

[PATCH 02/10] arm: vexpress: fix kconfig warning

[Felipe Contreras]

On Thu, Dec 22, 2011 at 1:39 PM, Dave Martin <dave.martin@linaro.org> wrote:
> NAK -- I hit this problem too, but in fact the errata configs should not
> really be dependent on SMP; these are fixed in a couple of patches
> currently queued in Russell's patch system:
>
> 7196/1 - errata: remove SMP dependency for erratum 720789
> 7197/1 - errata: remove SMP dependency for erratum 751472
>
> ...so we should avoid duplicating knowledge about exactly what these
> errata workarounds do, where it can be avoided.
>
> In the meantime, the warning is harmless.

All right.

-- 
Felipe Contreras

[PATCH 03/10] arm: omap2: fix omap3 touchbook kconfig warning

warning: (MACH_OMAP3_TOUCHBOOK && DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_ASUS && ACPI_CMPC && SAMSUNG_Q10) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)

A lot of boards need BACKLIGHT_CLASS_DEVICE for the framebuffers to
work, but it's not *needed* for the device itself. It might be nice to
enable it by default somewhoe if graphics stuff is enabled, but that's
another story.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index eb7c3ac..dbecb9a 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -211,7 +211,6 @@ config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
 	depends on ARCH_OMAP3
 	default y
-	select BACKLIGHT_CLASS_DEVICE
 
 config MACH_OMAP_3430SDP
 	bool "OMAP 3430 SDP board"
-- 
1.7.8.rc1.14.g248db

[PATCH 03/10] arm: omap2: fix omap3 touchbook kconfig warning

[Felipe Contreras]

warning: (MACH_OMAP3_TOUCHBOOK && DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_ASUS && ACPI_CMPC && SAMSUNG_Q10) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)

A lot of boards need BACKLIGHT_CLASS_DEVICE for the framebuffers to
work, but it's not *needed* for the device itself. It might be nice to
enable it by default somewhoe if graphics stuff is enabled, but that's
another story.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index eb7c3ac..dbecb9a 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -211,7 +211,6 @@ config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
 	depends on ARCH_OMAP3
 	default y
-	select BACKLIGHT_CLASS_DEVICE
 
 config MACH_OMAP_3430SDP
 	bool "OMAP 3430 SDP board"
-- 
1.7.8.rc1.14.g248db

[PATCH 03/10] arm: omap2: fix omap3 touchbook kconfig warning

[Felipe Contreras]

warning: (MACH_OMAP3_TOUCHBOOK && DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_ASUS && ACPI_CMPC && SAMSUNG_Q10) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)

A lot of boards need BACKLIGHT_CLASS_DEVICE for the framebuffers to
work, but it's not *needed* for the device itself. It might be nice to
enable it by default somewhoe if graphics stuff is enabled, but that's
another story.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index eb7c3ac..dbecb9a 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -211,7 +211,6 @@ config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
 	depends on ARCH_OMAP3
 	default y
-	select BACKLIGHT_CLASS_DEVICE
 
 config MACH_OMAP_3430SDP
 	bool "OMAP 3430 SDP board"
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 03/10] arm: omap2: fix omap3 touchbook kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (MACH_OMAP3_TOUCHBOOK && DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_ASUS && ACPI_CMPC && SAMSUNG_Q10) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)
> 
> A lot of boards need BACKLIGHT_CLASS_DEVICE for the framebuffers to
> work, but it's not *needed* for the device itself. It might be nice to
> enable it by default somewhoe if graphics stuff is enabled, but that's
> another story.

Thanks applying into fixes-non-critical-part3.

Tony

[PATCH 03/10] arm: omap2: fix omap3 touchbook kconfig warning

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (MACH_OMAP3_TOUCHBOOK && DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_ASUS && ACPI_CMPC && SAMSUNG_Q10) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)
> 
> A lot of boards need BACKLIGHT_CLASS_DEVICE for the framebuffers to
> work, but it's not *needed* for the device itself. It might be nice to
> enable it by default somewhoe if graphics stuff is enabled, but that's
> another story.

Thanks applying into fixes-non-critical-part3.

Tony

[PATCH 04/10] arm: omap2: fix regulator warnings

[Felipe Contreras]

warning: (MACH_OMAP_ZOOM2 && MACH_OMAP_ZOOM3 && MACH_OMAP_4430SDP && MACH_OMAP4_PANDA && TPS6105X) selects REGULATOR_FIXED_VOLTAGE which has unmet direct dependencies (REGULATOR)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dbecb9a..04734b5 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -205,7 +205,7 @@ config MACH_OMAP3_PANDORA
 	depends on ARCH_OMAP3
 	default y
 	select OMAP_PACKAGE_CBB
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
@@ -256,7 +256,7 @@ config MACH_OMAP_ZOOM2
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP_ZOOM3
 	bool "OMAP3630 Zoom3 board"
@@ -266,7 +266,7 @@ config MACH_OMAP_ZOOM3
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_CM_T35
 	bool "CompuLab CM-T35/CM-T3730 modules"
@@ -320,7 +320,7 @@ config MACH_OMAP_4430SDP
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP4_PANDA
 	bool "OMAP4 Panda Board"
@@ -328,7 +328,7 @@ config MACH_OMAP4_PANDA
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config OMAP3_EMU
 	bool "OMAP3 debugging peripherals"
-- 
1.7.8.rc1.14.g248db

[PATCH 04/10] arm: omap2: fix regulator warnings

[Felipe Contreras]

warning: (MACH_OMAP_ZOOM2 && MACH_OMAP_ZOOM3 && MACH_OMAP_4430SDP && MACH_OMAP4_PANDA && TPS6105X) selects REGULATOR_FIXED_VOLTAGE which has unmet direct dependencies (REGULATOR)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dbecb9a..04734b5 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -205,7 +205,7 @@ config MACH_OMAP3_PANDORA
 	depends on ARCH_OMAP3
 	default y
 	select OMAP_PACKAGE_CBB
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
@@ -256,7 +256,7 @@ config MACH_OMAP_ZOOM2
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP_ZOOM3
 	bool "OMAP3630 Zoom3 board"
@@ -266,7 +266,7 @@ config MACH_OMAP_ZOOM3
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_CM_T35
 	bool "CompuLab CM-T35/CM-T3730 modules"
@@ -320,7 +320,7 @@ config MACH_OMAP_4430SDP
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP4_PANDA
 	bool "OMAP4 Panda Board"
@@ -328,7 +328,7 @@ config MACH_OMAP4_PANDA
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config OMAP3_EMU
 	bool "OMAP3 debugging peripherals"
-- 
1.7.8.rc1.14.g248db

[PATCH 04/10] arm: omap2: fix regulator warnings

[Felipe Contreras]

warning: (MACH_OMAP_ZOOM2 && MACH_OMAP_ZOOM3 && MACH_OMAP_4430SDP && MACH_OMAP4_PANDA && TPS6105X) selects REGULATOR_FIXED_VOLTAGE which has unmet direct dependencies (REGULATOR)

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 arch/arm/mach-omap2/Kconfig |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dbecb9a..04734b5 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -205,7 +205,7 @@ config MACH_OMAP3_PANDORA
 	depends on ARCH_OMAP3
 	default y
 	select OMAP_PACKAGE_CBB
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP3_TOUCHBOOK
 	bool "OMAP3 Touch Book"
@@ -256,7 +256,7 @@ config MACH_OMAP_ZOOM2
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP_ZOOM3
 	bool "OMAP3630 Zoom3 board"
@@ -266,7 +266,7 @@ config MACH_OMAP_ZOOM3
 	select SERIAL_8250
 	select SERIAL_CORE_CONSOLE
 	select SERIAL_8250_CONSOLE
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_CM_T35
 	bool "CompuLab CM-T35/CM-T3730 modules"
@@ -320,7 +320,7 @@ config MACH_OMAP_4430SDP
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config MACH_OMAP4_PANDA
 	bool "OMAP4 Panda Board"
@@ -328,7 +328,7 @@ config MACH_OMAP4_PANDA
 	depends on ARCH_OMAP4
 	select OMAP_PACKAGE_CBL
 	select OMAP_PACKAGE_CBS
-	select REGULATOR_FIXED_VOLTAGE
+	select REGULATOR_FIXED_VOLTAGE if REGULATOR
 
 config OMAP3_EMU
 	bool "OMAP3 debugging peripherals"
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 04/10] arm: omap2: fix regulator warnings

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (MACH_OMAP_ZOOM2 && MACH_OMAP_ZOOM3 && MACH_OMAP_4430SDP && MACH_OMAP4_PANDA && TPS6105X) selects REGULATOR_FIXED_VOLTAGE which has unmet direct dependencies (REGULATOR)

Thanks applying into fixes-non-critical-part3.

Tony

[PATCH 04/10] arm: omap2: fix regulator warnings

[Tony Lindgren]

* Felipe Contreras <felipe.contreras@gmail.com> [111219 16:11]:
> warning: (MACH_OMAP_ZOOM2 && MACH_OMAP_ZOOM3 && MACH_OMAP_4430SDP && MACH_OMAP4_PANDA && TPS6105X) selects REGULATOR_FIXED_VOLTAGE which has unmet direct dependencies (REGULATOR)

Thanks applying into fixes-non-critical-part3.

Tony

[PATCH 05/10] usb: musb: trivial Kconfig cleanups

[Felipe Contreras]

Shuffle the code a bit so the description is at the top.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Acked-by: Felipe Balbi <balbi@ti.com>
---
 drivers/usb/musb/Kconfig |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig
index 07a0346..b0a46e4 100644
--- a/drivers/usb/musb/Kconfig
+++ b/drivers/usb/musb/Kconfig
@@ -5,6 +5,7 @@
 
 # (M)HDRC = (Multipoint) Highspeed Dual-Role Controller
 config USB_MUSB_HDRC
+	tristate 'Inventra Highspeed Dual Role Controller (TI, ADI, ...)'
 	depends on USB && USB_GADGET
 	depends on (ARM || (BF54x && !BF544) || (BF52x && !BF522 && !BF523))
 	select NOP_USB_XCEIV if (ARCH_DAVINCI || MACH_OMAP3EVM || BLACKFIN)
@@ -12,7 +13,6 @@ config USB_MUSB_HDRC
 	select TWL6030_USB if MACH_OMAP_4430SDP || MACH_OMAP4_PANDA
 	select USB_OTG_UTILS
 	select USB_GADGET_DUALSPEED
-	tristate 'Inventra Highspeed Dual Role Controller (TI, ADI, ...)'
 	help
 	  Say Y here if your system has a dual role high speed USB
 	  controller based on the Mentor Graphics silicon IP.  Then
@@ -107,4 +107,3 @@ config USB_TUSB_OMAP_DMA
 	default y
 	help
 	  Enable DMA transfers on TUSB 6010 when OMAP DMA is available.
-
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 05/10] usb: musb: trivial Kconfig cleanups

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 269 bytes --]

On Tue, Dec 20, 2011 at 02:42:22AM +0200, Felipe Contreras wrote:
> Shuffle the code a bit so the description is at the top.
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> Acked-by: Felipe Balbi <balbi@ti.com>

applied, thanks

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[PATCH 06/10] usb: core: trivial Kconfig cleanups

[Felipe Contreras]

The default is 'n', so specifying it is a no-op.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 drivers/usb/core/Kconfig |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/core/Kconfig b/drivers/usb/core/Kconfig
index 18d02e3..1b1d371 100644
--- a/drivers/usb/core/Kconfig
+++ b/drivers/usb/core/Kconfig
@@ -12,7 +12,6 @@ config USB_DEBUG
 config USB_ANNOUNCE_NEW_DEVICES
 	bool "USB announce new devices"
 	depends on USB
-	default N
 	help
 	  Say Y here if you want the USB core to always announce the
 	  idVendor, idProduct, Manufacturer, Product, and SerialNumber
@@ -110,7 +109,6 @@ config USB_OTG
 	bool "OTG support"
 	depends on USB && EXPERIMENTAL
 	depends on USB_SUSPEND
-	default n
 	help
 	  The most notable feature of USB OTG is support for a
 	  "Dual-Role" device, which can act as either a device
@@ -125,7 +123,6 @@ config USB_OTG_WHITELIST
 	bool "Rely on OTG Targeted Peripherals List"
 	depends on USB_OTG || EXPERT
 	default y if USB_OTG
-	default n if EXPERT
 	help
 	  If you say Y here, the "otg_whitelist.h" file will be used as a
 	  product whitelist, so USB peripherals not listed there will be
@@ -147,4 +144,3 @@ config USB_OTG_BLACKLIST_HUB
 	  external hubs.  OTG hosts are allowed to reduce hardware
 	  and software costs by not supporting external hubs.  So
 	  are "Embedded Hosts" that don't offer OTG support.
-
-- 
1.7.8.rc1.14.g248db

[PATCH 07/10] usb: gadget: remove useless depends on Kconfig

[Felipe Contreras]

Where are inside an 'if USB_GADGET'.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 drivers/usb/gadget/Kconfig |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
index 23a4473..6207f1a 100644
--- a/drivers/usb/gadget/Kconfig
+++ b/drivers/usb/gadget/Kconfig
@@ -124,7 +124,6 @@ config USB_GADGET_STORAGE_NUM_BUFFERS
 #
 choice
 	prompt "USB Peripheral Controller"
-	depends on USB_GADGET
 	help
 	   A USB device uses a controller to talk to its host.
 	   Systems should have only one such upstream link.
@@ -544,12 +543,10 @@ endchoice
 # Selected by UDC drivers that support high-speed operation.
 config USB_GADGET_DUALSPEED
 	bool
-	depends on USB_GADGET
 
 # Selected by UDC drivers that support super-speed opperation
 config USB_GADGET_SUPERSPEED
 	bool
-	depends on USB_GADGET
 	depends on USB_GADGET_DUALSPEED
 
 #
@@ -557,7 +554,6 @@ config USB_GADGET_SUPERSPEED
 #
 choice
 	tristate "USB Gadget Drivers"
-	depends on USB_GADGET
 	default USB_ETH
 	help
 	  A Linux "Gadget Driver" talks to the USB Peripheral Controller
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 07/10] usb: gadget: remove useless depends on Kconfig

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 215 bytes --]

Hi,

On Tue, Dec 20, 2011 at 02:42:24AM +0200, Felipe Contreras wrote:
> Where are inside an 'if USB_GADGET'.
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

applied, thanks

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[PATCH 08/10] usb: gadget: remove useless dependency in config

[Felipe Contreras]

USB_OTG_UTILS is already selected by NOP_USB_XCEIV.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 drivers/usb/gadget/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
index 6207f1a..78fe02d 100644
--- a/drivers/usb/gadget/Kconfig
+++ b/drivers/usb/gadget/Kconfig
@@ -181,7 +181,6 @@ config USB_OMAP
 	tristate "OMAP USB Device Controller"
 	depends on ARCH_OMAP
 	select ISP1301_OMAP if MACH_OMAP_H2 || MACH_OMAP_H3 || MACH_OMAP_H4_OTG
-	select USB_OTG_UTILS if ARCH_OMAP
 	help
 	   Many Texas Instruments OMAP processors have flexible full
 	   speed USB device controllers, with support for up to 30
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 08/10] usb: gadget: remove useless dependency in config

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 935 bytes --]

On Tue, Dec 20, 2011 at 02:42:25AM +0200, Felipe Contreras wrote:
> USB_OTG_UTILS is already selected by NOP_USB_XCEIV.

did you mean it's selected by the isp1301 driver ?

> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  drivers/usb/gadget/Kconfig |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> index 6207f1a..78fe02d 100644
> --- a/drivers/usb/gadget/Kconfig
> +++ b/drivers/usb/gadget/Kconfig
> @@ -181,7 +181,6 @@ config USB_OMAP
>  	tristate "OMAP USB Device Controller"
>  	depends on ARCH_OMAP
>  	select ISP1301_OMAP if MACH_OMAP_H2 || MACH_OMAP_H3 || MACH_OMAP_H4_OTG
> -	select USB_OTG_UTILS if ARCH_OMAP
>  	help
>  	   Many Texas Instruments OMAP processors have flexible full
>  	   speed USB device controllers, with support for up to 30
> -- 
> 1.7.8.rc1.14.g248db
> 

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 08/10] usb: gadget: remove useless dependency in config

[Felipe Contreras]

2011/12/20 Felipe Balbi <balbi@ti.com>:
> On Tue, Dec 20, 2011 at 02:42:25AM +0200, Felipe Contreras wrote:
>> USB_OTG_UTILS is already selected by NOP_USB_XCEIV.
>
> did you mean it's selected by the isp1301 driver ?

No.

config NOP_USB_XCEIV
	tristate "NOP USB Transceiver Driver"
	select USB_OTG_UTILS

-- 
Felipe Contreras

Re: [PATCH 08/10] usb: gadget: remove useless dependency in config

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 573 bytes --]

On Thu, Dec 22, 2011 at 06:14:34PM +0200, Felipe Contreras wrote:
> 2011/12/20 Felipe Balbi <balbi@ti.com>:
> > On Tue, Dec 20, 2011 at 02:42:25AM +0200, Felipe Contreras wrote:
> >> USB_OTG_UTILS is already selected by NOP_USB_XCEIV.
> >
> > did you mean it's selected by the isp1301 driver ?
> 
> No.
> 
> config NOP_USB_XCEIV
> 	tristate "NOP USB Transceiver Driver"
> 	select USB_OTG_UTILS

that "select" line is on all transceiver drivers. The legacy omap_udc
doesn't use Nop-Xceiv. All known boards use isp1301_omap, that's why I
asked.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[PATCH 09/10] usb: otg: trivial cleanups

[Felipe Contreras]

Spaces to tabs, proper alignment, and start sentences as they should.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 drivers/usb/otg/Kconfig |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/usb/otg/Kconfig b/drivers/usb/otg/Kconfig
index c66481a..5e913d7 100644
--- a/drivers/usb/otg/Kconfig
+++ b/drivers/usb/otg/Kconfig
@@ -82,9 +82,9 @@ config NOP_USB_XCEIV
 	tristate "NOP USB Transceiver Driver"
 	select USB_OTG_UTILS
 	help
-	 this driver is to be used by all the usb transceiver which are either
-	 built-in with usb ip or which are autonomous and doesn't require any
-	 phy programming such as ISP1x04 etc.
+	  This driver is to be used by all the usb transceiver which are either
+	  built-in with usb ip or which are autonomous and doesn't require any
+	  phy programming such as ISP1x04 etc.
 
 config USB_LANGWELL_OTG
 	tristate "Intel Langwell USB OTG dual-role support"
@@ -114,13 +114,13 @@ config USB_MSM_OTG
 	  has an external PHY.
 
 config AB8500_USB
-        tristate "AB8500 USB Transceiver Driver"
-        depends on AB8500_CORE
-        select USB_OTG_UTILS
-        help
-          Enable this to support the USB OTG transceiver in AB8500 chip.
-          This transceiver supports high and full speed devices plus,
-          in host mode, low speed.
+	tristate "AB8500 USB Transceiver Driver"
+	depends on AB8500_CORE
+	select USB_OTG_UTILS
+	help
+	  Enable this to support the USB OTG transceiver in AB8500 chip.
+	  This transceiver supports high and full speed devices plus,
+	  in host mode, low speed.
 
 config FSL_USB2_OTG
 	bool "Freescale USB OTG Transceiver Driver"
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 09/10] usb: otg: trivial cleanups

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 241 bytes --]

On Tue, Dec 20, 2011 at 02:42:26AM +0200, Felipe Contreras wrote:
> Spaces to tabs, proper alignment, and start sentences as they should.
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

applied, thanks

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[PATCH 10/10] musb: cleanup kconfig

[Felipe Contreras]

The whole thing depends on USB_MUSB_HDRC, just add an 'if'.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 drivers/usb/musb/Kconfig |   14 ++++++++------
 1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/usb/musb/Kconfig b/drivers/usb/musb/Kconfig
index b0a46e4..0844b8f 100644
--- a/drivers/usb/musb/Kconfig
+++ b/drivers/usb/musb/Kconfig
@@ -31,9 +31,10 @@ config USB_MUSB_HDRC
 	  To compile this driver as a module, choose M here; the
 	  module will be called "musb-hdrc".
 
+if USB_MUSB_HDRC
+
 choice
 	prompt "Platform Glue Layer"
-	depends on USB_MUSB_HDRC
 
 config USB_MUSB_DAVINCI
 	tristate "DaVinci"
@@ -67,7 +68,6 @@ endchoice
 
 config MUSB_PIO_ONLY
 	bool 'Disable DMA (always use PIO)'
-	depends on USB_MUSB_HDRC
 	default USB_MUSB_TUSB6010 || USB_MUSB_DA8XX || USB_MUSB_AM35X
 	help
 	  All data is copied between memory and FIFO by the CPU.
@@ -80,30 +80,32 @@ config MUSB_PIO_ONLY
 
 config USB_UX500_DMA
 	bool
-	depends on USB_MUSB_HDRC && !MUSB_PIO_ONLY
+	depends on !MUSB_PIO_ONLY
 	default USB_MUSB_UX500
 	help
 	  Enable DMA transfers on UX500 platforms.
 
 config USB_INVENTRA_DMA
 	bool
-	depends on USB_MUSB_HDRC && !MUSB_PIO_ONLY
+	depends on !MUSB_PIO_ONLY
 	default USB_MUSB_OMAP2PLUS || USB_MUSB_BLACKFIN
 	help
 	  Enable DMA transfers using Mentor's engine.
 
 config USB_TI_CPPI_DMA
 	bool
-	depends on USB_MUSB_HDRC && !MUSB_PIO_ONLY
+	depends on !MUSB_PIO_ONLY
 	default USB_MUSB_DAVINCI
 	help
 	  Enable DMA transfers when TI CPPI DMA is available.
 
 config USB_TUSB_OMAP_DMA
 	bool
-	depends on USB_MUSB_HDRC && !MUSB_PIO_ONLY
+	depends on !MUSB_PIO_ONLY
 	depends on USB_MUSB_TUSB6010
 	depends on ARCH_OMAP
 	default y
 	help
 	  Enable DMA transfers on TUSB 6010 when OMAP DMA is available.
+
+endif # USB_MUSB_HDRC
-- 
1.7.8.rc1.14.g248db

Re: [PATCH 10/10] musb: cleanup kconfig

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 238 bytes --]

Hi,

On Tue, Dec 20, 2011 at 02:42:27AM +0200, Felipe Contreras wrote:
> The whole thing depends on USB_MUSB_HDRC, just add an 'if'.
> 
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

applied, thanks

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]