From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7EAB444C7C; Sat, 22 Feb 2025 17:25:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740245149; cv=none; b=ZDPiR2IHGFsov9nXrQkSxB6sPLb37/d1Rg8qoWDlg+MxRVI5Wy4Zqy5h78HrUPfkJD5+IvQm2mMip3SoXS94ErmFGsz4tuEE2wge9WZdYO584lyBhpPZ1d4A/razd7VySjMveupETu8psoIKESNUIHmMlff3FaUjfJhFzRJCpCY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740245149; c=relaxed/simple; bh=fx3DRTGRkmKbnCXlKZ+ZwK/6h3hhdblLg2AxFF2wD44=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=hKYbDryJVhN9wt2PIdtOaG/YueoV9QhPSYtO0pboY4Zpv4rLkXDxjiPvz1xnqGIHujyLRRk6uRf166jBosaQ1yE9PfhhtU4X1Ijbu46Z46HdFlF+D4qMFs7jJ+16v3It9nyB38SNVY9O0bSd1sgqVkX3sNr4ViGq61kJUNq2R10= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=AW/05HQj; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AW/05HQj" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 42F9EC4CED1; Sat, 22 Feb 2025 17:25:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740245148; bh=fx3DRTGRkmKbnCXlKZ+ZwK/6h3hhdblLg2AxFF2wD44=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=AW/05HQj1cqYDn0v6OshMq3HXaCIsSmiZJfuXWKkuTBSnKiKRpt4EYiZZrZc937H4 VQMK67wpWr2WRUM/iz2FJnQiciJxIqe3WTrlR6cKX3HAeSpNgaN0OxmHuhCojWmBme qKAidpYlyNCgX4fGQhszSBH3b6MUwDoyR1nSpQV/hq/gySmIlfIZGkCJLUmHzISdAZ dE5Xptv+JeDCLapSokeNN8AOXGxY5ohkNjQ0RPeDTPTcgNj7fsCAArHAd+CE5mgCzI ONx4ZZLLNShIB8ePk8logz262hSp7a12efV9y6nToPrYyrCqcjdJYHOIQJ1M3sEPFQ b4DE9uxTgRD7Q== Date: Sat, 22 Feb 2025 17:25:36 +0000 From: Jonathan Cameron To: Svyatoslav Ryhel Cc: Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Thierry Reding , Jonathan Hunter , Javier Carrasco , Matti Vaittinen , Andy Shevchenko , Emil Gedenryd , Arthur Becker , Mudit Sharma , Per-Daniel Olsson , Subhajit Ghosh , Ivan Orlov , David Heidelberg , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-tegra@vger.kernel.org Subject: Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor Message-ID: <20250222172536.5fb73658@jic23-huawei> In-Reply-To: References: <20250217140336.107476-1-clamor95@gmail.com> <20250217140336.107476-3-clamor95@gmail.com> <20250222125335.177fc746@jic23-huawei> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.48; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sat, 22 Feb 2025 14:56:41 +0200 Svyatoslav Ryhel wrote: > =D1=81=D0=B1, 22 =D0=BB=D1=8E=D1=82. 2025=E2=80=AF=D1=80. =D0=BE 14:53 Jo= nathan Cameron =D0=BF=D0=B8=D1=88=D0=B5: > > > > On Mon, 17 Feb 2025 16:03:35 +0200 > > Svyatoslav Ryhel wrote: > > =20 > > > AL3000a is a simple I2C-based ambient light sensor, which is > > > closely related to AL3010 and AL3320a, but has significantly > > > different way of processing data generated by the sensor. > > > > > > Tested-by: Robert Eckelmann > > > Tested-by: Antoni Aloy Torrens > > > Signed-off-by: Svyatoslav Ryhel > > > Reviewed-by: David Heidelberg =20 > > > > Whilst I am confused by earlier statements about not > > having information on the conversion to illuminance values, I'm > > going to assume the look up table in here is based on some > > reasonable data from somewhere and hence that this is a sensor > > with appropriate filtering of the light to be able to do a non linear > > conversion from the value read and standard light curves. > > > > As such the IIO_LIGHT channel type is fine for this device. > > =20 >=20 > Thank you, but IIO_INTENSITY may be proper channel as well, after your > explanations. If you wish, I may upload v5 with swapping LIGHT with > INTENSITY. Where does the lux_table set of values come from? That seems to be key question for this driver. >=20 > > Applied patches 1 and 2 to the togreg branch of iio.git. > > Note that I'll initially push this out as testing to allow > > the autobuilders to see if they can find any issues that we missed. > > Patch 3 will need to go via the appropriate SoC tree as normal. > > > > Jonathan > > =20 > > > --- > > > drivers/iio/light/Kconfig | 10 ++ > > > drivers/iio/light/Makefile | 1 + > > > drivers/iio/light/al3000a.c | 209 ++++++++++++++++++++++++++++++++++= ++ > > > 3 files changed, 220 insertions(+) > > > create mode 100644 drivers/iio/light/al3000a.c > > > > > > diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig > > > index 29ffa8491927..37f83e1d8893 100644 > > > --- a/drivers/iio/light/Kconfig > > > +++ b/drivers/iio/light/Kconfig > > > @@ -43,6 +43,16 @@ config ADUX1020 > > > To compile this driver as a module, choose M here: the > > > module will be called adux1020. > > > > > > +config AL3000A > > > + tristate "AL3000a ambient light sensor" > > > + depends on I2C > > > + help > > > + Say Y here if you want to build a driver for the Dyna Image A= L3000a > > > + ambient light sensor. > > > + > > > + To compile this driver as a module, choose M here: the > > > + module will be called al3000a. > > > + > > > config AL3010 > > > tristate "AL3010 ambient light sensor" > > > depends on I2C > > > diff --git a/drivers/iio/light/Makefile b/drivers/iio/light/Makefile > > > index f14a37442712..03f10786273a 100644 > > > --- a/drivers/iio/light/Makefile > > > +++ b/drivers/iio/light/Makefile > > > @@ -7,6 +7,7 @@ > > > obj-$(CONFIG_ACPI_ALS) +=3D acpi-als.o > > > obj-$(CONFIG_ADJD_S311) +=3D adjd_s311.o > > > obj-$(CONFIG_ADUX1020) +=3D adux1020.o > > > +obj-$(CONFIG_AL3000A) +=3D al3000a.o > > > obj-$(CONFIG_AL3010) +=3D al3010.o > > > obj-$(CONFIG_AL3320A) +=3D al3320a.o > > > obj-$(CONFIG_APDS9300) +=3D apds9300.o > > > diff --git a/drivers/iio/light/al3000a.c b/drivers/iio/light/al3000a.c > > > new file mode 100644 > > > index 000000000000..e2fbb1270040 > > > --- /dev/null > > > +++ b/drivers/iio/light/al3000a.c > > > @@ -0,0 +1,209 @@ > > > +// SPDX-License-Identifier: GPL-2.0-only > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +#include > > > + > > > +#define AL3000A_REG_SYSTEM 0x00 > > > +#define AL3000A_REG_DATA 0x05 > > > + > > > +#define AL3000A_CONFIG_ENABLE 0x00 > > > +#define AL3000A_CONFIG_DISABLE 0x0b > > > +#define AL3000A_CONFIG_RESET 0x0f > > > +#define AL3000A_GAIN_MASK GENMASK(5, 0) > > > + > > > +/* > > > + * These are pre-calculated lux values based on possible output of s= ensor > > > + * (range 0x00 - 0x3F) > > > + */ > > > +static const u32 lux_table[] =3D { > > > + 1, 1, 1, 2, 2, 2, 3, 4, /* 0 - = 7 */ > > > + 4, 5, 6, 7, 9, 11, 13, 16, /* 8 - = 15 */ > > > + 19, 22, 27, 32, 39, 46, 56, 67, /* 16 -= 23 */ > > > + 80, 96, 116, 139, 167, 200, 240, 289, /* 24 -= 31 */ > > > + 347, 416, 499, 600, 720, 864, 1037, 1245, /* 32 -= 39 */ > > > + 1495, 1795, 2155, 2587, 3105, 3728, 4475, 5373, /* 40 -= 47 */ > > > + 6450, 7743, 9296, 11160, 13397, 16084, 19309, 23180, /* 48 -= 55 */ > > > + 27828, 33408, 40107, 48148, 57803, 69393, 83306, 100000 /* 56 -= 63 */ > > > +}; =20 > > =20