Re: [PATCH] ACPI: add "Fujitsu laptop extras" module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I was the person who cleaned up the code a couple years back.  I don't own a
Fujitsu notebook anymore, but here's some information from what I remember.

Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote:
> > +static int acpi_fuj02b1_get_volume_state(void *data)
> 
> This probably wants to be exported as an ALSA mixer rather than a 
> special device in /proc.

My S6010 had hardware volume control (it was separate from what was ALSA
controlled).  I'm not sure if this is still the case with the Fujitsu's S and
P-series notebooks.
 
> > +static int acpi_fuj02b1_get_pointer_state(void *data)
> 
> Less sure about this one, but I'm not sure exactly what it's meant to 
> do - control whether the mouse pointer is enabled?

I never used this, but I think it gives the user the ability to disable the
built-in touchpad.  Probably driver dependent.

> > +				acpi_bus_generate_event(device, event, (u32) fuj02b1->volume_level |
> (fuj02b1->mute_state * 0x1000000) | 0x10000000);
> 
> ALSA is capable of generating events when the mixer volume changes, so I 
> don't think there's a need to explicitly generate an event here 
> (assuming you move the mixer over to ALSA)

Since the buttons control the hardware volume control, I believe the events were
more for displaying OSD volume status.

Adrian
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux