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.pinchart@ideasonboard.com
I think this qualifies as a v5.7 fix.
Signed-off-by: Sam Ravnborg sam@ravnborg.org Cc: Laurent Pinchart laurent.pinchart@ideasonboard.com Cc: Maxime Ripard mripard@kernel.org Cc: Rob Herring robh@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 connectedto 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 {