On Thu, May 17, 2012 at 7:27 AM, Laurent Pinchart <laurent.pinchart at ideasonboard.com> wrote: > Hello, > > While studying the DRM core I came across a couple of issues or strange > constructs. Here are 6 patches that try to fix them. > > Criticism is as usual welcome. Please bear with my lack of experience with the > DRM core :-) > > Laurent Pinchart (6): > ?drm: Delete the vblank timer synchronously at cleanup time > ?drm: Miscellaneous typo fixes and documentation updates > ?drm: Don't initialize local ret variable when not needed > ?drm: Constify gem_vm_ops pointer > ?drm: Constify drm_mode_config_funcs pointer > ?drm: Make the CRTC gamma_set operation optional
For the series: Reviewed-by: Alex Deucher <alexander.deucher at amd.com> > > ?drivers/gpu/drm/drm_crtc.c ? ? ? ? ? ? ? ?| ? 19 ++++++++++++------- > ?drivers/gpu/drm/drm_crtc_helper.c ? ? ? ? | ? ?2 +- > ?drivers/gpu/drm/drm_edid_load.c ? ? ? ? ? | ? ?8 ++++---- > ?drivers/gpu/drm/drm_fb_helper.c ? ? ? ? ? | ? ?8 ++++---- > ?drivers/gpu/drm/drm_gem.c ? ? ? ? ? ? ? ? | ? ?2 +- > ?drivers/gpu/drm/drm_irq.c ? ? ? ? ? ? ? ? | ? 23 +++++++++-------------- > ?drivers/gpu/drm/drm_lock.c ? ? ? ? ? ? ? ?| ? ?2 +- > ?drivers/gpu/drm/drm_stub.c ? ? ? ? ? ? ? ?| ? ?2 +- > ?drivers/gpu/drm/drm_sysfs.c ? ? ? ? ? ? ? | ? 10 +++++----- > ?drivers/gpu/drm/exynos/exynos_drm_drv.c ? | ? ?2 +- > ?drivers/gpu/drm/exynos/exynos_drm_fb.c ? ?| ? ?2 +- > ?drivers/gpu/drm/gma500/framebuffer.c ? ? ?| ? ?4 ++-- > ?drivers/gpu/drm/gma500/psb_drv.c ? ? ? ? ?| ? ?2 +- > ?drivers/gpu/drm/i915/i915_drv.c ? ? ? ? ? | ? ?2 +- > ?drivers/gpu/drm/i915/intel_display.c ? ? ?| ? ?2 +- > ?drivers/gpu/drm/nouveau/nouveau_display.c | ? ?2 +- > ?drivers/gpu/drm/radeon/radeon_display.c ? | ? ?2 +- > ?drivers/gpu/drm/udl/udl_drv.c ? ? ? ? ? ? | ? ?2 +- > ?drivers/gpu/drm/udl/udl_modeset.c ? ? ? ? | ? ?2 +- > ?drivers/gpu/drm/vmwgfx/vmwgfx_kms.c ? ? ? | ? ?2 +- > ?drivers/staging/omapdrm/omap_crtc.c ? ? ? | ? ?7 ------- > ?drivers/staging/omapdrm/omap_drv.c ? ? ? ?| ? ?4 ++-- > ?include/drm/drmP.h ? ? ? ? ? ? ? ? ? ? ? ?| ? 12 ++++++------ > ?include/drm/drm_crtc.h ? ? ? ? ? ? ? ? ? ?| ? 23 ++++++++--------------- > ?include/drm/drm_crtc_helper.h ? ? ? ? ? ? | ? 21 +++++++++++++++++++++ > ?25 files changed, 87 insertions(+), 80 deletions(-) > > -- > Regards, > > Laurent Pinchart > > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel