From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C5D42C433E0 for ; Mon, 1 Jun 2020 12:39:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 99AD3206C3 for ; Mon, 1 Jun 2020 12:39:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726067AbgFAMjr (ORCPT ); Mon, 1 Jun 2020 08:39:47 -0400 Received: from asavdk3.altibox.net ([109.247.116.14]:33170 "EHLO asavdk3.altibox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725838AbgFAMjq (ORCPT ); Mon, 1 Jun 2020 08:39:46 -0400 Received: from ravnborg.org (unknown [158.248.194.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by asavdk3.altibox.net (Postfix) with ESMTPS id 6B82820026; Mon, 1 Jun 2020 14:39:36 +0200 (CEST) Date: Mon, 1 Jun 2020 14:39:34 +0200 From: Sam Ravnborg To: Peter Ujfalusi Cc: dri-devel@lists.freedesktop.org, Jingoo Han , Lee Jones , Daniel Thompson , Andy Gross , Bartlomiej Zolnierkiewicz , Bjorn Andersson , Daniel Vetter , David Airlie , Emil Velikov , Jonathan Corbet , linux-arm-msm@vger.kernel.org, linux-pwm@vger.kernel.org, Maarten Lankhorst , Maxime Ripard , Michael Hennerich , patches@opensource.cirrus.com, Support Opensource , Thierry Reding , Thomas Zimmermann , Uwe Kleine-Konig , Tomi Valkeinen Subject: Re: [PATCH v3 13/13] backlight: use backlight_is_blank() in all backlight drivers Message-ID: <20200601123934.GA7560@ravnborg.org> References: <20200601065207.492614-1-sam@ravnborg.org> <20200601065207.492614-14-sam@ravnborg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-CMAE-Score: 0 X-CMAE-Analysis: v=2.3 cv=edQTgYMH c=1 sm=1 tr=0 a=UWs3HLbX/2nnQ3s7vZ42gw==:117 a=UWs3HLbX/2nnQ3s7vZ42gw==:17 a=8nJEP1OIZ-IA:10 a=sozttTNsAAAA:8 a=7gkXJVJtAAAA:8 a=KKAkSRfTAAAA:8 a=pGLkceISAAAA:8 a=gAnH3GRIAAAA:8 a=X3a0BiUdAAAA:8 a=VwQbUJbxAAAA:8 a=w1d2syhTAAAA:8 a=Hz_N3PfyXqQW7AwsI2gA:9 a=wPNLvfGTeEIA:10 a=aeg5Gbbo78KNqacMgKqU:22 a=E9Po1WZjFZOl8hwRPBS3:22 a=cvBusfyB2V15izCimMoJ:22 a=oVHKYsEdi7-vN-J5QA_j:22 a=aWu3muFgq6otJbhCXpL5:22 a=AjGcO6oz07-iQ99wixmX:22 a=YXXWInSmI4Sqt1AkVdoW:22 Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Mon, Jun 01, 2020 at 03:28:01PM +0300, Peter Ujfalusi wrote: > > > On 01/06/2020 9.52, Sam Ravnborg wrote: > > Replaces the open-coded checks of the state etc., > > with the backlight_is_blank() helper. > > This increases readability of the code and align > > the functionality across the drivers. > > Reviewed-by: Peter Ujfalusi Thanks, I missed your previsous r-b. Sorry. Sam > > > v3: > > - Dropped as3711_bl, it will be modified in another patch > > > > v2: > > - Fixed so changelog is readable > > > > Signed-off-by: Sam Ravnborg > > Reviewed-by: Daniel Thompson > > Cc: Lee Jones > > Cc: Daniel Thompson > > Cc: Jingoo Han > > Cc: Michael Hennerich > > Cc: Support Opensource > > Cc: Thierry Reding > > Cc: "Uwe Kleine-König" > > Cc: Andy Gross > > Cc: Bjorn Andersson > > Cc: linux-pwm@vger.kernel.org > > Cc: linux-arm-msm@vger.kernel.org > > Cc: patches@opensource.cirrus.com > > --- > > drivers/video/backlight/88pm860x_bl.c | 8 +------- > > drivers/video/backlight/adp5520_bl.c | 5 +---- > > drivers/video/backlight/adp8860_bl.c | 5 +---- > > drivers/video/backlight/adp8870_bl.c | 5 +---- > > drivers/video/backlight/bd6107.c | 4 +--- > > drivers/video/backlight/corgi_lcd.c | 5 +---- > > drivers/video/backlight/cr_bllcd.c | 22 +++++++--------------- > > drivers/video/backlight/da903x_bl.c | 8 +------- > > drivers/video/backlight/ep93xx_bl.c | 3 +-- > > drivers/video/backlight/gpio_backlight.c | 4 +--- > > drivers/video/backlight/hp680_bl.c | 4 +--- > > drivers/video/backlight/jornada720_bl.c | 2 +- > > drivers/video/backlight/kb3886_bl.c | 4 +--- > > drivers/video/backlight/led_bl.c | 4 +--- > > drivers/video/backlight/lm3533_bl.c | 4 +--- > > drivers/video/backlight/locomolcd.c | 4 +--- > > drivers/video/backlight/lv5207lp.c | 4 +--- > > drivers/video/backlight/max8925_bl.c | 8 +------- > > drivers/video/backlight/pwm_bl.c | 4 +--- > > drivers/video/backlight/qcom-wled.c | 4 +--- > > drivers/video/backlight/tps65217_bl.c | 4 +--- > > drivers/video/backlight/wm831x_bl.c | 8 +------- > > 22 files changed, 28 insertions(+), 95 deletions(-) > > > > diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c > > index 20d96a5ac384..162c83ab0f5a 100644 > > --- a/drivers/video/backlight/88pm860x_bl.c > > +++ b/drivers/video/backlight/88pm860x_bl.c > > @@ -123,13 +123,7 @@ static int pm860x_backlight_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.state & BL_CORE_SUSPENDED) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return pm860x_backlight_set(bl, brightness); > > diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c > > index 0f63f76723a5..d817b0d95c9d 100644 > > --- a/drivers/video/backlight/adp5520_bl.c > > +++ b/drivers/video/backlight/adp5520_bl.c > > @@ -67,10 +67,7 @@ static int adp5520_bl_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return adp5520_bl_set(bl, brightness); > > diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c > > index 19968104fc47..a0ce2a3701fa 100644 > > --- a/drivers/video/backlight/adp8860_bl.c > > +++ b/drivers/video/backlight/adp8860_bl.c > > @@ -363,10 +363,7 @@ static int adp8860_bl_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return adp8860_bl_set(bl, brightness); > > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c > > index 4c0032010cfe..ae4269fdb189 100644 > > --- a/drivers/video/backlight/adp8870_bl.c > > +++ b/drivers/video/backlight/adp8870_bl.c > > @@ -401,10 +401,7 @@ static int adp8870_bl_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return adp8870_bl_set(bl, brightness); > > diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c > > index d5d5fb457e78..f6a5c1dba3bc 100644 > > --- a/drivers/video/backlight/bd6107.c > > +++ b/drivers/video/backlight/bd6107.c > > @@ -84,9 +84,7 @@ static int bd6107_backlight_update_status(struct backlight_device *backlight) > > struct bd6107 *bd = bl_get_data(backlight); > > int brightness = backlight->props.brightness; > > > > - if (backlight->props.power != FB_BLANK_UNBLANK || > > - backlight->props.fb_blank != FB_BLANK_UNBLANK || > > - backlight->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) > > + if (backlight_is_blank(backlight)) > > brightness = 0; > > > > if (brightness) { > > diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c > > index 25ef0cbd7583..c9adf4e26355 100644 > > --- a/drivers/video/backlight/corgi_lcd.c > > +++ b/drivers/video/backlight/corgi_lcd.c > > @@ -422,10 +422,7 @@ static int corgi_bl_update_status(struct backlight_device *bd) > > struct corgi_lcd *lcd = bl_get_data(bd); > > int intensity = bd->props.brightness; > > > > - if (bd->props.power != FB_BLANK_UNBLANK) > > - intensity = 0; > > - > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bd)) > > intensity = 0; > > > > if (corgibl_flags & CORGIBL_SUSPENDED) > > diff --git a/drivers/video/backlight/cr_bllcd.c b/drivers/video/backlight/cr_bllcd.c > > index 4624b7b7c6a6..d5ab7675f55c 100644 > > --- a/drivers/video/backlight/cr_bllcd.c > > +++ b/drivers/video/backlight/cr_bllcd.c > > @@ -59,26 +59,18 @@ struct cr_panel { > > > > static int cr_backlight_set_intensity(struct backlight_device *bd) > > { > > - int intensity = bd->props.brightness; > > u32 addr = gpio_bar + CRVML_PANEL_PORT; > > u32 cur = inl(addr); > > > > - if (bd->props.power == FB_BLANK_UNBLANK) > > - intensity = FB_BLANK_UNBLANK; > > - if (bd->props.fb_blank == FB_BLANK_UNBLANK) > > - intensity = FB_BLANK_UNBLANK; > > - if (bd->props.power == FB_BLANK_POWERDOWN) > > - intensity = FB_BLANK_POWERDOWN; > > - if (bd->props.fb_blank == FB_BLANK_POWERDOWN) > > - intensity = FB_BLANK_POWERDOWN; > > - > > - if (intensity == FB_BLANK_UNBLANK) { /* FULL ON */ > > - cur &= ~CRVML_BACKLIGHT_OFF; > > - outl(cur, addr); > > - } else if (intensity == FB_BLANK_POWERDOWN) { /* OFF */ > > + if (backlight_is_blank(bd)) { > > + /* OFF */ > > cur |= CRVML_BACKLIGHT_OFF; > > outl(cur, addr); > > - } /* anything else, don't bother */ > > + } else { > > + /* FULL ON */ > > + cur &= ~CRVML_BACKLIGHT_OFF; > > + outl(cur, addr); > > + } > > > > return 0; > > } > > diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c > > index 62540e4bdedb..ca351badfdcf 100644 > > --- a/drivers/video/backlight/da903x_bl.c > > +++ b/drivers/video/backlight/da903x_bl.c > > @@ -79,13 +79,7 @@ static int da903x_backlight_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.state & BL_CORE_SUSPENDED) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return da903x_backlight_set(bl, brightness); > > diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c > > index 4149e0b2f83c..491185df1411 100644 > > --- a/drivers/video/backlight/ep93xx_bl.c > > +++ b/drivers/video/backlight/ep93xx_bl.c > > @@ -38,8 +38,7 @@ static int ep93xxbl_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK || > > - bl->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return ep93xxbl_set(bl, brightness); > > diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c > > index 75409ddfba3e..94b65e4d2aa0 100644 > > --- a/drivers/video/backlight/gpio_backlight.c > > +++ b/drivers/video/backlight/gpio_backlight.c > > @@ -25,9 +25,7 @@ static int gpio_backlight_get_next_brightness(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK || > > - bl->props.fb_blank != FB_BLANK_UNBLANK || > > - bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return brightness; > > diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c > > index 8ea42b8d9bc8..01d805ca8415 100644 > > --- a/drivers/video/backlight/hp680_bl.c > > +++ b/drivers/video/backlight/hp680_bl.c > > @@ -35,9 +35,7 @@ static void hp680bl_send_intensity(struct backlight_device *bd) > > u16 v; > > int intensity = bd->props.brightness; > > > > - if (bd->props.power != FB_BLANK_UNBLANK) > > - intensity = 0; > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bd)) > > intensity = 0; > > if (hp680bl_suspended) > > intensity = 0; > > diff --git a/drivers/video/backlight/jornada720_bl.c b/drivers/video/backlight/jornada720_bl.c > > index f0385f9cf9da..996f7ba3b373 100644 > > --- a/drivers/video/backlight/jornada720_bl.c > > +++ b/drivers/video/backlight/jornada720_bl.c > > @@ -54,7 +54,7 @@ static int jornada_bl_update_status(struct backlight_device *bd) > > jornada_ssp_start(); > > > > /* If backlight is off then really turn it off */ > > - if ((bd->props.power != FB_BLANK_UNBLANK) || (bd->props.fb_blank != FB_BLANK_UNBLANK)) { > > + if (backlight_is_blank(bd)) { > > ret = jornada_ssp_byte(BRIGHTNESSOFF); > > if (ret != TXDUMMY) { > > dev_info(&bd->dev, "brightness off timeout\n"); > > diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c > > index 1dfe13c18925..a0fd5d3d82f5 100644 > > --- a/drivers/video/backlight/kb3886_bl.c > > +++ b/drivers/video/backlight/kb3886_bl.c > > @@ -89,9 +89,7 @@ static int kb3886bl_send_intensity(struct backlight_device *bd) > > { > > int intensity = bd->props.brightness; > > > > - if (bd->props.power != FB_BLANK_UNBLANK) > > - intensity = 0; > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bd)) > > intensity = 0; > > if (kb3886bl_flags & KB3886BL_SUSPENDED) > > intensity = 0; > > diff --git a/drivers/video/backlight/led_bl.c b/drivers/video/backlight/led_bl.c > > index 3f66549997c8..c655ddd99cfb 100644 > > --- a/drivers/video/backlight/led_bl.c > > +++ b/drivers/video/backlight/led_bl.c > > @@ -56,9 +56,7 @@ static int led_bl_update_status(struct backlight_device *bl) > > struct led_bl_data *priv = bl_get_data(bl); > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK || > > - bl->props.fb_blank != FB_BLANK_UNBLANK || > > - bl->props.state & BL_CORE_FBBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > if (brightness > 0) > > diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c > > index ee09d1bd02b9..476146b62c4e 100644 > > --- a/drivers/video/backlight/lm3533_bl.c > > +++ b/drivers/video/backlight/lm3533_bl.c > > @@ -41,9 +41,7 @@ static int lm3533_bl_update_status(struct backlight_device *bd) > > struct lm3533_bl *bl = bl_get_data(bd); > > int brightness = bd->props.brightness; > > > > - if (bd->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bd)) > > brightness = 0; > > > > return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness); > > diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c > > index cdc02e04f89d..8064cad8d683 100644 > > --- a/drivers/video/backlight/locomolcd.c > > +++ b/drivers/video/backlight/locomolcd.c > > @@ -113,9 +113,7 @@ static int locomolcd_set_intensity(struct backlight_device *bd) > > { > > int intensity = bd->props.brightness; > > > > - if (bd->props.power != FB_BLANK_UNBLANK) > > - intensity = 0; > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK) > > + if (backlight_is_blank(bd)) > > intensity = 0; > > if (locomolcd_flags & LOCOMOLCD_SUSPENDED) > > intensity = 0; > > diff --git a/drivers/video/backlight/lv5207lp.c b/drivers/video/backlight/lv5207lp.c > > index c6ad73a784e2..ef8aa9803577 100644 > > --- a/drivers/video/backlight/lv5207lp.c > > +++ b/drivers/video/backlight/lv5207lp.c > > @@ -48,9 +48,7 @@ static int lv5207lp_backlight_update_status(struct backlight_device *backlight) > > struct lv5207lp *lv = bl_get_data(backlight); > > int brightness = backlight->props.brightness; > > > > - if (backlight->props.power != FB_BLANK_UNBLANK || > > - backlight->props.fb_blank != FB_BLANK_UNBLANK || > > - backlight->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) > > + if (backlight_is_blank(backlight)) > > brightness = 0; > > > > if (brightness) { > > diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c > > index 97cc260ff9d1..b8af2c6407d3 100644 > > --- a/drivers/video/backlight/max8925_bl.c > > +++ b/drivers/video/backlight/max8925_bl.c > > @@ -66,13 +66,7 @@ static int max8925_backlight_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.state & BL_CORE_SUSPENDED) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return max8925_backlight_set(bl, brightness); > > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c > > index 82b8d7594701..7d0ffcd37f07 100644 > > --- a/drivers/video/backlight/pwm_bl.c > > +++ b/drivers/video/backlight/pwm_bl.c > > @@ -111,9 +111,7 @@ static int pwm_backlight_update_status(struct backlight_device *bl) > > int brightness = bl->props.brightness; > > struct pwm_state state; > > > > - if (bl->props.power != FB_BLANK_UNBLANK || > > - bl->props.fb_blank != FB_BLANK_UNBLANK || > > - bl->props.state & BL_CORE_FBBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > if (pb->notify) > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > > index 3d276b30a78c..9600f5d58ece 100644 > > --- a/drivers/video/backlight/qcom-wled.c > > +++ b/drivers/video/backlight/qcom-wled.c > > @@ -261,9 +261,7 @@ static int wled_update_status(struct backlight_device *bl) > > u16 brightness = bl->props.brightness; > > int rc = 0; > > > > - if (bl->props.power != FB_BLANK_UNBLANK || > > - bl->props.fb_blank != FB_BLANK_UNBLANK || > > - bl->props.state & BL_CORE_FBBLANK) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > mutex_lock(&wled->lock); > > diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c > > index 762e3feed097..1041e5e62ee3 100644 > > --- a/drivers/video/backlight/tps65217_bl.c > > +++ b/drivers/video/backlight/tps65217_bl.c > > @@ -82,9 +82,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl) > > if (bl->props.state & BL_CORE_SUSPENDED) > > brightness = 0; > > > > - if ((bl->props.power != FB_BLANK_UNBLANK) || > > - (bl->props.fb_blank != FB_BLANK_UNBLANK)) > > - /* framebuffer in low power mode or blanking active */ > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > if (brightness > 0) { > > diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c > > index e55977d54c15..dc2ab6c8b7f9 100644 > > --- a/drivers/video/backlight/wm831x_bl.c > > +++ b/drivers/video/backlight/wm831x_bl.c > > @@ -93,13 +93,7 @@ static int wm831x_backlight_update_status(struct backlight_device *bl) > > { > > int brightness = bl->props.brightness; > > > > - if (bl->props.power != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.fb_blank != FB_BLANK_UNBLANK) > > - brightness = 0; > > - > > - if (bl->props.state & BL_CORE_SUSPENDED) > > + if (backlight_is_blank(bl)) > > brightness = 0; > > > > return wm831x_backlight_set(bl, brightness); > > > > - Péter > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki