mbox series

[0/3] drm/panel: ABT Y030XX067A panel support

Message ID 20201101093150.8071-1-paul@crapouillou.net (mailing list archive)
Headers show
Series drm/panel: ABT Y030XX067A panel support | expand

Message

Paul Cercueil Nov. 1, 2020, 9:31 a.m. UTC
Hi,

This patchset is for adding support for the Asia Better Technology (aka.
ABT) Y030XX067A 3.0" 320x480 24-bit LCD IPS panel.

While being 320x480 it is actually 4:3 with non-square pixels, and
requires a specific bus format, as the pixel ordering changes each line
(RGB on odd lines, GRB on even lines).

Patch #1 adds the abt,* vendor prefix.
Patch #2 adds the abt,y030xx067a panel binding documentation.
Patch #3 adds the MEDIA_BUS_FMT_RGB888_3X8_DELTA media bus format.
Patch #4 adds the driver itself.

Cheers,
-Paul

Paul Cercueil (4):
  dt-bindings: vendor-prefixes: Add abt vendor prefix
  dt-bindings: display: Add ABT Y030XX067A panel bindings
  media: uapi: Add MEDIA_BUS_FMT_RGB888_3X8_DELTA media bus format
  drm/panel: Add ABT Y030XX067A 3.0" 320x480 panel

 .../display/panel/abt,y030xx067a.yaml         |  54 +++
 .../devicetree/bindings/vendor-prefixes.yaml  |   2 +
 drivers/gpu/drm/panel/Kconfig                 |   9 +
 drivers/gpu/drm/panel/Makefile                |   1 +
 drivers/gpu/drm/panel/panel-abt-y030xx067a.c  | 363 ++++++++++++++++++
 include/uapi/linux/media-bus-format.h         |   3 +-
 6 files changed, 431 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/devicetree/bindings/display/panel/abt,y030xx067a.yaml
 create mode 100644 drivers/gpu/drm/panel/panel-abt-y030xx067a.c

Comments

Sam Ravnborg Dec. 17, 2020, 6:03 a.m. UTC | #1
On Thu, Dec 17, 2020 at 12:59:45AM +0000, Paul Cercueil wrote:
> The binding should use "unevaluatedProperties" instead of
> "additionalProperties", since it is a SPI device and may have
> SPI-related Device Tree properties, for instance the "spi-max-frequency"
> property that is present in the example.
> 
> Fixes: e366a644c69d ("dt-bindings: display: Add ABT Y030XX067A panel bindings")
> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Reviewed-by: Sam Ravnborg <sam@ravnborg.org>