[alsa-devel] [PATCH] ALSA: hda/realtek: Enable HP amp and mute LED on HP Folio 9480m

Keith Packard keithp at keithp.com
Wed Jul 15 21:14:39 CEST 2015


Takashi Iwai <tiwai at suse.de> writes:

> Thanks!  The new patch looks good, but I think we don't have to use
> the headset code, as your case looks more like a headphone, not a
> combo headset that needs the special handling.  If so, the change can
> be reduced something like below.  Could you check whether this is
> enough?

Yes, that seems a lot simpler and works fine. I did remove the
led_power_filter setting; that doesn't seem relevant for GPIO-based
leds, and isn't necessary on my device in any case.

This patch is now otherwise essentially the same as yours, with the
addition of comments.

Thanks again for your help; it's always interesting to dive into some
different area of the kernel and see how it works.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ALSA-hda-realtek-Enable-HP-amp-and-mute-LED-on-HP-Fo.patch
Type: text/x-diff
Size: 3691 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20150715/c82fb74d/attachment-0001.bin>
-------------- next part --------------

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20150715/c82fb74d/attachment-0001.sig>


More information about the Alsa-devel mailing list