[Intel-gfx] [PATCH 02/14] drm: Hide hw.lock cleanup in filp->release better
Emil Velikov
emil.l.velikov at gmail.com
Wed Jun 15 12:10:31 UTC 2016
On 14 June 2016 at 19:50, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> A few things:
> - Rename the cleanup function from drm_master_release to
> drm_legacy_lock_release. It doesn't relase any master stuff, but
> just the legacy hw lock.
> - Hide it in drm_lock.c, which allows us to make a few more functions
> static in there. To avoid forward decl we need to shuffle the code a
> bit though.
> - Push the check for ->master into the function itself.
> - Only call this for !DRIVER_MODESET.
>
As per the multiple points above - it would have been great to split
out the drm_lock.c code reshuffle as separate patch.
Regardless of my suggestion:
Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
Side note: I think we don't need/bother with kernel doc for static
functions, do we ?
-Emil
More information about the Intel-gfx
mailing list