| Message ID | 20230929-topic-8280_ahbdisp-v1-0-72bdc38309b9@linaro.org |
|---|---|
| Headers | show |
| Series | 8280 display ahb clk fixes | expand |
On 9/30/23 11:39, Johan Hovold wrote: > On Fri, Sep 29, 2023 at 03:38:53PM +0200, Konrad Dybcio wrote: >> These clocks are consumed by the dispcc[01] clock controllers, so there's >> no reason to keep them on from gcc probe. Remove that hack. > > Eh, how did you test this patch? Oehh you're right, I didn't notice that I still had clk_ignore_unused :/ > > The GCC_DISP_AHB_CLK clocks are not modelled by the clock driver > currently so nothing is guaranteeing them to be enabled if we were to > apply this patch. They just happen to be left on by the bootloader on > some machines currently (well at least one of them is on one machine). What fooled me is that despite not being modeled by the clock driver, it is defined in bindings and referenced in the device tree. Another thing I'll fix up! > So this series is broken and should not be applied. Agreed, I'll revisit this tonight or next week. > > NAK. > > Also, please CC me on sc8280xp and X13s related patches. Omitting you was not intentional, sorry! Konrad
Little fixes to shut down disp ahb clk when dispss is not in use. If possible, the dt patch should theoretically go in first, but x13s worked fine regardless. Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> --- Konrad Dybcio (2): arm64: dts: qcom: sc8280xp: Use the correct AHB clock for DISPSS1 clk: qcom: gcc-sc8280xp: Don't keep display AHB clocks always-on arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 4 ++-- drivers/clk/qcom/gcc-sc8280xp.c | 6 ++---- 2 files changed, 4 insertions(+), 6 deletions(-) --- base-commit: df964ce9ef9fea10cf131bf6bad8658fde7956f6 change-id: 20230929-topic-8280_ahbdisp-e170e0ca2442 Best regards,