From: Zhang Rui <rui.zhang@intel.com>
To: Marek Vasut <marex@denx.de>
Cc: Corentin Chary <corentin.chary@gmail.com>,
platform-driver-x86@vger.kernel.org, joeyli <jlee@suse.com>,
Len Brown <len.brown@intel.com>,
pavel@denx.de
Subject: Re: [PATCH 2/2] ACER: Add support for ambient light sensor
Date: Wed, 06 Jun 2012 09:22:35 +0800 [thread overview]
Message-ID: <1338945755.1492.20.camel@rui.sh.intel.com> (raw)
In-Reply-To: <201206060055.59421.marex@denx.de>
On 三, 2012-06-06 at 00:55 +0200, Marek Vasut wrote:
> Dear Corentin Chary,
>
> [...]
>
> > >> This is a generic device, and should be handled by ACPI core
> > >> (Samsung laptop have this too). See "ACPI ALS" on linux-acpi.
> > >
> > > Saying this would be enough. Lee explained to me what this ACPI0008
> > > means, I'm quite new to the ACPI land so pardon my ignorance here. But
> > > it then boils down to Lee's question, why wasn't the patch for ACPI ALS
> > > that was already proposed accepted? And what can be done to get it in?
> >
> > Yep, I've seen Lee's message latter. Regarding ACPI ALS status, it was
> > forgotten for a long time, and recently Zhang Rui told that he will
> > take another look at it, but nothing sure. If you want to know more
> > about the status, you can do a quick search on gmane using "ACPI ALS"
> > query on linux.acpi
>
> Actually, going through the ACPI0008 patch, I see it introduced new sysfs class.
> Maybe we should rather integrate it with Linux-IIO these days?
>
yes.
please refer to this thread
http://marc.info/?l=linux-acpi&m=132060809120911&w=2
I do not have time to do this right now, and it will be great if you are
interested in introducing the ACPI ALS driver.
thanks,
rui
next prev parent reply other threads:[~2012-06-06 1:21 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-01 17:11 [PATCH 1/2] ACER: Add support for accelerometer sensor Marek Vasut
2012-06-01 17:11 ` [PATCH 2/2] ACER: Add support for ambient light sensor Marek Vasut
2012-06-04 9:08 ` joeyli
2012-06-04 9:29 ` Marek Vasut
2012-06-04 11:13 ` Corentin Chary
2012-06-04 17:14 ` Marek Vasut
2012-06-05 12:49 ` Corentin Chary
2012-06-05 13:06 ` Marek Vasut
2012-06-05 22:55 ` Marek Vasut
2012-06-06 1:22 ` Zhang Rui [this message]
2012-06-22 20:20 ` Marek Vasut
2012-06-04 8:23 ` [PATCH 1/2] ACER: Add support for accelerometer sensor joeyli
2012-06-05 22:52 ` Marek Vasut
2012-06-06 0:15 ` joeyli
2012-06-06 10:13 ` Marek Vasut
2012-06-26 18:41 ` Matthew Garrett
2012-06-26 18:52 ` Marek Vasut
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=1338945755.1492.20.camel@rui.sh.intel.com \
--to=rui.zhang@intel.com \
--cc=corentin.chary@gmail.com \
--cc=jlee@suse.com \
--cc=len.brown@intel.com \
--cc=marex@denx.de \
--cc=pavel@denx.de \
--cc=platform-driver-x86@vger.kernel.org \
/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