[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <SJ0PR03MB6778660019C2553E5E18F8EC99099@SJ0PR03MB6778.namprd03.prod.outlook.com>
Date: Fri, 18 Nov 2022 10:13:25 +0000
From: "Sa, Nuno" <Nuno.Sa@...log.com>
To: "Bolboaca, Ramona" <Ramona.Bolboaca@...log.com>,
"jic23@...nel.org" <jic23@...nel.org>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC: "Bolboaca, Ramona" <Ramona.Bolboaca@...log.com>
Subject: RE: [PATCH v3 01/10] iio: adis: add '__adis_enable_irq()'
implementation
> -----Original Message-----
> From: Ramona Bolboaca <ramona.bolboaca@...log.com>
> Sent: Friday, November 18, 2022 9:31 AM
> To: jic23@...nel.org; Sa, Nuno <Nuno.Sa@...log.com>; linux-
> iio@...r.kernel.org; linux-kernel@...r.kernel.org
> Cc: Bolboaca, Ramona <Ramona.Bolboaca@...log.com>
> Subject: [PATCH v3 01/10] iio: adis: add '__adis_enable_irq()' implementation
>
> Add '__adis_enable_irq()' implementation which is the unlocked
> version of 'adis_enable_irq()'.
>
> Signed-off-by: Ramona Bolboaca <ramona.bolboaca@...log.com>
> ---
> changes in v3:
> - new patch
> drivers/iio/imu/adis.c | 26 +++++++++-----------------
> include/linux/iio/imu/adis.h | 12 +++++++++++-
> 2 files changed, 20 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
> index f7fcfd04f659..cfcb4abeb5db 100644
> --- a/drivers/iio/imu/adis.c
> +++ b/drivers/iio/imu/adis.c
> @@ -270,23 +270,19 @@ EXPORT_SYMBOL_NS(adis_debugfs_reg_access,
> IIO_ADISLIB);
> #endif
>
> /**
> - * adis_enable_irq() - Enable or disable data ready IRQ
> + * __adis_enable_irq() - Enable or disable data ready IRQ (unlocked)
> * @adis: The adis device
> * @enable: Whether to enable the IRQ
> *
> * Returns 0 on success, negative error code otherwise
> */
> -int adis_enable_irq(struct adis *adis, bool enable)
> +int __adis_enable_irq(struct adis *adis, bool enable)
> {
> - int ret = 0;
> + int ret;
This change was not really needed as long as we 'return ret'
In the ' unmasked_drdy' case. Anyways, not a big deal for me...
- Nuno Sá
Powered by blists - more mailing lists