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 <[EMAIL PROTECTED]> 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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to