On Sat, Oct 27, 2018 at 12:36 PM Joe Perches <joe@xxxxxxxxxxx> wrote: > > In fairness, it seems many of the HID drivers do exactly that > and this could have been a "copy from example" addition. Interesting, and I think you're right. I wonder why I haven't noticed this before. Some of those might be hidden by other dependencies - what I do is to just check how "make oldconfig" changes my normal fairly minimal configuration, and then complain when I notice that somebody enables some odd new hardware. Anyway, those other "!EXPERT" defaults look pretty questionable too. HID itself beind default y and the HID_GENERIC driver defaulting on seems eminently sane. And HID_LOGITECH might fall under both "very common" and "enables other config options". But even then, the !EXPERT seems an odd choice. Why not just 'y' for the truly common cases? I wonder if there is some truly old historical legacy there, ie the old PC keyboard support would have been configurable out only for expert users to avoid errors, and maybe the HID Kconfig file started getting ideas from that... Linus