Hi Dmitry, On Mon, 24 Apr 2017 10:09:53 -0700, Dmitry Torokhov wrote: > On Mon, Apr 24, 2017 at 01:56:06PM +0200, Krzysztof Kozlowski wrote: > > >> > On Mon, 24 Apr 2017 10:00:32 +0200, Krzysztof Kozlowski wrote: > > >> > > On Mon, Apr 24, 2017 at 9:42 AM, Jean Delvare <jdelvare@xxxxxxx> wrote: > > >> > > > The tm2-touchkey driver is only useful on specific platforms. Add the > > >> > > > missing hardware dependency so that the driver is not proposed on > > >> > > > systems where the device does not exist. > > >> > > > > >> > > Although the device exists in only two upstreamed Exynos boards but > > >> > > there is no hardware dependency on Exynos. The hardware does not > > >> > > depend on Exynos. > > >> > (...) > > Workaround might be using default = N, unless ARCH_EXYNOS. Something like: > > default y if ARCH_EXYNOS > > This would strongly imply that the device is available on all Exynos > boards. I am fairly certain that our Chromebooks, for example, do not > have it. > > Besides, I do not see what is wrong with driver being offered on a > platform that does not have it. It makes the life of distribution kernel maintainers a lot harder, by asking them questions that are irrelevant. They may eventually figure it out and say "no", but it costs them time to investigate each option. Worst case they will accidentally say yes and bloat their kernel. > This happens all the time. That doesn't make it right. The trend is changing, and you should embrace that change. What was acceptable 10 days ago when we had 7000 kernel options no longer works today with 17000. > Consider I2C stuff: Trying to make it personal? ;-) > config I2C_I801 > tristate "Intel 82801 (ICH/PCH)" > depends on PCI > select CHECK_SIGNATURE if X86 && DMI > select I2C_SMBUS > > This is an X86 device, but is offered everywhere where we have PCI. In fact the 82801 family of south bridges is not limited to X86, it can be found on IA64 hardware as well. It was discussed here: https://patchwork.ozlabs.org/patch/635865/ and as you can see I would have liked the hardware dependency to be added, it's Andy Shevchenko who declined :-( -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
