linux-kernel - Re: [PATCH] i2c: busses: i2c-nomadik: Remove a useless call in the remove function

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8ac04af8-eff8-bfe1-647a-2d04f5739702@wanadoo.fr>
Date:   Fri, 30 Jun 2023 22:10:40 +0200
From:   Christophe JAILLET <christophe.jaillet@...adoo.fr>
To:     linus.walleij@...aro.org, Andi Shyti <andi.shyti@...nel.org>
Cc:     linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        kernel-janitors@...r.kernel.org,
        "linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>
Subject: Re: [PATCH] i2c: busses: i2c-nomadik: Remove a useless call in the
 remove function

Hi,

"drivers/i2c/busses/i2c-nomadik.c" just got some loved, so I was 
wondering if the below patch could be reviewed.

It does not apply anymore because of the recent change, but the 
explanation in the commit message and the proposed change still make 
sense to me.

If it looks valid, I could send a V2.

So any feedback would be appreciated.

CJ


Le 21/08/2021 à 09:17, Christophe JAILLET a écrit :
> Since commit a410963ba4c0 ("Merge branch 'i2c-embedded/for-next' of
> git://git.pengutronix.de/git/wsa/linux"), there is no more
> 'request_mem_region()' call in this driver.
> 
> So remove the 'release_mem_region()' call from the remove function which is
> likely a left over.
> 
> There is no details in the above commit log, but at its end we can read:
>     Conflicts:
> 	   drivers/i2c/busses/i2c-nomadik.c
> 
> This may explain why this call has been left here.
> 
> Fixes: a410963ba4c0 ("Merge branch 'i2c-embedded/for-next' of git://git.pengutronix.de/git/wsa/linux
> Signed-off-by: Christophe JAILLET <christophe.jaillet@...adoo.fr>
> ---
> This patch is completely speculative and without details about commit
> a410963ba4c0 it's hard to be sure of the intent.
> 
> All I can say is that it looks logical to me and that it compiles!
> ---
>   drivers/i2c/busses/i2c-nomadik.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/i2c/busses/i2c-nomadik.c b/drivers/i2c/busses/i2c-nomadik.c
> index a2d12a5b1c34..e215a7357873 100644
> --- a/drivers/i2c/busses/i2c-nomadik.c
> +++ b/drivers/i2c/busses/i2c-nomadik.c
> @@ -1057,7 +1057,6 @@ static int nmk_i2c_probe(struct amba_device *adev, const struct amba_id *id)
>   
>   static void nmk_i2c_remove(struct amba_device *adev)
>   {
> -	struct resource *res = &adev->res;
>   	struct nmk_i2c_dev *dev = amba_get_drvdata(adev);
>   
>   	i2c_del_adapter(&dev->adap);
> @@ -1067,7 +1066,6 @@ static void nmk_i2c_remove(struct amba_device *adev)
>   	/* disable the controller */
>   	i2c_clr_bit(dev->virtbase + I2C_CR, I2C_CR_PE);
>   	clk_disable_unprepare(dev->clk);
> -	release_mem_region(res->start, resource_size(res));
>   }
>   
>   static struct i2c_vendor_data vendor_stn8815 = {