Messages in this thread Patch in this message |  | | Subject | [PATCH 5/5] radeon_backlight: Delete a jump target in radeonfb_bl_init() | From | SF Markus Elfring <> | Date | Mon, 27 Nov 2017 22:28:31 +0100 |
| |
From: Markus Elfring <elfring@users.sourceforge.net> Date: Mon, 27 Nov 2017 22:03:03 +0100
* Replace a goto statement by a direct call of the function "kfree" in an if branch.
* Delete the jump label "error" which became unnecessary with this refactoring.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> --- drivers/video/fbdev/aty/radeon_backlight.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/video/fbdev/aty/radeon_backlight.c b/drivers/video/fbdev/aty/radeon_backlight.c index 7f7ed6f72ddb..3c5b8291e773 100644 --- a/drivers/video/fbdev/aty/radeon_backlight.c +++ b/drivers/video/fbdev/aty/radeon_backlight.c @@ -157,7 +157,8 @@ void radeonfb_bl_init(struct radeonfb_info *rinfo) if (IS_ERR(bd)) { rinfo->info->bl_dev = NULL; printk("radeonfb: Backlight registration failed\n"); - goto error; + kfree(pdata); + return; } pdata->rinfo = rinfo; @@ -188,11 +189,6 @@ void radeonfb_bl_init(struct radeonfb_info *rinfo) backlight_update_status(bd); printk("radeonfb: Backlight initialized (%s)\n", name); - - return; - -error: - kfree(pdata); } void radeonfb_bl_exit(struct radeonfb_info *rinfo) -- 2.15.0
|  |