mbox series

[v2,0/4] clk: hisilicon: Migrate devm APIs

Message ID 20230329075104.165176-1-mmyangfl@gmail.com
Headers
Series clk: hisilicon: Migrate devm APIs |

Message

Yangfl March 29, 2023, 7:50 a.m. UTC 
  Migrate devm APIs for HiSilicon clock drivers and remove redundant codes.

This series is a partial improvement of [1]

v2: fix test robot error

Links:
[1]: https://lore.kernel.org/r/20230322164201.2454771-1-mmyangfl@gmail.com
v1: https://lore.kernel.org/r/20230326052757.297551-1-mmyangfl@gmail.com

David Yang (4):
  clk: hisilicon: Add helper functions for platform driver
  clk: hisilicon: Use helper functions
  clk: hisilicon: Convert to platform driver
  clk: hisilicon: Migrate devm APIs

 drivers/clk/hisilicon/clk-hi3519.c        | 134 +------
 drivers/clk/hisilicon/clk-hi3559a.c       | 253 +++----------
 drivers/clk/hisilicon/clk-hi3620.c        | 215 +++++------
 drivers/clk/hisilicon/clk-hi3660.c        | 207 ++++-------
 drivers/clk/hisilicon/clk-hi3670.c        | 270 +++++---------
 drivers/clk/hisilicon/clk-hi6220-stub.c   |   9 +-
 drivers/clk/hisilicon/clk-hi6220.c        | 230 ++++++------
 drivers/clk/hisilicon/clk-hip04.c         |  41 ++-
 drivers/clk/hisilicon/clk-hisi-phase.c    |  13 +-
 drivers/clk/hisilicon/clk-hix5hd2.c       |  96 +++--
 drivers/clk/hisilicon/clk.c               | 429 +++++++++++-----------
 drivers/clk/hisilicon/clk.h               | 147 +++++---
 drivers/clk/hisilicon/clkdivider-hi6220.c |  24 +-
 drivers/clk/hisilicon/clkgate-separated.c |  26 +-
 drivers/clk/hisilicon/crg-hi3516cv300.c   | 177 +--------
 drivers/clk/hisilicon/crg-hi3798cv200.c   | 206 ++---------
 drivers/clk/hisilicon/crg.h               |  11 +-
 drivers/clk/hisilicon/reset.c             |  47 ++-
 18 files changed, 973 insertions(+), 1562 deletions(-)


base-commit: fff5a5e7f528b2ed2c335991399a766c2cf01103

Comments

Stephen Boyd April 5, 2023, 7:39 p.m. UTC | #1 
Quoting David Yang (2023-03-29 00:50:49)
> Migrate devm APIs for HiSilicon clock drivers and remove redundant codes.
> 
> This series is a partial improvement of [1]
> 

Run checkpatch.pl 

WARNING: Macros with flow control statements should be avoided
#591: FILE: drivers/clk/hisilicon/clk.c:146:
+#define hisi_clk_register_fn(fn, type, stmt) \
+int fn(struct device *dev, const struct type *clks, \
+       int num, struct hisi_clock_data *data) \
+{ \
+       void __iomem *base = data->base; \
+       int i; \
+\
+       for (i = 0; i < num; i++) { \
+               const struct type *p_clk = &clks[i]; \
+               struct clk_hw *clk = stmt; \
+\
+               if (IS_ERR(clk)) { \
+                       pr_err("%s: failed to register clock %s\n", \
+                              __func__, p_clk->name); \
+                       return PTR_ERR(clk); \
+               } \
+\
+               if (p_clk->alias) \
+                       clk_hw_register_clkdev(clk, p_clk->alias, NULL); \
+\
+               data->clk_data->hws[p_clk->id] = clk; \
+       } \
+\
+       return 0; \
+} \
+EXPORT_SYMBOL_GPL(fn);

WARNING: macros should not use a trailing semicolon
#591: FILE: drivers/clk/hisilicon/clk.c:146:
+#define hisi_clk_register_fn(fn, type, stmt) \
+int fn(struct device *dev, const struct type *clks, \
+       int num, struct hisi_clock_data *data) \
+{ \
+       void __iomem *base = data->base; \
+       int i; \
+\
+       for (i = 0; i < num; i++) { \
+               const struct type *p_clk = &clks[i]; \
+               struct clk_hw *clk = stmt; \
+\
+               if (IS_ERR(clk)) { \
+                       pr_err("%s: failed to register clock %s\n", \
+                              __func__, p_clk->name); \
+                       return PTR_ERR(clk); \
+               } \
+\
+               if (p_clk->alias) \
+                       clk_hw_register_clkdev(clk, p_clk->alias, NULL); \
+\
+               data->clk_data->hws[p_clk->id] = clk; \
+       } \
+\
+       return 0; \
+} \
+EXPORT_SYMBOL_GPL(fn);

total: 0 errors, 2 warnings, 1019 lines checked
in patch 'clk: hisilicon: Migrate devm APIs' (8d960cbef61a)
WARNING: Prefer "GPL" over "GPL v2" - see commit bf7fbeeae6db ("module: Cure the MODULE_LICENSE "GPL" vs. "GPL v2" bogosity")
#627: FILE: drivers/clk/hisilicon/clk-hi6220.c:291:
+MODULE_LICENSE("GPL v2");

WARNING: DT compatible string "hisilicon,hip04-clock" appears un-documented -- check ./Documentation/devicetree/bindings/
#661: FILE: drivers/clk/hisilicon/clk-hip04.c:33:
+       { .compatible = "hisilicon,hip04-clock",

WARNING: Prefer "GPL" over "GPL v2" - see commit bf7fbeeae6db ("module: Cure the MODULE_LICENSE "GPL" vs. "GPL v2" bogosity")
#842: FILE: drivers/clk/hisilicon/clk-hix5hd2.c:337:
+MODULE_LICENSE("GPL v2");

total: 0 errors, 3 warnings, 779 lines checked
in patch 'clk: hisilicon: Convert to platform driver' (2681347c2636)
WARNING: Prefer "GPL" over "GPL v2" - see commit bf7fbeeae6db ("module: Cure the MODULE_LICENSE "GPL" vs. "GPL v2" bogosity")
#1044: FILE: drivers/clk/hisilicon/clk-hi3670.c:935:
+MODULE_LICENSE("GPL v2");

total: 0 errors, 1 warnings, 1481 lines checked