* [PATCH v6 0/1] LTR301 ALS support @ 2015-04-11 0:59 Kuppuswamy Sathyanarayanan 2015-04-11 0:59 ` [PATCH v6 1/1] iio: ltr501: Add support for ltr301 Kuppuswamy Sathyanarayanan 0 siblings, 1 reply; 4+ messages in thread From: Kuppuswamy Sathyanarayanan @ 2015-04-11 0:59 UTC (permalink / raw) To: jic23, pmeerw Cc: daniel.baluta, linux-iio, srinivas.pandruvada, sathyanarayanan.kuppuswamy Following patch adds support for Liteon 301 Ambient light sensor. Please let me know your review comments. v1: Extended LTR501 driver to support both LTR301 and LTR501 device. v2: 1. Handled device id NULL case in probe 2. Changed pr_warn to dev_warn v3: 1. Sent v2 before by mistake. So sending the final version of v2 as v3. v4: 1. Addressed minor comments from Peter Meerwald 2. Changed invalid chip id errno from ENOSYS to ENODEV 3. Fixed channel id index number 4. Removed unused variable. v5: 1. Rebased this patch on top of Daniel's LTR559 patchset (http://marc.info/?l=linux-kernel&m=142779827617036&w=2) 2. Added info and channel members to chip_info struture. v6: 1. Changed the title in commit log Kuppuswamy Sathyanarayanan (1): iio: ltr501: Add support for ltr301 drivers/iio/light/Kconfig | 2 +- drivers/iio/light/ltr501.c | 99 +++++++++++++++++++++++++++++++++------------- 2 files changed, 72 insertions(+), 29 deletions(-) -- 1.9.1 ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v6 1/1] iio: ltr501: Add support for ltr301 2015-04-11 0:59 [PATCH v6 0/1] LTR301 ALS support Kuppuswamy Sathyanarayanan @ 2015-04-11 0:59 ` Kuppuswamy Sathyanarayanan 2015-04-12 16:17 ` Jonathan Cameron 0 siblings, 1 reply; 4+ messages in thread From: Kuppuswamy Sathyanarayanan @ 2015-04-11 0:59 UTC (permalink / raw) To: jic23, pmeerw Cc: daniel.baluta, linux-iio, srinivas.pandruvada, sathyanarayanan.kuppuswamy Added support for Liteon 301 Ambient light sensor. Since LTR-301 and LTR-501 are register compatible(and even have same part id), LTR-501 driver has been extended to support both devices. LTR-501 is similar to LTR-301 in ALS sensing, But the only difference is, LTR-501 also supports proximity sensing. LTR-501 - ALS + Proximity combo LTR-301 - ALS sensor. Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> --- drivers/iio/light/Kconfig | 2 +- drivers/iio/light/ltr501.c | 99 +++++++++++++++++++++++++++++++++------------- 2 files changed, 72 insertions(+), 29 deletions(-) diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig index a224afd..215e4a3 100644 --- a/drivers/iio/light/Kconfig +++ b/drivers/iio/light/Kconfig @@ -159,7 +159,7 @@ config LTR501 select IIO_TRIGGERED_BUFFER help If you say yes here you get support for the Lite-On LTR-501ALS-01 - ambient light and proximity sensor. + ambient light and proximity sensor or LTR-301 ambient light sensor. This driver can also be built as a module. If so, the module will be called ltr501. diff --git a/drivers/iio/light/ltr501.c b/drivers/iio/light/ltr501.c index 5939cda..4945405 100644 --- a/drivers/iio/light/ltr501.c +++ b/drivers/iio/light/ltr501.c @@ -50,6 +50,7 @@ enum { ltr501 = 0, ltr559, + ltr301, }; struct ltr501_gain { @@ -96,6 +97,8 @@ struct ltr501_chip_info { u8 als_mode_active; u8 als_gain_mask; u8 als_gain_shift; + struct iio_chan_spec const *channels; + const struct iio_info *info; }; struct ltr501_data { @@ -105,29 +108,6 @@ struct ltr501_data { u8 als_contr, ps_contr; }; -static struct ltr501_chip_info ltr501_chip_info_tbl[] = { - [ltr501] = { - .chip_id = 0x08, - .als_gain = ltr501_als_gain_tbl, - .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), - .ps_gain = ltr501_ps_gain_tbl, - .ps_gain_tbl_size = ARRAY_SIZE(ltr501_ps_gain_tbl), - .als_mode_active = BIT(0) | BIT(1), - .als_gain_mask = BIT(3), - .als_gain_shift = 3, - }, - [ltr559] = { - .chip_id = 0x09, - .als_gain = ltr559_als_gain_tbl, - .als_gain_tbl_size = ARRAY_SIZE(ltr559_als_gain_tbl), - .ps_gain = ltr559_ps_gain_tbl, - .ps_gain_tbl_size = ARRAY_SIZE(ltr559_ps_gain_tbl), - .als_mode_active = BIT(1), - .als_gain_mask = BIT(2) | BIT(3) | BIT(4), - .als_gain_shift = 2, - }, -}; - static int ltr501_drdy(struct ltr501_data *data, u8 drdy_mask) { int tries = 100; @@ -202,6 +182,13 @@ static const struct iio_chan_spec ltr501_channels[] = { IIO_CHAN_SOFT_TIMESTAMP(3), }; +static const struct iio_chan_spec ltr301_channels[] = { + LTR501_INTENSITY_CHANNEL(0, LTR501_ALS_DATA0, IIO_MOD_LIGHT_BOTH, 0), + LTR501_INTENSITY_CHANNEL(1, LTR501_ALS_DATA1, IIO_MOD_LIGHT_IR, + BIT(IIO_CHAN_INFO_SCALE)), + IIO_CHAN_SOFT_TIMESTAMP(2), +}; + static int ltr501_read_raw(struct iio_dev *indio_dev, struct iio_chan_spec const *chan, int *val, int *val2, long mask) @@ -367,10 +354,19 @@ static struct attribute *ltr501_attributes[] = { NULL }; +static struct attribute *ltr301_attributes[] = { + &iio_dev_attr_in_intensity_scale_available.dev_attr.attr, + NULL +}; + static const struct attribute_group ltr501_attribute_group = { .attrs = ltr501_attributes, }; +static const struct attribute_group ltr301_attribute_group = { + .attrs = ltr301_attributes, +}; + static const struct iio_info ltr501_info = { .read_raw = ltr501_read_raw, .write_raw = ltr501_write_raw, @@ -378,6 +374,50 @@ static const struct iio_info ltr501_info = { .driver_module = THIS_MODULE, }; +static const struct iio_info ltr301_info = { + .read_raw = ltr501_read_raw, + .write_raw = ltr501_write_raw, + .attrs = <r301_attribute_group, + .driver_module = THIS_MODULE, +}; + +static struct ltr501_chip_info ltr501_chip_info_tbl[] = { + [ltr501] = { + .chip_id = 0x08, + .als_gain = ltr501_als_gain_tbl, + .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), + .ps_gain = ltr501_ps_gain_tbl, + .ps_gain_tbl_size = ARRAY_SIZE(ltr501_ps_gain_tbl), + .als_mode_active = BIT(0) | BIT(1), + .als_gain_mask = BIT(3), + .als_gain_shift = 3, + .info = <r501_info, + .channels = ltr501_channels, + }, + [ltr559] = { + .chip_id = 0x09, + .als_gain = ltr559_als_gain_tbl, + .als_gain_tbl_size = ARRAY_SIZE(ltr559_als_gain_tbl), + .ps_gain = ltr559_ps_gain_tbl, + .ps_gain_tbl_size = ARRAY_SIZE(ltr559_ps_gain_tbl), + .als_mode_active = BIT(1), + .als_gain_mask = BIT(2) | BIT(3) | BIT(4), + .als_gain_shift = 2, + .info = <r501_info, + .channels = ltr501_channels, + }, + [ltr301] = { + .chip_id = 0x08, + .als_gain = ltr501_als_gain_tbl, + .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), + .als_mode_active = BIT(0) | BIT(1), + .als_gain_mask = BIT(3), + .als_gain_shift = 3, + .info = <r301_info, + .channels = ltr301_channels, + }, +}; + static int ltr501_write_contr(struct i2c_client *client, u8 als_val, u8 ps_val) { int ret = i2c_smbus_write_byte_data(client, LTR501_ALS_CONTR, als_val); @@ -463,7 +503,7 @@ static int ltr501_probe(struct i2c_client *client, struct ltr501_data *data; struct iio_dev *indio_dev; const char *name = NULL; - int chip_id = 0; + int chip_id; int ret; indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data)); @@ -476,10 +516,12 @@ static int ltr501_probe(struct i2c_client *client, mutex_init(&data->lock_als); mutex_init(&data->lock_ps); + /* TODO: Add condition for ACPI */ if (id) { - name = id->name; + name = id->name ? id->name : LTR501_DRV_NAME; chip_id = id->driver_data; - } + } else + return -ENODEV; data->chip_info = <r501_chip_info_tbl[chip_id]; @@ -490,8 +532,8 @@ static int ltr501_probe(struct i2c_client *client, return -ENODEV; indio_dev->dev.parent = &client->dev; - indio_dev->info = <r501_info; - indio_dev->channels = ltr501_channels; + indio_dev->info = data->chip_info->info; + indio_dev->channels = data->chip_info->channels; indio_dev->num_channels = ARRAY_SIZE(ltr501_channels); indio_dev->name = name; indio_dev->modes = INDIO_DIRECT_MODE; @@ -557,6 +599,7 @@ static SIMPLE_DEV_PM_OPS(ltr501_pm_ops, ltr501_suspend, ltr501_resume); static const struct i2c_device_id ltr501_id[] = { { "ltr501", ltr501}, { "ltr559", ltr559}, + { "ltr301", ltr301 }, { } }; MODULE_DEVICE_TABLE(i2c, ltr501_id); -- 1.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v6 1/1] iio: ltr501: Add support for ltr301 2015-04-11 0:59 ` [PATCH v6 1/1] iio: ltr501: Add support for ltr301 Kuppuswamy Sathyanarayanan @ 2015-04-12 16:17 ` Jonathan Cameron 2015-04-14 12:07 ` Daniel Baluta 0 siblings, 1 reply; 4+ messages in thread From: Jonathan Cameron @ 2015-04-12 16:17 UTC (permalink / raw) To: Kuppuswamy Sathyanarayanan, pmeerw Cc: daniel.baluta, linux-iio, srinivas.pandruvada On 11/04/15 01:59, Kuppuswamy Sathyanarayanan wrote: > Added support for Liteon 301 Ambient light sensor. Since > LTR-301 and LTR-501 are register compatible(and even have same > part id), LTR-501 driver has been extended to support both > devices. LTR-501 is similar to LTR-301 in ALS sensing, But the > only difference is, LTR-501 also supports proximity sensing. > > LTR-501 - ALS + Proximity combo > LTR-301 - ALS sensor. > > Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> Looks good to me. Will either pick this up with Daniel's final series (trivial bits outstanding there) or separately if he doesn't pick it up. After his has gone in obviously! Should I miss it by accident (it's been known to happen, particularly if a large delay happens for some reason :( then feel free to poke me! Jonathan > --- > drivers/iio/light/Kconfig | 2 +- > drivers/iio/light/ltr501.c | 99 +++++++++++++++++++++++++++++++++------------- > 2 files changed, 72 insertions(+), 29 deletions(-) > > diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig > index a224afd..215e4a3 100644 > --- a/drivers/iio/light/Kconfig > +++ b/drivers/iio/light/Kconfig > @@ -159,7 +159,7 @@ config LTR501 > select IIO_TRIGGERED_BUFFER > help > If you say yes here you get support for the Lite-On LTR-501ALS-01 > - ambient light and proximity sensor. > + ambient light and proximity sensor or LTR-301 ambient light sensor. > > This driver can also be built as a module. If so, the module > will be called ltr501. > diff --git a/drivers/iio/light/ltr501.c b/drivers/iio/light/ltr501.c > index 5939cda..4945405 100644 > --- a/drivers/iio/light/ltr501.c > +++ b/drivers/iio/light/ltr501.c > @@ -50,6 +50,7 @@ > enum { > ltr501 = 0, > ltr559, > + ltr301, > }; > > struct ltr501_gain { > @@ -96,6 +97,8 @@ struct ltr501_chip_info { > u8 als_mode_active; > u8 als_gain_mask; > u8 als_gain_shift; > + struct iio_chan_spec const *channels; > + const struct iio_info *info; > }; > > struct ltr501_data { > @@ -105,29 +108,6 @@ struct ltr501_data { > u8 als_contr, ps_contr; > }; > > -static struct ltr501_chip_info ltr501_chip_info_tbl[] = { > - [ltr501] = { > - .chip_id = 0x08, > - .als_gain = ltr501_als_gain_tbl, > - .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), > - .ps_gain = ltr501_ps_gain_tbl, > - .ps_gain_tbl_size = ARRAY_SIZE(ltr501_ps_gain_tbl), > - .als_mode_active = BIT(0) | BIT(1), > - .als_gain_mask = BIT(3), > - .als_gain_shift = 3, > - }, > - [ltr559] = { > - .chip_id = 0x09, > - .als_gain = ltr559_als_gain_tbl, > - .als_gain_tbl_size = ARRAY_SIZE(ltr559_als_gain_tbl), > - .ps_gain = ltr559_ps_gain_tbl, > - .ps_gain_tbl_size = ARRAY_SIZE(ltr559_ps_gain_tbl), > - .als_mode_active = BIT(1), > - .als_gain_mask = BIT(2) | BIT(3) | BIT(4), > - .als_gain_shift = 2, > - }, > -}; > - > static int ltr501_drdy(struct ltr501_data *data, u8 drdy_mask) > { > int tries = 100; > @@ -202,6 +182,13 @@ static const struct iio_chan_spec ltr501_channels[] = { > IIO_CHAN_SOFT_TIMESTAMP(3), > }; > > +static const struct iio_chan_spec ltr301_channels[] = { > + LTR501_INTENSITY_CHANNEL(0, LTR501_ALS_DATA0, IIO_MOD_LIGHT_BOTH, 0), > + LTR501_INTENSITY_CHANNEL(1, LTR501_ALS_DATA1, IIO_MOD_LIGHT_IR, > + BIT(IIO_CHAN_INFO_SCALE)), > + IIO_CHAN_SOFT_TIMESTAMP(2), > +}; > + > static int ltr501_read_raw(struct iio_dev *indio_dev, > struct iio_chan_spec const *chan, > int *val, int *val2, long mask) > @@ -367,10 +354,19 @@ static struct attribute *ltr501_attributes[] = { > NULL > }; > > +static struct attribute *ltr301_attributes[] = { > + &iio_dev_attr_in_intensity_scale_available.dev_attr.attr, > + NULL > +}; > + > static const struct attribute_group ltr501_attribute_group = { > .attrs = ltr501_attributes, > }; > > +static const struct attribute_group ltr301_attribute_group = { > + .attrs = ltr301_attributes, > +}; > + > static const struct iio_info ltr501_info = { > .read_raw = ltr501_read_raw, > .write_raw = ltr501_write_raw, > @@ -378,6 +374,50 @@ static const struct iio_info ltr501_info = { > .driver_module = THIS_MODULE, > }; > > +static const struct iio_info ltr301_info = { > + .read_raw = ltr501_read_raw, > + .write_raw = ltr501_write_raw, > + .attrs = <r301_attribute_group, > + .driver_module = THIS_MODULE, > +}; > + > +static struct ltr501_chip_info ltr501_chip_info_tbl[] = { > + [ltr501] = { > + .chip_id = 0x08, > + .als_gain = ltr501_als_gain_tbl, > + .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), > + .ps_gain = ltr501_ps_gain_tbl, > + .ps_gain_tbl_size = ARRAY_SIZE(ltr501_ps_gain_tbl), > + .als_mode_active = BIT(0) | BIT(1), > + .als_gain_mask = BIT(3), > + .als_gain_shift = 3, > + .info = <r501_info, > + .channels = ltr501_channels, > + }, > + [ltr559] = { > + .chip_id = 0x09, > + .als_gain = ltr559_als_gain_tbl, > + .als_gain_tbl_size = ARRAY_SIZE(ltr559_als_gain_tbl), > + .ps_gain = ltr559_ps_gain_tbl, > + .ps_gain_tbl_size = ARRAY_SIZE(ltr559_ps_gain_tbl), > + .als_mode_active = BIT(1), > + .als_gain_mask = BIT(2) | BIT(3) | BIT(4), > + .als_gain_shift = 2, > + .info = <r501_info, > + .channels = ltr501_channels, > + }, > + [ltr301] = { > + .chip_id = 0x08, > + .als_gain = ltr501_als_gain_tbl, > + .als_gain_tbl_size = ARRAY_SIZE(ltr501_als_gain_tbl), > + .als_mode_active = BIT(0) | BIT(1), > + .als_gain_mask = BIT(3), > + .als_gain_shift = 3, > + .info = <r301_info, > + .channels = ltr301_channels, > + }, > +}; > + > static int ltr501_write_contr(struct i2c_client *client, u8 als_val, u8 ps_val) > { > int ret = i2c_smbus_write_byte_data(client, LTR501_ALS_CONTR, als_val); > @@ -463,7 +503,7 @@ static int ltr501_probe(struct i2c_client *client, > struct ltr501_data *data; > struct iio_dev *indio_dev; > const char *name = NULL; > - int chip_id = 0; > + int chip_id; > int ret; > > indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data)); > @@ -476,10 +516,12 @@ static int ltr501_probe(struct i2c_client *client, > mutex_init(&data->lock_als); > mutex_init(&data->lock_ps); > > + /* TODO: Add condition for ACPI */ > if (id) { > - name = id->name; > + name = id->name ? id->name : LTR501_DRV_NAME; > chip_id = id->driver_data; > - } > + } else > + return -ENODEV; > > data->chip_info = <r501_chip_info_tbl[chip_id]; > > @@ -490,8 +532,8 @@ static int ltr501_probe(struct i2c_client *client, > return -ENODEV; > > indio_dev->dev.parent = &client->dev; > - indio_dev->info = <r501_info; > - indio_dev->channels = ltr501_channels; > + indio_dev->info = data->chip_info->info; > + indio_dev->channels = data->chip_info->channels; > indio_dev->num_channels = ARRAY_SIZE(ltr501_channels); > indio_dev->name = name; > indio_dev->modes = INDIO_DIRECT_MODE; > @@ -557,6 +599,7 @@ static SIMPLE_DEV_PM_OPS(ltr501_pm_ops, ltr501_suspend, ltr501_resume); > static const struct i2c_device_id ltr501_id[] = { > { "ltr501", ltr501}, > { "ltr559", ltr559}, > + { "ltr301", ltr301 }, > { } > }; > MODULE_DEVICE_TABLE(i2c, ltr501_id); > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v6 1/1] iio: ltr501: Add support for ltr301 2015-04-12 16:17 ` Jonathan Cameron @ 2015-04-14 12:07 ` Daniel Baluta 0 siblings, 0 replies; 4+ messages in thread From: Daniel Baluta @ 2015-04-14 12:07 UTC (permalink / raw) To: Jonathan Cameron Cc: Kuppuswamy Sathyanarayanan, Peter Meerwald, Daniel Baluta, linux-iio@vger.kernel.org, Srinivas Pandruvada On Sun, Apr 12, 2015 at 7:17 PM, Jonathan Cameron <jic23@kernel.org> wrote: > On 11/04/15 01:59, Kuppuswamy Sathyanarayanan wrote: >> Added support for Liteon 301 Ambient light sensor. Since >> LTR-301 and LTR-501 are register compatible(and even have same >> part id), LTR-501 driver has been extended to support both >> devices. LTR-501 is similar to LTR-301 in ALS sensing, But the >> only difference is, LTR-501 also supports proximity sensing. >> >> LTR-501 - ALS + Proximity combo >> LTR-301 - ALS sensor. >> >> Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> > Looks good to me. Will either pick this up with Daniel's final series > (trivial bits outstanding there) or separately if he doesn't pick it up. > After his has gone in obviously! > > Should I miss it by accident (it's been known to happen, particularly if > a large delay happens for some reason :( then feel free to poke me! > > Jonathan I will pick this up and resend together with patches for LTR559. thanks, Daniel. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-04-14 12:07 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-04-11 0:59 [PATCH v6 0/1] LTR301 ALS support Kuppuswamy Sathyanarayanan 2015-04-11 0:59 ` [PATCH v6 1/1] iio: ltr501: Add support for ltr301 Kuppuswamy Sathyanarayanan 2015-04-12 16:17 ` Jonathan Cameron 2015-04-14 12:07 ` Daniel Baluta
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).