Message ID | 20230703214715.623447-3-paul@crapouillou.net |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9f45:0:b0:3ea:f831:8777 with SMTP id v5csp811512vqx; Mon, 3 Jul 2023 14:58:18 -0700 (PDT) X-Google-Smtp-Source: APBJJlHC6Qf2c9D4jck7A8CJZD0wpqSVGdtR7HC0Z7c61vOxG+6hjCjWb5ghWICYndWlUymUgdd6 X-Received: by 2002:a17:90a:be16:b0:262:c2a1:c029 with SMTP id a22-20020a17090abe1600b00262c2a1c029mr12791764pjs.2.1688421497901; Mon, 03 Jul 2023 14:58:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688421497; cv=none; d=google.com; s=arc-20160816; b=aFKRiatHblw0E3y40kwLESxg6Y6/bAFK5n3L6GgR7MbUZBJ7cbwaFGUXDhXVvBrleT snToM9t0rhkRajaPVBtlXFy6F0Z2BIBMhWeFFxo9l8yquiB8/KNDJZbjVmu+fA6h59MI AL+al5zt/uI7gGnZR/gf1GLz5u1PXZ/e4Y4EBReNKHxsRQSeyzCwLIRkPtck98VTBDKY B49KIpE7zDqnxZ8YyyMsGWnppwsj9RVlR5PblWOqLcwqglPowQhKD1yU2L5gh3wbW6cc bTWs6ZHOlW32WyBU5BcIkrbAYIg5yXOmy0k1X3UsmlKuYBjpwBuNziJMEc8F/XmEKlVU OekQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=oSVu9zN3Z7FGQFi9DqYFaRkee0OgOav2Bk1wOeBtGYc=; fh=ZGUIriixHxc1iGKObIsgEzb0Ckv+dumFOC4kNAscsoI=; b=tBFzwL2NnGi9z0jWBW6ykkTdWF39h8t1LfZ9ReP/d70tA4G36xfCEk1jadv3lskQw4 Ty1cXPVF6fSSLwcWbx/bhlHAva5YPyjrwmTUChY8GN1W5X8a98ThQJqF6jpG4HSKheaZ 0hjkeyqcBp9DRlBJtt2B5tSMHjb8eTjHssmpO8KtsaBw+SnPoWEaCl6kE8pJUh8jEvyY OdB8IQBUH94/FpK8KjcC6K+1u17s39NnhEjeWJn5MLQpI1btDvWVEDMh6F1ee17iHXvB YzSiEJSAbeqlHAfkClauMtXH6oZVRLtNZg4JfuajCcNHZ+K3d4PfkXFentXZY9KeLDLS Gn8w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@crapouillou.net header.s=mail header.b=RKg1Gt9z; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=crapouillou.net Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id mq3-20020a17090b380300b002611518b4aesi7748280pjb.101.2023.07.03.14.58.05; Mon, 03 Jul 2023 14:58:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@crapouillou.net header.s=mail header.b=RKg1Gt9z; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=crapouillou.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229535AbjGCVsC (ORCPT <rfc822;ivan.orlov0322@gmail.com> + 99 others); Mon, 3 Jul 2023 17:48:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57420 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231316AbjGCVrz (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 3 Jul 2023 17:47:55 -0400 Received: from aposti.net (aposti.net [89.234.176.197]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CA61910EC; Mon, 3 Jul 2023 14:47:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=crapouillou.net; s=mail; t=1688420847; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=oSVu9zN3Z7FGQFi9DqYFaRkee0OgOav2Bk1wOeBtGYc=; b=RKg1Gt9zR7825mopAtoJWrfdnCagok+w3KhRM/9TQ6vlc7X2WIX60WCSQvaDK/OzN6JpH2 3koPY79DI0JglsrJ281eXLYen7CfnzVT+2av3neFJ3WTzRz/XHx+r9jizQaCboZr1DoAHc i0AGWAOqJJxR7X25X9t2hoo2hSGHpMs= From: Paul Cercueil <paul@crapouillou.net> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Rob Herring <robh+dt@kernel.org>, Conor Dooley <conor+dt@kernel.org>, Alim Akhtar <alim.akhtar@samsung.com>, Neil Armstrong <neil.armstrong@linaro.org>, Sam Ravnborg <sam@ravnborg.org> Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Paul Cercueil <paul@crapouillou.net> Subject: [PATCH 2/3] drm/panel: ld9040: Register a backlight device Date: Mon, 3 Jul 2023 23:47:14 +0200 Message-Id: <20230703214715.623447-3-paul@crapouillou.net> In-Reply-To: <20230703214715.623447-1-paul@crapouillou.net> References: <20230703214715.623447-1-paul@crapouillou.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam: Yes X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1770438260571977341?= X-GMAIL-MSGID: =?utf-8?q?1770438260571977341?= |
Series |
Galaxy S2 (i9100) panel updates
|
|
Commit Message
Paul Cercueil
July 3, 2023, 9:47 p.m. UTC
Register a backlight device to be able to switch between all the gamma
levels.
Signed-off-by: Paul Cercueil <paul@crapouillou.net>
---
drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 ++++++++++++++++++++
1 file changed, 40 insertions(+)
Comments
Hi, > -----Original Message----- > From: dri-devel <dri-devel-bounces@lists.freedesktop.org> On Behalf Of > Paul Cercueil > Sent: Tuesday, July 4, 2023 6:47 AM > To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Rob Herring > <robh+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>; Alim Akhtar > <alim.akhtar@samsung.com>; Neil Armstrong <neil.armstrong@linaro.org>; Sam > Ravnborg <sam@ravnborg.org> > Cc: devicetree@vger.kernel.org; linux-samsung-soc@vger.kernel.org; linux- > kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; Paul Cercueil > <paul@crapouillou.net>; linux-arm-kernel@lists.infradead.org > Subject: [PATCH 2/3] drm/panel: ld9040: Register a backlight device > > Register a backlight device to be able to switch between all the gamma > levels. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 ++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > index 7fd9444b42c5..b4f87d6244cb 100644 > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > @@ -8,6 +8,7 @@ > * Andrzej Hajda <a.hajda@samsung.com> > */ > > +#include <linux/backlight.h> > #include <linux/delay.h> > #include <linux/gpio/consumer.h> > #include <linux/module.h> > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > return 0; > } > > +static int ld9040_bl_update_status(struct backlight_device *dev) > +{ > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); > + > + ctx->brightness = dev->props.brightness; > + ld9040_brightness_set(ctx); > + > + return 0; > +} > + > +static int ld9040_bl_get_intensity(struct backlight_device *dev) > +{ > + if (dev->props.fb_blank == FB_BLANK_UNBLANK && fb_blank member is deprecated according to the description of backlight.h file so you could drop above condition I think. Thanks, Inki Dae
Hi Inki, Le mardi 04 juillet 2023 à 08:49 +0900, 대인기/Tizen Platform Lab(SR)/삼성전자 a écrit : > Hi, > > > -----Original Message----- > > From: dri-devel <dri-devel-bounces@lists.freedesktop.org> On Behalf > > Of > > Paul Cercueil > > Sent: Tuesday, July 4, 2023 6:47 AM > > To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Rob > > Herring > > <robh+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>; Alim > > Akhtar > > <alim.akhtar@samsung.com>; Neil Armstrong > > <neil.armstrong@linaro.org>; Sam > > Ravnborg <sam@ravnborg.org> > > Cc: devicetree@vger.kernel.org; linux-samsung-soc@vger.kernel.org; > > linux- > > kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; Paul > > Cercueil > > <paul@crapouillou.net>; linux-arm-kernel@lists.infradead.org > > Subject: [PATCH 2/3] drm/panel: ld9040: Register a backlight device > > > > Register a backlight device to be able to switch between all the > > gamma > > levels. > > > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > > --- > > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 > > ++++++++++++++++++++ > > 1 file changed, 40 insertions(+) > > > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > index 7fd9444b42c5..b4f87d6244cb 100644 > > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > @@ -8,6 +8,7 @@ > > * Andrzej Hajda <a.hajda@samsung.com> > > */ > > > > +#include <linux/backlight.h> > > #include <linux/delay.h> > > #include <linux/gpio/consumer.h> > > #include <linux/module.h> > > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > > return 0; > > } > > > > +static int ld9040_bl_update_status(struct backlight_device *dev) > > +{ > > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); > > + > > + ctx->brightness = dev->props.brightness; > > + ld9040_brightness_set(ctx); > > + > > + return 0; > > +} > > + > > +static int ld9040_bl_get_intensity(struct backlight_device *dev) > > +{ > > + if (dev->props.fb_blank == FB_BLANK_UNBLANK && > > fb_blank member is deprecated according to the description of > backlight.h > file so you could drop above condition I think. Thanks. I'll send a V2. Cheers, -Paul
On 03/07/2023 23:47, Paul Cercueil wrote: > Register a backlight device to be able to switch between all the gamma > levels. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 ++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > index 7fd9444b42c5..b4f87d6244cb 100644 > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > @@ -8,6 +8,7 @@ > * Andrzej Hajda <a.hajda@samsung.com> > */ > > +#include <linux/backlight.h> > #include <linux/delay.h> > #include <linux/gpio/consumer.h> > #include <linux/module.h> > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > return 0; > } > > +static int ld9040_bl_update_status(struct backlight_device *dev) > +{ > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); > + > + ctx->brightness = dev->props.brightness; > + ld9040_brightness_set(ctx); > + > + return 0; > +} > + > +static int ld9040_bl_get_intensity(struct backlight_device *dev) > +{ > + if (dev->props.fb_blank == FB_BLANK_UNBLANK && > + dev->props.power == FB_BLANK_UNBLANK) > + return dev->props.brightness; > + > + return 0; > +} You can totally drop the _get_brightness. Neil > + > +static const struct backlight_ops ld9040_bl_ops = { > + .get_brightness = ld9040_bl_get_intensity, > + .update_status = ld9040_bl_update_status, > +}; > + > +static const struct backlight_properties ld9040_bl_props = { > + .type = BACKLIGHT_RAW, > + .scale = BACKLIGHT_SCALE_NON_LINEAR, > + .max_brightness = ARRAY_SIZE(ld9040_gammas) - 1, > + .brightness = ARRAY_SIZE(ld9040_gammas) - 1, > +}; > + > static int ld9040_probe(struct spi_device *spi) > { > + struct backlight_device *bldev; > struct device *dev = &spi->dev; > struct ld9040 *ctx; > int ret; > @@ -354,6 +387,13 @@ static int ld9040_probe(struct spi_device *spi) > drm_panel_init(&ctx->panel, dev, &ld9040_drm_funcs, > DRM_MODE_CONNECTOR_DPI); > > + > + bldev = devm_backlight_device_register(dev, dev_name(dev), dev, > + ctx, &ld9040_bl_ops, > + &ld9040_bl_props); > + if (IS_ERR(bldev)) > + return PTR_ERR(bldev); > + > drm_panel_add(&ctx->panel); > > return 0;
Hi Neil, Le mercredi 05 juillet 2023 à 15:57 +0200, Neil Armstrong a écrit : > On 03/07/2023 23:47, Paul Cercueil wrote: > > Register a backlight device to be able to switch between all the > > gamma > > levels. > > > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > > --- > > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 > > ++++++++++++++++++++ > > 1 file changed, 40 insertions(+) > > > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > index 7fd9444b42c5..b4f87d6244cb 100644 > > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > @@ -8,6 +8,7 @@ > > * Andrzej Hajda <a.hajda@samsung.com> > > */ > > > > +#include <linux/backlight.h> > > #include <linux/delay.h> > > #include <linux/gpio/consumer.h> > > #include <linux/module.h> > > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > > return 0; > > } > > > > +static int ld9040_bl_update_status(struct backlight_device *dev) > > +{ > > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); > > + > > + ctx->brightness = dev->props.brightness; > > + ld9040_brightness_set(ctx); > > + > > + return 0; > > +} > > + > > +static int ld9040_bl_get_intensity(struct backlight_device *dev) > > +{ > > + if (dev->props.fb_blank == FB_BLANK_UNBLANK && > > + dev->props.power == FB_BLANK_UNBLANK) > > + return dev->props.brightness; > > + > > + return 0; > > +} > > You can totally drop the _get_brightness. The current behaviour is to return 0 when the framebuffer is blanked. A few drivers do that so I thought it was the norm; and the backlight core doesn't do that by default (and just uses dev->props.brightness). It is not clear to me if that's the preferred behaviour. The "backlight_get_brightness" function in backlight.h seems to suggest that the current behaviour is correct, unless it is not supposed to be used in the backlight_ops.get_brightness() callback. Then in that case some other drivers get it wrong too. Cheers, -Paul > Neil > > > + > > +static const struct backlight_ops ld9040_bl_ops = { > > + .get_brightness = ld9040_bl_get_intensity, > > + .update_status = ld9040_bl_update_status, > > +}; > > + > > +static const struct backlight_properties ld9040_bl_props = { > > + .type = BACKLIGHT_RAW, > > + .scale = BACKLIGHT_SCALE_NON_LINEAR, > > + .max_brightness = ARRAY_SIZE(ld9040_gammas) - 1, > > + .brightness = ARRAY_SIZE(ld9040_gammas) - 1, > > +}; > > + > > static int ld9040_probe(struct spi_device *spi) > > { > > + struct backlight_device *bldev; > > struct device *dev = &spi->dev; > > struct ld9040 *ctx; > > int ret; > > @@ -354,6 +387,13 @@ static int ld9040_probe(struct spi_device > > *spi) > > drm_panel_init(&ctx->panel, dev, &ld9040_drm_funcs, > > DRM_MODE_CONNECTOR_DPI); > > > > + > > + bldev = devm_backlight_device_register(dev, dev_name(dev), > > dev, > > + ctx, &ld9040_bl_ops, > > + &ld9040_bl_props); > > + if (IS_ERR(bldev)) > > + return PTR_ERR(bldev); > > + > > drm_panel_add(&ctx->panel); > > > > return 0; >
Hi Paul, On Wed, Jul 05, 2023 at 04:38:05PM +0200, Paul Cercueil wrote: > Hi Neil, > > Le mercredi 05 juillet 2023 à 15:57 +0200, Neil Armstrong a écrit : > > On 03/07/2023 23:47, Paul Cercueil wrote: > > > Register a backlight device to be able to switch between all the > > > gamma > > > levels. > > > > > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > > > --- > > > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 > > > ++++++++++++++++++++ > > > 1 file changed, 40 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > > b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > > index 7fd9444b42c5..b4f87d6244cb 100644 > > > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > > > @@ -8,6 +8,7 @@ > > > * Andrzej Hajda <a.hajda@samsung.com> > > > */ > > > > > > +#include <linux/backlight.h> > > > #include <linux/delay.h> > > > #include <linux/gpio/consumer.h> > > > #include <linux/module.h> > > > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > > > return 0; > > > } > > > > > > +static int ld9040_bl_update_status(struct backlight_device *dev) > > > +{ > > > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); > > > + > > > + ctx->brightness = dev->props.brightness; Use backlight_get_brightness(dev); > > > + ld9040_brightness_set(ctx); > > > + > > > + return 0; > > > +} > > > + > > > +static int ld9040_bl_get_intensity(struct backlight_device *dev) > > > +{ > > > + if (dev->props.fb_blank == FB_BLANK_UNBLANK && > > > + dev->props.power == FB_BLANK_UNBLANK) > > > + return dev->props.brightness; > > > + > > > + return 0; > > > +} > > > > You can totally drop the _get_brightness. > > The current behaviour is to return 0 when the framebuffer is blanked. A > few drivers do that so I thought it was the norm; and the backlight > core doesn't do that by default (and just uses dev->props.brightness). > > It is not clear to me if that's the preferred behaviour. The > "backlight_get_brightness" function in backlight.h seems to suggest > that the current behaviour is correct, unless it is not supposed to be > used in the backlight_ops.get_brightness() callback. Then in that case > some other drivers get it wrong too. Several drivers get it wrong. You are supposed to provide get_brightness only when you read back a value from the HW, which is not the case here so just drop it is the right choice. Sam
Hi Paul, On Mon, Jul 03, 2023 at 11:47:14PM +0200, Paul Cercueil wrote: > Register a backlight device to be able to switch between all the gamma > levels. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > drivers/gpu/drm/panel/panel-samsung-ld9040.c | 40 ++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > index 7fd9444b42c5..b4f87d6244cb 100644 > --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c > +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c > @@ -8,6 +8,7 @@ > * Andrzej Hajda <a.hajda@samsung.com> > */ > > +#include <linux/backlight.h> > #include <linux/delay.h> > #include <linux/gpio/consumer.h> > #include <linux/module.h> > @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) > return 0; > } > > +static int ld9040_bl_update_status(struct backlight_device *dev) > +{ > + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); There is also the helper bl_get_data() - that do the same. Sam
diff --git a/drivers/gpu/drm/panel/panel-samsung-ld9040.c b/drivers/gpu/drm/panel/panel-samsung-ld9040.c index 7fd9444b42c5..b4f87d6244cb 100644 --- a/drivers/gpu/drm/panel/panel-samsung-ld9040.c +++ b/drivers/gpu/drm/panel/panel-samsung-ld9040.c @@ -8,6 +8,7 @@ * Andrzej Hajda <a.hajda@samsung.com> */ +#include <linux/backlight.h> #include <linux/delay.h> #include <linux/gpio/consumer.h> #include <linux/module.h> @@ -311,8 +312,40 @@ static int ld9040_parse_dt(struct ld9040 *ctx) return 0; } +static int ld9040_bl_update_status(struct backlight_device *dev) +{ + struct ld9040 *ctx = dev_get_drvdata(&dev->dev); + + ctx->brightness = dev->props.brightness; + ld9040_brightness_set(ctx); + + return 0; +} + +static int ld9040_bl_get_intensity(struct backlight_device *dev) +{ + if (dev->props.fb_blank == FB_BLANK_UNBLANK && + dev->props.power == FB_BLANK_UNBLANK) + return dev->props.brightness; + + return 0; +} + +static const struct backlight_ops ld9040_bl_ops = { + .get_brightness = ld9040_bl_get_intensity, + .update_status = ld9040_bl_update_status, +}; + +static const struct backlight_properties ld9040_bl_props = { + .type = BACKLIGHT_RAW, + .scale = BACKLIGHT_SCALE_NON_LINEAR, + .max_brightness = ARRAY_SIZE(ld9040_gammas) - 1, + .brightness = ARRAY_SIZE(ld9040_gammas) - 1, +}; + static int ld9040_probe(struct spi_device *spi) { + struct backlight_device *bldev; struct device *dev = &spi->dev; struct ld9040 *ctx; int ret; @@ -354,6 +387,13 @@ static int ld9040_probe(struct spi_device *spi) drm_panel_init(&ctx->panel, dev, &ld9040_drm_funcs, DRM_MODE_CONNECTOR_DPI); + + bldev = devm_backlight_device_register(dev, dev_name(dev), dev, + ctx, &ld9040_bl_ops, + &ld9040_bl_props); + if (IS_ERR(bldev)) + return PTR_ERR(bldev); + drm_panel_add(&ctx->panel); return 0;