From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
andrzej.hajda@intel.com, neil.armstrong@linaro.org,
rfoss@kernel.org, Laurent.pinchart@ideasonboard.com,
jonas@kwiboo.se, jernej.skrabec@gmail.com,
maarten.lankhorst@linux.intel.com, mripard@kernel.org,
tzimmermann@suse.de, airlied@gmail.com, simona@ffwll.ch,
Liu Ying <victor.liu@nxp.com>
Subject: Re: [PATCH 0/5] drm/bridge: simple-bridge: Add DPI color encoder support
Date: Tue, 04 Mar 2025 16:00:21 +0100 [thread overview]
Message-ID: <22618349.EfDdHjke4D@steina-w> (raw)
In-Reply-To: <20250304101530.969920-1-victor.liu@nxp.com>
Hi,
Am Dienstag, 4. März 2025, 11:15:25 CET schrieb Liu Ying:
> Hi,
>
> This patch series aims to add DPI color encoder support as a simple DRM
> bridge. A DPI color encoder simply converts input DPI color coding to
> output DPI color coding, like Adafruit Kippah DPI hat[1] which converts
> input 18-bit pixel data to 24-bit pixel data(with 2 low padding bits in
> every color component though). A real use case is that NXP i.MX93 11x11
> EVK[2] and i.MX93 9x9 QSB[3] boards may connect a 24-bit DPI panel through
> the Adafruit Kippah DPI hat. The display pipeline is
>
> i.MX93 LCDIF display controller(RGB888) ->
> i.MX93 parallel display format configuration(RGB666) ->
> on-board Raspiberry Pi compatible interface(RPi)(RGB666) ->
> Adafruit Kippah DPI hat(RGB888 with 2 low padding bits in color components) ->
> 24-bit "ontat,kd50g21-40nt-a1" DPI panel
>
> [1] https://learn.adafruit.com/adafruit-dpi-display-kippah-ttl-tft/downloads
> [2] https://www.nxp.com/design/design-center/development-boards-and-designs/i.MX93EVK
> [3] https://www.nxp.com/design/design-center/development-boards-and-designs/IMX93QSB
Thanks for this series.
Actually I was about to create a similar (dumb) bridge. My use case is wrong
wiring on hardware for DPI displays. The current workaround was to use a
"new" display compatible with bus_format changes from
MEDIA_BUS_FMT_RGB666_1X18 -> MEDIA_BUS_FMT_RGB888_1X24.
I added this new bridge and changed my DT and it works flawlessly.
Best regards
Alexander
> Liu Ying (5):
> dt-bindings: display: Document DPI color codings
> drm/of: Add drm_of_dpi_get_color_coding()
> dt-bindings: display: simple-bridge: Document DPI color encoder
> drm/bridge: simple-bridge: Add DPI color encoder support
> drm/bridge: simple-bridge: Add next panel support
>
> .../display/bridge/simple-bridge.yaml | 89 +++++++++++-
> .../bindings/display/dpi-color-coding.yaml | 90 ++++++++++++
> drivers/gpu/drm/bridge/Kconfig | 1 +
> drivers/gpu/drm/bridge/simple-bridge.c | 132 ++++++++++++++++--
> drivers/gpu/drm/drm_of.c | 43 ++++++
> include/drm/drm_of.h | 7 +
> 6 files changed, 348 insertions(+), 14 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/display/dpi-color-coding.yaml
>
>
--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
next prev parent reply other threads:[~2025-03-04 15:00 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-04 10:15 [PATCH 0/5] drm/bridge: simple-bridge: Add DPI color encoder support Liu Ying
2025-03-04 10:15 ` [PATCH 1/5] dt-bindings: display: Document DPI color codings Liu Ying
2025-03-04 10:33 ` Maxime Ripard
2025-03-05 7:51 ` Krzysztof Kozlowski
2025-03-05 8:26 ` Maxime Ripard
2025-03-05 12:16 ` Krzysztof Kozlowski
2025-03-06 7:13 ` Liu Ying
2025-03-04 10:15 ` [PATCH 2/5] drm/of: Add drm_of_dpi_get_color_coding() Liu Ying
2025-03-04 10:15 ` [PATCH 3/5] dt-bindings: display: simple-bridge: Document DPI color encoder Liu Ying
2025-03-04 10:38 ` Maxime Ripard
2025-03-06 5:49 ` Liu Ying
2025-03-04 11:27 ` Rob Herring (Arm)
2025-03-04 15:23 ` Rob Herring
2025-03-05 9:35 ` Alexander Stein
2025-03-05 16:38 ` Rob Herring
2025-03-06 7:02 ` Liu Ying
2025-03-06 11:35 ` Maxime Ripard
2025-03-06 20:34 ` Rob Herring
2025-03-07 3:25 ` Liu Ying
2025-03-10 9:53 ` Maxime Ripard
2025-03-11 2:29 ` Liu Ying
2025-03-11 7:44 ` Maxime Ripard
2025-03-07 3:10 ` Liu Ying
2025-03-10 9:44 ` Maxime Ripard
2025-03-11 2:38 ` Liu Ying
2025-03-11 7:52 ` Maxime Ripard
2025-03-04 10:15 ` [PATCH 4/5] drm/bridge: simple-bridge: Add DPI color encoder support Liu Ying
2025-03-04 10:40 ` Maxime Ripard
2025-03-06 5:57 ` Liu Ying
2025-03-04 10:15 ` [PATCH 5/5] drm/bridge: simple-bridge: Add next panel support Liu Ying
2025-03-04 10:41 ` Maxime Ripard
2025-03-06 6:17 ` Liu Ying
2025-03-04 15:00 ` Alexander Stein [this message]
2026-01-12 9:31 ` [PATCH 0/5] drm/bridge: simple-bridge: Add DPI color encoder support Francesco Dolcini
2026-01-12 9:44 ` Liu Ying
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=22618349.EfDdHjke4D@steina-w \
--to=alexander.stein@ew.tq-group.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=robh@kernel.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
--cc=victor.liu@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.