mbox series

[0/4] clk: qcom: use module_platform_driver() to register drivers

Message ID 20240206-clk-module-platform-driver-v1-0-db799bd2feeb@linaro.org
Headers show
Series clk: qcom: use module_platform_driver() to register drivers | expand

Message

Dmitry Baryshkov Feb. 6, 2024, 3:25 p.m. UTC
Several Qualcomm clock controller drivers use subsys_initcall to
register the driver early. This makes sense for GCC or TCSRcc. However
there is no need to register camera, display, GPU or venus clocks
earlier. Shift registration to driver init level, where they belong.

Note, I didn't touch lpass clock drivers, they need to be analysed
separately.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
Dmitry Baryshkov (4):
      clk: qcom: camcc-*: switch to module_platform_driver
      clk: qcom: dispcc-*: switch to module_platform_driver
      clk: qcom: gpucc-*: switch to module_platform_driver
      clk: qcom: videocc-*: switch to module_platform_driver

 drivers/clk/qcom/camcc-sc7180.c    | 12 +-----------
 drivers/clk/qcom/camcc-sc7280.c    | 12 +-----------
 drivers/clk/qcom/camcc-sdm845.c    | 12 +-----------
 drivers/clk/qcom/camcc-sm6350.c    | 12 +-----------
 drivers/clk/qcom/dispcc-qcm2290.c  | 12 +-----------
 drivers/clk/qcom/dispcc-sc7180.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sc7280.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sc8280xp.c | 12 +-----------
 drivers/clk/qcom/dispcc-sdm845.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm6125.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm6350.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm6375.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm8250.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm8450.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm8550.c   | 12 +-----------
 drivers/clk/qcom/dispcc-sm8650.c   | 12 +-----------
 drivers/clk/qcom/gpucc-sa8775p.c   | 12 +-----------
 drivers/clk/qcom/gpucc-sc7180.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sc7280.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sdm845.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sm8150.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sm8250.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sm8350.c    | 12 +-----------
 drivers/clk/qcom/gpucc-sm8550.c    | 12 +-----------
 drivers/clk/qcom/videocc-sc7180.c  | 12 +-----------
 drivers/clk/qcom/videocc-sc7280.c  | 12 +-----------
 drivers/clk/qcom/videocc-sdm845.c  | 12 +-----------
 drivers/clk/qcom/videocc-sm8150.c  | 12 +-----------
 drivers/clk/qcom/videocc-sm8250.c  | 12 +-----------
 drivers/clk/qcom/videocc-sm8450.c  | 12 +-----------
 drivers/clk/qcom/videocc-sm8550.c  | 12 +-----------
 31 files changed, 31 insertions(+), 341 deletions(-)
---
base-commit: 076d56d74f17e625b3d63cf4743b3d7d02180379
change-id: 20240206-clk-module-platform-driver-4161aee8d197

Best regards,

Comments

Konrad Dybcio Feb. 6, 2024, 7:06 p.m. UTC | #1
On 6.02.2024 16:25, Dmitry Baryshkov wrote:
> There is no need to register display clock controllers during subsys init
> calls. Use module_platform_driver() instead.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
Bjorn Andersson Feb. 7, 2024, 4:46 a.m. UTC | #2
On Tue, 06 Feb 2024 17:25:11 +0200, Dmitry Baryshkov wrote:
> Several Qualcomm clock controller drivers use subsys_initcall to
> register the driver early. This makes sense for GCC or TCSRcc. However
> there is no need to register camera, display, GPU or venus clocks
> earlier. Shift registration to driver init level, where they belong.
> 
> Note, I didn't touch lpass clock drivers, they need to be analysed
> separately.
> 
> [...]

Applied, thanks!

[1/4] clk: qcom: camcc-*: switch to module_platform_driver
      commit: 0ac31d8c2bb90312071e299a23b3299adde13736
[2/4] clk: qcom: dispcc-*: switch to module_platform_driver
      commit: f4ccb184940247c784e229a739344678e487046d
[3/4] clk: qcom: gpucc-*: switch to module_platform_driver
      commit: a5119f7dbf90d12d169f3e3088908ead1955c1b0
[4/4] clk: qcom: videocc-*: switch to module_platform_driver
      commit: 95dd41487aecd1e8feeca9e10c6b28243b2f6145

Best regards,