On Fri, Apr 01, 2022 at 01:36:53PM -0500, Rob Herring wrote: > On Fri, Apr 1, 2022 at 11:38 AM Marek Vasut <marex@xxxxxxx> wrote: > > > > Make the width-mm/height-mm panel properties mandatory > > to correctly report the panel dimensions to the OS. > > Adding required properties is an ABI break. Only if drivers don't provide backward compatibility. There's still the issue of validation warnings of course. We can ensure that DT sources in the kernel are updated. For out-of-tree DT, I'll argue that a validation warning is a good thing, it will notify that an update is needed. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > > Cc: Christoph Niedermaier <cniedermaier@xxxxxxxxxxxxxxxxxx> > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > > Cc: Dmitry Osipenko <digetx@xxxxxxxxx> > > Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > Cc: Noralf Trønnes <noralf@xxxxxxxxxxx> > > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > > Cc: Robert Foss <robert.foss@xxxxxxxxxx> > > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> > > Cc: Thomas Zimmermann <tzimmermann@xxxxxxx> > > Cc: devicetree@xxxxxxxxxxxxxxx > > To: dri-devel@xxxxxxxxxxxxxxxxxxxxx > > --- > > .../devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml b/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml > > index f29789994b18..c2df8d28aaf5 100644 > > --- a/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml > > +++ b/Documentation/devicetree/bindings/display/panel/panel-mipi-dbi-spi.yaml > > @@ -83,6 +83,8 @@ properties: > > required: > > - compatible > > - reg > > + - width-mm > > + - height-mm > > - panel-timing > > > > unevaluatedProperties: false -- Regards, Laurent Pinchart
