Re: [PATCH v2 2/3] drm/bridge: simple-bridge: Support data-enable-active endpoint property

[Frank Li <Frank.li@oss.nxp.com>]

On Tue, Oct 20, 2026 at 08:15:57AM -0300, Leonardo Costa wrote:
> On Mon, Oct 19, 2026 at 02:50:36PM -0500, Frank Li wrote:
> > On Mon, Oct 19, 2026 at 07:17:15AM -0300, Leonardo Costa wrote:
> > > From: Leonardo Costa <leonardo.costa@toradex.com>
> > >
> > > Some simple bridges, such as a resistor ladder DAC, may include a
> > > data-enable input that must be driven at a specific polarity.
> > >
> > > Read the data-enable-active property from the input endpoint and fill
> > > the bridge's input bus flags accordingly, so the upstream encoder drives
> > > DE with the required polarity. The property is applied only if the
> > > bridge doesn't provide its own timings struct.
> > >
> > > Signed-off-by: Leonardo Costa <leonardo.costa@toradex.com>
> > > ---
> >
> > Reviewed-by: Frank Li <Frank.Li@nxp.com>
>
> Hi Frank,
>
> Sashiko's comment regarding atomic_check was a false positive, but I
> wasn't too sure about the one regarding devm_kzalloc. It seemed to make
> sense to me, but I'm not too familiar with this subsystem yet. Do you
> think this was also a false positive?

I am also not familiar it, it looks make sense to me also.

Frank

>
> Best regards,
> Leonardo