Hi Sam, Thank you for the patch.
On Sat, Mar 14, 2020 at 04:30:45PM +0100, Sam Ravnborg wrote: > data-mapping may not be the best way to describe the > data format used between panels and display interface. > > Drop it from the panel-dpi binding so we do not start to rely on it. > We can then work out how to best describe this mapping and when > we know it, we can add it to this binding. I certainly welcome that, as we need to define how to express this information in a more detailed way, taking into account all use cases. Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> I think this qualifies as a v5.7 fix. > Signed-off-by: Sam Ravnborg <s...@ravnborg.org> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Cc: Maxime Ripard <mrip...@kernel.org> > Cc: Rob Herring <r...@kernel.org> > --- > .../devicetree/bindings/display/panel/panel-dpi.yaml | 10 ---------- > 1 file changed, 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/panel/panel-dpi.yaml > b/Documentation/devicetree/bindings/display/panel/panel-dpi.yaml > index f63870384c00..0cd74c8dab42 100644 > --- a/Documentation/devicetree/bindings/display/panel/panel-dpi.yaml > +++ b/Documentation/devicetree/bindings/display/panel/panel-dpi.yaml > @@ -21,15 +21,6 @@ properties: > - {} > - const: panel-dpi > > - data-mapping: > - enum: > - - rgb24 > - - rgb565 > - - bgr666 > - description: | > - Describes the media format, how the display panel is connected > - to the display interface. > - > backlight: true > enable-gpios: true > height-mm: true > @@ -52,7 +43,6 @@ examples: > compatible = "osddisplays,osd057T0559-34ts", "panel-dpi"; > label = "osddisplay"; > power-supply = <&vcc_supply>; > - data-mapping = "rgb565"; > backlight = <&backlight>; > > port { -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel