From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753929AbbCIUiu (ORCPT ); Mon, 9 Mar 2015 16:38:50 -0400 Received: from mail-wg0-f43.google.com ([74.125.82.43]:36105 "EHLO mail-wg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752475AbbCIUir (ORCPT ); Mon, 9 Mar 2015 16:38:47 -0400 MIME-Version: 1.0 In-Reply-To: <54FDC005.2000905@kernel.org> References: <1421921412-7710-1-git-send-email-daniel.baluta@intel.com> <54C4C510.5090306@kernel.org> <000d01d0551b$9d424bb0$d7c6e310$@capellamicro.com> <54FDC005.2000905@kernel.org> Date: Mon, 9 Mar 2015 22:38:45 +0200 X-Google-Sender-Auth: npQOF9nYsUQ8l6CnpVuJXTe8BOA Message-ID: Subject: Re: [PATCH] iio: light: Add support for Capella CM3323 color/light sensor From: Daniel Baluta To: Jonathan Cameron Cc: Kevin Tsai , Daniel Baluta , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald , "linux-iio@vger.kernel.org" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 9, 2015 at 5:45 PM, Jonathan Cameron wrote: > On 02/03/15 19:03, Kevin Tsai wrote: >> Hi Daniel, >> >> Ambient light sensor is trying to match the brightness sensitivity of human visual system. Please see the following links: >> http://en.wikipedia.org/wiki/Color_vision#mediaviewer/File:Eyesensitivity.svg >> http://en.wikipedia.org/wiki/Color_vision >> >> You can compare the spectrum with the datasheet. The green channel is matched with ALS spectrum. > It's close (ish) to the right measurement, but it isn't the right measurement. As such > I'd argue we leave it up to userspace to make the jump and use Green if a true estimate > if illuminance isn't available, but don't suggest it is available from the kernel. Ok, I will respin the patch with this in mind. thanks, Daniel.