[PATCH v5 2/4] drm/bridge: microchip-lvds: migrate to atomic bridge ops
Maxime Ripard
mripard at kernel.org
Wed Jun 25 06:54:15 UTC 2025
On Wed, Jun 25, 2025 at 10:26:10AM +0530, Dharma Balasubiramani wrote:
> Replace legacy .enable and .disable callbacks with their atomic
> counterparts .atomic_enable and .atomic_disable.
>
> Also, add turn off the serialiser inside atomic_disable().
>
> Signed-off-by: Dharma Balasubiramani <dharma.b at microchip.com>
As a rule of thumb, if you have "Also, do X" in your commit log, you
need a separate patch.
And you need to explain why turning off the serialiser inside
atomic_disable() is needed. It might make sense to you, it's not really
obvious to me from that patch, and it will definitely not be to someone
trying to identify fixes and doing backports.
Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 273 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250625/23b51a0d/attachment.sig>
More information about the dri-devel
mailing list