[PATCH v3 0/3] Support non-default LVDS data mapping for simple panel
Johannes Zink
j.zink at pengutronix.de
Fri Jul 28 11:54:37 UTC 2023
Some LVDS panels, such as the innolux,g101ice-l01 support multiple LVDS
data mapping modes, which can be configured by strapping a dataformat
pin on the display to a specific voltage.
This can be particularly useful for using the jeida-18 format, which
requires only 3 instead of 4 LVDS lanes.
This series moves the data-mapping property for LVDS panels in a
separate file and optionally adds it to simple-panel when matching to
the innolux,g101ice-l01 compatible. This property allows to override
the default data mapping set in the panel description in simple-panel.
The last patch in this series actually adds the driver support for
parsing the data format override device tree property and modifying the
corresponding values for bit per color and media bus format in the panel
descriptor.
Best regards
Johannes
---
Changelog:
v2 -> v3: - dt bindings: Worked in Conor's and Laurent's Feedback
(thanks for your review): Drop the chomping indicator
- dt bindings: Worked in Laurent's Feedback: fix typos
- driver: worked in Laurent's review findings:
- extract function for fixing up the bus format
- only call this function on LVDS panels
- fix typo
- Link to v2: https://lore.kernel.org/r/20230523-simplepanel_support_nondefault_datamapping-v2-0-87196f0d0b64@pengutronix.de
v1 -> v2: - dt bindings: Worked in Rob's review findings (thanks for your
review), refactored to use common include instead of duplication
- driver: added missing error unwinding goto, as found by Dan
Carpenter's test robot:
Reported-by: kernel test robot <lkp at intel.com>
Reported-by: Dan Carpenter <error27 at gmail.com>
Link: https://lore.kernel.org/r/202304160359.4LHmFOlU-lkp@intel.com/
To: David Airlie <airlied at gmail.com>
To: Daniel Vetter <daniel at ffwll.ch>
To: Rob Herring <robh+dt at kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
To: Conor Dooley <conor+dt at kernel.org>
To: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
To: Thierry Reding <thierry.reding at gmail.com>
To: Neil Armstrong <neil.armstrong at linaro.org>
To: Sam Ravnborg <sam at ravnborg.org>
Cc: patchwork-jzi at pengutronix.de
Cc: kernel at pengutronix.de
Cc: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>
Cc: dri-devel at lists.freedesktop.org
Cc: devicetree at vger.kernel.org
Cc: linux-kernel at vger.kernel.org
Signed-off-by: Johannes Zink <j.zink at pengutronix.de>
---
Johannes Zink (3):
dt-bindings: display: move LVDS data-mapping definition to separate file
dt-bindings: display: simple: support non-default data-mapping
drm/panel-simple: allow LVDS format override
.../bindings/display/lvds-data-mapping.yaml | 84 ++++++++++++++++++++++
.../devicetree/bindings/display/lvds.yaml | 77 +++-----------------
.../bindings/display/panel/panel-simple.yaml | 26 ++++++-
drivers/gpu/drm/panel/panel-simple.c | 55 +++++++++++++-
4 files changed, 172 insertions(+), 70 deletions(-)
---
base-commit: 52920704df878050123dfeb469aa6ab8022547c1
change-id: 20230523-simplepanel_support_nondefault_datamapping-13c3f2ea28f8
Best regards,
--
Johannes Zink <j.zink at pengutronix.de>
More information about the dri-devel
mailing list