On 11/9/23 05:55, Igor Prusov wrote:
This adds dump function to struct clk_ops which should replace soc_clk_dump. It allows clock drivers to provide custom dump implementation without overriding generic CCF dump function.
Reviewed-by: Patrice Chotard patrice.chotard@foss.st.com Tested-by: Patrice Chotard patrice.chotard@foss.st.com Reviewed-by: Sean Anderson seanga2@gmail.com Signed-off-by: Igor Prusov ivprusov@sberdevices.ru
include/clk-uclass.h | 13 +++++++++++++ 1 file changed, 13 insertions(+)
diff --git a/include/clk-uclass.h b/include/clk-uclass.h index a22f1a5d84..f10dd213ff 100644 --- a/include/clk-uclass.h +++ b/include/clk-uclass.h @@ -25,6 +25,7 @@ struct ofnode_phandle_args;
- @set_parent: Set current clock parent
- @enable: Enable a clock.
- @disable: Disable a clock.
*/
- @dump: Print clock information.
- The individual methods are described more fully below.
@@ -39,6 +40,9 @@ struct clk_ops { int (*set_parent)(struct clk *clk, struct clk *parent); int (*enable)(struct clk *clk); int (*disable)(struct clk *clk); +#if IS_ENABLED(CONFIG_CMD_CLK)
- void (*dump)(struct udevice *dev);
+#endif };
#if 0 /* For documentation only */ @@ -135,6 +139,15 @@ int enable(struct clk *clk);
- Return: zero on success, or -ve error code.
*/ int disable(struct clk *clk);
+/**
- dump() - Print clock information.
- @clk: The clock device to dump.
The correct member here is @dev. I fixed this when applying.
--Sean
- If present, this function is called by "clk dump" command for each
- bound device.
- */
+void dump(struct udevice *dev); #endif
#endif