dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Luca Ceresoli <luca.ceresoli@bootlin.com>
To: Maxime Ripard <mripard@kernel.org>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Andrzej Hajda <andrzej.hajda@intel.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Robert Foss <rfoss@kernel.org>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Jonas Karlman <jonas@kwiboo.se>,
	Louis Chauvet <louis.chauvet@bootlin.com>,
	Dmitry Baryshkov <lumag@kernel.org>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH v3 02/22] drm: convert many bridge drivers from devm_kzalloc() to devm_drm_bridge_alloc() API
Date: Mon, 12 May 2025 16:02:01 +0200	[thread overview]
Message-ID: <20250512160201.7d0b21d8@booty> (raw)
In-Reply-To: <4yeqvg3wnlr2bhb54zutgqpkehrodat5w5x4rr5qjlrc2ts3pz@gr2iosycclpl>

Hi Maxime,

On Sat, 10 May 2025 10:43:04 +0200
Maxime Ripard <mripard@kernel.org> wrote:

> Acked-by: Maxime Ripard <mripard@kernel.org>

Thanks for looking at this series!

Unfortunately the exynos patch is still needing a decision. Quick recap
of the story:

 1. patch was sent in v2, nobody acked/reviewed it [1]
 2. was applied by mistake to drm-misc-next [2]
 3. you pinged maintainers, no reply do far -> should be reverted [3]
 4. Louis (fomys) asked on IRC how to handle it, no reply [4]
 5. in this v3 I added a revert (patch 1), no acks/reviews on it [5]

[1] https://lore.kernel.org/lkml/20250424-drm-bridge-convert-to-alloc-api-v2-14-8f91a404d86b@bootlin.com/
[2] https://lore.kernel.org/lkml/832a9db0-cf8a-4d35-8a98-08053fbd6723@bootlin.com/
[3] https://lore.kernel.org/lkml/20250430-arrogant-marmoset-of-justice-92ced3@houat/
[4] https://dri.freedesktop.org/~cbrill/dri-log/index.php?channel=dri-devel&highlight_names=&date=2025-05-07
[5] https://lore.kernel.org/lkml/20250509-drm-bridge-convert-to-alloc-api-v3-1-b8bc1f16d7aa@bootlin.com/

What to do? Proposed plan:

 1. Louis immediately applies the revert (patch 1)
 2. I send v4 with the original patch to hopefully be reviewed/acked

Plan is OK?

I'm assuming Louis doesn't need an Acked/Reviewed-by for that, but I might
be wrong, and we both would like to avoid further mess.

This annoying issue is taking much more effort to be fixed than it took
to develop it. I'd like to get past it and think about the next steps
in bridge lifetime management.

Thanks for your understanding.

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2025-05-12 14:02 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-09 13:53 [PATCH v3 00/22] drm: convert all bridges to devm_drm_bridge_alloc() Luca Ceresoli
2025-05-09 13:53 ` [PATCH v3 01/22] Revert "drm/exynos: mic: convert to devm_drm_bridge_alloc() API" Luca Ceresoli
2025-05-09 13:53 ` [PATCH v3 02/22] drm: convert many bridge drivers from devm_kzalloc() to devm_drm_bridge_alloc() API Luca Ceresoli
2025-05-10  8:43   ` Maxime Ripard
2025-05-12 14:02     ` Luca Ceresoli [this message]
2025-05-22 13:08       ` Dmitry Baryshkov
2025-05-22 16:09         ` Luca Ceresoli
2025-05-25 19:40   ` Adam Ford
2025-05-27  9:27     ` Luca Ceresoli
2025-05-09 13:53 ` [PATCH v3 03/22] drm/bridge: anx7625: convert " Luca Ceresoli
2025-05-09 18:26   ` Anusha Srivatsa
2025-05-10  8:43   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 04/22] drm/bridge: cdns-dsi: " Luca Ceresoli
2025-05-09 18:31   ` Anusha Srivatsa
2025-05-10  8:44   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 05/22] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: " Luca Ceresoli
2025-05-09 18:48   ` Anusha Srivatsa
2025-05-10  8:52   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 06/22] drm/bridge: nxp-ptn3460: " Luca Ceresoli
2025-05-09 19:57   ` Anusha Srivatsa
2025-05-10  8:52   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 07/22] drm/bridge: sii902x: " Luca Ceresoli
2025-05-09 20:04   ` Anusha Srivatsa
2025-05-10  8:52   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 08/22] drm/omap: dss: dpi: " Luca Ceresoli
2025-05-09 20:12   ` Anusha Srivatsa
2025-05-10  8:54   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 09/22] drm/omap: dss: dsi: " Luca Ceresoli
2025-05-09 20:19   ` Anusha Srivatsa
2025-05-10  8:54   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 10/22] drm/omap: dss: hdmi4: " Luca Ceresoli
2025-05-10  8:54   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 11/22] drm/omap: dss: hdmi5: " Luca Ceresoli
2025-05-10  8:54   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 12/22] drm/omap: dss: sdi: " Luca Ceresoli
2025-05-10  8:58   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 13/22] drm/omap: dss: venc: " Luca Ceresoli
2025-05-10  8:55   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 14/22] drm/rcar-du: dsi: " Luca Ceresoli
2025-05-09 20:26   ` Anusha Srivatsa
2025-05-10  8:56   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 15/22] drm/bridge: stm_lvds: " Luca Ceresoli
2025-05-09 20:32   ` Anusha Srivatsa
2025-05-10  8:56   ` Maxime Ripard
2025-05-20 11:37   ` Raphael Gallais-Pou
2025-05-09 13:53 ` [PATCH v3 16/22] drm/sti: dvo: " Luca Ceresoli
2025-05-10  8:57   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 17/22] drm: zynqmp_dp: " Luca Ceresoli
2025-05-10  8:57   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 18/22] drm/bridge: imx8qxp-pixel-combiner: " Luca Ceresoli
2025-05-27  1:48   ` Liu Ying
2025-05-09 13:53 ` [PATCH v3 19/22] drm/bridge: tc358767: " Luca Ceresoli
2025-05-09 13:53 ` [PATCH v3 20/22] drm/bridge: add devm_drm_put_bridge() Luca Ceresoli
2025-05-10  9:15   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 21/22] drm/bridge: panel: convert to devm_drm_bridge_alloc() API Luca Ceresoli
2025-05-10  9:42   ` Maxime Ripard
2025-05-09 13:53 ` [PATCH v3 22/22] drm/todo: add entry to remove devm_drm_put_bridge() Luca Ceresoli
2025-05-10  9:42   ` Maxime Ripard
2025-05-21 14:22 ` [PATCH v3 00/22] drm: convert all bridges to devm_drm_bridge_alloc() Luca Ceresoli
2025-05-22  3:20   ` Liu Ying
2025-05-22  7:28     ` Luca Ceresoli
2025-05-22 15:01     ` Maxime Ripard
2025-05-22 14:57   ` Maxime Ripard
2025-05-22 15:56     ` Luca Ceresoli
2025-05-22 15:11   ` Inki Dae
2025-05-23 13:23     ` Luca Ceresoli
2025-05-27 10:21 ` (subset) " Luca Ceresoli

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=20250512160201.7d0b21d8@booty \
    --to=luca.ceresoli@bootlin.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=airlied@gmail.com \
    --cc=andrzej.hajda@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jonas@kwiboo.se \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=louis.chauvet@bootlin.com \
    --cc=lumag@kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=rfoss@kernel.org \
    --cc=simona@ffwll.ch \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=tzimmermann@suse.de \
    /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).