Re: [PATCH v2] dt-bindings: rng: rockchip,rk3588-rng: Drop unnecessary status from example

[Rob Herring]

On Mon, Mar 10, 2025 at 7:49 AM Rob Herring <robh@xxxxxxxxxx> wrote:
>
> On Fri, Mar 07, 2025 at 03:05:50PM +0100, Nicolas Frattaroli wrote:
> > On Friday, 7 March 2025 10:33:09 Central European Standard Time Krzysztof
> > Kozlowski wrote:
> > > Device nodes are enabled by default, so no need for 'status = "okay"' in
> > > the DTS example.
> > >
> > > Reviewed-by: Heiko Stuebner <heiko@xxxxxxxxx>
> > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> > >
> > > ---
> > >
> > > Changes in v2:
> > > 1. Drop unnecessary full stop in subject prefix after ':'.
> > > 2. Add Rb tag.
> > > ---
> > >  Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml
> > > b/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml index
> > > 757967212f55..ca71b400bcae 100644
> > > --- a/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml
> > > +++ b/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml
> > > @@ -53,7 +53,6 @@ examples:
> > >          interrupts = <GIC_SPI 400 IRQ_TYPE_LEVEL_HIGH 0>;
> > >          clocks = <&scmi_clk SCMI_HCLK_SECURE_NS>;
> > >          resets = <&scmi_reset SCMI_SRST_H_TRNG_NS>;
> > > -        status = "okay";
> > >        };
> > >      };
> >
> > Hi,
> >
> > is there the possibility we could make dtschema as invoked by `make
> > dt_binding_check W=1` (or W=2) add a warning for examples that have disabled
> > or explicitly listed status properties when not needed? Or is this something
> > better handled in, say, checkpatch.pl?
> >
> > The question arises because dumb mistakes by me like this should ideally be
> > caught before they waste precious maintainer time.
> >
> > If it's best handled in dtschema, I can look into working on that so you guys
> > don't have to do even more work due to me.
>
> Trust me, I'm always looking for things to check automatically. :)
>
> I have something like that which adds a schema for examples only. I
> think there were some corner cases. I'll take a fresh look at it.

I got my check working again. i2c/i2c-demux-pinctrl.yaml is the
problematic one. Not sure what to do there.

Rob