Hi Heiko, Heiko Stübner <heiko@xxxxxxxxx> wrote on Wed, 04 Dec 2019 18:31:46 +0100: > 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: He he, no problem :) I was not actually looking at the right branch (v5.5-armsoc/dts64). > 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. Thanks for the confirmation! Miquèl
