Export some APIs for module drivers.
Fix the clock config to support module build.
Fix the clk driver init, add module author, description
and license to support building RK3399 SoC clock driver as module.
Change in V2:
[PATCH v2 1/6]: remove "clk",and check "hw" isn't an error value.
[PATCH v2 6/6]: store a function pointer in the match data.
Change in V3:
[PATCH v3 1/6]: fix up the compiler warning.
drivers/clk/rockchip/clk.c: In function 'rockchip_clk_register_branch':
>> drivers/clk/rockchip/clk.c:52:6: warning: variable 'ret' set but not
>> used [-Wunused-but-set-variable]
52 | int ret;
| ^~~
Change in V4:
[PATCH v4 2/6]: Use EXPORT_SYMBOL_GPL instead of EXPORT_SYMBOL.
[PATCH v4 3/6]: Use EXPORT_SYMBOL_GPL instead of EXPORT_SYMBOL.
[PATCH v4 4/6]: Use EXPORT_SYMBOL_GPL instead of EXPORT_SYMBOL.
[PATCH v4 5/6]: Mark CONFIG_CLK_xxx to "bool".
[PATCH v4 6/6]: add .suppress_bind_attrs = true
Elaine Zhang (6):
clk: rockchip: Use clk_hw_register_composite instead of
clk_register_composite calls
clk: rockchip: Export rockchip_clk_register_ddrclk()
clk: rockchip: Export rockchip_register_softrst()
clk: rockchip: Export some clock common APIs for module drivers
clk: rockchip: fix the clk config to support module build
clk: rockchip: rk3399: Support module build
drivers/clk/Kconfig | 1 +
drivers/clk/rockchip/Kconfig | 78 ++++++++++++++++
drivers/clk/rockchip/Makefile | 42 ++++-----
drivers/clk/rockchip/clk-ddr.c | 1 +
drivers/clk/rockchip/clk-half-divider.c | 18 ++--
drivers/clk/rockchip/clk-rk3399.c | 56 ++++++++++++
drivers/clk/rockchip/clk.c | 113 +++++++++++++-----------
drivers/clk/rockchip/softrst.c | 7 +-
8 files changed, 232 insertions(+), 84 deletions(-)
create mode 100644 drivers/clk/rockchip/Kconfig
base-commit: b36c969764ab12faebb74711c942fa3e6eaf1e96
--
2.17.1
On Mon, 14 Sep 2020 10:22:19 +0800, Elaine Zhang wrote:
> Export some APIs for module drivers.
> Fix the clock config to support module build.
> Fix the clk driver init, add module author, description
> and license to support building RK3399 SoC clock driver as module.
>
> Change in V2:
> [PATCH v2 1/6]: remove "clk",and check "hw" isn't an error value.
> [PATCH v2 6/6]: store a function pointer in the match data.
>
> [...]
Applied, thanks!
[1/6] clk: rockchip: Use clk_hw_register_composite instead of clk_register_composite calls
commit: 63207c37eac4f15fdebac14685a315c259c0a780
[2/6] clk: rockchip: Export rockchip_clk_register_ddrclk()
commit: f73907de3493b94d80af5122bcacc98f0e7b295b
[3/6] clk: rockchip: Export rockchip_register_softrst()
commit: 37353491d1a8c207685c138c3640bd43864b70d9
[4/6] clk: rockchip: Export some clock common APIs for module drivers
commit: ea650c26611dd61adfcc8647d6144f2c9f453d90
[5/6] clk: rockchip: fix the clk config to support module build
commit: 4d98ed1e126495016f2a3ef4db6379855c4aacf2
[6/6] clk: rockchip: rk3399: Support module build
commit: 70d839e2761d22eba6facdb3b65faea4d57f355d
I did some minor tweaks:
- in the Kconfig texts "Rk" -> "RK"
- made the rk3399 entry bool in patch 5
so that it stays correct when a bisection lands between
patches 5 and 6
- made in tristate in patch6 which adds the rest of module
elements
Best regards,
--
Heiko Stuebner <[email protected]>