Re: [PATCH v2 09/20] staging:iio:lis2l02dq: Share threshold value between axis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/12/2013 01:45 PM, Jonathan Cameron wrote:
> On 10/07/13 15:11, Lars-Peter Clausen wrote:
>> The threshold event can be enabled/disabled separately, but the threshold value
>> is shared between all three axis.
>>
>> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
> Build fix applied as below.
>> ---
>>  drivers/staging/iio/accel/lis3l02dq_core.c | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging/iio/accel/lis3l02dq_core.c
>> index 78187f1..b2d1dfa 100644
>> --- a/drivers/staging/iio/accel/lis3l02dq_core.c
>> +++ b/drivers/staging/iio/accel/lis3l02dq_core.c
>> @@ -518,13 +518,13 @@ static const struct iio_event_spec lis3l02dq_event[] = {
>>  	{
>>  		.type = IIO_EV_TYPE_THRESH,
>>  		.dir = IIO_EV_DIR_RISING,
>> -		.mask_separate = BIT(IIO_EV_INFO_VALUE) |
>> -			BIT(IIO_EV_INFO_ENABLE),
>> +		.mask_separate = BIT(IIO_EV_INFO_ENABLE);
> semicolons?

Hm, I might have send out the wrong version of that branch. I think the only
other difference was slightly better wording for the hysteresis sysfs
documentation.

>> +		.mask_shared_by_type = BIT(IIO_EV_INFO_VALUE);
>>  	}, {
>>  		.type = IIO_EV_TYPE_THRESH,
>>  		.dir = IIO_EV_DIR_FALLING,
>> -		.mask_separate = BIT(IIO_EV_INFO_VALUE) |
>> -			BIT(IIO_EV_INFO_ENABLE),
>> +		.mask_separate = BIT(IIO_EV_INFO_ENABLE);
>> +		.mask_shared_by_type = BIT(IIO_EV_INFO_VALUE);
>>  	}
>>  };
>>  
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux