diff options
| author | Wambui Karuga <wambui.karugax@gmail.com> | 2020-03-10 16:31:19 +0300 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2020-03-18 16:45:22 +0100 |
| commit | e196e140f95cbe1cbd654f8529346fcdfe496cb7 (patch) | |
| tree | 786ca6c99e657bc6fa4f99bff326f300c972253f | |
| parent | f06b2052f74d7b42ff8cbfe1ac0b78be7668c37d (diff) | |
| download | linux-stable-e196e140f95cbe1cbd654f8529346fcdfe496cb7.tar.gz linux-stable-e196e140f95cbe1cbd654f8529346fcdfe496cb7.tar.bz2 linux-stable-e196e140f95cbe1cbd654f8529346fcdfe496cb7.zip | |
drm: make various debugfs_init() functions return 0
Since commit 987d65d01356 (drm: debugfs: make
drm_debugfs_create_files() never fail), drm_debugfs_create_files() never
fails and should return void. Therefore, remove its use as the
return value of various debugfs_init() functions in drm, and have these
functions return 0 directly.
v2: convert debugfs_init() functions to return 0 instead of void to
avoid build breakage.
References: https://lists.freedesktop.org/archives/dri-devel/2020-February/257183.html
Signed-off-by: Wambui Karuga <wambui.karugax@gmail.com>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20200310133121.27913-16-wambui.karugax@gmail.com
| -rw-r--r-- | drivers/gpu/drm/drm_atomic.c | 7 | ||||
| -rw-r--r-- | drivers/gpu/drm/drm_client.c | 8 | ||||
| -rw-r--r-- | drivers/gpu/drm/drm_framebuffer.c | 8 |
3 files changed, 14 insertions, 9 deletions
diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c index 9ccfbf213d72..c0056d9cc139 100644 --- a/drivers/gpu/drm/drm_atomic.c +++ b/drivers/gpu/drm/drm_atomic.c @@ -1643,8 +1643,9 @@ static const struct drm_info_list drm_atomic_debugfs_list[] = { int drm_atomic_debugfs_init(struct drm_minor *minor) { - return drm_debugfs_create_files(drm_atomic_debugfs_list, - ARRAY_SIZE(drm_atomic_debugfs_list), - minor->debugfs_root, minor); + drm_debugfs_create_files(drm_atomic_debugfs_list, + ARRAY_SIZE(drm_atomic_debugfs_list), + minor->debugfs_root, minor); + return 0; } #endif diff --git a/drivers/gpu/drm/drm_client.c b/drivers/gpu/drm/drm_client.c index 6b0c6ef8b9b3..82fbdee407b2 100644 --- a/drivers/gpu/drm/drm_client.c +++ b/drivers/gpu/drm/drm_client.c @@ -459,8 +459,10 @@ static const struct drm_info_list drm_client_debugfs_list[] = { int drm_client_debugfs_init(struct drm_minor *minor) { - return drm_debugfs_create_files(drm_client_debugfs_list, - ARRAY_SIZE(drm_client_debugfs_list), - minor->debugfs_root, minor); + drm_debugfs_create_files(drm_client_debugfs_list, + ARRAY_SIZE(drm_client_debugfs_list), + minor->debugfs_root, minor); + + return 0; } #endif diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c index 57ac94ce9b9e..46be88271fe5 100644 --- a/drivers/gpu/drm/drm_framebuffer.c +++ b/drivers/gpu/drm/drm_framebuffer.c @@ -1209,8 +1209,10 @@ static const struct drm_info_list drm_framebuffer_debugfs_list[] = { int drm_framebuffer_debugfs_init(struct drm_minor *minor) { - return drm_debugfs_create_files(drm_framebuffer_debugfs_list, - ARRAY_SIZE(drm_framebuffer_debugfs_list), - minor->debugfs_root, minor); + drm_debugfs_create_files(drm_framebuffer_debugfs_list, + ARRAY_SIZE(drm_framebuffer_debugfs_list), + minor->debugfs_root, minor); + + return 0; } #endif |