linux-kernel - Re: [PATCH] Fujitsu U810 keyboard light support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <200806092341.m59NfOFu022668@turbo.physics.adelaide.edu.au>
Date:	Tue, 10 Jun 2008 09:11:24 +0930 (CST)
From:	Jonathan Woithe <jwoithe@...sics.adelaide.edu.au>
To:	jules@...ic.cs-bristol.org.uk (Julian Brown)
Cc:	linux-kernel@...r.kernel.org,
	jwoithe@...sics.adelaide.edu.au (Jonathan Woithe)
Subject: Re: [PATCH] Fujitsu U810 keyboard light support

Hi Julian

Thanks for your patch.

> This patch contains support for the Fujitsu U810 mini-laptops'
> keyboard-illuminating "headlights". Information on how to control these
> was reverse-engineered from the ACPI DSDT and from disassembling the
> Windows driver for the device (i.e. it's not 100% certain that this
> patch won't make your laptop explode, but it works for me. A more likely
> problem is that I might have broken support for other Fujitsu laptops,
> but I can't test that).
> 
> The patched fujitsu-laptop module creates two entities in the /sys
> hierarchy:
> 
>   /sys/devices/platform/fujitsu-laptop/kbd_light
>   /sys/class/leds/kbd_light/*
> 
> So that then, the lights can be controlled by reading/writing to these
> files. (Probably not all the LED class things are relevant, though).
> 
> The fujitsu-laptop module before patching only uses the FUJ02B1 ACPI "HID"
> (namespace? Identifier?), but the keyboard LEDs (as well as several other
> possibly-interesting things) hang off the FUJ02E3 ("FEXT") HID. I'm not
> sure if adding the latter to the existing driver (vs. creating a new one)
> was the right thing to do, but I think it's probably sensible.
> 
> Any comments/suggestions?

There is a later version of the fujitsu-laptop module floating around the
linux-acpi list (v0.4.1) which I have submitted for inclusion in the next
merge window.  This version adds support additional backlight controls used
on some Fujitsu laptops, and for some of the fujitsu hotkeys.  The latter is
done using the FUJ02E3 HID.  It has been extensively tested on the fujitsu
laptops we currently have access to.  As it stands your current patch will
clash horribly with the new fujitsu-laptop version.

IMHO the easiest way forward would be for you to rebase your patch against
this newer fujitsu-laptop version and see where things go from there.  As
maintainer of the fujitsu-laptop module I would rather get the current
well-tested hotkeys/backlight code in and use that as the base for your
"headlights" patch.  That way we can build on the work that's already been
done rather than reinventing the wheel (so to speak).  You'll find the patch
on the linux-acpi list - contact me off-list if you can't locate it.

FYI the fujitsu-laptop development generally happens through the linux-acpi
subsystem/list.

Regards
  jonathan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/