Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1557478 > unrolled thread

[PATCH -next] drm/atomic: make release_crtc_commit() static

Back to article view | Back to linux.kernel

Contents

  [PATCH -next] drm/atomic: make release_crtc_commit() static Wei Yongjun <weiyj.lk@gmail.com> - 2017-01-12 15:30 +0100
    Re: [PATCH -next] drm/atomic: make release_crtc_commit() static Gustavo Padovan <gustavo@padovan.org> - 2017-01-16 22:40 +0100
      Re: [PATCH -next] drm/atomic: make release_crtc_commit() static Daniel Vetter <daniel@ffwll.ch> - 2017-01-23 09:10 +0100

#1557478 — [PATCH -next] drm/atomic: make release_crtc_commit() static

From: Wei Yongjun <weiyongjun1@huawei.com>

Fixes the following sparse warning:

drivers/gpu/drm/drm_atomic_helper.c:1360:6: warning:
 symbol 'release_crtc_commit' was not declared. Should it be static?

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
 drivers/gpu/drm/drm_atomic_helper.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
index 7b71ac4..e2e5a74 100644
--- a/drivers/gpu/drm/drm_atomic_helper.c
+++ b/drivers/gpu/drm/drm_atomic_helper.c
@@ -1357,7 +1357,7 @@ static int stall_checks(struct drm_crtc *crtc, bool nonblock)
 	return ret < 0 ? ret : 0;
 }
 
-void release_crtc_commit(struct completion *completion)
+static void release_crtc_commit(struct completion *completion)
 {
 	struct drm_crtc_commit *commit = container_of(completion,
 						      typeof(*commit),

[toc] | [next] | [standalone]

#1560099

Hi Wei,

2017-01-12 Wei Yongjun <weiyj.lk@gmail.com>:

> From: Wei Yongjun <weiyongjun1@huawei.com>
> 
> Fixes the following sparse warning:
> 
> drivers/gpu/drm/drm_atomic_helper.c:1360:6: warning:
>  symbol 'release_crtc_commit' was not declared. Should it be static?
> 
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---
>  drivers/gpu/drm/drm_atomic_helper.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>

Gustavo

[toc] | [prev] | [next] | [standalone]

#1564751

On Mon, Jan 16, 2017 at 07:36:27PM -0200, Gustavo Padovan wrote:
> Hi Wei,
> 
> 2017-01-12 Wei Yongjun <weiyj.lk@gmail.com>:
> 
> > From: Wei Yongjun <weiyongjun1@huawei.com>
> > 
> > Fixes the following sparse warning:
> > 
> > drivers/gpu/drm/drm_atomic_helper.c:1360:6: warning:
> >  symbol 'release_crtc_commit' was not declared. Should it be static?
> > 
> > Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> > ---
> >  drivers/gpu/drm/drm_atomic_helper.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>

Thanks for the patch, applied to drm-misc.

Gustavo, for next time around please just use your commit rights when
reviewing a simple patch like this :-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web