Hi Vasily,
On 2024-12-14 at 21:34:56 -0800, Vasily Khoruzhick <anarsoul@xxxxxxxxx> wrote:
> Since commit ca1170b69968 ("clk: sunxi-ng: a64: force select PLL_MIPI in TCON0 mux"),
> TCON0 clock parent is always set to PLL_MIPI, but unfortunately it breaks
> video output on Pinebook.
>
> I did an experiment: I manually configured PLL_MIPI and PLL_VIDEO0_2X
> to the same clock rate and flipped the switch with devmem. Experiment clearly
> showed that whenever PLL_MIPI is selected as TCON0 clock parent, the video
> output stops working.
>
> To fix the issue, I partially reverted mentioned commit and added explicit
> TCON0 clock parent assignment to device tree. By default, it will be
> PLL_MIPI, and the only users with RGB output - Pinebook and Teres-I will
> override it in their dts.
I've successfully tested this series on my pinephone where it still
correctly selects PLL_MIPI.
Hence,
Tested-by: Frank Oltmanns <frank@xxxxxxxxxxxx> # on pinephone
I've also tested it on Ondřej's downstream kernel (added him to cc),
where also the HDMI output continues to work.
Thank you and best regards,
Frank
> Vasily Khoruzhick (3):
> dt-bindings: clock: sunxi: Export PLL_VIDEO_2X and PLL_MIPI
> arm64: dts: allwinner: a64: explicitly assign clock parent for TCON0
> clk: sunxi-ng: a64: stop force-selecting PLL-MIPI as TCON0 parent
>
> arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 ++
> arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts | 2 ++
> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 2 ++
> drivers/clk/sunxi-ng/ccu-sun50i-a64.c | 11 -----------
> drivers/clk/sunxi-ng/ccu-sun50i-a64.h | 2 --
> include/dt-bindings/clock/sun50i-a64-ccu.h | 2 ++
> 6 files changed, 8 insertions(+), 13 deletions(-)
