[PATCH] ARM: OMAP2+: omap2plus_defconfig: EHCI driver is not stable, disable it

Munegowda, Keshava keshava_mgowda at ti.com
Thu Jul 12 04:33:40 EDT 2012


On Fri, Jul 6, 2012 at 11:50 PM, Kevin Hilman <khilman at ti.com> wrote:
> The EHCI driver is not stable enough to be enabled by default.  In v3.5,
> it has at least the following problems:
>
> - warning dump during bootup
> - hang during suspend
> - prevents CORE powerdomain from entering retention during idle (even
>   when no USB devices connected.)
>
> This demonstrates that this driver has not been thoroughly tested and
> therfore should not be enabled in the default defconfig.
>
> In addition, the problems above cause new PM regressions which need be
> addressed before this driver should be enabled in the default
> defconfig.
>
> Signed-off-by: Kevin Hilman <khilman at ti.com>
> ---
> Tony, this applies to your current fixes branch.   Please queue up for v3.5-rc
> so this PM regression is fixed in v3.5.  Thanks.
>
>  arch/arm/configs/omap2plus_defconfig |    1 -
>  1 file changed, 1 deletion(-)
>
> 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
> --
> 1.7.9.2
>


hi Kevin
   This patch is good. But , in  my earlier patch I was disabling the
dependent configuration of ehci too.

- CONFIG_USB=y
                - removes the usb host  ( EHCI is the only host),
hence keeping it enabled with ehci does not make sense for
                        omap2 defconfig

-CONFIG_USB_DEBUG=y
                   - Enables the debug messages for usb host

-CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
-CONFIG_USB_DEVICEFS=y
                    Announces the new devices attached to ehci; if usb
host it self is not there, then we dont need it.

-CONFIG_USB_SUSPEND=y
                   - suspend resume of usb host; not required if ehci
itself is not existing

-CONFIG_USB_MON=y
                   - usb monitor

-CONFIG_USB_WDM=y
-CONFIG_USB_STORAGE=y
-CONFIG_USB_LIBUSUAL=y
-CONFIG_USB_TEST=y
              - usb storage and test ; again dependent on ehci only in
omap2plusdefconfig.

for omap build, if you disabling ehci , its good have not include
these modules; yet including these modules does not harm anything.

regards
keshava



More information about the linux-arm-kernel mailing list