[lm-sensors] Ambient Light sensor for Intersil-ISL29020 device

Trisal, Kalhan kalhan.trisal at intel.com
Thu Aug 13 12:52:29 CEST 2009


This driver used Hwmon interface , I don't find the ils29003 registering with hwmon. I still believe this should be part of hwmon group.

Thanks 
Kalhan 

-----Original Message-----
From: Cory T. Tusar [mailto:ctusar at videon-central.com] 
Sent: Tuesday, August 11, 2009 7:00 PM
To: Trisal, Kalhan
Cc: lm-sensors at lm-sensors.org; alan at linux.intel.com
Subject: Re: [lm-sensors] Ambient Light sensor for Intersil-ISL29020 device

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Kalhan Trisal wrote:
>>From 2d11a9f666b7477cc8faca0f055f4477f328516f Mon Sep 17 00:00:00 2001
> From: Kalhan Trisal <kalhan.trisal at intel.com>
> Date: Tue, 11 Aug 2009 14:28:32 -0400
> Subject: [PATCH] Intersil ISL29020 ALS driver
> ALS driver will read the latest Lux measurement based on the light brightness and will report the LUX output through sysfs interface.
> 
> Signed-off-by: Kalhan Trisal <kalhan.trisal at intel.com>
> 
> ---
>  drivers/hwmon/Kconfig    |   11 ++
>  drivers/hwmon/Makefile   |    1 +
>  drivers/hwmon/isl29020.c |  237 ++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 249 insertions(+), 0 deletions(-)
>  create mode 100755 drivers/hwmon/isl29020.c
> 
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index 2d50166..c89f1f6 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -1017,6 +1017,17 @@ config SENSORS_APPLESMC
>           Say Y here if you have an applicable laptop and want to experience
>           the awesome power of applesmc.
> 
> +config SENSORS_ISL29020
> +       tristate "Intersil ISL29020 ALS"
> +       depends on I2C_MRST

Is there a reason you're hardcoding a Moorestown dependency here?

> +       default n
> +       help
> +         If you say yes here you get support for the ALS Devices
> +         Ambient Light Sensor monitoring chip.
> +         Range values can be configured using sysfs.
> +         Lux Data are  accessible via sysfs.
> +
> +
>  config HWMON_DEBUG_CHIP
>         bool "Hardware Monitoring Chip debugging messages"
>         default n
> diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile
> index b793dce..3b1e424 100644
> --- a/drivers/hwmon/Makefile
> +++ b/drivers/hwmon/Makefile
> @@ -89,6 +89,7 @@ obj-$(CONFIG_SENSORS_VT8231)  += vt8231.o
>  obj-$(CONFIG_SENSORS_W83627EHF)        += w83627ehf.o
>  obj-$(CONFIG_SENSORS_W83L785TS)        += w83l785ts.o
>  obj-$(CONFIG_SENSORS_W83L786NG)        += w83l786ng.o
> +obj-$(CONFIG_SENSORS_LIS331DL) += lis331dl.o

This appears to be incorrect.

> 
>  ifeq ($(CONFIG_HWMON_DEBUG_CHIP),y)
>  EXTRA_CFLAGS += -DDEBUG
> diff --git a/drivers/hwmon/isl29020.c b/drivers/hwmon/isl29020.c
> new file mode 100755
> index 0000000..162dcb1
> --- /dev/null
> +++ b/drivers/hwmon/isl29020.c

Would this driver be better placed in drivers/misc, alongside the
similar ils29003 driver?

- -Cory


- -- 
Cory T. Tusar
Senior Software Engineer
Videon Central, Inc.
2171 Sandy Drive
State College, PA 16803
(814) 235-1111 x316
(814) 235-1118 fax


"There are two ways of constructing a software design.  One way is to
 make it so simple that there are obviously no deficiencies, and the
 other way is to make it so complicated that there are no obvious
 deficiencies."  --Sir Charles Anthony Richard Hoare

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)

iEYEARECAAYFAkqBckwACgkQHT1tsfGwHJ/68QCfY9RgbwotuijyABAFacG9ogQi
NvwAoJl2u9cunjoBEJBST4GqKpHiID9a
=ldQN
-----END PGP SIGNATURE-----




More information about the lm-sensors mailing list