On Thu, 11 Feb 2021 12:27:52 +0800, Liu Ying wrote:
To complement panel-simple.yaml, create panel-simple-lvds-dual-ports.yaml. panel-simple-lvds-dual-ports.yaml is for all simple LVDS panels that have dual LVDS ports and require only a single power-supply. The first port receives odd pixels, and the second port receives even pixels. Optionally, a backlight and an enable GPIO can be specified as properties.
Panels with swapped pixel order, if any, need dedicated bindings.
Migrate 'auo,g133han01', 'auo,g185han01', 'auo,g190ean01', 'koe,tx26d202vm0bwa' and 'nlt,nl192108ac18-02d' over to the new file.
The objectives with one file for all the simple LVDS panels with dual ports are:
- Make it simpler to add bindings for this kind of LVDS panels
- Keep the number of bindings file lower
- Keep the binding documentation for this kind of LVDS panels more consistent
- Make it possible for drivers to get pixel order via drm_of_lvds_get_dual_link_pixel_order(), as the 'ports' property is required
Suggested-by: Sam Ravnborg sam@ravnborg.org Cc: Thierry Reding thierry.reding@gmail.com Cc: Sam Ravnborg sam@ravnborg.org Cc: David Airlie airlied@linux.ie Cc: Daniel Vetter daniel@ffwll.ch Cc: Rob Herring robh+dt@kernel.org Cc: Lucas Stach l.stach@pengutronix.de Cc: Sebastian Reichel sebastian.reichel@collabora.com Signed-off-by: Liu Ying victor.liu@nxp.com
v6->v7:
- Reference '#/$defs/port-base' instead of '#/properties/port'. (Rob)
v5->v6:
- Use OF graph schema.
- Drop Rob's R-b tag, as review is needed.
v4->v5:
- Require the 'ports' property and update commit message accordingly. (Rob)
- Add Rob's R-b tag.
v3->v4:
- Add type and descriptions for dual-lvds-{odd,even}-pixels properties. Also, update descriptions for port@0 and port@1 properties accordingly. (Rob)
v2->v3:
- Do not allow 'port' property. (Rob)
- Define port number. (Rob)
- Specify 'dual-lvds-odd-pixels' and 'dual-lvds-even-pixels' properties. (Rob)
v1->v2:
- Correct pixel order in example LVDS panel node.
.../panel/panel-simple-lvds-dual-ports.yaml | 118 +++++++++++++++++++++ .../bindings/display/panel/panel-simple.yaml | 10 -- 2 files changed, 118 insertions(+), 10 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/panel/panel-simple-lvds-dual-ports.yaml
Reviewed-by: Rob Herring robh@kernel.org