Re: [PATCH 00/37] drm: more doc work&cleanup, mostly vblank related

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

 



On Wed, May 24, 2017 at 04:51:35PM +0200, Daniel Vetter wrote:
> Hi all,
> 
> So the first part of this patch series is the usual deal of decrufting some part
> of drm and attempting to improve the docs. Mostly around vblank. I seem to be
> the ever constant 5% of finally being able to document drm_driver/device
> properly, oh well.
> 
> By patch count the bigger part is trying to get rid of drm_vblank_cleanup. That
> was initially exported to drivers for a radeon ums irq failover hack, and then
> seems to have been cargo-culted all over the place. It's a bit of churn, but it
> will get us one step closer to tracking vblanks as part of drm_crtc. The other
> bit would be to get rid of drm_vblank_init, and instead auto-setup the relevant
> vblank stuff in drm_crtc_init (iff the driver supports vblanks ofc).

Whilst I laud the motive, afaict drm_vblank_cleanup() is still required
after drm_vblank_init(), if only to kfree(dev->vblank). I didn't see
that change in the earlier patches...
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux