[PATCH v4 0/3] iio: light: add al3000a als support

[PATCH v4 0/3] iio: light: add al3000a als support

[Svyatoslav Ryhel]

AL3000a is an illuminance sensor found in ASUS TF101 tablet.

---
Changes on switching from v3 to v4:
- return write function directly
- clean up and fix i2c_device_id

Changes on switching from v2 to v3:
- droped linux/iio/sysfs.h
- set driver name directly
- switched to IIO_CHAN_INFO_PROCESSED
- split al3000a_set_pwr into 2 functions
- added i2c_device_id
- improved code formatting 

Changes on switching from v1 to v2:
- sort compatible alphabetically in schema
- clarify commit descriptions
- convert to use regmap
- arrangle lux conversion table in rows of 8
- add more used headers
- improve code formatting 
---

Svyatoslav Ryhel (3):
  dt-bindings: iio: light: al3010: add al3000a support
  iio: light: Add support for AL3000a illuminance sensor
  ARM: tegra: tf101: Add al3000a illuminance sensor node

 .../bindings/iio/light/dynaimage,al3010.yaml  |   6 +-
 .../boot/dts/nvidia/tegra20-asus-tf101.dts    |  11 +
 drivers/iio/light/Kconfig                     |  10 +
 drivers/iio/light/Makefile                    |   1 +
 drivers/iio/light/al3000a.c                   | 209 ++++++++++++++++++
 5 files changed, 235 insertions(+), 2 deletions(-)
 create mode 100644 drivers/iio/light/al3000a.c

-- 
2.43.0

[PATCH v4 1/3] dt-bindings: iio: light: al3010: add al3000a support

[Svyatoslav Ryhel]

AL3000a is an ambient light sensor quite closely related to
exising AL3010 and can reuse exising schema for AL3010.

Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
Reviewed-by: David Heidelberg <david@ixit.cz>
---
 .../devicetree/bindings/iio/light/dynaimage,al3010.yaml     | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/light/dynaimage,al3010.yaml b/Documentation/devicetree/bindings/iio/light/dynaimage,al3010.yaml
index a3a979553e32..f1048c30e73e 100644
--- a/Documentation/devicetree/bindings/iio/light/dynaimage,al3010.yaml
+++ b/Documentation/devicetree/bindings/iio/light/dynaimage,al3010.yaml
@@ -4,14 +4,16 @@
 $id: http://devicetree.org/schemas/iio/light/dynaimage,al3010.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
-title: Dyna-Image AL3010 sensor
+title: Dyna-Image AL3000a/AL3010 sensor
 
 maintainers:
   - David Heidelberg <david@ixit.cz>
 
 properties:
   compatible:
-    const: dynaimage,al3010
+    enum:
+      - dynaimage,al3000a
+      - dynaimage,al3010
 
   reg:
     maxItems: 1
-- 
2.43.0

Re: [PATCH v4 1/3] dt-bindings: iio: light: al3010: add al3000a support

[Krzysztof Kozlowski]

On Mon, Feb 17, 2025 at 04:03:34PM +0200, Svyatoslav Ryhel wrote:
> AL3000a is an ambient light sensor quite closely related to
> exising AL3010 and can reuse exising schema for AL3010.
> 
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> Reviewed-by: David Heidelberg <david@ixit.cz>
> ---
>  .../devicetree/bindings/iio/light/dynaimage,al3010.yaml     | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

[PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Svyatoslav Ryhel]

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 <longnoserob@gmail.com>
Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
Reviewed-by: David Heidelberg <david@ixit.cz>
---
 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 AL3000a
+	  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)		+= acpi-als.o
 obj-$(CONFIG_ADJD_S311)		+= adjd_s311.o
 obj-$(CONFIG_ADUX1020)		+= adux1020.o
+obj-$(CONFIG_AL3000A)		+= al3000a.o
 obj-$(CONFIG_AL3010)		+= al3010.o
 obj-$(CONFIG_AL3320A)		+= al3320a.o
 obj-$(CONFIG_APDS9300)		+= 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 <linux/array_size.h>
+#include <linux/bitfield.h>
+#include <linux/device.h>
+#include <linux/err.h>
+#include <linux/i2c.h>
+#include <linux/mod_devicetable.h>
+#include <linux/module.h>
+#include <linux/pm.h>
+#include <linux/regmap.h>
+#include <linux/regulator/consumer.h>
+#include <linux/types.h>
+
+#include <linux/iio/iio.h>
+
+#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 sensor
+ * (range 0x00 - 0x3F)
+ */
+static const u32 lux_table[] = {
+	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 */
+};
+
+static const struct regmap_config al3000a_regmap_config = {
+	.reg_bits = 8,
+	.val_bits = 8,
+	.max_register = AL3000A_REG_DATA,
+};
+
+struct al3000a_data {
+	struct regmap *regmap;
+	struct regulator *vdd_supply;
+};
+
+static const struct iio_chan_spec al3000a_channels[] = {
+	{
+		.type = IIO_LIGHT,
+		.info_mask_separate = BIT(IIO_CHAN_INFO_PROCESSED),
+	},
+};
+
+static int al3000a_set_pwr_on(struct al3000a_data *data)
+{
+	struct device *dev = regmap_get_device(data->regmap);
+	int ret;
+
+	ret = regulator_enable(data->vdd_supply);
+	if (ret) {
+		dev_err(dev, "failed to enable vdd power supply\n");
+		return ret;
+	}
+
+	return regmap_write(data->regmap, AL3000A_REG_SYSTEM, AL3000A_CONFIG_ENABLE);
+}
+
+static void al3000a_set_pwr_off(void *_data)
+{
+	struct al3000a_data *data = _data;
+	struct device *dev = regmap_get_device(data->regmap);
+	int ret;
+
+	ret = regmap_write(data->regmap, AL3000A_REG_SYSTEM, AL3000A_CONFIG_DISABLE);
+	if (ret)
+		dev_err(dev, "failed to write system register\n");
+
+	ret = regulator_disable(data->vdd_supply);
+	if (ret)
+		dev_err(dev, "failed to disable vdd power supply\n");
+}
+
+static int al3000a_init(struct al3000a_data *data)
+{
+	int ret;
+
+	ret = al3000a_set_pwr_on(data);
+	if (ret)
+		return ret;
+
+	ret = regmap_write(data->regmap, AL3000A_REG_SYSTEM, AL3000A_CONFIG_RESET);
+	if (ret)
+		return ret;
+
+	return regmap_write(data->regmap, AL3000A_REG_SYSTEM, AL3000A_CONFIG_ENABLE);
+}
+
+static int al3000a_read_raw(struct iio_dev *indio_dev,
+			    struct iio_chan_spec const *chan, int *val,
+			    int *val2, long mask)
+{
+	struct al3000a_data *data = iio_priv(indio_dev);
+	int ret, gain;
+
+	switch (mask) {
+	case IIO_CHAN_INFO_PROCESSED:
+		ret = regmap_read(data->regmap, AL3000A_REG_DATA, &gain);
+		if (ret)
+			return ret;
+
+		*val = lux_table[gain & AL3000A_GAIN_MASK];
+
+		return IIO_VAL_INT;
+	default:
+		return -EINVAL;
+	}
+}
+
+static const struct iio_info al3000a_info = {
+	.read_raw = al3000a_read_raw,
+};
+
+static int al3000a_probe(struct i2c_client *client)
+{
+	struct al3000a_data *data;
+	struct device *dev = &client->dev;
+	struct iio_dev *indio_dev;
+	int ret;
+
+	indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
+	if (!indio_dev)
+		return -ENOMEM;
+
+	data = iio_priv(indio_dev);
+	i2c_set_clientdata(client, indio_dev);
+
+	data->regmap = devm_regmap_init_i2c(client, &al3000a_regmap_config);
+	if (IS_ERR(data->regmap))
+		return dev_err_probe(dev, PTR_ERR(data->regmap),
+				     "cannot allocate regmap\n");
+
+	data->vdd_supply = devm_regulator_get(dev, "vdd");
+	if (IS_ERR(data->vdd_supply))
+		return dev_err_probe(dev, PTR_ERR(data->vdd_supply),
+				     "failed to get vdd regulator\n");
+
+	indio_dev->info = &al3000a_info;
+	indio_dev->name = "al3000a";
+	indio_dev->channels = al3000a_channels;
+	indio_dev->num_channels = ARRAY_SIZE(al3000a_channels);
+	indio_dev->modes = INDIO_DIRECT_MODE;
+
+	ret = al3000a_init(data);
+	if (ret)
+		return dev_err_probe(dev, ret, "failed to init ALS\n");
+
+	ret = devm_add_action_or_reset(dev, al3000a_set_pwr_off, data);
+	if (ret)
+		return dev_err_probe(dev, ret, "failed to add action\n");
+
+	return devm_iio_device_register(dev, indio_dev);
+}
+
+static int al3000a_suspend(struct device *dev)
+{
+	struct al3000a_data *data = iio_priv(dev_get_drvdata(dev));
+
+	al3000a_set_pwr_off(data);
+	return 0;
+}
+
+static int al3000a_resume(struct device *dev)
+{
+	struct al3000a_data *data = iio_priv(dev_get_drvdata(dev));
+
+	return al3000a_set_pwr_on(data);
+}
+
+static DEFINE_SIMPLE_DEV_PM_OPS(al3000a_pm_ops, al3000a_suspend, al3000a_resume);
+
+static const struct i2c_device_id al3000a_id[] = {
+	{ "al3000a" },
+	{ }
+};
+MODULE_DEVICE_TABLE(i2c, al3000a_id);
+
+static const struct of_device_id al3000a_of_match[] = {
+	{ .compatible = "dynaimage,al3000a" },
+	{ /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, al3000a_of_match);
+
+static struct i2c_driver al3000a_driver = {
+	.driver = {
+		.name = "al3000a",
+		.of_match_table = al3000a_of_match,
+		.pm = pm_sleep_ptr(&al3000a_pm_ops),
+	},
+	.probe = al3000a_probe,
+	.id_table = al3000a_id,
+};
+module_i2c_driver(al3000a_driver);
+
+MODULE_AUTHOR("Svyatolsav Ryhel <clamor95@gmail.com>");
+MODULE_DESCRIPTION("al3000a Ambient Light Sensor driver");
+MODULE_LICENSE("GPL");
-- 
2.43.0

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Andy Shevchenko]

On Mon, Feb 17, 2025 at 04:03:35PM +0200, Svyatoslav Ryhel wrote:
> 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.

LGTM, FWIW,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

...

> +static void al3000a_set_pwr_off(void *_data)
> +{
> +	struct al3000a_data *data = _data;
> +	struct device *dev = regmap_get_device(data->regmap);
> +	int ret;
> +
> +	ret = regmap_write(data->regmap, AL3000A_REG_SYSTEM, AL3000A_CONFIG_DISABLE);
> +	if (ret)
> +		dev_err(dev, "failed to write system register\n");
> +
> +	ret = regulator_disable(data->vdd_supply);
> +	if (ret)
> +		dev_err(dev, "failed to disable vdd power supply\n");

I dunno what we can do on errors, but at least user will be informed what was
failed.

> +}

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Jonathan Cameron]

On Mon, 17 Feb 2025 16:03:35 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> 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 <longnoserob@gmail.com>
> Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> Reviewed-by: David Heidelberg <david@ixit.cz>

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.

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

> ---
>  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 AL3000a
> +	  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)		+= acpi-als.o
>  obj-$(CONFIG_ADJD_S311)		+= adjd_s311.o
>  obj-$(CONFIG_ADUX1020)		+= adux1020.o
> +obj-$(CONFIG_AL3000A)		+= al3000a.o
>  obj-$(CONFIG_AL3010)		+= al3010.o
>  obj-$(CONFIG_AL3320A)		+= al3320a.o
>  obj-$(CONFIG_APDS9300)		+= 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 <linux/array_size.h>
> +#include <linux/bitfield.h>
> +#include <linux/device.h>
> +#include <linux/err.h>
> +#include <linux/i2c.h>
> +#include <linux/mod_devicetable.h>
> +#include <linux/module.h>
> +#include <linux/pm.h>
> +#include <linux/regmap.h>
> +#include <linux/regulator/consumer.h>
> +#include <linux/types.h>
> +
> +#include <linux/iio/iio.h>
> +
> +#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 sensor
> + * (range 0x00 - 0x3F)
> + */
> +static const u32 lux_table[] = {
> +	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 */
> +};

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Svyatoslav Ryhel]

сб, 22 лют. 2025 р. о 14:53 Jonathan Cameron <jic23@kernel.org> пише:
>
> On Mon, 17 Feb 2025 16:03:35 +0200
> Svyatoslav Ryhel <clamor95@gmail.com> wrote:
>
> > 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 <longnoserob@gmail.com>
> > Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> > Reviewed-by: David Heidelberg <david@ixit.cz>
>
> 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.
>

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.

> 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
>
> > ---
> >  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 AL3000a
> > +       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)               += acpi-als.o
> >  obj-$(CONFIG_ADJD_S311)              += adjd_s311.o
> >  obj-$(CONFIG_ADUX1020)               += adux1020.o
> > +obj-$(CONFIG_AL3000A)                += al3000a.o
> >  obj-$(CONFIG_AL3010)         += al3010.o
> >  obj-$(CONFIG_AL3320A)                += al3320a.o
> >  obj-$(CONFIG_APDS9300)               += 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 <linux/array_size.h>
> > +#include <linux/bitfield.h>
> > +#include <linux/device.h>
> > +#include <linux/err.h>
> > +#include <linux/i2c.h>
> > +#include <linux/mod_devicetable.h>
> > +#include <linux/module.h>
> > +#include <linux/pm.h>
> > +#include <linux/regmap.h>
> > +#include <linux/regulator/consumer.h>
> > +#include <linux/types.h>
> > +
> > +#include <linux/iio/iio.h>
> > +
> > +#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 sensor
> > + * (range 0x00 - 0x3F)
> > + */
> > +static const u32 lux_table[] = {
> > +     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 */
> > +};
>

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Jonathan Cameron]

On Sat, 22 Feb 2025 14:56:41 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> сб, 22 лют. 2025 р. о 14:53 Jonathan Cameron <jic23@kernel.org> пише:
> >
> > On Mon, 17 Feb 2025 16:03:35 +0200
> > Svyatoslav Ryhel <clamor95@gmail.com> wrote:
> >  
> > > 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 <longnoserob@gmail.com>
> > > Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> > > Reviewed-by: David Heidelberg <david@ixit.cz>  
> >
> > 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.
> >  
> 
> 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.

> 
> > 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
> >  
> > > ---
> > >  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 AL3000a
> > > +       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)               += acpi-als.o
> > >  obj-$(CONFIG_ADJD_S311)              += adjd_s311.o
> > >  obj-$(CONFIG_ADUX1020)               += adux1020.o
> > > +obj-$(CONFIG_AL3000A)                += al3000a.o
> > >  obj-$(CONFIG_AL3010)         += al3010.o
> > >  obj-$(CONFIG_AL3320A)                += al3320a.o
> > >  obj-$(CONFIG_APDS9300)               += 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 <linux/array_size.h>
> > > +#include <linux/bitfield.h>
> > > +#include <linux/device.h>
> > > +#include <linux/err.h>
> > > +#include <linux/i2c.h>
> > > +#include <linux/mod_devicetable.h>
> > > +#include <linux/module.h>
> > > +#include <linux/pm.h>
> > > +#include <linux/regmap.h>
> > > +#include <linux/regulator/consumer.h>
> > > +#include <linux/types.h>
> > > +
> > > +#include <linux/iio/iio.h>
> > > +
> > > +#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 sensor
> > > + * (range 0x00 - 0x3F)
> > > + */
> > > +static const u32 lux_table[] = {
> > > +     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 */
> > > +};  
> >  

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Svyatoslav Ryhel]

сб, 22 лют. 2025 р. о 19:25 Jonathan Cameron <jic23@kernel.org> пише:
>
> On Sat, 22 Feb 2025 14:56:41 +0200
> Svyatoslav Ryhel <clamor95@gmail.com> wrote:
>
> > сб, 22 лют. 2025 р. о 14:53 Jonathan Cameron <jic23@kernel.org> пише:
> > >
> > > On Mon, 17 Feb 2025 16:03:35 +0200
> > > Svyatoslav Ryhel <clamor95@gmail.com> wrote:
> > >
> > > > 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 <longnoserob@gmail.com>
> > > > Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> > > > Reviewed-by: David Heidelberg <david@ixit.cz>
> > >
> > > 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.
> > >
> >
> > 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.
>

3.1.10 kernel driver for this device

> >
> > > 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
> > >
> > > > ---
> > > >  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 AL3000a
> > > > +       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)               += acpi-als.o
> > > >  obj-$(CONFIG_ADJD_S311)              += adjd_s311.o
> > > >  obj-$(CONFIG_ADUX1020)               += adux1020.o
> > > > +obj-$(CONFIG_AL3000A)                += al3000a.o
> > > >  obj-$(CONFIG_AL3010)         += al3010.o
> > > >  obj-$(CONFIG_AL3320A)                += al3320a.o
> > > >  obj-$(CONFIG_APDS9300)               += 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 <linux/array_size.h>
> > > > +#include <linux/bitfield.h>
> > > > +#include <linux/device.h>
> > > > +#include <linux/err.h>
> > > > +#include <linux/i2c.h>
> > > > +#include <linux/mod_devicetable.h>
> > > > +#include <linux/module.h>
> > > > +#include <linux/pm.h>
> > > > +#include <linux/regmap.h>
> > > > +#include <linux/regulator/consumer.h>
> > > > +#include <linux/types.h>
> > > > +
> > > > +#include <linux/iio/iio.h>
> > > > +
> > > > +#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 sensor
> > > > + * (range 0x00 - 0x3F)
> > > > + */
> > > > +static const u32 lux_table[] = {
> > > > +     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 */
> > > > +};
> > >
>

Re: [PATCH v4 2/3] iio: light: Add support for AL3000a illuminance sensor

[Jonathan Cameron]

On Sat, 22 Feb 2025 19:27:22 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> сб, 22 лют. 2025 р. о 19:25 Jonathan Cameron <jic23@kernel.org> пише:
> >
> > On Sat, 22 Feb 2025 14:56:41 +0200
> > Svyatoslav Ryhel <clamor95@gmail.com> wrote:
> >  
> > > сб, 22 лют. 2025 р. о 14:53 Jonathan Cameron <jic23@kernel.org> пише:  
> > > >
> > > > On Mon, 17 Feb 2025 16:03:35 +0200
> > > > Svyatoslav Ryhel <clamor95@gmail.com> wrote:
> > > >  
> > > > > 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 <longnoserob@gmail.com>
> > > > > Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> > > > > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> > > > > Reviewed-by: David Heidelberg <david@ixit.cz>  
> > > >
> > > > 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.
> > > >  
> > >
> > > 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.
> >  
> 
> 3.1.10 kernel driver for this device
Ok. So until we know otherwise let us trust that data as being
correct.

Jonathan

> 
> > >  
> > > > 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
> > > >  
> > > > > ---
> > > > >  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 AL3000a
> > > > > +       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)               += acpi-als.o
> > > > >  obj-$(CONFIG_ADJD_S311)              += adjd_s311.o
> > > > >  obj-$(CONFIG_ADUX1020)               += adux1020.o
> > > > > +obj-$(CONFIG_AL3000A)                += al3000a.o
> > > > >  obj-$(CONFIG_AL3010)         += al3010.o
> > > > >  obj-$(CONFIG_AL3320A)                += al3320a.o
> > > > >  obj-$(CONFIG_APDS9300)               += 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 <linux/array_size.h>
> > > > > +#include <linux/bitfield.h>
> > > > > +#include <linux/device.h>
> > > > > +#include <linux/err.h>
> > > > > +#include <linux/i2c.h>
> > > > > +#include <linux/mod_devicetable.h>
> > > > > +#include <linux/module.h>
> > > > > +#include <linux/pm.h>
> > > > > +#include <linux/regmap.h>
> > > > > +#include <linux/regulator/consumer.h>
> > > > > +#include <linux/types.h>
> > > > > +
> > > > > +#include <linux/iio/iio.h>
> > > > > +
> > > > > +#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 sensor
> > > > > + * (range 0x00 - 0x3F)
> > > > > + */
> > > > > +static const u32 lux_table[] = {
> > > > > +     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 */
> > > > > +};  
> > > >  
> >  
> 

[PATCH v4 3/3] ARM: tegra: tf101: Add al3000a illuminance sensor node

[Svyatoslav Ryhel]

Bind al3000a illuminance sensor found in ASUS TF101

Tested-by: Robert Eckelmann <longnoserob@gmail.com>
Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
---
 arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts b/arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts
index 975ec24195ca..fcf3d6dd64a2 100644
--- a/arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts
+++ b/arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts
@@ -1123,6 +1123,17 @@ smart-battery@b {
 				sbs,poll-retry-count = <10>;
 				power-supplies = <&mains>;
 			};
+
+			/* Dynaimage ambient light sensor */
+			light-sensor@1c {
+				compatible = "dynaimage,al3000a";
+				reg = <0x1c>;
+
+				interrupt-parent = <&gpio>;
+				interrupts = <TEGRA_GPIO(Z, 2) IRQ_TYPE_LEVEL_HIGH>;
+
+				vdd-supply = <&vdd_1v8_sys>;
+			};
 		};
 	};
 
-- 
2.43.0

Re: [PATCH v4 3/3] ARM: tegra: tf101: Add al3000a illuminance sensor node

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 440 bytes --]

On Mon, Feb 17, 2025 at 04:03:36PM +0200, Svyatoslav Ryhel wrote:
> Bind al3000a illuminance sensor found in ASUS TF101
> 
> Tested-by: Robert Eckelmann <longnoserob@gmail.com>
> Tested-by: Antoni Aloy Torrens <aaloytorrens@gmail.com>
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> ---
>  arch/arm/boot/dts/nvidia/tegra20-asus-tf101.dts | 11 +++++++++++
>  1 file changed, 11 insertions(+)

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH v4 0/3] iio: light: add al3000a als support

[Jonathan Cameron]

On Mon, 17 Feb 2025 16:03:33 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> AL3000a is an illuminance sensor found in ASUS TF101 tablet.
Hi Svyatoslav,

Just a small request.  Please reduce rate of sending new versions.
That tends to give time for discussions on earlier versions to progress
and for more reviewers to see the current version.

I'd suggest at least 3-4 days between versions.  It's great to
responsive to reviewers of course!

Jonathan


> 
> ---
> Changes on switching from v3 to v4:
> - return write function directly
> - clean up and fix i2c_device_id
> 
> Changes on switching from v2 to v3:
> - droped linux/iio/sysfs.h
> - set driver name directly
> - switched to IIO_CHAN_INFO_PROCESSED
> - split al3000a_set_pwr into 2 functions
> - added i2c_device_id
> - improved code formatting 
> 
> Changes on switching from v1 to v2:
> - sort compatible alphabetically in schema
> - clarify commit descriptions
> - convert to use regmap
> - arrangle lux conversion table in rows of 8
> - add more used headers
> - improve code formatting 
> ---
> 
> Svyatoslav Ryhel (3):
>   dt-bindings: iio: light: al3010: add al3000a support
>   iio: light: Add support for AL3000a illuminance sensor
>   ARM: tegra: tf101: Add al3000a illuminance sensor node
> 
>  .../bindings/iio/light/dynaimage,al3010.yaml  |   6 +-
>  .../boot/dts/nvidia/tegra20-asus-tf101.dts    |  11 +
>  drivers/iio/light/Kconfig                     |  10 +
>  drivers/iio/light/Makefile                    |   1 +
>  drivers/iio/light/al3000a.c                   | 209 ++++++++++++++++++
>  5 files changed, 235 insertions(+), 2 deletions(-)
>  create mode 100644 drivers/iio/light/al3000a.c
> 

Re: [PATCH v4 0/3] iio: light: add al3000a als support

[Svyatoslav Ryhel]

пн, 17 лют. 2025 р. о 16:30 Jonathan Cameron <jic23@kernel.org> пише:
>
> On Mon, 17 Feb 2025 16:03:33 +0200
> Svyatoslav Ryhel <clamor95@gmail.com> wrote:
>
> > AL3000a is an illuminance sensor found in ASUS TF101 tablet.
> Hi Svyatoslav,
>
> Just a small request.  Please reduce rate of sending new versions.
> That tends to give time for discussions on earlier versions to progress
> and for more reviewers to see the current version.
>
> I'd suggest at least 3-4 days between versions.  It's great to
> responsive to reviewers of course!
>
> Jonathan
>
>

Once you are satisfied with amount of time patch will hang, let me
know, I will upload new version. Thank you.

> >
> > ---
> > Changes on switching from v3 to v4:
> > - return write function directly
> > - clean up and fix i2c_device_id
> >
> > Changes on switching from v2 to v3:
> > - droped linux/iio/sysfs.h
> > - set driver name directly
> > - switched to IIO_CHAN_INFO_PROCESSED
> > - split al3000a_set_pwr into 2 functions
> > - added i2c_device_id
> > - improved code formatting
> >
> > Changes on switching from v1 to v2:
> > - sort compatible alphabetically in schema
> > - clarify commit descriptions
> > - convert to use regmap
> > - arrangle lux conversion table in rows of 8
> > - add more used headers
> > - improve code formatting
> > ---
> >
> > Svyatoslav Ryhel (3):
> >   dt-bindings: iio: light: al3010: add al3000a support
> >   iio: light: Add support for AL3000a illuminance sensor
> >   ARM: tegra: tf101: Add al3000a illuminance sensor node
> >
> >  .../bindings/iio/light/dynaimage,al3010.yaml  |   6 +-
> >  .../boot/dts/nvidia/tegra20-asus-tf101.dts    |  11 +
> >  drivers/iio/light/Kconfig                     |  10 +
> >  drivers/iio/light/Makefile                    |   1 +
> >  drivers/iio/light/al3000a.c                   | 209 ++++++++++++++++++
> >  5 files changed, 235 insertions(+), 2 deletions(-)
> >  create mode 100644 drivers/iio/light/al3000a.c
> >
>

Re: [PATCH v4 0/3] iio: light: add al3000a als support

[Jonathan Cameron]

On Mon, 17 Feb 2025 16:33:45 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> пн, 17 лют. 2025 р. о 16:30 Jonathan Cameron <jic23@kernel.org> пише:
> >
> > On Mon, 17 Feb 2025 16:03:33 +0200
> > Svyatoslav Ryhel <clamor95@gmail.com> wrote:
> >  
> > > AL3000a is an illuminance sensor found in ASUS TF101 tablet.  
> > Hi Svyatoslav,
> >
> > Just a small request.  Please reduce rate of sending new versions.
> > That tends to give time for discussions on earlier versions to progress
> > and for more reviewers to see the current version.
> >
> > I'd suggest at least 3-4 days between versions.  It's great to
> > responsive to reviewers of course!
> >
> > Jonathan
> >
> >  
> 
> Once you are satisfied with amount of time patch will hang, let me
> know, I will upload new version. Thank you.

That approach will never work.   Just use 1 week between versions
if you need a simple number.

Jonathan

> 
> > >
> > > ---
> > > Changes on switching from v3 to v4:
> > > - return write function directly
> > > - clean up and fix i2c_device_id
> > >
> > > Changes on switching from v2 to v3:
> > > - droped linux/iio/sysfs.h
> > > - set driver name directly
> > > - switched to IIO_CHAN_INFO_PROCESSED
> > > - split al3000a_set_pwr into 2 functions
> > > - added i2c_device_id
> > > - improved code formatting
> > >
> > > Changes on switching from v1 to v2:
> > > - sort compatible alphabetically in schema
> > > - clarify commit descriptions
> > > - convert to use regmap
> > > - arrangle lux conversion table in rows of 8
> > > - add more used headers
> > > - improve code formatting
> > > ---
> > >
> > > Svyatoslav Ryhel (3):
> > >   dt-bindings: iio: light: al3010: add al3000a support
> > >   iio: light: Add support for AL3000a illuminance sensor
> > >   ARM: tegra: tf101: Add al3000a illuminance sensor node
> > >
> > >  .../bindings/iio/light/dynaimage,al3010.yaml  |   6 +-
> > >  .../boot/dts/nvidia/tegra20-asus-tf101.dts    |  11 +
> > >  drivers/iio/light/Kconfig                     |  10 +
> > >  drivers/iio/light/Makefile                    |   1 +
> > >  drivers/iio/light/al3000a.c                   | 209 ++++++++++++++++++
> > >  5 files changed, 235 insertions(+), 2 deletions(-)
> > >  create mode 100644 drivers/iio/light/al3000a.c
> > >  
> >