2023-06-15 10:31:52

by Claudiu Beznea

Subject: [PATCH] clk: fix typo in clk_hw_register_fixed_rate_parent_data() macro

clk_hw_register_fixed_rate_parent_data() 3rd parameter is parent_data
not parent_hw. Inner function (__clk_hw_register_fixed_rate()) is called
with parent_data parameter as valid. To have this parameter taken into
account update the name of the 3rd parameter of
clk_hw_register_fixed_rate_parent_data() macro to parent_data.

Signed-off-by: Claudiu Beznea <[email protected]>
---
include/linux/clk-provider.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index 842e72a5348f..0b18a17dca3e 100644
--- a/include/linux/clk-provider.h
+++ b/include/linux/clk-provider.h
@@ -415,7 +415,7 @@ struct clk *clk_register_fixed_rate(struct device *dev, const char *name,
* @flags: framework-specific flags
* @fixed_rate: non-adjustable clock rate
*/
-#define clk_hw_register_fixed_rate_parent_data(dev, name, parent_hw, flags, \
+#define clk_hw_register_fixed_rate_parent_data(dev, name, parent_data, flags, \
fixed_rate) \
__clk_hw_register_fixed_rate((dev), NULL, (name), NULL, NULL, \
(parent_data), (flags), (fixed_rate), 0, \
--
2.34.1

2023-06-20 19:54:45

by Stephen Boyd

Subject: Re: [PATCH] clk: fix typo in clk_hw_register_fixed_rate_parent_data() macro

Quoting Claudiu Beznea (2023-06-15 03:19:31)
> clk_hw_register_fixed_rate_parent_data() 3rd parameter is parent_data
> not parent_hw. Inner function (__clk_hw_register_fixed_rate()) is called
> with parent_data parameter as valid. To have this parameter taken into
> account update the name of the 3rd parameter of
> clk_hw_register_fixed_rate_parent_data() macro to parent_data.
>
> Signed-off-by: Claudiu Beznea <[email protected]>
> ---

Applied to clk-next