From: Emmanuel Fleury <fleury@cs.aau.dk>
To: Stelian Pop <stelian@popies.net>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
acpi-devel@lists.sourceforge.net
Subject: Re: [PATCH, new ACPI driver] new sony_acpi driver
Date: Wed, 16 Feb 2005 20:12:44 +0100 [thread overview]
Message-ID: <42139B2C.6040709@cs.aau.dk> (raw)
In-Reply-To: <20050211113636.GI3263@crusoe.alcove-fr>
Stelian Pop wrote:
> Based on feedback from Jean Delvare and Pekka Enberg, here is an
> updated version.
>
> Changes from the previous version include:
> - do not initialize to 0 a static variable
> - trim to 80 columns
> - do not do spurious void * casts
> - use c99 style struct initialization
> - use simple_strtoul instead of sscanf
> - move documentation to new directory Documentation/acpi
> - name the file 'brightness' instead of 'brt'
Works straight for me (on a Vaio PCG-C1MZX), I can adjust brightness by
feeding the proc/ interface.
I have to try the debug mode.
Regards
--
Emmanuel Fleury
Computer Science Department, | Office: B1-201
Aalborg University, | Phone: +45 96 35 72 23
Fredriks Bajersvej 7E, | Fax: +45 98 15 98 89
9220 Aalborg East, Denmark | Email: fleury@cs.aau.dk
next prev parent reply other threads:[~2005-02-16 19:37 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-02-10 16:18 [PATCH, new ACPI driver] new sony_acpi driver Stelian Pop
2005-02-10 19:39 ` [ACPI] " Bruno Ducrot
2005-02-11 9:16 ` Stelian Pop
2005-02-11 10:30 ` Jean Delvare
2005-02-11 11:17 ` Stelian Pop
2005-02-11 11:05 ` Pekka Enberg
2005-02-11 11:36 ` Stelian Pop
2005-02-11 12:02 ` Pekka Enberg
2005-02-12 13:21 ` Jean Delvare
2005-02-14 10:07 ` Stelian Pop
2005-02-14 12:13 ` Jean Delvare
2005-02-14 12:38 ` Stelian Pop
2005-02-14 18:42 ` Jean Delvare
2005-02-16 15:39 ` Stelian Pop
2005-02-18 18:38 ` Jean Delvare
2005-02-16 19:12 ` Emmanuel Fleury [this message]
2005-02-14 10:53 ` Matthew Garrett
2005-02-14 10:58 ` Stelian Pop
2005-02-14 20:32 ` Vojtech Pavlik
2005-02-15 16:14 ` Romano Giannetti
2005-02-16 14:41 ` Stelian Pop
2005-02-16 15:40 ` Romano Giannetti
2005-02-16 23:18 ` [ACPI] " Pavel Machek
2005-02-15 15:30 ` [ACPI] " Len Brown
2005-02-15 15:39 ` Stelian Pop
2005-02-16 19:40 ` Bruno Ducrot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=42139B2C.6040709@cs.aau.dk \
--to=fleury@cs.aau.dk \
--cc=acpi-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=stelian@popies.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox