Commit d978bfdd authored by Peter Meerwald-Stadler's avatar Peter Meerwald-Stadler Committed by Jonathan Cameron

iio: light: vcnl4000: Mention and check support for VCNL4010 and VCNL4020

VCNL4000, VCNL4010 and VCNL4020 chips are fairly compatible from a software
point of view, added features are not yet supported by the driver

patch adds a check for the product ID and demotes the corresponding
dev_info() to dev_dbg()
Signed-off-by: default avatarPeter Meerwald-Stadler <pmeerw@pmeerw.net>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
parent 1f25ca11
...@@ -333,11 +333,11 @@ config US5182D ...@@ -333,11 +333,11 @@ config US5182D
will be called us5182d. will be called us5182d.
config VCNL4000 config VCNL4000
tristate "VCNL4000 combined ALS and proximity sensor" tristate "VCNL4000/4010/4020 combined ALS and proximity sensor"
depends on I2C depends on I2C
help help
Say Y here if you want to build a driver for the Vishay VCNL4000 Say Y here if you want to build a driver for the Vishay VCNL4000,
combined ambient light and proximity sensor. VCNL4010, VCNL4020 combined ambient light and proximity sensor.
To compile this driver as a module, choose M here: the To compile this driver as a module, choose M here: the
module will be called vcnl4000. module will be called vcnl4000.
......
/* /*
* vcnl4000.c - Support for Vishay VCNL4000 combined ambient light and * vcnl4000.c - Support for Vishay VCNL4000/4010/4020 combined ambient
* proximity sensor * light and proximity sensor
* *
* Copyright 2012 Peter Meerwald <pmeerw@pmeerw.net> * Copyright 2012 Peter Meerwald <pmeerw@pmeerw.net>
* *
...@@ -13,6 +13,8 @@ ...@@ -13,6 +13,8 @@
* TODO: * TODO:
* allow to adjust IR current * allow to adjust IR current
* proximity threshold and event handling * proximity threshold and event handling
* periodic ALS/proximity measurement (VCNL4010/20)
* interrupts (VCNL4010/20)
*/ */
#include <linux/module.h> #include <linux/module.h>
...@@ -24,6 +26,8 @@ ...@@ -24,6 +26,8 @@
#include <linux/iio/sysfs.h> #include <linux/iio/sysfs.h>
#define VCNL4000_DRV_NAME "vcnl4000" #define VCNL4000_DRV_NAME "vcnl4000"
#define VCNL4000_ID 0x01
#define VCNL4010_ID 0x02 /* for VCNL4020, VCNL4010 */
#define VCNL4000_COMMAND 0x80 /* Command register */ #define VCNL4000_COMMAND 0x80 /* Command register */
#define VCNL4000_PROD_REV 0x81 /* Product ID and Revision ID */ #define VCNL4000_PROD_REV 0x81 /* Product ID and Revision ID */
...@@ -155,7 +159,7 @@ static int vcnl4000_probe(struct i2c_client *client, ...@@ -155,7 +159,7 @@ static int vcnl4000_probe(struct i2c_client *client,
{ {
struct vcnl4000_data *data; struct vcnl4000_data *data;
struct iio_dev *indio_dev; struct iio_dev *indio_dev;
int ret; int ret, prod_id;
indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data)); indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
if (!indio_dev) if (!indio_dev)
...@@ -169,8 +173,13 @@ static int vcnl4000_probe(struct i2c_client *client, ...@@ -169,8 +173,13 @@ static int vcnl4000_probe(struct i2c_client *client,
if (ret < 0) if (ret < 0)
return ret; return ret;
dev_info(&client->dev, "VCNL4000 Ambient light/proximity sensor, Prod %02x, Rev: %02x\n", prod_id = ret >> 4;
ret >> 4, ret & 0xf); if (prod_id != VCNL4010_ID && prod_id != VCNL4000_ID)
return -ENODEV;
dev_dbg(&client->dev, "%s Ambient light/proximity sensor, Rev: %02x\n",
(prod_id == VCNL4010_ID) ? "VCNL4010/4020" : "VCNL4000",
ret & 0xf);
indio_dev->dev.parent = &client->dev; indio_dev->dev.parent = &client->dev;
indio_dev->info = &vcnl4000_info; indio_dev->info = &vcnl4000_info;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment