On Mon, Mar 13, 2017 at 01:53:28PM -0400, Sean Paul wrote: > On Wed, Mar 08, 2017 at 03:12:44PM +0100, Daniel Vetter wrote: > > Well, mostly drm_file.h, and clean up all related things: > > > > - I didnt' figure out the difference between preclose and postclose. > > The existing explanation in drm-internals.rst didn't convince me, > > since it's also really outdated - we clean up pending DRM events in > > the core nowadays. I put a FIXME in for the future. > > > > - Another FIXME is to have a macro for default fops. > > > > - Lots of links all around, main areas are to tie the overview in > > drm_file.c more into the callbacks in struct drm_device, and the > > other is to link render/primary node code to the right sections in > > drm-uapi.rst. > > > > - Also moved the open/close stuff to drm_drv.h from drm-internals.rst, > > seems like the better place for that information. Since that section > > was rather outdated this amounted to full-on rewrite. > > > > A big missing piece here is some overview graph, but I think better to > > wait with that one until drm_device and drm_driver are also fully > > documented. > > > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > Overall, awesome patch, really does a good job explaining the parameters and how > they interact with each other (bonus points for the lock breadcrumbs). I just > have a few optional nits and some spelling errors. > > Reviewed-by: Sean Paul <seanpaul@xxxxxxxxxxxx> Fixed all the nits, except for /* private: */, as explained, that's just kernel-doc markup to annotate internal structure stuff that users shouldn't look at. Thanks a lot for your careful read-through, 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
