On Sat, 21 Sept 2024 at 20:23, Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > On 12/09/2024 09:14, Mahadevan wrote: > > > > + clocks = <&dispcc0 MDSS_DISP_CC_MDSS_AHB_CLK>, > > + <&gcc GCC_DISP_HF_AXI_CLK>, > > + <&dispcc0 MDSS_DISP_CC_MDSS_MDP_CLK>; > > + > > + interrupts = <GIC_SPI 92 IRQ_TYPE_LEVEL_HIGH>; > > + interrupt-controller; > > + #interrupt-cells = <1>; > > + > > + iommus = <&apps_smmu 0x1000 0x402>; > > + > > + #address-cells = <2>; > > + #size-cells = <2>; > > + ranges; > > + > > + status = "disabled"; > > Uh no, it cannot be disabled. What would be the point of it? Please > reach to your colleagues for some internal review before posting (see > also go/upstream in internal systems). Rob, can we make it part of dt-validate maybe? Error out if schema _examples_ have disabled nodes. -- With best wishes Dmitry
