mbox series

[v3,0/4] soc: qcom: rpmhpd: Cleanups and fixups for sc7280/sm8450

Message ID 1639063917-9011-1-git-send-email-quic_rjendra@quicinc.com (mailing list archive)
Headers show
Series soc: qcom: rpmhpd: Cleanups and fixups for sc7280/sm8450 | expand

Message

Rajendra Nayak Dec. 9, 2021, 3:31 p.m. UTC 
v3:
* used the _w_<parent-name>_parent suffix instead of _no_parent
* Added a minor fix for sm8450 while rebasing 

v2:
* Fixed the wrong assumption in v1 that only sdm845 needed mx to be
parent of cx, turned out all existing upstream SoCs need it except sc7280
* Added another cleanup patch to sort power-domain defines and lists in
alphabetical order as suggested by Matthias

Mostly cleanups, with a fixup to remove the parent/child relationship
across mx/cx for sc7280 SoC, and a fixup to add missing .peer for sm8450

Rajendra Nayak (4):
  soc: qcom: rpmhpd: sm8450: Add the missing .peer for sm8450_cx_ao
  soc: qcom: rpmhpd: Rename rpmhpd struct names
  soc: qcom: rpmhpd: Remove mx/cx relationship on sc7280
  soc: qcom: rpmhpd: Sort power-domain definitions and lists

 drivers/soc/qcom/rpmhpd.c | 358 +++++++++++++++++++++++-----------------------
 1 file changed, 181 insertions(+), 177 deletions(-)

Comments

Bjorn Andersson Dec. 9, 2021, 3:53 p.m. UTC | #1 
On Thu 09 Dec 09:31 CST 2021, Rajendra Nayak wrote:

> v3:
> * used the _w_<parent-name>_parent suffix instead of _no_parent
> * Added a minor fix for sm8450 while rebasing 
> 
> v2:
> * Fixed the wrong assumption in v1 that only sdm845 needed mx to be
> parent of cx, turned out all existing upstream SoCs need it except sc7280
> * Added another cleanup patch to sort power-domain defines and lists in
> alphabetical order as suggested by Matthias
> 
> Mostly cleanups, with a fixup to remove the parent/child relationship
> across mx/cx for sc7280 SoC, and a fixup to add missing .peer for sm8450
> 

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

> Rajendra Nayak (4):
>   soc: qcom: rpmhpd: sm8450: Add the missing .peer for sm8450_cx_ao
>   soc: qcom: rpmhpd: Rename rpmhpd struct names
>   soc: qcom: rpmhpd: Remove mx/cx relationship on sc7280
>   soc: qcom: rpmhpd: Sort power-domain definitions and lists
> 
>  drivers/soc/qcom/rpmhpd.c | 358 +++++++++++++++++++++++-----------------------
>  1 file changed, 181 insertions(+), 177 deletions(-)
> 
> -- 
> 2.7.4
>
Bjorn Andersson Dec. 15, 2021, 10:27 p.m. UTC | #2 
On Thu, 9 Dec 2021 21:01:53 +0530, Rajendra Nayak wrote:
> v3:
> * used the _w_<parent-name>_parent suffix instead of _no_parent
> * Added a minor fix for sm8450 while rebasing
> 
> v2:
> * Fixed the wrong assumption in v1 that only sdm845 needed mx to be
> parent of cx, turned out all existing upstream SoCs need it except sc7280
> * Added another cleanup patch to sort power-domain defines and lists in
> alphabetical order as suggested by Matthias
> 
> [...]

Applied, thanks!

[1/4] soc: qcom: rpmhpd: sm8450: Add the missing .peer for sm8450_cx_ao
      commit: 7e1377712ba2da42edd354dc8da363f0fd68d33e
[2/4] soc: qcom: rpmhpd: Rename rpmhpd struct names
      commit: 8f3d4dd65abd03a5edcf7b5d5a7a3e2a4866db15
[3/4] soc: qcom: rpmhpd: Remove mx/cx relationship on sc7280
      commit: 65e7b31cc48581e32bee4546b59cea404252a138
[4/4] soc: qcom: rpmhpd: Sort power-domain definitions and lists
      commit: 8beb290d17f280f690857897af11dc90ac2e1f3d

Best regards,