Re: [PATCH] iio: adc: stm32-dfsdm: Remove redundant null check before clk_disable_unprepare
From: Jonathan Cameron <jic23@kernel.org>
Date: 2021-01-17 12:36:06
Also in:
linux-arm-kernel, lkml
On Thu, 31 Dec 2020 08:53:22 +0000 Xu Wang [off-list ref] wrote:
ecause clk_disable_unprepare() already checked NULL clock parameter, so the additional check is unnecessary, just remove it. Signed-off-by: Xu Wang <redacted>
Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to see if we missed anything. thanks, Jonathan
quoted hunk ↗ jump to hunk
--- 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); }