mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2] drm/bridge: lt9611c: use drm_atomic_commit in callbacks
@ 2026-07-07  8:18 Mohit Dsor
  2026-07-07 12:35 ` Maxime Ripard
  0 siblings, 1 reply; 2+ messages in thread
From: Mohit Dsor @ 2026-07-07  8:18 UTC (permalink / raw)
  To: Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
	Jonas Karlman, Jernej Skrabec, Luca Ceresoli, Maarten Lankhorst,
	Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
	Sunyun Yang, Dmitry Baryshkov, Mohit Dsor
  Cc: dri-devel, linux-kernel, venkata.valluru, Jessica Zhang, Mohit Dsor

DRM bridge atomic callbacks now pass struct drm_atomic_commit, and
the atomic helper functions also expect that type. The lt9611c
driver still declares its callbacks with struct drm_atomic_state,
which makes the callback assignments and helper calls fail to build
with incompatible pointer type errors.

Update the lt9611c callback signatures to match the current DRM
bridge API.

This will fix compilation issue in v7.2.rc1.

Fixes: 96d1cfec419e ("drm/bridge: Add Lontium LT9611C(EX/UXD) MIPI DSI to HDMI driver")
Signed-off-by: Mohit Dsor <mohit.dsor@oss.qualcomm.com>
---
Changes in v2:
- EDITME: describe what is new in this series revision.
- EDITME: use bulletpoints and terse descriptions.
- Link to v1: https://patch.msgid.link/20260707-lt9611c-v7-v1-1-3f48568b6e56@oss.qualcomm.com
---
 drivers/gpu/drm/bridge/lontium-lt9611c.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/bridge/lontium-lt9611c.c b/drivers/gpu/drm/bridge/lontium-lt9611c.c
index cb584855fd8f..022205177a4d 100644
--- a/drivers/gpu/drm/bridge/lontium-lt9611c.c
+++ b/drivers/gpu/drm/bridge/lontium-lt9611c.c
@@ -614,7 +614,7 @@ static void lt9611c_video_setup(struct lt9611c *lt9611c,
 }
 
 static void lt9611c_bridge_atomic_pre_enable(struct drm_bridge *bridge,
-					     struct drm_atomic_state *state)
+					     struct drm_atomic_commit *state)
 {
 	struct lt9611c *lt9611c = bridge_to_lt9611c(bridge);
 	int ret;
@@ -626,7 +626,7 @@ static void lt9611c_bridge_atomic_pre_enable(struct drm_bridge *bridge,
 }
 
 static void lt9611c_bridge_atomic_enable(struct drm_bridge *bridge,
-					 struct drm_atomic_state *state)
+					 struct drm_atomic_commit *state)
 {
 	struct lt9611c *lt9611c = bridge_to_lt9611c(bridge);
 	struct drm_connector *connector;
@@ -652,7 +652,7 @@ static void lt9611c_bridge_atomic_enable(struct drm_bridge *bridge,
 }
 
 static void lt9611c_bridge_atomic_post_disable(struct drm_bridge *bridge,
-					       struct drm_atomic_state *state)
+					       struct drm_atomic_commit *state)
 {
 	struct lt9611c *lt9611c = bridge_to_lt9611c(bridge);
 	int ret;
@@ -1238,4 +1238,3 @@ MODULE_AUTHOR("SunYun Yang <syyang@lontium.com>");
 MODULE_DESCRIPTION("Lontium LT9611C(EX/UXD) MIPI DSI to HDMI driver");
 MODULE_LICENSE("GPL");
 MODULE_FIRMWARE(FW_FILE);
-

---
base-commit: a9f7d19810c7719ca55134a386dfc48160a1bcfc
change-id: 20260707-lt9611c-v7-4c6579cdb267
prerequisite-message-id: 20260508134009.4582-3-syyang@lontium.com
prerequisite-patch-id: e9ce15b4a5842cfde130a813afecf136798179f4
prerequisite-patch-id: 569618a60e8c3615e7347c79b48c4756b7bf85d0

Best regards,
--  
Mohit Dsor <mohit.dsor@oss.qualcomm.com>


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] drm/bridge: lt9611c: use drm_atomic_commit in callbacks
  2026-07-07  8:18 [PATCH v2] drm/bridge: lt9611c: use drm_atomic_commit in callbacks Mohit Dsor
@ 2026-07-07 12:35 ` Maxime Ripard
  0 siblings, 0 replies; 2+ messages in thread
From: Maxime Ripard @ 2026-07-07 12:35 UTC (permalink / raw)
  To: Mohit Dsor
  Cc: Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
	Jonas Karlman, Jernej Skrabec, Luca Ceresoli, Maarten Lankhorst,
	Thomas Zimmermann, David Airlie, Simona Vetter, Sunyun Yang,
	Dmitry Baryshkov, Mohit Dsor, dri-devel, linux-kernel,
	venkata.valluru, Jessica Zhang

[-- Attachment #1: Type: text/plain, Size: 741 bytes --]

Hi,

On Tue, Jul 07, 2026 at 01:48:35PM +0530, Mohit Dsor wrote:
> DRM bridge atomic callbacks now pass struct drm_atomic_commit, and
> the atomic helper functions also expect that type. The lt9611c
> driver still declares its callbacks with struct drm_atomic_state,
> which makes the callback assignments and helper calls fail to build
> with incompatible pointer type errors.
> 
> Update the lt9611c callback signatures to match the current DRM
> bridge API.
> 
> This will fix compilation issue in v7.2.rc1.
> 
> Fixes: 96d1cfec419e ("drm/bridge: Add Lontium LT9611C(EX/UXD) MIPI DSI to HDMI driver")
> Signed-off-by: Mohit Dsor <mohit.dsor@oss.qualcomm.com>

This commit, or driver, doesn't exist in 7.2-rc1.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-07-07 12:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-07-07  8:18 [PATCH v2] drm/bridge: lt9611c: use drm_atomic_commit in callbacks Mohit Dsor
2026-07-07 12:35 ` Maxime Ripard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®