lkml.org 
[lkml]   [2014]   [Feb]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] thinkpad_acpi: Fix inconsistent mute LED after resume
Hi,

this patch doesn't work. The sound is now muted after the resume (light is on).
But the userspace knows nothing about it. (kmix shows the volume as umuted)

Peter


On Thu, 13 Feb 2014 09:35:30 +0100
Takashi Iwai <tiwai@suse.de> wrote:

> At Wed, 12 Feb 2014 20:37:36 +0000,
> Matthew Garrett wrote:
> >
> > On Wed, 2014-02-12 at 16:32 +0100, Takashi Iwai wrote:
> > > The mute LED states have to be restored after resume.
> >
> > Oh, never mind, we're not doing this through the LED class, are we?
>
> Good point. It seems that leds_class has pm_ops but it's enabled only
> when a flag is set.
>
> I added the original bug reporter to Cc. (Don't know the name,
> Bugzilla shows only the email address.)
> Could you check the patch below works instead of the previous fix?
>
>
> Takashi
>
> ---
> diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
> index defb6afc1409..16edf75d9c00 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -4796,6 +4796,7 @@ static struct tpacpi_led_classdev tpacpi_led_thinklight = {
> .name = "tpacpi::thinklight",
> .brightness_set = &light_sysfs_set,
> .brightness_get = &light_sysfs_get,
> + .flags = LED_CORE_SUSPENDRESUME,
> }
> };
>


\
 
 \ /
  Last update: 2014-02-13 19:41    [W:0.041 / U:0.520 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site