On Sat, Feb 02, 2019 at 09:41:53AM -0600, Rob Herring wrote: > This series implements the todo to add reservation_object to > drm_gem_object. I converted the easy drivers, but not Intel or AMD. The > series is build tested only. > > Rob > > v2: > - Fix missing kerneldoc > - Reword todo with what is left todo. > - Fix timeout error handling (added to drm_gem_reservation_object_wait) I like. On the series: Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx> for msm just ping Sean Paul on irc, he's usually quick with taking a lock. And I'd merge the entire series through drm-misc, least amount of trouble. -Daniel > > Rob Herring (5): > drm: Add reservation_object to drm_gem_object > drm: etnaviv: Switch to use drm_gem_object reservation_object > drm: msm: Switch to use drm_gem_object reservation_object > drm: v3d: Switch to use drm_gem_object reservation_object > drm: vc4: Switch to use drm_gem_object reservation_object > > Documentation/gpu/todo.rst | 8 ++-- > drivers/gpu/drm/drm_gem.c | 43 ++++++++++++++++++++ > drivers/gpu/drm/drm_prime.c | 1 + > drivers/gpu/drm/etnaviv/etnaviv_drv.c | 1 - > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - > drivers/gpu/drm/etnaviv/etnaviv_gem.c | 16 +++----- > drivers/gpu/drm/etnaviv/etnaviv_gem.h | 4 -- > drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 7 ---- > drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 22 +++++----- > drivers/gpu/drm/msm/msm_drv.c | 1 - > drivers/gpu/drm/msm/msm_drv.h | 1 - > drivers/gpu/drm/msm/msm_gem.c | 27 ++++-------- > drivers/gpu/drm/msm/msm_gem_prime.c | 7 ---- > drivers/gpu/drm/msm/msm_gem_submit.c | 8 ++-- > drivers/gpu/drm/v3d/v3d_bo.c | 14 +------ > drivers/gpu/drm/v3d/v3d_drv.c | 1 - > drivers/gpu/drm/v3d/v3d_drv.h | 6 --- > drivers/gpu/drm/v3d/v3d_gem.c | 35 ++++------------ > drivers/gpu/drm/vc4/vc4_bo.c | 15 +------ > drivers/gpu/drm/vc4/vc4_drv.c | 1 - > drivers/gpu/drm/vc4/vc4_drv.h | 6 --- > drivers/gpu/drm/vc4/vc4_gem.c | 18 ++++---- > drivers/gpu/drm/vc4/vc4_plane.c | 2 +- > include/drm/drm_gem.h | 21 ++++++++++ > 24 files changed, 119 insertions(+), 147 deletions(-) > > -- > 2.19.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch