Increase readability of fb_notifier_callback() by removing
a few indent levels.
No functional change.

Signed-off-by: Sam Ravnborg <s...@ravnborg.org>
Cc: Lee Jones <lee.jo...@linaro.org>
Cc: Daniel Thompson <daniel.thomp...@linaro.org>
Cc: Jingoo Han <jingooh...@gmail.com>
---
 drivers/video/backlight/backlight.c | 43 +++++++++++++++--------------
 1 file changed, 22 insertions(+), 21 deletions(-)

diff --git a/drivers/video/backlight/backlight.c 
b/drivers/video/backlight/backlight.c
index 91dfcf4a2087..5e08f4f1c99a 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -58,28 +58,29 @@ static int fb_notifier_callback(struct notifier_block *self,
 
        bd = container_of(self, struct backlight_device, fb_notif);
        mutex_lock(&bd->ops_lock);
-       if (bd->ops)
-               if (!bd->ops->check_fb ||
-                   bd->ops->check_fb(bd, evdata->info)) {
-                       fb_blank = *(int *)evdata->data;
-                       if (fb_blank == FB_BLANK_UNBLANK &&
-                           !bd->fb_bl_on[node]) {
-                               bd->fb_bl_on[node] = true;
-                               if (!bd->use_count++) {
-                                       bd->props.state &= ~BL_CORE_FBBLANK;
-                                       bd->props.fb_blank = FB_BLANK_UNBLANK;
-                                       backlight_update_status(bd);
-                               }
-                       } else if (fb_blank != FB_BLANK_UNBLANK &&
-                                  bd->fb_bl_on[node]) {
-                               bd->fb_bl_on[node] = false;
-                               if (!(--bd->use_count)) {
-                                       bd->props.state |= BL_CORE_FBBLANK;
-                                       bd->props.fb_blank = fb_blank;
-                                       backlight_update_status(bd);
-                               }
-                       }
+
+       if (!bd->ops)
+               goto out;
+       if (bd->ops->check_fb && !bd->ops->check_fb(bd, evdata->info))
+               goto out;
+
+       fb_blank = *(int *)evdata->data;
+       if (fb_blank == FB_BLANK_UNBLANK && !bd->fb_bl_on[node]) {
+               bd->fb_bl_on[node] = true;
+               if (!bd->use_count++) {
+                       bd->props.state &= ~BL_CORE_FBBLANK;
+                       bd->props.fb_blank = FB_BLANK_UNBLANK;
+                       backlight_update_status(bd);
+               }
+       } else if (fb_blank != FB_BLANK_UNBLANK && bd->fb_bl_on[node]) {
+               bd->fb_bl_on[node] = false;
+               if (!(--bd->use_count)) {
+                       bd->props.state |= BL_CORE_FBBLANK;
+                       bd->props.fb_blank = fb_blank;
+                       backlight_update_status(bd);
                }
+       }
+out:
        mutex_unlock(&bd->ops_lock);
        return 0;
 }
-- 
2.25.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to