All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Corentin Chary <corentin.chary@gmail.com>
Cc: platform-driver-x86@vger.kernel.org, joeyli <jlee@suse.com>,
	"Zhang, Rui" <rui.zhang@intel.com>,
	Len Brown <len.brown@intel.com>,
	pavel@denx.de
Subject: Re: [PATCH 2/2] ACER: Add support for ambient light sensor
Date: Wed, 6 Jun 2012 00:55:59 +0200	[thread overview]
Message-ID: <201206060055.59421.marex@denx.de> (raw)
In-Reply-To: <CAHR064jYAFd06VYZvi90xZCyJCKEmEdKkmrim9mEn584CqRmzw@mail.gmail.com>

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?

Best regards,
Marek Vasut

  parent reply	other threads:[~2012-06-05 22:56 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 [this message]
2012-06-06  1:22           ` Zhang Rui
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=201206060055.59421.marex@denx.de \
    --to=marex@denx.de \
    --cc=corentin.chary@gmail.com \
    --cc=jlee@suse.com \
    --cc=len.brown@intel.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.