Re: [PATCH] drm/omapdrm: don't call drm_helper_disable_unused_functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 10, 2016 at 09:26:42AM +0300, Tomi Valkeinen wrote:
> On 10/06/16 09:24, Tomi Valkeinen wrote:
> > On 10/06/16 01:14, Daniel Vetter wrote:
> >> It's a legacy helper function which won't do good with atomic helpers.
> >>
> >> Cc: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
> >> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> >> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> >> ---
> >>  drivers/gpu/drm/omapdrm/omap_fbdev.c | 3 ---
> >>  1 file changed, 3 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/omapdrm/omap_fbdev.c b/drivers/gpu/drm/omapdrm/omap_fbdev.c
> >> index 31dfa0893416..adb10fbe918d 100644
> >> --- a/drivers/gpu/drm/omapdrm/omap_fbdev.c
> >> +++ b/drivers/gpu/drm/omapdrm/omap_fbdev.c
> >> @@ -279,9 +279,6 @@ struct drm_fb_helper *omap_fbdev_init(struct drm_device *dev)
> >>  	if (ret)
> >>  		goto fini;
> >>  
> >> -	/* disable all the possible outputs/crtcs before entering KMS mode */
> >> -	drm_helper_disable_unused_functions(dev);
> >> -
> >>  	ret = drm_fb_helper_initial_config(helper, 32);
> >>  	if (ret)
> >>  		goto fini;
> >>
> > 
> > Acked-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
> 
> Ops, my mail client was too eager to send this... I was about to ask if
> I should pick this up?

Sorry for the long delay. I merged this into drm-misc so that I can merge
the patch to WARN_ON when using this function for atomic drivers, too. Too
many people hit this little snag ;-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux