| 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
| ||
|
Message-Id: <20150723140516.A3C9E4314B@saturn.retrosnub.co.uk>
Date: Thu, 23 Jul 2015 15:05:16 +0100
From: jic23@...23.retrosnub.co.uk
To: Octavian Purdila <octavian.purdila@...el.com>
Cc: Jonathan Cameron <jic23@...nel.org>,
Mika Westerberg <mika.westerberg@...ux.intel.com>,
Linus Walleij <linus.walleij@...aro.org>,
Hartmut Knaack <knaack.h@....de>,
Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald <pmeerw@...erw.net>,
Alessandro Zummo <a.zummo@...ertech.it>,
Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
rtc-linux@...glegroups.com, lkml <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 1/2] iio: fix drivers that consider 0 as a valid
IRQ in client->irq
Octavian Purdila writes:
> On Fri, Jun 5, 2015 at 4:59 PM, Octavian Purdila
> <octavian.purdila@...el.com> wrote:
>> Since patch "i2c / ACPI: Use 0 to indicate that device does not have
>> interrupt assigned" [1], 0 is not a valid i2c client irq anymore, so
>> change all driver's checks accordingly.
>>
>> The same issue occurs when the device is instantiated via device tree
>> with no IRQ, or from the i2c sysfs interface, even before the patch
>> above.
>>
>> [1] http://lkml.kernel.org/g/<1430908148-201129-3-git-send-email-mika.westerberg@...ux.intel.com>
>>
>> Signed-off-by: Octavian Purdila <octavian.purdila@...el.com>
>> Reviewed-by: Mika Westerberg <mika.westerberg@...ux.intel.com>
>
> Hi Jonathan,
>
> Does this look OK to you? If so, could you pleas ACK the patch so that
> Linus can pick it up in its for-next branch if/when needed?
>
> Thanks,
> Tavi
>
Hi Tavi,
This is fine, but is there a particular rush to get it in?
Otherwise I'll just take it through the IIO tree.
Acked-by: Jonathan Cameron <jic23@...nel.org>
Thanks,
Jonathan
>
>> ---
>> drivers/iio/accel/bmc150-accel.c | 2 +-
>> drivers/iio/accel/kxcjk-1013.c | 2 +-
>> drivers/iio/accel/mma9553.c | 2 +-
>> drivers/iio/imu/kmx61.c | 8 ++++----
>> 4 files changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/iio/accel/bmc150-accel.c b/drivers/iio/accel/bmc150-accel.c
>> index 4e70f51..55751d9 100644
>> --- a/drivers/iio/accel/bmc150-accel.c
>> +++ b/drivers/iio/accel/bmc150-accel.c
>> @@ -1663,7 +1663,7 @@ static int bmc150_accel_probe(struct i2c_client *client,
>> if (client->irq < 0)
>> client->irq = bmc150_accel_gpio_probe(client, data);
>>
>> - if (client->irq >= 0) {
>> + if (client->irq > 0) {
>> ret = devm_request_threaded_irq(
>> &client->dev, client->irq,
>> bmc150_accel_irq_handler,
>> diff --git a/drivers/iio/accel/kxcjk-1013.c b/drivers/iio/accel/kxcjk-1013.c
>> index 0d9bd35..aa93cbd 100644
>> --- a/drivers/iio/accel/kxcjk-1013.c
>> +++ b/drivers/iio/accel/kxcjk-1013.c
>> @@ -1243,7 +1243,7 @@ static int kxcjk1013_probe(struct i2c_client *client,
>> if (client->irq < 0)
>> client->irq = kxcjk1013_gpio_probe(client, data);
>>
>> - if (client->irq >= 0) {
>> + if (client->irq > 0) {
>> ret = devm_request_threaded_irq(&client->dev, client->irq,
>> kxcjk1013_data_rdy_trig_poll,
>> kxcjk1013_event_handler,
>> diff --git a/drivers/iio/accel/mma9553.c b/drivers/iio/accel/mma9553.c
>> index 9d649c4..df043b3 100644
>> --- a/drivers/iio/accel/mma9553.c
>> +++ b/drivers/iio/accel/mma9553.c
>> @@ -1143,7 +1143,7 @@ static int mma9553_probe(struct i2c_client *client,
>> if (client->irq < 0)
>> client->irq = mma9553_gpio_probe(client);
>>
>> - if (client->irq >= 0) {
>> + if (client->irq > 0) {
>> ret = devm_request_threaded_irq(&client->dev, client->irq,
>> mma9553_irq_handler,
>> mma9553_event_handler,
>> diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
>> index 462a010..82cdf50 100644
>> --- a/drivers/iio/imu/kmx61.c
>> +++ b/drivers/iio/imu/kmx61.c
>> @@ -1363,7 +1363,7 @@ static int kmx61_probe(struct i2c_client *client,
>> if (client->irq < 0)
>> client->irq = kmx61_gpio_probe(client, data);
>>
>> - if (client->irq >= 0) {
>> + if (client->irq > 0) {
>> ret = devm_request_threaded_irq(&client->dev, client->irq,
>> kmx61_data_rdy_trig_poll,
>> kmx61_event_handler,
>> @@ -1445,10 +1445,10 @@ err_iio_unregister_mag:
>> err_iio_unregister_acc:
>> iio_device_unregister(data->acc_indio_dev);
>> err_buffer_cleanup_mag:
>> - if (client->irq >= 0)
>> + if (client->irq > 0)
>> iio_triggered_buffer_cleanup(data->mag_indio_dev);
>> err_buffer_cleanup_acc:
>> - if (client->irq >= 0)
>> + if (client->irq > 0)
>> iio_triggered_buffer_cleanup(data->acc_indio_dev);
>> err_trigger_unregister_motion:
>> iio_trigger_unregister(data->motion_trig);
>> @@ -1472,7 +1472,7 @@ static int kmx61_remove(struct i2c_client *client)
>> iio_device_unregister(data->acc_indio_dev);
>> iio_device_unregister(data->mag_indio_dev);
>>
>> - if (client->irq >= 0) {
>> + if (client->irq > 0) {
>> iio_triggered_buffer_cleanup(data->acc_indio_dev);
>> iio_triggered_buffer_cleanup(data->mag_indio_dev);
>> iio_trigger_unregister(data->acc_dready_trig);
>> --
>> 1.9.1
>>
--
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/
Powered by blists - more mailing lists