Skip to content

Make clk_hw::init NULL after clk registration

Stephen Boyd linux-clk lkml 1 version Applied

Status from this message: Applied to clk-next

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 26 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/9clk: actions: Don't reference clk_init_data after registrationdrivers/clk/actions/owl-common.c+2 -14 replies
2/9clk: lochnagar: Don't reference clk_init_data after registrationdrivers/clk/clk-lochnagar.c+1 -12 replies
3/9clk: meson: axg-audio: Don't reference clk_init_data after registrationdrivers/clk/meson/axg-audio.c+5 -25 replies
4/9clk: qcom: Don't reference clk_init_data after registrationdrivers/clk/qcom/clk-rpmh.c+2 -22 replies
5/9clk: sirf: Don't reference clk_init_data after registrationdrivers/clk/sirf/clk-common.c+8 -41 reply
6/9clk: socfpga: Don't reference clk_init_data after registrationdrivers/clk/socfpga/clk-gate.c drivers/clk/socfpga/clk-periph-a10.c+15 -133 replies
7/9clk: sprd: Don't reference clk_init_data after registrationdrivers/clk/sprd/common.c+3 -23 replies
8/9phy: ti: am654-serdes: Don't reference clk_init_data after registrationdrivers/phy/ti/phy-am654-serdes.c+2 -22 replies
9/9clk: Overwrite clk_hw::init with NULL during clk_register()drivers/clk/clk.c include/linux/clk-provider.h+18 -94 replies

Reviews trailers per patch

8 of 9 patches have a Reviewed-by or Acked-by.

PatchNeil ArmstrongBaolin WangCharles KeepaxDinh NguyenChunyan ZhangKishon Vijay Abraham IManivannan SadhasivamTaniya DasSylwester Nawrocki
1/9A-b
2/9A-b
3/9A-b
4/9A-b
5/9
6/9A-b
7/9A-b A-b
8/9A-b
9/9R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 9 reviewers.

Read the v1 thread

Cover letter [PATCH 0/9] Make clk_hw::init NULL after clk registration

We don't want clk provider drivers using the init member of struct clk_hw after the clk is registered. It isn't guaranteed to be a valid pointer and all the necessary information inside the structure is copied over into struct clk_core anyway. This patch series fixes up the handful of users that do this and then overwrites the pointer to NULL within the clk registration path.

Stephen Boyd (9):
clk: actions: Don't reference clk_init_data after registration
clk: lochnagar: Don't reference clk_init_data after registration
clk: meson: axg-audio: Don't reference clk_init_data after
registration
clk: qcom: Don't reference clk_init_data after registration
clk: sirf: Don't reference clk_init_data after registration
clk: socfpga: Don't reference clk_init_data after registration
clk: sprd: Don't reference clk_init_data after registration
phy: ti: am654-serdes: Don't reference clk_init_data after
registration
clk: Overwrite clk_hw::init with NULL during clk_register()

Cc Andy Gross <[email protected]>
Cc Baolin Wang <[email protected]>
Cc Barry Song <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Charles Keepax <[email protected]>
Cc Chunyan Zhang <[email protected]>
Cc Dinh Nguyen <[email protected]>
Cc Doug Anderson <[email protected]>
Cc Guo Zeng <[email protected]>
Cc Jerome Brunet <[email protected]>
Cc Kishon Vijay Abraham I <[email protected]>
Cc Manivannan Sadhasivam <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Richard Fitzgerald <[email protected]>
Cc Roger Quadros <[email protected]>
Cc Taniya Das <[email protected]>
 drivers/clk/actions/owl-common.c     |  3 ++-
 drivers/clk/clk-lochnagar.c          |  2 +-
 drivers/clk/clk.c                    | 24 ++++++++++++++++--------
 drivers/clk/meson/axg-audio.c        |  7 +++++--
 drivers/clk/qcom/clk-rpmh.c          |  4 ++--
 drivers/clk/sirf/clk-common.c        | 12 ++++++++----
 drivers/clk/socfpga/clk-gate.c       | 21 +++++++++++----------
 drivers/clk/socfpga/clk-periph-a10.c |  7 ++++---
 drivers/clk/sprd/common.c            |  5 +++--
 drivers/phy/ti/phy-am654-serdes.c    |  4 ++--
 include/linux/clk-provider.h         |  3 ++-
 11 files changed, 56 insertions(+), 36 deletions(-)

base-commit: 5f9e832c137075045d15cd6899ab0505cfb2ca4b

Signature · 1 line
Sent by a computer through tubes