[PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[Uwe Kleine-König]

Returning an error value in an i2c remove callback results in an error
message being emitted by the i2c core, but otherwise it doesn't make a
difference.

As afe4404_remove() already emits an error message on failure and the
additional error message by the i2c core doesn't add any useful
information, change the return value to zero to suppress this message.

This patch is a preparation for making i2c remove callbacks return void.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
Hello,

this is similar to the patches I sent recently.
(https://lore.kernel.org/linux-iio/20220430081607.15078-1-u.kleine-koenig@pengutronix.de/)
Just missed this driver when I intended to address this problem type for
all of drivers/iio/.

Best regards
Uwe

 drivers/iio/health/afe4404.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/iio/health/afe4404.c b/drivers/iio/health/afe4404.c
index aa9311e1e655..1bb7de60f8ca 100644
--- a/drivers/iio/health/afe4404.c
+++ b/drivers/iio/health/afe4404.c
@@ -591,10 +591,8 @@ static int afe4404_remove(struct i2c_client *client)
 		iio_trigger_unregister(afe->trig);
 
 	ret = regulator_disable(afe->regulator);
-	if (ret) {
+	if (ret)
 		dev_err(afe->dev, "Unable to disable regulator\n");
-		return ret;
-	}
 
 	return 0;
 }

base-commit: 3123109284176b1532874591f7c81f3837bbdc17
-- 
2.35.1

Re: [PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[Jonathan Cameron]

On Sat, 14 May 2022 15:04:32 +0200
Uwe Kleine-König         <u.kleine-koenig@pengutronix.de> wrote:

> Returning an error value in an i2c remove callback results in an error
> message being emitted by the i2c core, but otherwise it doesn't make a
> difference.
> 
> As afe4404_remove() already emits an error message on failure and the
> additional error message by the i2c core doesn't add any useful
> information, change the return value to zero to suppress this message.
> 
> This patch is a preparation for making i2c remove callbacks return void.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Applied.

Like the others this will sit in testing until I can rebase the togreg
branch.

Thanks,

Jonathan

> ---
> Hello,
> 
> this is similar to the patches I sent recently.
> (https://lore.kernel.org/linux-iio/20220430081607.15078-1-u.kleine-koenig@pengutronix.de/)
> Just missed this driver when I intended to address this problem type for
> all of drivers/iio/.
> 
> Best regards
> Uwe
> 
>  drivers/iio/health/afe4404.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/iio/health/afe4404.c b/drivers/iio/health/afe4404.c
> index aa9311e1e655..1bb7de60f8ca 100644
> --- a/drivers/iio/health/afe4404.c
> +++ b/drivers/iio/health/afe4404.c
> @@ -591,10 +591,8 @@ static int afe4404_remove(struct i2c_client *client)
>  		iio_trigger_unregister(afe->trig);
>  
>  	ret = regulator_disable(afe->regulator);
> -	if (ret) {
> +	if (ret)
>  		dev_err(afe->dev, "Unable to disable regulator\n");
> -		return ret;
> -	}
>  
>  	return 0;
>  }
> 
> base-commit: 3123109284176b1532874591f7c81f3837bbdc17

Re: [PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1139 bytes --]

Hello Jonathan,

On Sat, May 14, 2022 at 02:38:12PM +0100, Jonathan Cameron wrote:
> On Sat, 14 May 2022 15:04:32 +0200
> Uwe Kleine-König         <u.kleine-koenig@pengutronix.de> wrote:
> 
> > Returning an error value in an i2c remove callback results in an error
> > message being emitted by the i2c core, but otherwise it doesn't make a
> > difference.
> > 
> > As afe4404_remove() already emits an error message on failure and the
> > additional error message by the i2c core doesn't add any useful
> > information, change the return value to zero to suppress this message.
> > 
> > This patch is a preparation for making i2c remove callbacks return void.
> > 
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> Applied.
> 
> Like the others this will sit in testing until I can rebase the togreg
> branch.

Don't know if it's to late to fix already, but I just noticed we'd want
s/io/iio/ in the Subject line.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[Jonathan Cameron]

On Sun, 22 May 2022 21:58:42 +0200
Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote:

> Hello Jonathan,
> 
> On Sat, May 14, 2022 at 02:38:12PM +0100, Jonathan Cameron wrote:
> > On Sat, 14 May 2022 15:04:32 +0200
> > Uwe Kleine-König         <u.kleine-koenig@pengutronix.de> wrote:
> >   
> > > Returning an error value in an i2c remove callback results in an error
> > > message being emitted by the i2c core, but otherwise it doesn't make a
> > > difference.
> > > 
> > > As afe4404_remove() already emits an error message on failure and the
> > > additional error message by the i2c core doesn't add any useful
> > > information, change the return value to zero to suppress this message.
> > > 
> > > This patch is a preparation for making i2c remove callbacks return void.
> > > 
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>  
> > Applied.
> > 
> > Like the others this will sit in testing until I can rebase the togreg
> > branch.  
> 
> Don't know if it's to late to fix already, but I just noticed we'd want
> s/io/iio/ in the Subject line.
Good spot. Fixed up as haven't pushed the tree out as non rebasing yet
(I'm waiting as I normally rebase on rc1).

Jonathan

> 
> Best regards
> Uwe
> 

Re: [PATCH] io:health:afe4404: Remove duplicated error reporting in .remove()

[Jonathan Cameron]

On Fri, 3 Jun 2022 16:16:38 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> On Sun, 22 May 2022 21:58:42 +0200
> Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote:
> 
> > Hello Jonathan,
> > 
> > On Sat, May 14, 2022 at 02:38:12PM +0100, Jonathan Cameron wrote:  
> > > On Sat, 14 May 2022 15:04:32 +0200
> > > Uwe Kleine-König         <u.kleine-koenig@pengutronix.de> wrote:
> > >     
> > > > Returning an error value in an i2c remove callback results in an error
> > > > message being emitted by the i2c core, but otherwise it doesn't make a
> > > > difference.
> > > > 
> > > > As afe4404_remove() already emits an error message on failure and the
> > > > additional error message by the i2c core doesn't add any useful
> > > > information, change the return value to zero to suppress this message.
> > > > 
> > > > This patch is a preparation for making i2c remove callbacks return void.
> > > > 
> > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>    
> > > Applied.
> > > 
> > > Like the others this will sit in testing until I can rebase the togreg
> > > branch.    
> > 
> > Don't know if it's to late to fix already, but I just noticed we'd want
> > s/io/iio/ in the Subject line.  
> Good spot. Fixed up as haven't pushed the tree out as non rebasing yet
> (I'm waiting as I normally rebase on rc1).
> 

Might get delayed a little longer than that as I've just seen Greg has
sent a late pull request for char-misc (unrelated to the IIO changes),
so might need to wait a little past rc1.

Jonathan

> Jonathan
> 
> > 
> > Best regards
> > Uwe
> >   
>