[PATCH 2/3] i2c: designware_i2c: Remove clk disabled codes
Heiko Schocher
hs at nabladev.com
Wed Apr 29 08:10:16 CEST 2026
Hello Dinesh,
On 29.04.26 07:11, dinesh.maniyam at altera.com wrote:
> From: Dinesh Maniyam <dinesh.maniyam at altera.com>
>
> l4_sp_clk is disabled when i2c device is removed, this would
> causing unexpected failing behavior for other peripherals which
> are still running with l4_sp_clk. The patch removes the codes
> implementation for disabling the lp_spl_clk.
>
> Signed-off-by: Tien Fong Chee <tien.fong.chee at altera.com>
> Signed-off-by: Dinesh Maniyam <dinesh.maniyam at altera.com>
> ---
> drivers/i2c/designware_i2c.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/drivers/i2c/designware_i2c.c b/drivers/i2c/designware_i2c.c
> index 8ad716f410e..40153c72545 100644
> --- a/drivers/i2c/designware_i2c.c
> +++ b/drivers/i2c/designware_i2c.c
> @@ -807,10 +807,6 @@ int designware_i2c_remove(struct udevice *dev)
> {
> struct dw_i2c *priv = dev_get_priv(dev);
>
> -#if CONFIG_IS_ENABLED(CLK)
> - clk_disable(&priv->clk);
> -#endif
Hmm... clock framework should have a counter how many times clock
is enabled... with your patch, clock never gets disabled...
bye,
Heiko
> -
> return reset_release_bulk(&priv->resets);
> }
>
>
--
Nabla Software Engineering
HRB 40522 Augsburg
Phone: +49 821 45592596
E-Mail: office at nabladev.com
Geschäftsführer : Stefano Babic
More information about the U-Boot
mailing list