mbox series

[v6,00/13] clk: hisilicon: Migrate devm APIs

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

Message

Yangfl July 31, 2023, 12:18 p.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
v3:
  * size_t for all these num types
  * hisi_clk_unregister() change into separate patch
  * keep relevant header inclusions
  * split driver files changes into separate patches
  * explain hisi_clk_register_fn() checkpatch warnings
  * not fixed: MODULE_LICENSE("GPL v2"), as stated in SPDX-License-Identifier
  * not fixed: "hisilicon,hip04-clock" un-documented, as none of dts files in
    arch/ use it, better to ask hisi people why they pushed this driver
v4:
  * typo: hisi_clocks_get_nr() should check clks->nr first
  * unexport hisi_clk_unregister_fn() as no one use them outside
v5: catch up with remove_new refactoring
v6: fix compilation error and expand macros

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
v2: https://lore.kernel.org/r/20230329075104.165176-1-mmyangfl@gmail.com
v3: https://lore.kernel.org/r/20230410110733.192151-1-mmyangfl@gmail.com
v4: https://lore.kernel.org/r/20230411174329.424763-1-mmyangfl@gmail.com
v5: https://lore.kernel.org/r/20230723162245.35033-1-mmyangfl@gmail.com

David Yang (13):
  clk: hisilicon: Add helper functions for platform driver
  clk: hisilicon: hi3516cv300: Use helper functions
  clk: hisilicon: hi3798cv200: Use helper functions
  clk: hisilicon: Remove hisi_crg_funcs
  clk: hisilicon: hi3519: Use helper functions
  clk: hisilicon: hi3559a: Use helper functions
  clk: hisilicon: hi3660: Convert into module
  clk: hisilicon: hi3670: Convert into module
  clk: hisilicon: hi3620: Convert into platform driver module
  clk: hisilicon: hi6220: Convert into platform driver module
  clk: hisilicon: hip04: Convert into platform driver module
  clk: hisilicon: hix5hd2: Convert into platform driver module
  clk: hisilicon: Migrate devm APIs

 drivers/clk/hisilicon/clk-hi3519.c        | 127 +-----
 drivers/clk/hisilicon/clk-hi3559a.c       | 253 +++--------
 drivers/clk/hisilicon/clk-hi3620.c        | 212 +++++-----
 drivers/clk/hisilicon/clk-hi3660.c        | 192 +++------
 drivers/clk/hisilicon/clk-hi3670.c        | 248 ++++-------
 drivers/clk/hisilicon/clk-hi6220-stub.c   |   9 +-
 drivers/clk/hisilicon/clk-hi6220.c        | 229 +++++-----
 drivers/clk/hisilicon/clk-hip04.c         |  39 +-
 drivers/clk/hisilicon/clk-hisi-phase.c    |  13 +-
 drivers/clk/hisilicon/clk-hix5hd2.c       |  98 +++--
 drivers/clk/hisilicon/clk.c               | 490 +++++++++++++---------
 drivers/clk/hisilicon/clk.h               | 141 ++++---
 drivers/clk/hisilicon/clkdivider-hi6220.c |  24 +-
 drivers/clk/hisilicon/clkgate-separated.c |  26 +-
 drivers/clk/hisilicon/crg-hi3516cv300.c   | 171 +-------
 drivers/clk/hisilicon/crg-hi3798cv200.c   | 196 +--------
 drivers/clk/hisilicon/crg.h               |  11 +-
 drivers/clk/hisilicon/reset.c             |  42 ++
 18 files changed, 998 insertions(+), 1523 deletions(-)


base-commit: 5d0c230f1de8c7515b6567d9afba1f196fb4e2f4

Comments

Stephen Boyd Dec. 18, 2023, 6:42 a.m. UTC | #1 
Quoting David Yang (2023-07-31 05:18:00)
> Migrate devm APIs for HiSilicon clock drivers and remove redundant codes.
> 
> This series is a partial improvement of [1]
> 
> v2: fix test robot error
> v3:
>   * size_t for all these num types
>   * hisi_clk_unregister() change into separate patch
>   * keep relevant header inclusions
>   * split driver files changes into separate patches
>   * explain hisi_clk_register_fn() checkpatch warnings
>   * not fixed: MODULE_LICENSE("GPL v2"), as stated in SPDX-License-Identifier
>   * not fixed: "hisilicon,hip04-clock" un-documented, as none of dts files in
>     arch/ use it, better to ask hisi people why they pushed this driver
> v4:
>   * typo: hisi_clocks_get_nr() should check clks->nr first
>   * unexport hisi_clk_unregister_fn() as no one use them outside
> v5: catch up with remove_new refactoring
> v6: fix compilation error and expand macros

This needs to be resent. Sorry. Use mod_devicetable.h instead of
of_device.h please and fix compilation errors.