On 30/06/2020 12:13, Jon Hunter wrote: > > On 16/06/2020 14:52, Thierry Reding wrote: >> From: Thierry Reding <treding@xxxxxxxxxx> >> >> The standard way to do this is to list out the clocks at the top-level. >> Adopt the standard way to fix validation. >> >> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> >> --- >> arch/arm64/boot/dts/nvidia/tegra132-norrin.dts | 15 ++++----------- >> arch/arm64/boot/dts/nvidia/tegra210-p2180.dtsi | 15 ++++----------- >> arch/arm64/boot/dts/nvidia/tegra210-p2530.dtsi | 15 ++++----------- >> arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi | 15 ++++----------- >> .../arm64/boot/dts/nvidia/tegra210-p3450-0000.dts | 15 ++++----------- >> arch/arm64/boot/dts/nvidia/tegra210-smaug.dts | 15 ++++----------- >> 6 files changed, 24 insertions(+), 66 deletions(-) > > After applying this patch I see the following build breakage ... > > DTC arch/arm64/boot/dts/nvidia/tegra132-norrin.dtb > arch/arm64/boot/dts/nvidia/tegra132.dtsi:649.15-682.4: ERROR > (phandle_references): /usb@70090000: Reference to non-existent node or > label "vdd_3v3_lp0" > > also defined at > arch/arm64/boot/dts/nvidia/tegra132-norrin.dts:895.15-909.4 > arch/arm64/boot/dts/nvidia/tegra132.dtsi:649.15-682.4: ERROR > (phandle_references): /usb@70090000: Reference to non-existent node or > label "vdd_3v3_lp0" > > also defined at > arch/arm64/boot/dts/nvidia/tegra132-norrin.dts:895.15-909.4 > arch/arm64/boot/dts/nvidia/tegra132.dtsi:684.26-812.4: ERROR > (phandle_references): /padctl@7009f000: Reference to non-existent node > or label "vdd_3v3_lp0" Same here is might not be this patch, but somewhere during this series I see a few build breakages. Jon -- nvpublic