[PATCH] clk: fixed-rate: Remove redundant if statement

Li Zhengyu posted 1 patch 3 years, 12 months ago
drivers/clk/clk-fixed-rate.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] clk: fixed-rate: Remove redundant if statement
Posted by Li Zhengyu 3 years, 12 months ago
(np) is always true when (dev || !np) is false, so just remove
the check.

Signed-off-by: Li Zhengyu <lizhengyu3@huawei.com>
---
 drivers/clk/clk-fixed-rate.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/clk-fixed-rate.c b/drivers/clk/clk-fixed-rate.c
index 45501637705c..ac68a6b40f0e 100644
--- a/drivers/clk/clk-fixed-rate.c
+++ b/drivers/clk/clk-fixed-rate.c
@@ -87,7 +87,7 @@ struct clk_hw *__clk_hw_register_fixed_rate(struct device *dev,
 	hw = &fixed->hw;
 	if (dev || !np)
 		ret = clk_hw_register(dev, hw);
-	else if (np)
+	else
 		ret = of_clk_hw_register(np, hw);
 	if (ret) {
 		kfree(fixed);
-- 
2.17.1
Re: [PATCH] clk: fixed-rate: Remove redundant if statement
Posted by Stephen Boyd 3 years, 11 months ago
Quoting Li Zhengyu (2022-05-09 02:21:02)
> (np) is always true when (dev || !np) is false, so just remove
> the check.
> 
> Signed-off-by: Li Zhengyu <lizhengyu3@huawei.com>
> ---

Applied to clk-next