mbox series

[v2,00/11] Hook up ACV enable_mask for everybody

Message ID 20230811-topic-acv-v2-0-765ad70e539a@linaro.org (mailing list archive)
Headers show
Series Hook up ACV enable_mask for everybody | expand

Message

Konrad Dybcio Aug. 11, 2023, 11:20 p.m. UTC
In the downstream kernel, ACV enable_mask has not been mentioned
explicitly, rather being handled by a sneaky if-condition [1], [2].

Add it to all RPMh platforms to actually enable that BCM.

Based atop [3].

[1] https://git.codelinaro.org/clo/la/kernel/msm-4.19/-/blob/LA.UM.10.2.1.c25/drivers/soc/qcom/msm_bus/msm_bus_arb_rpmh.c#L556-567
[2] https://git.codelinaro.org/clo/la/kernel/msm-4.19/-/blob/LA.UM.10.2.1.c25/drivers/soc/qcom/msm_bus/msm_bus_arb_rpmh.c#L475-495
[3] https://lore.kernel.org/linux-arm-msm/20230811-topic-icc_retire_macrosd-v1-0-c03aaeffc769@linaro.org/T/#t

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Changes in v2:
- Remove the define and use raw BIT(3) as the APPS mask may change (Mike)
- Pick up tags (Bjorn)
- Link to v1: https://lore.kernel.org/r/20230811-topic-acv-v1-0-a85b6e94f46d@linaro.org

---
Konrad Dybcio (11):
      interconnect: qcom: qdu1000: Set ACV enable_mask
      interconnect: qcom: sc7180: Set ACV enable_mask
      interconnect: qcom: sc7280: Set ACV enable_mask
      interconnect: qcom: sc8180x: Set ACV enable_mask
      interconnect: qcom: sc8280xp: Set ACV enable_mask
      interconnect: qcom: sdm670: Set ACV enable_mask
      interconnect: qcom: sdm845: Set ACV enable_mask
      interconnect: qcom: sm6350: Set ACV enable_mask
      interconnect: qcom: sm8150: Set ACV enable_mask
      interconnect: qcom: sm8250: Set ACV enable_mask
      interconnect: qcom: sm8350: Set ACV enable_mask

 drivers/interconnect/qcom/qdu1000.c  | 1 +
 drivers/interconnect/qcom/sc7180.c   | 1 +
 drivers/interconnect/qcom/sc7280.c   | 1 +
 drivers/interconnect/qcom/sc8180x.c  | 1 +
 drivers/interconnect/qcom/sc8280xp.c | 1 +
 drivers/interconnect/qcom/sdm670.c   | 1 +
 drivers/interconnect/qcom/sdm845.c   | 1 +
 drivers/interconnect/qcom/sm6350.c   | 1 +
 drivers/interconnect/qcom/sm8150.c   | 1 +
 drivers/interconnect/qcom/sm8250.c   | 1 +
 drivers/interconnect/qcom/sm8350.c   | 1 +
 11 files changed, 11 insertions(+)
---
base-commit: ee4aa20e094643232438b896f49a405361406fbf
change-id: 20230811-topic-acv-72aca8ad6f41

Best regards,

Comments

Konrad Dybcio Aug. 11, 2023, 11:25 p.m. UTC | #1
On 12.08.2023 01:20, Konrad Dybcio wrote:
> In the downstream kernel, ACV enable_mask has not been mentioned
> explicitly, rather being handled by a sneaky if-condition [1], [2].
> 
> Add it to all RPMh platforms to actually enable that BCM.
> 
> Based atop [3].
> 
> [1] https://git.codelinaro.org/clo/la/kernel/msm-4.19/-/blob/LA.UM.10.2.1.c25/drivers/soc/qcom/msm_bus/msm_bus_arb_rpmh.c#L556-567
> [2] https://git.codelinaro.org/clo/la/kernel/msm-4.19/-/blob/LA.UM.10.2.1.c25/drivers/soc/qcom/msm_bus/msm_bus_arb_rpmh.c#L475-495
> [3] https://lore.kernel.org/linux-arm-msm/20230811-topic-icc_retire_macrosd-v1-0-c03aaeffc769@linaro.org/T/#t
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
> Changes in v2:
> - Remove the define and use raw BIT(3) as the APPS mask may change (Mike)
> - Pick up tags (Bjorn)
Whoops sorry, there were no tags (and none were picked up)!

Konrad