Re: [PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-17 Thread Harsha Sharma
On Tue, Oct 17, 2017 at 12:15 AM, Sean Paul <seanp...@chromium.org> wrote: > On Sat, Oct 14, 2017 at 2:36 PM, Harsha Sharma > <harshasharmai...@gmail.com> wrote: >> Replace instances of drm_framebuffer_reference/unreference() with >> *_get/put() suffixes and drm

[PATCH v4] drm/amd/powerplay: Remove unnecessary cast on void pointer

2017-10-15 Thread Harsha Sharma
Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v4: -Removed git diff warning "No newline at end of

[PATCH] drm/omap: Replace list_for_each with list_for_each_entry

2017-10-15 Thread Harsha Sharma
@ identifier s.i,s.f; expression s.e; statement S; @@ list_for_each_entry(i,e,f) - { S - } Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/drm/omapdrm/dss/display.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/g

[PATCH] drm/tinydrm: Replace list_for_each with list_for_each_entry

2017-10-15 Thread Harsha Sharma
@ identifier s.i,s.f; expression s.e; statement S; @@ list_for_each_entry(i,e,f) - { S - } Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/ti

[PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-15 Thread Harsha Sharma
; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v5: -rebase drm_dev_put change on drm-tip Changes in v4: -change one in

[PATCH v2] drm/amd/powerplay: Remove unnecessary cast on void pointer

2017-10-15 Thread Harsha Sharma
Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v2: -Remove unnecessary parentheses -Remove one more use

[PATCH v2] drm: Replace kzalloc with kcalloc

2017-10-15 Thread Harsha Sharma
Prefer kcalloc over kzalloc to allocate an array. This patch fixes checkcpatch issue. Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v2: -kcalloc will take 3 arguments drivers/gpu/drm/drm_crtc_helper.c | 4 ++-- drivers/gpu/drm/drm_fb_helper.c| 2 +- drive

[PATCH v3] drm/amd/powerplay: Remove unnecessary cast on void pointer

2017-10-15 Thread Harsha Sharma
Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v3: -Removed unnecessary lines -Remove more useless cast

Re: [PATCH v4] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-15 Thread Harsha Sharma
On Mon, Oct 9, 2017 at 5:36 PM, Harsha Sharma <harshasharmai...@gmail.com> wrote: > Replace instances of drm_framebuffer_reference/unreference() with > *_get/put() suffixes and drm_dev_unref with *_put() suffix > because get/put is shorter and consistent with the > kernel use of

[PATCH] drm/amd/powerplay: Remove unnecessary cast on void pointer

2017-10-15 Thread Harsha Sharma
Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c| 6 +++--- dr

[PATCH] drm: Replace kzalloc with kcalloc

2017-10-13 Thread Harsha Sharma
Prefer kcalloc over kzalloc to allocate an array. This patch fixes checkcpatch issue. Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/drm/drm_crtc_helper.c | 4 ++-- drivers/gpu/drm/drm_fb_helper.c| 2 +- drivers/gpu/drm/drm_plane_helper.c | 2 +- 3 files c

[PATCH v4] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-10 Thread Harsha Sharma
; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v4: -change one instance of *_put to *_get Changes in v3: -Removed c

[PATCH v3] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

2017-10-08 Thread Harsha Sharma
Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v3: -Solve merge conflicts Changes in v2: -Fix alignment issues drivers/gpu/drm/tinydrm/mi0283qt.

[PATCH v3] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-08 Thread Harsha Sharma
; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v3: -Removed changes in selftests Changes in v2: -Added cocinelle patch

Re: [PATCH v2] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-08 Thread Harsha Sharma
On Fri, Sep 29, 2017 at 10:00 AM, Harsha Sharma <harshasharmai...@gmail.com> wrote: > Replace instances of drm_framebuffer_reference/unreference() with > *_get/put() suffixes and drm_dev_unref with *_put() suffix > because get/put is shorter and consistent with the > kerne

[PATCH v2] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

2017-09-30 Thread Harsha Sharma
Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v2: -Fix alignment issues drivers/gpu/drm/tinydrm/mi0283qt.c | 8 drivers/gpu/drm/tinydrm/repaper.c

[PATCH] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-09-29 Thread Harsha Sharma
Replace instances of drm_framebuffer_reference/unreference() with *_get/put() suffixes and drm_dev_unref with *_put() suffix because get/put is shorter and consistent with the kernel use of *_get/put suffixes . Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/dr

[PATCH v2] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-09-29 Thread Harsha Sharma
; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- Changes in v2: -Added coccinelle patch in log message -cc to all

Kernel GPU subsytem: Better manual upload for atomic

2017-09-26 Thread Harsha Sharma
an suggest something. Thanks a lot for your time. Regards, Harsha Sharma ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [Outreachy kernel] [PATCH v2] drm/tegra: Replace dev_* with DRM_DEV_*

2017-09-26 Thread Harsha Sharma
Hi, Thanks a lot for your feedback. I will send another version for this patch. Thanks for your time. Regards, Harsha Sharma On Sun, Sep 24, 2017 at 10:30 PM, Julia Lawall <julia.law...@lip6.fr> wrote: > > > On Sun, 24 Sep 2017, Harsha Sharma wrote: > > > Replace all oc

[PATCH v2] drm/tegra: Replace dev_* with DRM_DEV_*

2017-09-26 Thread Harsha Sharma
Replace all occurences of dev_info/err/dbg with DRM_DEV_INFO/ ERROR/DEBUG as we have DRM_DEV_* variants of drm print macros Done using following coccinelle semantic patch @r@ @@ ( -dev_info +DRM_DEV_INFO | -dev_err +DRM_DEV_ERROR | -dev_dbg +DRM_DEV_DEBUG ) Signed-off-by: Harsha Sharma

[PATCH] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

2017-09-24 Thread Harsha Sharma
Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma <harshasharmai...@gmail.com> --- drivers/gpu/drm/tinydrm/mi0283qt.c | 8 drivers/gpu/drm/tinydrm/repaper.c | 26 +- drive

[PATCH] drm/tegra: Replace dev_* with DRM_DEV_*

2017-09-24 Thread Harsha Sharma
Replace all occurences of dev_info/err/dbg with DRM_DEV_INFO/ ERROR/DEBUG as we have DRM_DEV_* variants of drm print macros Done using following coccinelle semantic patch @r@ @@ ( -dev_info +DRM_DEV_INFO | -dev_err +DRM_DEV_ERROR | -dev_dbg +DRM_DEV_DEBUG ) Signed-off-by: Harsha Sharma