Hi Miquel, Am Mittwoch, 4. Dezember 2019, 18:15:37 CET schrieb Miquel Raynal: > This clock has nothing to do in the PX30 DTSI as it is supposed to be > an input of the SoC. Moving it to the EVB DTS (only board file using > this DTSI) makes more sense. Also, when this clock is not a fixed > clock and comes from eg. a PMIC the situation can be described cleanly > in the device tree (avoids having to delete the fixed-clock node > first). > > This clock is not mandatory to boot so it should not break existing > users. > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> Sorry foo being the bearer of bad news again, but that issue got already fixed by: arm64: dts: rockchip: remove static xin32k from px30 https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v5.5-armsoc/dts64&id=00519137f7d4fc19ff27f3d3f4fc45b5b222ae82 arm64: dts: rockchip: fix the px30-evb power tree https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v5.5-armsoc/dts64&id=915b6a8b54a6d436885a458867e59fb20fc6356d On most/all Rockchip the xin32k clock is actually provided by the boards pmic - the rk809 in this case. Heiko
