On Thu, May 31, 2018 at 4:43 AM, Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: >> + depends on ASUS_WMI > > I'm not sure it's a good idea. > > Rather you may use ifdef inside header which also provides a stub for > the function. But be careful with corner cases, like ASUS_WMI=m && > HID_ASUS=y. What do you see as wrong with it? The driver already depends on e.g. LEDS_CLASS for the functionality that it sometimes uses there. In my opinion the ideal solution to the corner cases you mention is the Kconfig "depends on" which is exactly what the patch is proposing... Daniel
