Re: Re: [PATCH] ALSA: hda: generic: Always add LED controls

[Bernhard Seibold <mail@xxxxxxxxxxxxxxxxxxx>]

On Thu, Jan 04, 2024 at 05:41:48PM +0100, Takashi Iwai wrote:
> On Thu, 04 Jan 2024 16:58:38 +0100,
> Bernhard Seibold wrote:
> > 
> > LEDs for mute and microphone mute can be added to any system via the
> > audio-mute and audio-micmute triggers, e.g. via USB HID devices.
> > Therefore, add the LED controls unconditionally.
> 
> Hmm...  Won't your change lead to the sysfs entries always,
> i.e. creating sysfs entries even if the device has no LED at all?

This patch is not affecting create_mute_led_cdev, that function is still
only called for some special systems via fixups, to create the actual
LED classdevs that are present on there.

What I'm trying to change is to have the led-ctls always available. They
serve as input to the audio-mute and audio-micmute triggers, and these
triggers should work on any system with HD Audio, not just a few.

It would be even better to move the whole functionality into the sound
core so that it works with other drivers as well, but since yesterday was
the first time I even looked at sound code, I wouldn't even know where
to start.

Regard,
Bernhard