Re: [alsa-devel] [V2 PATCH] ALSA: hda - Enable mute/mic-mute LEDs for more Thinkpads with Conexant codec

[Henrique de Moraes Holschuh]

On Sat, 27 Jun 2015, Jan Kiszka wrote:
> > Please explain _in detail_ what you mean with "changing a power button to a
> > reset button by acessing the SSMS ACPI method in a X121e".
> > 
> > Are we trigering a bug somewhere that crashes the x121e and causes it to
> > reboot?
> 
> Well, there aren't much details to describe in this case: When this
> patch is applied and I press the power button, the device performs a
> hard reset. It doesn't reveal if the kernel crashes catastrophically,
> causing a triple fault or so, or if the firmware decides to reset the
> platform. Reverting the patch or preventing its effect via a quirk
> filter like I posted [1] makes the issue go away.

Ok, so we crashing the firmware, and on a box that is not likely to get
further updates.  Well, this is a pretty clear case: blacklist.

Since calling SSMS in the X121e is unsafe, it needs to be blacklisted in
thinkpad-acpi.  I will need dmidecode output for that (please XXX-out UUIDs
and serial numbers in the dmidecode output before you send it to me, but do
keep model numbers intact).

We can also have a blacklist in ALSA to avoid the whole symbol_request() for
something that will have to fail, of course.  I leave that for the ALSA
maintainers to decide.

That said, you blacklisted in ALSA by full codec->subsystem_id.  Which
thinkpads have the 0x17aa21ec subsystem ID on the codec?  Just the x121e?

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/