| Message ID | 20220819014758.v3.5.I6a1fca5d53c886c05ea3e24cd4282d31c9c0cd0b@changeid (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | clk/qcom: Support gdsc collapse polling using 'reset' inteface | expand |
On 18/08/2022 23:18, Akhil P Oommen wrote: > Add support for Reset using GPUCC driver for GPU. This helps to ensure > that GPU state is reset by making sure that CX head switch is collapsed. > > Signed-off-by: Akhil P Oommen <quic_akhilpo@quicinc.com> > --- > > (no changes since v1) > > arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi > index e66fc67..f5257d6 100644 > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi > @@ -2243,6 +2243,9 @@ > nvmem-cells = <&gpu_speed_bin>; > nvmem-cell-names = "speed_bin"; > > + resets = <&gpucc GPU_CX_COLLAPSE>; > + reset-names = "cx_collapse"; > + I think this is not allowed by bindings. Did you test your change with dtbs_check? Best regards, Krzysztof
On 8/19/2022 11:47 AM, Krzysztof Kozlowski wrote: > On 18/08/2022 23:18, Akhil P Oommen wrote: >> Add support for Reset using GPUCC driver for GPU. This helps to ensure >> that GPU state is reset by making sure that CX head switch is collapsed. >> >> Signed-off-by: Akhil P Oommen <quic_akhilpo@quicinc.com> >> --- >> >> (no changes since v1) >> >> arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> index e66fc67..f5257d6 100644 >> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi >> @@ -2243,6 +2243,9 @@ >> nvmem-cells = <&gpu_speed_bin>; >> nvmem-cell-names = "speed_bin"; >> >> + resets = <&gpucc GPU_CX_COLLAPSE>; >> + reset-names = "cx_collapse"; >> + > I think this is not allowed by bindings. Did you test your change with > dtbs_check? > > Best regards, > Krzysztof My bad! Thanks for pointing this out. Fixed in v4 with patch 5/6. -Akhil.
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi index e66fc67..f5257d6 100644 --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi @@ -2243,6 +2243,9 @@ nvmem-cells = <&gpu_speed_bin>; nvmem-cell-names = "speed_bin"; + resets = <&gpucc GPU_CX_COLLAPSE>; + reset-names = "cx_collapse"; + gpu_opp_table: opp-table { compatible = "operating-points-v2";
Add support for Reset using GPUCC driver for GPU. This helps to ensure that GPU state is reset by making sure that CX head switch is collapsed. Signed-off-by: Akhil P Oommen <quic_akhilpo@quicinc.com> --- (no changes since v1) arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++ 1 file changed, 3 insertions(+)