2024-04-13 13:55:05

by Christophe JAILLET

[permalink] [raw]
Subject: [PATCH] clk: highbank: Remove an unused field in struct hb_clk

In "struct hb_clk", the 'parent_name' field is unused.

Remove it.

Found with cppcheck, unusedStructMember.

Signed-off-by: Christophe JAILLET <[email protected]>
---
Compile tested only.

Apparently, it has never been used. It is not a left-over from a
refactoring.
---
drivers/clk/clk-highbank.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/clk/clk-highbank.c b/drivers/clk/clk-highbank.c
index 2a0cea2946f9..6e68a41a70a1 100644
--- a/drivers/clk/clk-highbank.c
+++ b/drivers/clk/clk-highbank.c
@@ -37,7 +37,6 @@
struct hb_clk {
struct clk_hw hw;
void __iomem *reg;
- char *parent_name;
};
#define to_hb_clk(p) container_of(p, struct hb_clk, hw)

--
2.44.0



2024-04-20 01:58:04

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] clk: highbank: Remove an unused field in struct hb_clk

Quoting Christophe JAILLET (2024-04-13 06:54:35)
> In "struct hb_clk", the 'parent_name' field is unused.
>
> Remove it.
>
> Found with cppcheck, unusedStructMember.
>
> Signed-off-by: Christophe JAILLET <[email protected]>
> ---

Applied to clk-next