Hi all, At 2025-03-21 17:48:04, "Maxime Ripard" <mrip...@kernel.org> wrote: >On Fri, Mar 21, 2025 at 04:53:38PM +0800, Andy Yan wrote: >> From: Andy Yan <andy....@rock-chips.com> >> >> In some application scenarios, we hope to get the corresponding >> connector when the bridge's detect hook is invoked. >> >> In most cases, we can get the connector by >> drm_atomic_get_connector_for_encoder >> if the encoder attached to the bridge is enabled, however there will >> still be some scenarios where the detect hook of the bridge is called >> but the corresponding encoder has not been enabled yet. For instance, >> this occurs when the device is hot plug in for the first time. >> >> Since the call to bridge's detect is initiated by the connector, passing >> down the corresponding connector directly will make things simpler. >> >> Signed-off-by: Andy Yan <andy....@rock-chips.com> > >FTR, I'm against it and would have appreciated that you wait for a >meaningful closure to the discussion we've had on this.
Can we start to review this patch? Since it's been almost a month now and no one has raised any objection to Dmitry's suggestion[1]. [1]https://lore.kernel.org/dri-devel/vw2ncdomx3rwltb2xlo6nf3rapgcdtcjcodofgmecrzzabf7ji@pybsfv27jkq2/ > >Maxime