Thread (6 messages) 6 messages, 1 author, 2017-11-27

[PATCH 3/5] radeon_backlight: Delete an unnecessary return statement in radeonfb_bl_init()

From: SF Markus Elfring <hidden>
Date: 2017-11-27 21:26:23
Also in: dri-devel, kernel-janitors, lkml
Subsystem: framebuffer layer, radeon framebuffer display driver, the rest · Maintainers: Helge Deller, Benjamin Herrenschmidt, Linus Torvalds

From: Markus Elfring <redacted>
Date: Mon, 27 Nov 2017 21:37:52 +0100

The script "checkpatch.pl" pointed information out like the following.

WARNING: void function return statements are not generally useful

Thus remove such a statement in the affected function.

Signed-off-by: Markus Elfring <redacted>
---
 drivers/video/fbdev/aty/radeon_backlight.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/video/fbdev/aty/radeon_backlight.c b/drivers/video/fbdev/aty/radeon_backlight.c
index 78c36c8d5e57..c5e0e3557cc2 100644
--- a/drivers/video/fbdev/aty/radeon_backlight.c
+++ b/drivers/video/fbdev/aty/radeon_backlight.c
@@ -193,7 +193,6 @@ void radeonfb_bl_init(struct radeonfb_info *rinfo)
 
 error:
 	kfree(pdata);
-	return;
 }
 
 void radeonfb_bl_exit(struct radeonfb_info *rinfo)
-- 
2.15.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help