linux-kernel - Re: [PATCH 11/20] iio: sensors-core: st: Support sensors which don't have a Data Ready pin

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <523D9A0D.50306@kernel.org>
Date:	Sat, 21 Sep 2013 14:07:25 +0100
From:	Jonathan Cameron <jic23@...nel.org>
To:	Lee Jones <lee.jones@...aro.org>
CC:	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	jic23@....ac.uk, arnd@...db.de, linus.walleij@...aro.org,
	denis.ciocca@...com, linux-iio@...r.kernel.org
Subject: Re: [PATCH 11/20] iio: sensors-core: st: Support sensors which don't
 have a Data Ready pin

On 09/21/13 13:51, Jonathan Cameron wrote:
> On 09/16/13 17:02, Lee Jones wrote:
>> Not all ST's sensors support data ready, so let's make the declaration
>> of one conditional.
>>
>> Signed-off-by: Lee Jones <lee.jones@...aro.org>
> Applied to the togreg branch of iio.git
> 
> Thanks Lee
Having done some extra build tests, noticed that
for st_snesors_set_drdy_int_pin should be static.

Fixed up and push out to the togreg branch of iio.git

>> ---
>>  drivers/iio/common/st_sensors/st_sensors_core.c | 33 ++++++++++++++++---------
>>  drivers/iio/pressure/st_pressure_core.c         |  3 ++-
>>  2 files changed, 24 insertions(+), 12 deletions(-)
>>
>> diff --git a/drivers/iio/common/st_sensors/st_sensors_core.c b/drivers/iio/common/st_sensors/st_sensors_core.c
>> index 793136a..2672630 100644
>> --- a/drivers/iio/common/st_sensors/st_sensors_core.c
>> +++ b/drivers/iio/common/st_sensors/st_sensors_core.c
>> @@ -198,21 +198,17 @@ int st_sensors_set_axis_enable(struct iio_dev *indio_dev, u8 axis_enable)
>>  }
>>  EXPORT_SYMBOL(st_sensors_set_axis_enable);
>>  
>> -int st_sensors_init_sensor(struct iio_dev *indio_dev,
>> -					struct st_sensors_platform_data *pdata)
>> +int st_sensors_set_drdy_int_pin(struct iio_dev *indio_dev,
>> +				struct st_sensors_platform_data *pdata)
>>  {
>> -	int err;
>>  	struct st_sensor_data *sdata = iio_priv(indio_dev);
>>  
>> -	mutex_init(&sdata->tb.buf_lock);
>> -
>>  	switch (pdata->drdy_int_pin) {
>>  	case 1:
>>  		if (sdata->sensor->drdy_irq.mask_int1 == 0) {
>>  			dev_err(&indio_dev->dev,
>>  					"DRDY on INT1 not available.\n");
>> -			err = -EINVAL;
>> -			goto init_error;
>> +			return -EINVAL;
>>  		}
>>  		sdata->drdy_int_pin = 1;
>>  		break;
>> @@ -220,17 +216,29 @@ int st_sensors_init_sensor(struct iio_dev *indio_dev,
>>  		if (sdata->sensor->drdy_irq.mask_int2 == 0) {
>>  			dev_err(&indio_dev->dev,
>>  					"DRDY on INT2 not available.\n");
>> -			err = -EINVAL;
>> -			goto init_error;
>> +			return -EINVAL;
>>  		}
>>  		sdata->drdy_int_pin = 2;
>>  		break;
>>  	default:
>>  		dev_err(&indio_dev->dev, "DRDY on pdata not valid.\n");
>> -		err = -EINVAL;
>> -		goto init_error;
>> +		return -EINVAL;
>>  	}
>>  
>> +	return 0;
>> +}
>> +
>> +int st_sensors_init_sensor(struct iio_dev *indio_dev,
>> +					struct st_sensors_platform_data *pdata)
>> +{
>> +	struct st_sensor_data *sdata = iio_priv(indio_dev);
>> +	int err = 0;
>> +
>> +	mutex_init(&sdata->tb.buf_lock);
>> +
>> +	if (pdata)
>> +		err = st_sensors_set_drdy_int_pin(indio_dev, pdata);
>> +
>>  	err = st_sensors_set_enable(indio_dev, false);
>>  	if (err < 0)
>>  		goto init_error;
>> @@ -266,6 +274,9 @@ int st_sensors_set_dataready_irq(struct iio_dev *indio_dev, bool enable)
>>  	u8 drdy_mask;
>>  	struct st_sensor_data *sdata = iio_priv(indio_dev);
>>  
>> +	if (!sdata->sensor->drdy_irq.addr)
>> +		return 0;
>> +
>>  	/* Enable/Disable the interrupt generator 1. */
>>  	if (sdata->sensor->drdy_irq.ig1.en_addr > 0) {
>>  		err = st_sensors_write_data_with_mask(indio_dev,
>> diff --git a/drivers/iio/pressure/st_pressure_core.c b/drivers/iio/pressure/st_pressure_core.c
>> index 93bff9b..3abada2 100644
>> --- a/drivers/iio/pressure/st_pressure_core.c
>> +++ b/drivers/iio/pressure/st_pressure_core.c
>> @@ -254,7 +254,8 @@ int st_press_common_probe(struct iio_dev *indio_dev,
>>  
>>  	pdata->odr = pdata->sensor->odr.odr_avl[0].hz;
>>  
>> -	if (!plat_data)
>> +	/* Some devices don't support a data ready pin. */
>> +	if (!plat_data && pdata->sensor->drdy_irq.addr)
>>  		plat_data =
>>  			(struct st_sensors_platform_data *)&default_press_pdata;
>>  
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/