Re: [PATCH v6 1/6] drm/bridge: cadence: cdns-mhdp8546-core: Remove legacy support for connector initialisation in bridge

[Harikrishna Shenoy]

On 9/9/25 19:39, Dmitry Baryshkov wrote:
> On Tue, Sep 09, 2025 at 02:38:19PM +0530, Harikrishna Shenoy wrote:
> > From: Jayesh Choudhary <j-choudhary@xxxxxx>
> >
> > Now that we have DBANC framework, remove the connector initialisation code
> > as that piece of code is not called if DRM_BRIDGE_ATTACH_NO_CONNECTOR flag
> > is used. Only TI K3 platforms consume this driver and tidss (their display
> > controller) has this flag set. So this legacy support can be dropped.
> >
> > Fixes: c932ced6b585 ("drm/tidss: Update encoder/bridge chain connect model")
> I'm not sure, why do you have this Fixes tag. Other than that:
>
>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
Hi Dmitry,

Have explained about fixes tag in cover letter, that these 4 commits

fix the bug of null pointer de-refernce (logs and bug described in cover 
letter).

also have given the rationale of sequencing of commits in cover letter.

Warm Regards,

Hari

> > Signed-off-by: Jayesh Choudhary <j-choudhary@xxxxxx>
> > ---
> >   .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 187 +-----------------
> >   1 file changed, 10 insertions(+), 177 deletions(-)