Hi Marek, On Thu, May 11, 2023 at 12:24 AM Marek Vasut <marex@xxxxxxx> wrote: > On 5/10/23 11:24, Liu Ying wrote: > > The single LCDIF embedded in i.MX93 SoC may drive multiple displays > > simultaneously. Look at LCDIF output port's remote port parents to > > find all enabled first bridges. Add an encoder for each found bridge > > and attach the bridge to the encoder. This is a preparation for > > adding i.MX93 LCDIF support. > > > > Tested-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> > > Acked-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> > > Signed-off-by: Liu Ying <victor.liu@xxxxxxx> > > --- > > v5->v6: > > * Drop MAX_DISPLAYS macro. (Marek) > > * Drop the encoder member in struct lcdif_drm_private. > > * Drop endpoint id check. > > It might be nice to check (based on driver data for each IP variant) the > encoder count, but that can be a separate patch. > > Reviewed-by: Marek Vasut <marex@xxxxxxx> > > Thanks ! > > btw if this doesn't get picked by someone in like a week or two, let me > know and I'll apply this via drm-misc . Thanks for your review. I don't see any more comments in two weeks. Can you please go ahead to apply this series? Regards, Liu Ying
