[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <171438653511.2566805.2252397252665472830.b4-ty@kernel.org>
Date: Mon, 29 Apr 2024 12:28:55 +0200
From: Robert Foss <rfoss@...nel.org>
To: Jonas Karlman <jonas@...boo.se>, David Airlie <airlied@...il.com>, Thomas Zimmermann <tzimmermann@...e.de>,
Luca Ceresoli <luca.ceresoli@...tlin.com>, Neil Armstrong <neil.armstrong@...aro.org>, Daniel Vetter <daniel@...ll.ch>,
Andrzej Hajda <andrzej.hajda@...el.com>, Jernej Skrabec <jernej.skrabec@...il.com>,
Alexander Stein <alexander.stein@...tq-group.com>, Maxime Ripard <mripard@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
Cc: dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org, Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH RESEND] Revert "drm/bridge: ti-sn65dsi83: Fix enable error path"
On Fri, 26 Apr 2024 14:22:59 +0200, Luca Ceresoli wrote:
> This reverts commit 8a91b29f1f50ce7742cdbe5cf11d17f128511f3f.
>
> The regulator_disable() added by the original commit solves one kind of
> regulator imbalance but adds another one as it allows the regulator to be
> disabled one more time than it is enabled in the following scenario:
>
> 1. Start video pipeline -> sn65dsi83_atomic_pre_enable -> regulator_enable
> 2. PLL lock fails -> regulator_disable
> 3. Stop video pipeline -> sn65dsi83_atomic_disable -> regulator_disable
>
> [...]
Applied, thanks!
[1/1] Revert "drm/bridge: ti-sn65dsi83: Fix enable error path"
https://cgit.freedesktop.org/drm/drm-misc/commit/?id=2940ee03b232
Rob
Powered by blists - more mailing lists