Hi Greg, On Mon, Nov 21, 2016 at 03:46:08PM +0100, Greg KH wrote: > > > > + > > > > +config TYPEC > > > > + tristate > > > > > > Hah, that says NOTHING about what this code is at all. > > > > Alone the class driver does nothing. Why would the user need to be > > aware of if it when selecting the Type-C drivers, and what can the > > user use that information for? > > If you see a blank Kconfig option, what are you supposed to do with it? > How do you know if you need to enable it or not? Are you just supposed > to guess? But you don't see anything when you are selecting the drivers and that is the point. You now can't select this separately. There is now no option for it. Why should we bother the user with this? The user is most likely only interested in the drivers and by selecting those the user will get the interface. The drivers will need to have the dependency to the class set correctly in any case. Thanks, -- heikki -- 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
