[PATCH v7 2/4] drm/kmb: Add support for KeemBay Display

Daniel Vetter daniel at ffwll.ch
Fri Sep 11 08:56:42 UTC 2020


On Thu, Sep 10, 2020 at 08:33:43PM +0200, Sam Ravnborg wrote:
> Hi Daniel.
> 
> > > +	}
> > > +
> > > +	/* Initialize MIPI DSI */
> > > +	ret = kmb_dsi_init(drm, adv_bridge);
> > 
> > Split up isn't correct here, this won't compile since the dsi code isn't
> > in this patch yet. So you need to have this ordered the other way round.
> 
> I have explicit asked for a simple file split as done here
> and told Anitha this was fine.
> 
> As the kbuild/kconfig bits are the last patch then no build breakage.
> The file based split was IMO fine to ease review.
> 
> If this is not OK then blame me :-)

Ah I missed that, sounds ok for me too then :-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list