[PATCH] drm/omap: dsi: Fix missing of_platform_depopulate()

Laurent Pinchart laurent.pinchart at ideasonboard.com
Thu Nov 8 17:08:57 UTC 2018


Hi Tony,

On Thursday, 8 November 2018 19:01:45 EET Tony Lindgren wrote:
> * Laurent Pinchart <laurent.pinchart at ideasonboard.com> [181107 05:30]:
> > On Tuesday, 6 November 2018 17:28:02 EET Tony Lindgren wrote:
> > > We're missing a call to of_platform_depopulate() on errors for dsi.
> > > Looks like dss is already doing this.
> > > 
> > > Signed-off-by: Tony Lindgren <tony at atomide.com>
> > 
> > This makes sense to me. You may however want to note in the commit message
> > that this patch also turns the of_platform_populate() failures into fatal
> > errors.
> 
> Well of_platform_populate() failure should be fatal here,right? :)

I'm not complaining about the functional change, just the lack of visibility 
:-)

> Tomi, let me know if you need a resend with updated comments or
> if you want to update the message yourself to your liking.

-- 
Regards,

Laurent Pinchart





More information about the dri-devel mailing list