On 09/19/2018 10:27 AM, Ley Foon Tan wrote: > Disable and free clock when remove driver. > > Signed-off-by: Ley Foon Tan <ley.foon....@intel.com> > > --- > v2: > - Check return value from clk_free() and return 0 when exit function > --- > drivers/spi/designware_spi.c | 16 +++++++++++++++- > 1 file changed, 15 insertions(+), 1 deletion(-) > > diff --git a/drivers/spi/designware_spi.c b/drivers/spi/designware_spi.c > index 5cca414..8f0c602 100644 > --- a/drivers/spi/designware_spi.c > +++ b/drivers/spi/designware_spi.c > @@ -516,8 +516,22 @@ static int dw_spi_set_mode(struct udevice *bus, uint > mode) > static int dw_spi_remove(struct udevice *bus) > { > struct dw_spi_priv *priv = dev_get_priv(bus); > + int ret; > + > + ret = reset_release_bulk(&priv->resets); > + if (ret) > + return ret; > > - return reset_release_bulk(&priv->resets); > +#if CONFIG_IS_ENABLED(CLK) > + ret = clk_disable(&priv->clk); > + if (ret) > + return ret; > + > + ret = clk_free(&priv->clk); > + if (ret) > + return ret; > +#endif > + return 0; > } > > static const struct dm_spi_ops dw_spi_ops = { >
Acked-by: Marek Vasut <ma...@denx.de> btw this doesn't apply to socfpga/master . -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot