* [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
@ 2012-07-06 11:49 Keshava Munegowda
2012-07-06 11:55 ` Munegowda, Keshava
2012-07-06 14:13 ` Kevin Hilman
0 siblings, 2 replies; 10+ messages in thread
From: Keshava Munegowda @ 2012-07-06 11:49 UTC (permalink / raw)
To: linux-omap, linux-usb; +Cc: Keshava Munegowda, balbi, sameo, parthab
The usb host is disabled in the omap2 build; This is because
usb host is causing the retention to break in cpu idle.
Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
---
arch/arm/configs/omap2plus_defconfig | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
index 9854ff4..7b32da6 100644
--- a/arch/arm/configs/omap2plus_defconfig
+++ b/arch/arm/configs/omap2plus_defconfig
@@ -170,17 +170,6 @@ CONFIG_SND_USB_AUDIO=m
CONFIG_SND_SOC=m
CONFIG_SND_OMAP_SOC=m
CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m
-CONFIG_USB=y
-CONFIG_USB_DEBUG=y
-CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
-CONFIG_USB_DEVICEFS=y
-CONFIG_USB_SUSPEND=y
-CONFIG_USB_MON=y
-CONFIG_USB_EHCI_HCD=y
-CONFIG_USB_WDM=y
-CONFIG_USB_STORAGE=y
-CONFIG_USB_LIBUSUAL=y
-CONFIG_USB_TEST=y
CONFIG_USB_GADGET=y
CONFIG_USB_GADGET_DEBUG=y
CONFIG_USB_GADGET_DEBUG_FILES=y
--
1.7.9.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 11:49 [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig Keshava Munegowda
@ 2012-07-06 11:55 ` Munegowda, Keshava
2012-07-06 11:59 ` Shilimkar, Santosh
2012-07-06 14:13 ` Kevin Hilman
1 sibling, 1 reply; 10+ messages in thread
From: Munegowda, Keshava @ 2012-07-06 11:55 UTC (permalink / raw)
To: linux-omap, Partha Basak, sameo; +Cc: Keshava Munegowda, balbi, Kevin Hilman
On Fri, Jul 6, 2012 at 5:19 PM, Keshava Munegowda <keshava_mgowda@ti.com> wrote:
> The usb host is disabled in the omap2 build; This is because
> usb host is causing the retention to break in cpu idle.
>
> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
> ---
> arch/arm/configs/omap2plus_defconfig | 11 -----------
> 1 file changed, 11 deletions(-)
>
> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> index 9854ff4..7b32da6 100644
> --- a/arch/arm/configs/omap2plus_defconfig
> +++ b/arch/arm/configs/omap2plus_defconfig
> @@ -170,17 +170,6 @@ CONFIG_SND_USB_AUDIO=m
> CONFIG_SND_SOC=m
> CONFIG_SND_OMAP_SOC=m
> CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m
> -CONFIG_USB=y
> -CONFIG_USB_DEBUG=y
> -CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
> -CONFIG_USB_DEVICEFS=y
> -CONFIG_USB_SUSPEND=y
> -CONFIG_USB_MON=y
> -CONFIG_USB_EHCI_HCD=y
> -CONFIG_USB_WDM=y
> -CONFIG_USB_STORAGE=y
> -CONFIG_USB_LIBUSUAL=y
> -CONFIG_USB_TEST=y
> CONFIG_USB_GADGET=y
> CONFIG_USB_GADGET_DEBUG=y
> CONFIG_USB_GADGET_DEBUG_FILES=y
> --
> 1.7.9.5
>
Sameo
please merge this patch ; this makes the core retention to work in
omap2plus_defconfig
regards
keshava
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 11:55 ` Munegowda, Keshava
@ 2012-07-06 11:59 ` Shilimkar, Santosh
2012-07-06 12:01 ` Munegowda, Keshava
0 siblings, 1 reply; 10+ messages in thread
From: Shilimkar, Santosh @ 2012-07-06 11:59 UTC (permalink / raw)
To: Munegowda, Keshava; +Cc: linux-omap, Partha Basak, sameo, balbi, Kevin Hilman
On Fri, Jul 6, 2012 at 5:25 PM, Munegowda, Keshava
<keshava_mgowda@ti.com> wrote:
> On Fri, Jul 6, 2012 at 5:19 PM, Keshava Munegowda <keshava_mgowda@ti.com> wrote:
>> The usb host is disabled in the omap2 build; This is because
>> usb host is causing the retention to break in cpu idle.
>>
>> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
>> ---
>> arch/arm/configs/omap2plus_defconfig | 11 -----------
>> 1 file changed, 11 deletions(-)
>>
>> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
>> index 9854ff4..7b32da6 100644
>> --- a/arch/arm/configs/omap2plus_defconfig
>> +++ b/arch/arm/configs/omap2plus_defconfig
>> @@ -170,17 +170,6 @@ CONFIG_SND_USB_AUDIO=m
>> CONFIG_SND_SOC=m
>> CONFIG_SND_OMAP_SOC=m
>> CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m
>> -CONFIG_USB=y
>> -CONFIG_USB_DEBUG=y
>> -CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
>> -CONFIG_USB_DEVICEFS=y
>> -CONFIG_USB_SUSPEND=y
>> -CONFIG_USB_MON=y
>> -CONFIG_USB_EHCI_HCD=y
>> -CONFIG_USB_WDM=y
>> -CONFIG_USB_STORAGE=y
>> -CONFIG_USB_LIBUSUAL=y
>> -CONFIG_USB_TEST=y
>> CONFIG_USB_GADGET=y
>> CONFIG_USB_GADGET_DEBUG=y
>> CONFIG_USB_GADGET_DEBUG_FILES=y
>> --
>> 1.7.9.5
>>
>
> Sameo
> please merge this patch ; this makes the core retention to work in
> omap2plus_defconfig
>
If at all this patch has to be merged, it should be via Tony's tree,
otherwise you
might have merge conflicts.
Regards
Santosh
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 11:59 ` Shilimkar, Santosh
@ 2012-07-06 12:01 ` Munegowda, Keshava
2012-07-06 12:07 ` Tony Lindgren
0 siblings, 1 reply; 10+ messages in thread
From: Munegowda, Keshava @ 2012-07-06 12:01 UTC (permalink / raw)
To: Shilimkar, Santosh, tony
Cc: linux-omap, Partha Basak, sameo, balbi, Kevin Hilman
On Fri, Jul 6, 2012 at 5:29 PM, Shilimkar, Santosh
<santosh.shilimkar@ti.com> wrote:
> On Fri, Jul 6, 2012 at 5:25 PM, Munegowda, Keshava
> <keshava_mgowda@ti.com> wrote:
>> On Fri, Jul 6, 2012 at 5:19 PM, Keshava Munegowda <keshava_mgowda@ti.com> wrote:
>>> The usb host is disabled in the omap2 build; This is because
>>> usb host is causing the retention to break in cpu idle.
>>>
>>> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
>>> ---
>>> arch/arm/configs/omap2plus_defconfig | 11 -----------
>>> 1 file changed, 11 deletions(-)
>>>
>>> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
>>> index 9854ff4..7b32da6 100644
>>> --- a/arch/arm/configs/omap2plus_defconfig
>>> +++ b/arch/arm/configs/omap2plus_defconfig
>>> @@ -170,17 +170,6 @@ CONFIG_SND_USB_AUDIO=m
>>> CONFIG_SND_SOC=m
>>> CONFIG_SND_OMAP_SOC=m
>>> CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m
>>> -CONFIG_USB=y
>>> -CONFIG_USB_DEBUG=y
>>> -CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
>>> -CONFIG_USB_DEVICEFS=y
>>> -CONFIG_USB_SUSPEND=y
>>> -CONFIG_USB_MON=y
>>> -CONFIG_USB_EHCI_HCD=y
>>> -CONFIG_USB_WDM=y
>>> -CONFIG_USB_STORAGE=y
>>> -CONFIG_USB_LIBUSUAL=y
>>> -CONFIG_USB_TEST=y
>>> CONFIG_USB_GADGET=y
>>> CONFIG_USB_GADGET_DEBUG=y
>>> CONFIG_USB_GADGET_DEBUG_FILES=y
>>> --
>>> 1.7.9.5
>>>
>>
>> Sameo
>> please merge this patch ; this makes the core retention to work in
>> omap2plus_defconfig
>>
> If at all this patch has to be merged, it should be via Tony's tree,
> otherwise you
> might have merge conflicts.
>
> Regards
> Santosh
Thanks Santosh
looping Tony........
regards
keshava
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 12:01 ` Munegowda, Keshava
@ 2012-07-06 12:07 ` Tony Lindgren
2012-07-06 12:25 ` Munegowda, Keshava
0 siblings, 1 reply; 10+ messages in thread
From: Tony Lindgren @ 2012-07-06 12:07 UTC (permalink / raw)
To: Munegowda, Keshava
Cc: Shilimkar, Santosh, linux-omap, Partha Basak, sameo, balbi,
Kevin Hilman
* Munegowda, Keshava <keshava_mgowda@ti.com> [120706 05:06]:
> On Fri, Jul 6, 2012 at 5:29 PM, Shilimkar, Santosh
> <santosh.shilimkar@ti.com> wrote:
> > On Fri, Jul 6, 2012 at 5:25 PM, Munegowda, Keshava
> > <keshava_mgowda@ti.com> wrote:
> >> On Fri, Jul 6, 2012 at 5:19 PM, Keshava Munegowda <keshava_mgowda@ti.com> wrote:
> >>> The usb host is disabled in the omap2 build; This is because
> >>> usb host is causing the retention to break in cpu idle.
...
> >> Sameo
> >> please merge this patch ; this makes the core retention to work in
> >> omap2plus_defconfig
> >>
> > If at all this patch has to be merged, it should be via Tony's tree,
> > otherwise you
> > might have merge conflicts.
I think it's best to fix things properly, even if it means that it will
take longer and go into next kernel version.
Tony
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 12:07 ` Tony Lindgren
@ 2012-07-06 12:25 ` Munegowda, Keshava
2012-07-06 12:39 ` Tony Lindgren
0 siblings, 1 reply; 10+ messages in thread
From: Munegowda, Keshava @ 2012-07-06 12:25 UTC (permalink / raw)
To: Tony Lindgren
Cc: Shilimkar, Santosh, linux-omap, Partha Basak, sameo, balbi,
Kevin Hilman
On Fri, Jul 6, 2012 at 5:37 PM, Tony Lindgren <tony@atomide.com> wrote:
> * Munegowda, Keshava <keshava_mgowda@ti.com> [120706 05:06]:
>> On Fri, Jul 6, 2012 at 5:29 PM, Shilimkar, Santosh
>> <santosh.shilimkar@ti.com> wrote:
>> > On Fri, Jul 6, 2012 at 5:25 PM, Munegowda, Keshava
>> > <keshava_mgowda@ti.com> wrote:
>> >> On Fri, Jul 6, 2012 at 5:19 PM, Keshava Munegowda <keshava_mgowda@ti.com> wrote:
>> >>> The usb host is disabled in the omap2 build; This is because
>> >>> usb host is causing the retention to break in cpu idle.
> ...
>> >> Sameo
>> >> please merge this patch ; this makes the core retention to work in
>> >> omap2plus_defconfig
>> >>
>> > If at all this patch has to be merged, it should be via Tony's tree,
>> > otherwise you
>> > might have merge conflicts.
>
> I think it's best to fix things properly, even if it means that it will
> take longer and go into next kernel version.
>
> Tony
Hi tony
This requires ehci remote wakeup feature to be implemented, which
will take at least to 3 months to
get reviewed in opensource.
hence for now kevin is also agreed to disable it by default;
regards
keshava
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 12:25 ` Munegowda, Keshava
@ 2012-07-06 12:39 ` Tony Lindgren
2012-07-06 13:34 ` Munegowda, Keshava
0 siblings, 1 reply; 10+ messages in thread
From: Tony Lindgren @ 2012-07-06 12:39 UTC (permalink / raw)
To: Munegowda, Keshava
Cc: Shilimkar, Santosh, linux-omap, Partha Basak, sameo, balbi,
Kevin Hilman
* Munegowda, Keshava <keshava_mgowda@ti.com> [120706 05:30]:
> On Fri, Jul 6, 2012 at 5:37 PM, Tony Lindgren <tony@atomide.com> wrote:
> >
> > I think it's best to fix things properly, even if it means that it will
> > take longer and go into next kernel version.
>
> This requires ehci remote wakeup feature to be implemented, which
> will take at least to 3 months to
> get reviewed in opensource.
> hence for now kevin is also agreed to disable it by default;
Why does echi need a "remote wakeup" feature? If you're talking about
echi wake-up events, I don't see how that would affect core retention.
Tony
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 12:39 ` Tony Lindgren
@ 2012-07-06 13:34 ` Munegowda, Keshava
0 siblings, 0 replies; 10+ messages in thread
From: Munegowda, Keshava @ 2012-07-06 13:34 UTC (permalink / raw)
To: Tony Lindgren
Cc: Shilimkar, Santosh, linux-omap, Partha Basak, sameo, balbi,
Kevin Hilman
On Fri, Jul 6, 2012 at 6:09 PM, Tony Lindgren <tony@atomide.com> wrote:
> * Munegowda, Keshava <keshava_mgowda@ti.com> [120706 05:30]:
>> On Fri, Jul 6, 2012 at 5:37 PM, Tony Lindgren <tony@atomide.com> wrote:
>> >
>> > I think it's best to fix things properly, even if it means that it will
>> > take longer and go into next kernel version.
>>
>> This requires ehci remote wakeup feature to be implemented, which
>> will take at least to 3 months to
>> get reviewed in opensource.
>> hence for now kevin is also agreed to disable it by default;
>
> Why does echi need a "remote wakeup" feature? If you're talking about
> echi wake-up events, I don't see how that would affect core retention.
Hi Tony
the cpu should hit the retention in idle; in this case, up on
usb bus suspend ( no device connected) , if I cut the clock, then later
device detection happen through I/O wakeup, so implementing the
ehci remote wakeup using I/O chain handler ensure the retention in idle too.
now, with the existing ehci driver in mainline, whether device connected or not
the clocks are not cut and prevents the retention in idle.
so, I need to implement the ehci clock gating mechanism through I/O interrupts
in bus spend /resume . This activity will take some time.
regards
keshava
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig
2012-07-06 11:49 [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig Keshava Munegowda
2012-07-06 11:55 ` Munegowda, Keshava
@ 2012-07-06 14:13 ` Kevin Hilman
[not found] ` <87d349c70t.fsf-l0cyMroinI0@public.gmane.org>
1 sibling, 1 reply; 10+ messages in thread
From: Kevin Hilman @ 2012-07-06 14:13 UTC (permalink / raw)
To: Keshava Munegowda; +Cc: linux-omap, linux-usb, balbi, sameo, parthab
Keshava Munegowda <keshava_mgowda@ti.com> writes:
> The usb host is disabled in the omap2 build; This is because
> usb host is causing the retention to break in cpu idle.
... and causes warnings during boot, and hangs in suspend, can't suspend
using NFSroot, etc. etc.
Thank you for disabling this by default, But I don't think this fix
is targetted enough (more below.)
Tony, when v3.5 came out, I asked them to fix this properly, but nothing
was done. So I've resorted to requesting that this feature be disabled
by default since it introduced PM regressions. Of course, I'd like to
see this fixed properly instead, but we've run out of time for v3.5 and
IMO, this PM regression needs to be fixed for v3.5. See my more
targetted patches below.
> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com>
> ---
> arch/arm/configs/omap2plus_defconfig | 11 -----------
> 1 file changed, 11 deletions(-)
>
> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> index 9854ff4..7b32da6 100644
> --- a/arch/arm/configs/omap2plus_defconfig
> +++ b/arch/arm/configs/omap2plus_defconfig
> @@ -170,17 +170,6 @@ CONFIG_SND_USB_AUDIO=m
> CONFIG_SND_SOC=m
> CONFIG_SND_OMAP_SOC=m
> CONFIG_SND_OMAP_SOC_OMAP3_PANDORA=m
> -CONFIG_USB=y
> -CONFIG_USB_DEBUG=y
> -CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
> -CONFIG_USB_DEVICEFS=y
> -CONFIG_USB_SUSPEND=y
> -CONFIG_USB_MON=y
> -CONFIG_USB_EHCI_HCD=y
> -CONFIG_USB_WDM=y
> -CONFIG_USB_STORAGE=y
> -CONFIG_USB_LIBUSUAL=y
> -CONFIG_USB_TEST=y
Why disable all of these? With that, it's not terribly clear what is
going on, or why.
Instead, what you need is a targetted fix to just disable the problem
driver by default:
Either this:
diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
index 9854ff4..11828e6 100644
--- a/arch/arm/configs/omap2plus_defconfig
+++ b/arch/arm/configs/omap2plus_defconfig
@@ -176,7 +176,6 @@ CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
CONFIG_USB_DEVICEFS=y
CONFIG_USB_SUSPEND=y
CONFIG_USB_MON=y
-CONFIG_USB_EHCI_HCD=y
CONFIG_USB_WDM=y
CONFIG_USB_STORAGE=y
CONFIG_USB_LIBUSUAL=y
or this:
diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
index e129c82..3747826 100644
--- a/drivers/mfd/Kconfig
+++ b/drivers/mfd/Kconfig
@@ -825,7 +825,7 @@ config MFD_WL1273_CORE
config MFD_OMAP_USB_HOST
bool "Support OMAP USBHS core driver"
depends on USB_EHCI_HCD_OMAP || USB_OHCI_HCD_OMAP3
- default y
+ default n
help
This is the core driver for the OAMP EHCI and OHCI drivers.
This MFD driver does the required setup functionalities for
Will have the inteded affect with the benefit of being much clearer
about what is actually being disabled, and why.
Kevin
^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-07-06 18:11 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-06 11:49 [PATCH] OMAP: config : disable the usb host configuration in omap2plus_defconfig Keshava Munegowda
2012-07-06 11:55 ` Munegowda, Keshava
2012-07-06 11:59 ` Shilimkar, Santosh
2012-07-06 12:01 ` Munegowda, Keshava
2012-07-06 12:07 ` Tony Lindgren
2012-07-06 12:25 ` Munegowda, Keshava
2012-07-06 12:39 ` Tony Lindgren
2012-07-06 13:34 ` Munegowda, Keshava
2012-07-06 14:13 ` Kevin Hilman
[not found] ` <87d349c70t.fsf-l0cyMroinI0@public.gmane.org>
2012-07-06 18:11 ` Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).