On Thu, 31 Dec 2020 08:53:22 +0000 Xu Wang <vulab@xxxxxxxxxxx> wrote: > ecause clk_disable_unprepare() already checked NULL clock parameter, > so the additional check is unnecessary, just remove it. > > Signed-off-by: Xu Wang <vulab@xxxxxxxxxxx> Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to see if we missed anything. thanks, Jonathan > --- > drivers/iio/adc/stm32-dfsdm-core.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/iio/adc/stm32-dfsdm-core.c b/drivers/iio/adc/stm32-dfsdm-core.c > index 42a7377704a4..bb925a11c8ae 100644 > --- a/drivers/iio/adc/stm32-dfsdm-core.c > +++ b/drivers/iio/adc/stm32-dfsdm-core.c > @@ -117,8 +117,7 @@ static void stm32_dfsdm_clk_disable_unprepare(struct stm32_dfsdm *dfsdm) > { > struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > > - if (priv->aclk) > - clk_disable_unprepare(priv->aclk); > + clk_disable_unprepare(priv->aclk); > clk_disable_unprepare(priv->clk); > } >