On Tue, May 13, 2014 at 10:19:27AM +0800, Peter Chen wrote:
> the user should only select it when the board supports HNP and SRP,
> it should NOT be selected if the board only supports dual-role
> switch through ID pin.
> 
> There is a discussion for it:
> http://marc.info/?l=linux-arm-kernel&m=139994896101516&w=2
> http://marc.info/?l=linux-usb&m=139994881701504&w=2
> 
> Signed-off-by: Peter Chen <peter.c...@freescale.com>
> ---
>  drivers/usb/core/Kconfig |   12 +++++++-----
>  1 files changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/usb/core/Kconfig b/drivers/usb/core/Kconfig
> index cb8e991..158048b 100644
> --- a/drivers/usb/core/Kconfig
> +++ b/drivers/usb/core/Kconfig
> @@ -54,13 +54,15 @@ config USB_OTG
>       default n
>       help
>         The most notable feature of USB OTG is support for a
> -       "Dual-Role" device, which can act as either a device
> -       or a host. The initial role is decided by the type of
> -       plug inserted and can be changed later when two dual
> -       role devices talk to each other.
> +       otg device, which can act as either a device or a host.
> +       The initial role is decided by the type of plug inserted
> +       and can be changed using HNP (Host Negotiation Protocol)
> +       later when two otg devices talk to each other.
>  
>         Select this only if your board has Mini-AB/Micro-AB
> -       connector.
> +       connector and supports HNP and SRP (Session Request Protocol).
> +       If the board only supports dual-role switch through ID
> +       pin, this configuration should not be selected.

That's really lame.  What happened to our "all modconfig should work"
goal?  Why can't you dynamically detect that this is not a config that
works and handle it?

And this is just the OTG "core", not any specific hardware, so why
should it matter?

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to