Oops. My bad marked it for stable when it was just for the current cycle.
Don't worry about the message from Greg.
Jonathan
On November 10, 2014 2:42:22 PM GMT+00:00, Daniel Baluta <daniel.baluta@xxxxxxxxx> wrote:
>Hi Greg,
>
>I am not sure why this error messages was generated.
>
>Current patch fixes: b4b491c083 (iio: accel: kxcjk-1013: Support
>threshold) but this commit doesn't seem to be in 3.17 stable kernel.
>
>Daniel.
>
>On 11/10/2014 06:22 AM, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
>>
>> The patch below does not apply to the 3.17-stable tree.
>> If someone wants it applied there, or to any other stable or longterm
>> tree, then please email the backport, including the original git
>commit
>> id to <stable@xxxxxxxxxxxxxxx>.
>>
>> thanks,
>>
>> greg k-h
>>
>> ------------------ original commit in Linus's tree ------------------
>>
>> From 25afffe16d07909197f99163ca7cd1f80fb5cbdd Mon Sep 17 00:00:00
>2001
>> From: Daniel Baluta <daniel.baluta@xxxxxxxxx>
>> Date: Fri, 10 Oct 2014 15:53:00 +0100
>> Subject: [PATCH] io: accel: kxcjk-1013: Fix iio_event_spec direction
>>
>> Because IIO_EV_DIR_* are not bitmasks but enums,
>> IIO_EV_DIR_RISING | IIO_EV_DIR_FALLING is not equal
>> with IIO_EV_DIR_EITHER.
>>
>> This could lead to potential misformatted sysfs attributes
>> like:
>> * in_accel_x_thresh_(null)_en
>> * in_accel_x_thresh_(null)_period
>> * in_accel_x_thresh_(null)_value
>>
>> or even memory corruption.
>>
>> Fixes: b4b491c083 (iio: accel: kxcjk-1013: Support threshold)
>> Signed-off-by: Daniel Baluta <daniel.baluta@xxxxxxxxx>
>> Cc: <Stable@xxxxxxxxxxxxxxx>
>> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxxx>
>>
>> diff --git a/drivers/iio/accel/kxcjk-1013.c
>b/drivers/iio/accel/kxcjk-1013.c
>> index 98909a9e284e..a23e58c4ed99 100644
>> --- a/drivers/iio/accel/kxcjk-1013.c
>> +++ b/drivers/iio/accel/kxcjk-1013.c
>> @@ -894,7 +894,7 @@ static const struct attribute_group
>kxcjk1013_attrs_group = {
>>
>> static const struct iio_event_spec kxcjk1013_event = {
>> .type = IIO_EV_TYPE_THRESH,
>> - .dir = IIO_EV_DIR_RISING | IIO_EV_DIR_FALLING,
>> + .dir = IIO_EV_DIR_EITHER,
>> .mask_separate = BIT(IIO_EV_INFO_VALUE) |
>> BIT(IIO_EV_INFO_ENABLE) |
>> BIT(IIO_EV_INFO_PERIOD)
>>
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html