From: Liu Ying <victor.liu@nxp.com>
To: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Cc: marex@denx.de, stefan@agner.ch, airlied@gmail.com,
daniel@ffwll.ch, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
shawnguo@kernel.org, s.hauer@pengutronix.de,
kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com,
krzysztof.kozlowski@linaro.org, LW@karo-electronics.de,
alexander.stein@ew.tq-group.com
Subject: [PATCH v5 0/6] drm: lcdif: Add i.MX93 LCDIF support
Date: Mon, 8 May 2023 13:57:34 +0800 [thread overview]
Message-ID: <20230508055740.635256-1-victor.liu@nxp.com> (raw)
Hi,
This patch set aims to add i.MX93 LCDIF display controller support
in the existing LCDIF DRM driver. The LCDIF embedded in i.MX93 SoC
is essentially the same to those embedded in i.MX8mp SoC. Through
internal bridges, i.MX93 LCDIF may drive a MIPI DSI display or a LVDS
display or a parallel display.
Patch 1/6 adds device tree binding support for i.MX93 LCDIF in the
existing fsl,lcdif.yaml.
Patch 2/6 drops lcdif->bridge NULL pointer check as a cleanup patch.
Patch 3/6~5/6 prepare for adding i.MX93 LCDIF support step by step.
Patch 6/6 adds i.MX93 LCDIF compatible string as the last step of
adding i.MX93 LCDIF support.
v4->v5:
* Rebase this patch set upon v6.4-rc1 and resolve a trivial conflict for
patch 5/6.
* Collect Alexander's R-b tag on patch 3/6, A-b tags on patch 4/6 and 5/6,
T-b tags on patch 2/6~6/6.
v3->v4:
* Improve warning message when ignoring invalid LCDIF OF endpoint ids in
patch 5/6. (Alexander)
* Use 'new_{c,p}state' instead of 'new_{crtc,plane}_state' in patch 3/6.
(Alexander)
* Simplify lcdif_crtc_reset() by calling lcdif_crtc_atomic_destroy_state()
in patch 3/6. (Alexander)
* Add '!crtc->state' check in lcdif_crtc_atomic_duplicate_state() in patch 3/6.
(Alexander)
* Collect Alexander's R-b tags on patch 1/6, 2/6 and 6/6.
v2->v3:
* Fix a trivial typo in patch 6/6's commit message.
v1->v2:
* Add Krzysztof's A-b and Marek's R-b tags on patch 1/6.
* Split patch 2/2 in v1 into patch 2/6~6/6 in v2. (Marek, Alexander)
* Drop '!remote ||' from lcdif_attach_bridge(). (Lothar)
* Add comment on the 'base' member of lcdif_crtc_state structure to
note it should always be the first member. (Lothar)
* Drop unneeded 'bridges' member from lcdif_drm_private structure.
* Drop a comment about bridge input bus format from lcdif_crtc_atomic_check().
Liu Ying (6):
dt-bindings: lcdif: Add i.MX93 LCDIF support
drm: lcdif: Drop unnecessary NULL pointer check on lcdif->bridge
drm: lcdif: Determine bus format and flags in ->atomic_check()
drm: lcdif: Check consistent bus format and flags across first bridges
drm: lcdif: Add multiple encoders and first bridges support
drm: lcdif: Add i.MX93 LCDIF compatible string
.../bindings/display/fsl,lcdif.yaml | 7 +-
drivers/gpu/drm/mxsfb/lcdif_drv.c | 71 ++++++-
drivers/gpu/drm/mxsfb/lcdif_drv.h | 5 +-
drivers/gpu/drm/mxsfb/lcdif_kms.c | 198 ++++++++++++------
4 files changed, 206 insertions(+), 75 deletions(-)
--
2.37.1
next reply other threads:[~2023-05-08 5:53 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-08 5:57 Liu Ying [this message]
2023-05-08 5:57 ` [PATCH v5 1/6] dt-bindings: lcdif: Add i.MX93 LCDIF support Liu Ying
2023-05-08 6:26 ` Rob Herring
2023-05-08 5:57 ` [PATCH v5 2/6] drm: lcdif: Drop unnecessary NULL pointer check on lcdif->bridge Liu Ying
2023-05-09 0:02 ` Marek Vasut
2023-05-10 8:30 ` Ying Liu
2023-05-10 8:40 ` Marek Vasut
2023-05-08 5:57 ` [PATCH v5 3/6] drm: lcdif: Determine bus format and flags in ->atomic_check() Liu Ying
2023-05-09 0:11 ` Marek Vasut
2023-05-08 5:57 ` [PATCH v5 4/6] drm: lcdif: Check consistent bus format and flags across first bridges Liu Ying
2023-05-09 0:13 ` Marek Vasut
2023-05-08 5:57 ` [PATCH v5 5/6] drm: lcdif: Add multiple encoders and first bridges support Liu Ying
2023-05-09 0:23 ` Marek Vasut
2023-05-10 8:44 ` Ying Liu
2023-05-08 5:57 ` [PATCH v5 6/6] drm: lcdif: Add i.MX93 LCDIF compatible string Liu Ying
2023-05-09 0:24 ` Marek Vasut
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=20230508055740.635256-1-victor.liu@nxp.com \
--to=victor.liu@nxp.com \
--cc=LW@karo-electronics.de \
--cc=airlied@gmail.com \
--cc=alexander.stein@ew.tq-group.com \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=festevam@gmail.com \
--cc=kernel@pengutronix.de \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=marex@denx.de \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=stefan@agner.ch \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).