Re: [PATCH] usb: host: add Kconfig option for EHSET

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 13, 2013 at 10:07:58AM -0400, Alan Stern wrote:
> On Mon, 12 Aug 2013, Jack Pham wrote:
> 
> > commit 9841f37a1c ("usb: ehci: Add support for SINGLE_STEP_SET_FEATURE
> > test of EHSET") added additional code to the EHCI hub driver but it is
> > anticipated to only have a limited audience (e.g. embedded silicon
> > vendors and integrators). Avoid subjecting all EHCI (and in the future
> > maybe xHCI/OHCI, etc.) HCD users to code bloat by conditionally
> > compiling the EHSET-specific additions with a new Kconfig option,
> > CONFIG_USB_HCD_TEST_MODE.
> > 
> > Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> > Signed-off-by: Jack Pham <jackp@xxxxxxxxxxxxxx>
> 
> Quick work, thank you.
> 
> Greg, do you object to this new Kconfig option?

No, as long as:

> > +	  include other tests that require support from a HCD driver.
> > +
> > +	  If unsure, say N.
> 
> There should be something along the lines of: "This option is of
> interest only to developers who need to validate their USB hardware
> designs.  It is not needed for normal use."

This needs to be added, to help the distro people out in determining if
the option should be enabled or not (I'm guessing they will all turn it
off, right?)

thanks,

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



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux