Hi Hans, On 8/7/23 15:58, Hans Verkuil wrote: > This is an automatic generated email to let you know that the following patch were queued: > > Subject: media: dt-bindings: drop unneeded status from examples > Author: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Date: Tue Jul 25 12:16:25 2023 +0200 > > Example DTS should not have 'status' property. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Reviewed-by: Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx> > Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > Acked-by: Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx> > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> Careful you have both my Reviewed-by and Acked-by tag. This is my fault as I initially sent the Reviewed-by, but it should be an Acked-By tag instead as pointed by Krzysztof, as it applies only to a part of the patch and not its entirety. Sorry for the mess. > > Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml | 1 - > Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml | 1 - > 2 files changed, 2 deletions(-) > > --- > > diff --git a/Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml b/Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml > index 369c48fd9bf9..a6b73498bc21 100644 > --- a/Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml > +++ b/Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml > @@ -53,6 +53,5 @@ examples: > interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&tegra_car TEGRA124_CLK_CEC>; > clock-names = "cec"; > - status = "disabled"; > hdmi-phandle = <&hdmi>; > }; > diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml b/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml > index 19a39d753aad..b68141264c0e 100644 > --- a/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml > @@ -143,7 +143,6 @@ examples: > mipid02: csi2rx@14 { > compatible = "st,st-mipid02"; > reg = <0x14>; > - status = "okay"; > clocks = <&clk_ext_camera_12>; > clock-names = "xclk"; > VDDE-supply = <&vdd>; -- Regards, Benjamin