[-next] clk: meson: Remove unneeded semicolon

Message ID 20230614083943.85660-1-yang.lee@linux.alibaba.com
State New
Headers
Series [-next] clk: meson: Remove unneeded semicolon |

Commit Message

Yang Li June 14, 2023, 8:39 a.m. UTC
  ./drivers/clk/meson/clk-pll.c:373:2-3: Unneeded semicolon

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5531
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/clk/meson/clk-pll.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Martin Blumenstingl June 14, 2023, 8:13 p.m. UTC | #1
On Wed, Jun 14, 2023 at 10:40 AM Yang Li <yang.lee@linux.alibaba.com> wrote:
>
> ./drivers/clk/meson/clk-pll.c:373:2-3: Unneeded semicolon
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5531
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
An identical patch was sent within minutes of this one: [0]
The other one has two Reviewed-by already. So I think this one should
be discarded.
Still, thanks for following up on this issue!


Best regards,
Martin


[0] https://lore.kernel.org/linux-amlogic/20230614084808.98819-1-jiapeng.chong@linux.alibaba.com/
  

Patch

diff --git a/drivers/clk/meson/clk-pll.c b/drivers/clk/meson/clk-pll.c
index 56ec2210f1ad..8fef90bf962f 100644
--- a/drivers/clk/meson/clk-pll.c
+++ b/drivers/clk/meson/clk-pll.c
@@ -370,7 +370,7 @@  static int meson_clk_pll_enable(struct clk_hw *hw)
 		usleep_range(10, 20);
 		meson_parm_write(clk->map, &pll->current_en, 1);
 		usleep_range(40, 50);
-	};
+	}
 
 	if (MESON_PARM_APPLICABLE(&pll->l_detect)) {
 		meson_parm_write(clk->map, &pll->l_detect, 1);