On Wed, 14 Oct 2020 09:54:59 +0200, Pavel Machek wrote: > > Hi! > > > >>> I.e.: it looks like I will lose some funcionality when I disable > > >>> SND_HDA_CODEC_REALTEK. > > >> > > >> OK. At present you can't have it both ways, i.e., SND_HDA_CODEC_REALTEK > > >> with no LEDS. That driver apparently wants LEDS. > > > > > > Thanks but why have I gone for years without LEDS? > > > I do not need LEDS, I do not want LEDS, I do not have LEDS (that are > > > visible, usable, etc). > > > > > > Please make this selectable instead of forcing more bulk into my > >> kernel. > > LED core is not that big, and this avoided some rather "interesting" > hacks IIRC. If Udo wants more config complexity, lets first make him > measure the benefits, second submit a patch. > > But I'd suggest to just live with it. > > And yes, we should probably get rid of "CONFIG_NEW_LEDS" symbol. That > one is actually useless. IIRC, this was needed for the reverse selection of CONFIG_LEDS_CLASS and co. But if it's really useless, I'll happily delete it. thanks, Takashi
