All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Zhang Rui <rui.zhang@intel.com>
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, "Martin Liška" <marxin.liska@gmail.com>
Subject: Re: [PATCH 2/2] ACER: Add support for ambient light sensor
Date: Fri, 22 Jun 2012 22:20:15 +0200	[thread overview]
Message-ID: <201206222220.15755.marex@denx.de> (raw)
In-Reply-To: <1338945755.1492.20.camel@rui.sh.intel.com>

Dear Zhang Rui,

> 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.

So, I finally raised from dead again ... sorry I didn't reply earlier. 
Apparently, one of my students is interested in fixing/writing such driver, 
therefore I'll land him my hardware and let him do it.

I CCed him, please treat him well.

Best regards,
Marek Vasut

  reply	other threads:[~2012-06-22 20:20 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
2012-06-22 20:20             ` Marek Vasut [this message]
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=201206222220.15755.marex@denx.de \
    --to=marex@denx.de \
    --cc=corentin.chary@gmail.com \
    --cc=jlee@suse.com \
    --cc=len.brown@intel.com \
    --cc=marxin.liska@gmail.com \
    --cc=pavel@denx.de \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=rui.zhang@intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.