projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
cef49e5
)
iio: frequency: adf4350: Remove redundant null check before clk_disable_unprepare
author
Xu Wang
<vulab@iscas.ac.cn>
Fri, 18 Dec 2020 09:46:47 +0000
(09:46 +0000)
committer
Jonathan Cameron
<Jonathan.Cameron@huawei.com>
Sat, 9 Jan 2021 21:52:46 +0000
(21:52 +0000)
Because clk_disable_unprepare() already checked NULL clock parameter,
so the additional check is unnecessary, just remove it.
Signed-off-by: Xu Wang <vulab@iscas.ac.cn>
Link:
https://lore.kernel.org/r/20201218094647.1386-1-vulab@iscas.ac.cn
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/frequency/adf4350.c
patch
|
blob
|
history
diff --git
a/drivers/iio/frequency/adf4350.c
b/drivers/iio/frequency/adf4350.c
index 82c050a3899d9f4f2558339979fd0ec7df7c6680..1462a6a5bc6dad73ea2157f0fc9dcfee521047f5 100644
(file)
--- a/
drivers/iio/frequency/adf4350.c
+++ b/
drivers/iio/frequency/adf4350.c
@@
-582,8
+582,7
@@
error_disable_reg:
if (!IS_ERR(st->reg))
regulator_disable(st->reg);
error_disable_clk:
- if (clk)
- clk_disable_unprepare(clk);
+ clk_disable_unprepare(clk);
return ret;
}
@@
-599,8
+598,7
@@
static int adf4350_remove(struct spi_device *spi)
iio_device_unregister(indio_dev);
- if (st->clk)
- clk_disable_unprepare(st->clk);
+ clk_disable_unprepare(st->clk);
if (!IS_ERR(reg))
regulator_disable(reg);