* [PATCH v4 01/20] backlight: refactor fb_notifier_callback()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 02/20] backlight: add backlight_is_blank() Sam Ravnborg
` (18 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Increase readability of fb_notifier_callback() by removing
a few indent levels.
No functional change.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@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 92d80aa0c0ef..7e249aa90b0e 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
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 02/20] backlight: add backlight_is_blank()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 01/20] backlight: refactor fb_notifier_callback() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-07 5:42 ` Daniel Vetter
2020-07-03 18:45 ` [PATCH v4 03/20] backlight: improve backlight_ops documentation Sam Ravnborg
` (17 subsequent siblings)
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
The backlight support has two properties that express the state:
- power
- state
It is un-documented and easy to get wrong.
Add backlight_is_blank() helper to make it simpler
for drivers to get the check of the state correct.
A lot of drivers also includes checks for fb_blank.
This check is redundant when the state is checked
and thus not needed in this helper function.
But added anyway to avoid introducing subtle bugs
due to the creative use of fb_blank in some drivers.
Introducing this helper will for some drivers results in
added support for fb_blank. This will be a change in
functionality, which will improve the backlight driver.
Rolling out this helper to all relevant backlight drivers
will eliminate almost all accesses to fb_blank.
v4:
- struct backlight_device * is now const
v3:
- Clarified that the fb_blank support in
backlight_is_blank() may result in functionality
changes for the users (Emil)
v2:
- Added fb_blank condition (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 56e4580d4f55..56e51ebab740 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -175,6 +175,25 @@ static inline void backlight_put(struct backlight_device *bd)
put_device(&bd->dev);
}
+/**
+ * backlight_is_blank - Return true if display is expected to be blank
+ * @bd: the backlight device
+ *
+ * Display is expected to be blank if any of these is true::
+ *
+ * 1) if power in not UNBLANK
+ * 2) if fb_blank is not UNBLANK
+ * 3) if state indicate BLANK or SUSPENDED
+ *
+ * Returns true if display is expected to be blank, false otherwise.
+ */
+static inline bool backlight_is_blank(const struct backlight_device *bd)
+{
+ return bd->props.power != FB_BLANK_UNBLANK ||
+ bd->props.fb_blank != FB_BLANK_UNBLANK ||
+ bd->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK);
+}
+
extern struct backlight_device *backlight_device_register(const char *name,
struct device *dev, void *devdata, const struct backlight_ops *ops,
const struct backlight_properties *props);
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 02/20] backlight: add backlight_is_blank()
2020-07-03 18:45 ` [PATCH v4 02/20] backlight: add backlight_is_blank() Sam Ravnborg
@ 2020-07-07 5:42 ` Daniel Vetter
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Vetter @ 2020-07-07 5:42 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas
On Fri, Jul 03, 2020 at 08:45:28PM +0200, Sam Ravnborg wrote:
> The backlight support has two properties that express the state:
> - power
> - state
Tiny nit, but please add "fb_blank" here too, commit message doesn't match
the code anymore. With that:
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> It is un-documented and easy to get wrong.
> Add backlight_is_blank() helper to make it simpler
> for drivers to get the check of the state correct.
>
> A lot of drivers also includes checks for fb_blank.
> This check is redundant when the state is checked
> and thus not needed in this helper function.
> But added anyway to avoid introducing subtle bugs
> due to the creative use of fb_blank in some drivers.
> Introducing this helper will for some drivers results in
> added support for fb_blank. This will be a change in
> functionality, which will improve the backlight driver.
>
> Rolling out this helper to all relevant backlight drivers
> will eliminate almost all accesses to fb_blank.
>
> v4:
> - struct backlight_device * is now const
>
> v3:
> - Clarified that the fb_blank support in
> backlight_is_blank() may result in functionality
> changes for the users (Emil)
>
> v2:
> - Added fb_blank condition (Daniel)
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
> Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> ---
> include/linux/backlight.h | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/include/linux/backlight.h b/include/linux/backlight.h
> index 56e4580d4f55..56e51ebab740 100644
> --- a/include/linux/backlight.h
> +++ b/include/linux/backlight.h
> @@ -175,6 +175,25 @@ static inline void backlight_put(struct backlight_device *bd)
> put_device(&bd->dev);
> }
>
> +/**
> + * backlight_is_blank - Return true if display is expected to be blank
> + * @bd: the backlight device
> + *
> + * Display is expected to be blank if any of these is true::
> + *
> + * 1) if power in not UNBLANK
> + * 2) if fb_blank is not UNBLANK
> + * 3) if state indicate BLANK or SUSPENDED
> + *
> + * Returns true if display is expected to be blank, false otherwise.
> + */
> +static inline bool backlight_is_blank(const struct backlight_device *bd)
> +{
> + return bd->props.power != FB_BLANK_UNBLANK ||
> + bd->props.fb_blank != FB_BLANK_UNBLANK ||
> + bd->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK);
> +}
> +
> extern struct backlight_device *backlight_device_register(const char *name,
> struct device *dev, void *devdata, const struct backlight_ops *ops,
> const struct backlight_properties *props);
> --
> 2.25.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 03/20] backlight: improve backlight_ops documentation
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 01/20] backlight: refactor fb_notifier_callback() Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 02/20] backlight: add backlight_is_blank() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 04/20] backlight: improve backlight_properties documentation Sam Ravnborg
` (16 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Improve the documentation for backlight_ops and
adapt it to kernel-doc style.
v2:
- Add intro for each field (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 59 +++++++++++++++++++++++++++++++++++----
1 file changed, 53 insertions(+), 6 deletions(-)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 56e51ebab740..dfb43ee02ea0 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -55,19 +55,66 @@ enum backlight_scale {
struct backlight_device;
struct fb_info;
+/**
+ * struct backlight_ops - backlight operations
+ *
+ * The backlight operations are specifed when the backlight device is registered.
+ */
struct backlight_ops {
+ /**
+ * @options: Configure how operations are called from the core.
+ *
+ * The options parameter is used to adjust the behaviour of the core.
+ * Set BL_CORE_SUSPENDRESUME to get the update_status() operation called
+ * upon suspend and resume.
+ */
unsigned int options;
#define BL_CORE_SUSPENDRESUME (1 << 0)
- /* Notify the backlight driver some property has changed */
+ /**
+ * @update_status: Operation called when properties have changed.
+ *
+ * Notify the backlight driver some property has changed.
+ * The update_status operation is protected by the update_lock.
+ *
+ * The backlight driver is expected to use backlight_is_blank()
+ * to check if the display is blanked and set brightness accordingly.
+ * update_status() is called when any of the properties has changed.
+ *
+ * RETURNS:
+ *
+ * 0 on sucees, negative error code if any failure occured.
+ */
int (*update_status)(struct backlight_device *);
- /* Return the current backlight brightness (accounting for power,
- fb_blank etc.) */
+
+ /**
+ * @get_brightness: Return the current backlight brightness.
+ *
+ * The driver may implement this as a readback from the HW.
+ * This operation is optional and if not present then the current
+ * brightness property value is used.
+ *
+ * RETURNS:
+ *
+ * A brightness value which is 0 or a positive numer.
+ * On failure a negative error code is returned.
+ */
int (*get_brightness)(struct backlight_device *);
- /* Check if given framebuffer device is the one bound to this backlight;
- return 0 if not, !=0 if it is. If NULL, backlight always matches the fb. */
- int (*check_fb)(struct backlight_device *, struct fb_info *);
+
+ /**
+ * @check_fb: Check the framebuffer device.
+ *
+ * Check if given framebuffer device is the one bound to this backlight.
+ * This operation is optional and if not implemented it is assumed that the
+ * fbdev is always the one bound to the backlight.
+ *
+ * RETURNS:
+ *
+ * If info is NULL or the info matches the fbdev bound to the backlight return true.
+ * If info does not match the fbdev bound to the backlight return false.
+ */
+ int (*check_fb)(struct backlight_device *bd, struct fb_info *info);
};
/* This structure defines all the properties of a backlight */
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 04/20] backlight: improve backlight_properties documentation
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (2 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 03/20] backlight: improve backlight_ops documentation Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 05/20] backlight: improve backlight_device documentation Sam Ravnborg
` (15 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Improve the documentation for backlight_properties and
adapt it to kernel-doc style.
v3:
- Added missing '@' in kernel-doc
v2:
- Added into for each field (Daniel)
- Re-written some parts to explain what to do, rather
than what not to do.
Partly based on suggestions from the review (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 96 ++++++++++++++++++++++++++++++++++-----
1 file changed, 85 insertions(+), 11 deletions(-)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index dfb43ee02ea0..10518b00b059 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -117,28 +117,102 @@ struct backlight_ops {
int (*check_fb)(struct backlight_device *bd, struct fb_info *info);
};
-/* This structure defines all the properties of a backlight */
+/**
+ * struct backlight_properties - backlight properties
+ *
+ * This structure defines all the properties of a backlight.
+ */
struct backlight_properties {
- /* Current User requested brightness (0 - max_brightness) */
+ /**
+ * @brightness: The current brightness requested by the user.
+ *
+ * The backlight core makes sure the range is (0 to max_brightness)
+ * when the brightness is set via the sysfs attribute:
+ * /sys/class/backlight/<backlight>/brightness.
+ *
+ * This value can be set in the backlight_properties passed
+ * to devm_backlight_device_register() to set a default brightness
+ * value.
+ */
int brightness;
- /* Maximal value for brightness (read-only) */
+
+ /**
+ * @max_brightness: The maximum brightness value.
+ *
+ * This value must be set in the backlight_properties passed to
+ * devm_backlight_device_register() and shall not be modified by the
+ * driver after registration.
+ */
int max_brightness;
- /* Current FB Power mode (0: full on, 1..3: power saving
- modes; 4: full off), see FB_BLANK_XXX */
+
+ /**
+ * @power: The current power mode.
+ *
+ * User space can configure the power mode using the sysfs
+ * attribute: /sys/class/backlight/<backlight>/bl_power
+ * When the power property is updated update_status() is called.
+ *
+ * The possible values are: (0: full on, 1 to 3: power saving
+ * modes; 4: full off), see FB_BLANK_XXX.
+ *
+ * When the backlight device is enabled @power is set
+ * to FB_BLANK_UNBLANK. When the backlight device is disabled
+ * @power is set to FB_BLANK_POWERDOWN.
+ */
int power;
- /* FB Blanking active? (values as for power) */
- /* Due to be removed, please use (state & BL_CORE_FBBLANK) */
+
+ /**
+ * @fb_blank: The power state from the FBIOBLANK ioclt.
+ *
+ * When the FBIOBLANK ioctl is called @fb_blank is set to the
+ * blank parameter and the update_status() operation is called.
+ *
+ * When the backlight device is enabled @fb_blank is set
+ * to FB_BLANK_UNBLANK. When the backlight device is disabled
+ * @fb_blank is set to FB_BLANK_POWERDOWN.
+ *
+ * Backlight drivers should avoid using this property. It has been
+ * replaced by state & BL_CORE_FBLANK (although most drivers should
+ * use backlight_is_blank() as the preferred means to get the blank
+ * state).
+ *
+ * fb_blank is deprecated and will be removed.
+ */
int fb_blank;
- /* Backlight type */
+
+ /**
+ * @type: The type of backlight supported.
+ *
+ * The backlight type allows userspace to make appropriate
+ * policy desicions based on the backlight type.
+ *
+ * This value must be set in the backlight_properties
+ * passed to devm_backlight_device_register().
+ */
enum backlight_type type;
- /* Flags used to signal drivers of state changes */
+
+ /**
+ * @state: The state of the backlight core.
+ *
+ * The state is a bitmask. BL_CORE_FBBLANK is set when the display
+ * is expected to be blank. BL_CORE_SUSPENDED is set when the
+ * driver is suspended.
+ *
+ * backlight drivers are excpected to use backlight_is_blank()
+ * in their update_status() operation rather than reading the
+ * state property.
+ *
+ * The state is maintained by the core and drivers may not modify it.
+ */
unsigned int state;
- /* Type of the brightness scale (linear, non-linear, ...) */
- enum backlight_scale scale;
#define BL_CORE_SUSPENDED (1 << 0) /* backlight is suspended */
#define BL_CORE_FBBLANK (1 << 1) /* backlight is under an fb blank event */
+ /**
+ * @scale: The type of the brightness scale.
+ */
+ enum backlight_scale scale;
};
struct backlight_device {
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (3 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 04/20] backlight: improve backlight_properties documentation Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 10:23 ` Daniel Thompson
` (2 more replies)
2020-07-03 18:45 ` [PATCH v4 06/20] backlight: document inline functions in backlight.h Sam Ravnborg
` (14 subsequent siblings)
19 siblings, 3 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Improve the documentation for backlight_device and
adapt it to kernel-doc style.
The updated documentation is more strict on how locking is used.
With the update neither update_lock nor ops_lock may be used
outside the backlight core.
This restriction was introduced to keep the locking simple
by keeping it in the core.
It was verified that this documents the current state by renaming
update_lock => bl_update_lock and ops_lock => bl_ops_lock.
The rename did not reveal any uses outside the backlight core.
The rename is NOT part of this patch.
v3:
- Update changelog to explain locking details (Daniel)
v2:
- Add short intro to all fields (Daniel)
- Updated description of update_lock (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 72 ++++++++++++++++++++++++++-------------
1 file changed, 49 insertions(+), 23 deletions(-)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 10518b00b059..7654fe5f1589 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -14,21 +14,6 @@
#include <linux/mutex.h>
#include <linux/notifier.h>
-/* Notes on locking:
- *
- * backlight_device->ops_lock is an internal backlight lock protecting the
- * ops pointer and no code outside the core should need to touch it.
- *
- * Access to update_status() is serialised by the update_lock mutex since
- * most drivers seem to need this and historically get it wrong.
- *
- * Most drivers don't need locking on their get_brightness() method.
- * If yours does, you need to implement it in the driver. You can use the
- * update_lock mutex if appropriate.
- *
- * Any other use of the locks below is probably wrong.
- */
-
enum backlight_update_reason {
BACKLIGHT_UPDATE_HOTKEY,
BACKLIGHT_UPDATE_SYSFS,
@@ -215,30 +200,71 @@ struct backlight_properties {
enum backlight_scale scale;
};
+/**
+ * struct backlight_device - backlight device data
+ *
+ * This structure holds all data required by a backlight device.
+ */
struct backlight_device {
- /* Backlight properties */
+ /**
+ * @props: Backlight properties
+ */
struct backlight_properties props;
- /* Serialise access to update_status method */
+ /**
+ * @update_lock: The lock used when calling the update_status() operation.
+ *
+ * update_lock is an internal backlight lock that serialise access
+ * to the update_status() operation. The backlight core holds the update_lock
+ * when calling the update_status() operation. The update_lock shall not
+ * be used by backlight drivers.
+ */
struct mutex update_lock;
- /* This protects the 'ops' field. If 'ops' is NULL, the driver that
- registered this device has been unloaded, and if class_get_devdata()
- points to something in the body of that driver, it is also invalid. */
+ /**
+ * @ops_lock: The lock used around everything related to backlight_ops.
+ *
+ * ops_lock is an internal backlight lock that protects the ops pointer
+ * and is used around all accesses to ops and when the operations are
+ * invoked. The ops_lock shall not be used by backlight drivers.
+ */
struct mutex ops_lock;
+
+ /**
+ * @ops: Pointer to the backlight operations.
+ *
+ * If ops is NULL, the driver that registered this device has been unloaded,
+ * and if class_get_devdata() points to something in the body of that driver,
+ * it is also invalid.
+ */
const struct backlight_ops *ops;
- /* The framebuffer notifier block */
+ /**
+ * @fb_notif: The framebuffer notifier block
+ */
struct notifier_block fb_notif;
- /* list entry of all registered backlight devices */
+ /**
+ * @entry: List entry of all registered backlight devices
+ */
struct list_head entry;
+ /**
+ * @dev: Parent device.
+ */
struct device dev;
- /* Multiple framebuffers may share one backlight device */
+ /**
+ * @fb_bl_on: The state of individual fbdev's.
+ *
+ * Multiple fbdev's may share one backlight device. The fb_bl_on
+ * records the state of the individual fbdev.
+ */
bool fb_bl_on[FB_MAX];
+ /**
+ * @use_count: The number of uses of fb_bl_on.
+ */
int use_count;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-03 18:45 ` [PATCH v4 05/20] backlight: improve backlight_device documentation Sam Ravnborg
@ 2020-07-08 10:23 ` Daniel Thompson
2020-07-16 14:39 ` Lee Jones
2020-07-18 5:18 ` Jingoo Han
2 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 10:23 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:31PM +0200, Sam Ravnborg wrote:
> Improve the documentation for backlight_device and
> adapt it to kernel-doc style.
>
> The updated documentation is more strict on how locking is used.
> With the update neither update_lock nor ops_lock may be used
> outside the backlight core.
> This restriction was introduced to keep the locking simple
> by keeping it in the core.
> It was verified that this documents the current state by renaming
> update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> The rename did not reveal any uses outside the backlight core.
> The rename is NOT part of this patch.
>
> v3:
> - Update changelog to explain locking details (Daniel)
>
> v2:
> - Add short intro to all fields (Daniel)
> - Updated description of update_lock (Daniel)
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-03 18:45 ` [PATCH v4 05/20] backlight: improve backlight_device documentation Sam Ravnborg
2020-07-08 10:23 ` Daniel Thompson
@ 2020-07-16 14:39 ` Lee Jones
2020-07-16 15:00 ` Sam Ravnborg
2020-07-18 5:18 ` Jingoo Han
2 siblings, 1 reply; 39+ messages in thread
From: Lee Jones @ 2020-07-16 14:39 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann
On Fri, 03 Jul 2020, Sam Ravnborg wrote:
> Improve the documentation for backlight_device and
> adapt it to kernel-doc style.
>
> The updated documentation is more strict on how locking is used.
> With the update neither update_lock nor ops_lock may be used
> outside the backlight core.
> This restriction was introduced to keep the locking simple
> by keeping it in the core.
> It was verified that this documents the current state by renaming
> update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> The rename did not reveal any uses outside the backlight core.
> The rename is NOT part of this patch.
>
> v3:
> - Update changelog to explain locking details (Daniel)
>
> v2:
> - Add short intro to all fields (Daniel)
> - Updated description of update_lock (Daniel)
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> ---
> include/linux/backlight.h | 72 ++++++++++++++++++++++++++-------------
> 1 file changed, 49 insertions(+), 23 deletions(-)
Some of these do not apply cleanly.
Please collect the *-bys already received, rebase and resubmit.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-16 14:39 ` Lee Jones
@ 2020-07-16 15:00 ` Sam Ravnborg
2020-07-16 15:09 ` Lee Jones
0 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-16 15:00 UTC (permalink / raw)
To: Lee Jones
Cc: dri-devel, Jingoo Han, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann
Hi Lee.
On Thu, Jul 16, 2020 at 03:39:41PM +0100, Lee Jones wrote:
> On Fri, 03 Jul 2020, Sam Ravnborg wrote:
>
> > Improve the documentation for backlight_device and
> > adapt it to kernel-doc style.
> >
> > The updated documentation is more strict on how locking is used.
> > With the update neither update_lock nor ops_lock may be used
> > outside the backlight core.
> > This restriction was introduced to keep the locking simple
> > by keeping it in the core.
> > It was verified that this documents the current state by renaming
> > update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> > The rename did not reveal any uses outside the backlight core.
> > The rename is NOT part of this patch.
> >
> > v3:
> > - Update changelog to explain locking details (Daniel)
> >
> > v2:
> > - Add short intro to all fields (Daniel)
> > - Updated description of update_lock (Daniel)
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
> > ---
> > include/linux/backlight.h | 72 ++++++++++++++++++++++++++-------------
> > 1 file changed, 49 insertions(+), 23 deletions(-)
>
> Some of these do not apply cleanly.
>
> Please collect the *-bys already received, rebase and resubmit.
Will do.
The patch-set is based on drm-misc-next. Are there another
tree that I should use?
Sam
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-16 15:00 ` Sam Ravnborg
@ 2020-07-16 15:09 ` Lee Jones
0 siblings, 0 replies; 39+ messages in thread
From: Lee Jones @ 2020-07-16 15:09 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann
On Thu, 16 Jul 2020, Sam Ravnborg wrote:
> Hi Lee.
>
> On Thu, Jul 16, 2020 at 03:39:41PM +0100, Lee Jones wrote:
> > On Fri, 03 Jul 2020, Sam Ravnborg wrote:
> >
> > > Improve the documentation for backlight_device and
> > > adapt it to kernel-doc style.
> > >
> > > The updated documentation is more strict on how locking is used.
> > > With the update neither update_lock nor ops_lock may be used
> > > outside the backlight core.
> > > This restriction was introduced to keep the locking simple
> > > by keeping it in the core.
> > > It was verified that this documents the current state by renaming
> > > update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> > > The rename did not reveal any uses outside the backlight core.
> > > The rename is NOT part of this patch.
> > >
> > > v3:
> > > - Update changelog to explain locking details (Daniel)
> > >
> > > v2:
> > > - Add short intro to all fields (Daniel)
> > > - Updated description of update_lock (Daniel)
> > >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > ---
> > > include/linux/backlight.h | 72 ++++++++++++++++++++++++++-------------
> > > 1 file changed, 49 insertions(+), 23 deletions(-)
> >
> > Some of these do not apply cleanly.
> >
> > Please collect the *-bys already received, rebase and resubmit.
>
> Will do.
> The patch-set is based on drm-misc-next. Are there another
> tree that I should use?
I don't have anything to do with that tree.
Either Backlight [0] or Next would be fine.
[0] https://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git/
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-03 18:45 ` [PATCH v4 05/20] backlight: improve backlight_device documentation Sam Ravnborg
2020-07-08 10:23 ` Daniel Thompson
2020-07-16 14:39 ` Lee Jones
@ 2020-07-18 5:18 ` Jingoo Han
2020-07-18 6:00 ` Sam Ravnborg
2 siblings, 1 reply; 39+ messages in thread
From: Jingoo Han @ 2020-07-18 5:18 UTC (permalink / raw)
To: Sam Ravnborg, dri-devel, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig
On 7/3/20, 2:46 PM, Sam Ravnborg wrote:
>
> Improve the documentation for backlight_device and
> adapt it to kernel-doc style.
>
> The updated documentation is more strict on how locking is used.
> With the update neither update_lock nor ops_lock may be used
> outside the backlight core.
> This restriction was introduced to keep the locking simple
> by keeping it in the core.
> It was verified that this documents the current state by renaming
> update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> The rename did not reveal any uses outside the backlight core.
> The rename is NOT part of this patch.
>
> v3:
> - Update changelog to explain locking details (Daniel)
>
> v2:
> - Add short intro to all fields (Daniel)
> - Updated description of update_lock (Daniel)
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
It looks good!
Reviewed-by: Jingoo Han <jingoohan1@gmail.com>
For the rebase, if you don't know which branch of maintainer's git can be used,
linux-next tree [1] is useful. The linux-next git collects all next branches from
other maintainers' git every day.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/
Thank you.
Best regards,
Jingoo Han
> ---
> include/linux/backlight.h | 72 ++++++++++++++++++++++++++-------------
> 1 file changed, 49 insertions(+), 23 deletions(-)
.....
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 05/20] backlight: improve backlight_device documentation
2020-07-18 5:18 ` Jingoo Han
@ 2020-07-18 6:00 ` Sam Ravnborg
0 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-18 6:00 UTC (permalink / raw)
To: Jingoo Han
Cc: dri-devel, Lee Jones, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support
Hi Jingoo
On Sat, Jul 18, 2020 at 05:18:39AM +0000, Jingoo Han wrote:
> On 7/3/20, 2:46 PM, Sam Ravnborg wrote:
> >
> > Improve the documentation for backlight_device and
> > adapt it to kernel-doc style.
> >
> > The updated documentation is more strict on how locking is used.
> > With the update neither update_lock nor ops_lock may be used
> > outside the backlight core.
> > This restriction was introduced to keep the locking simple
> > by keeping it in the core.
> > It was verified that this documents the current state by renaming
> > update_lock => bl_update_lock and ops_lock => bl_ops_lock.
> > The rename did not reveal any uses outside the backlight core.
> > The rename is NOT part of this patch.
> >
> > v3:
> > - Update changelog to explain locking details (Daniel)
> >
> > v2:
> > - Add short intro to all fields (Daniel)
> > - Updated description of update_lock (Daniel)
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
>
> It looks good!
> Reviewed-by: Jingoo Han <jingoohan1@gmail.com>
Thanks!
>
> For the rebase, if you don't know which branch of maintainer's git can be used,
> linux-next tree [1] is useful. The linux-next git collects all next branches from
> other maintainers' git every day.
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/
I had used drm-misc-next because the original focus was to clean up
drivers in gpu/drm/ - and then I just continued to use this wrong tree.
linux-next is indeed a good place to catch the latest and greatest - but
as I now have the URL for the backlight tree (thanks to Lee) I will use it here.
Will try to find time this weekend so we can land these.
Sam
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 06/20] backlight: document inline functions in backlight.h
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (4 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 05/20] backlight: improve backlight_device documentation Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 07/20] backlight: document enums " Sam Ravnborg
` (13 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Add documentation for the inline functions in backlight.h
v2:
- Fix spelling (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 7654fe5f1589..7d6cb61e10f5 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -268,6 +268,10 @@ struct backlight_device {
int use_count;
};
+/**
+ * backlight_update_status - force an update of the backlight device status
+ * @bd: the backlight device
+ */
static inline int backlight_update_status(struct backlight_device *bd)
{
int ret = -ENOENT;
@@ -361,6 +365,18 @@ extern int backlight_device_set_brightness(struct backlight_device *bd, unsigned
#define to_backlight_device(obj) container_of(obj, struct backlight_device, dev)
+/**
+ * bl_get_data - access devdata
+ * @bl_dev: pointer to backlight device
+ *
+ * When a backlight device is registered the driver has the possibility
+ * to supply a void * devdata. bl_get_data() return a pointer to the
+ * devdata.
+ *
+ * RETURNS:
+ *
+ * pointer to devdata stored while registering the backlight device.
+ */
static inline void * bl_get_data(struct backlight_device *bl_dev)
{
return dev_get_drvdata(&bl_dev->dev);
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 07/20] backlight: document enums in backlight.h
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (5 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 06/20] backlight: document inline functions in backlight.h Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 08/20] backlight: remove the unused backlight_bl driver Sam Ravnborg
` (12 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Add kernel-doc documentation for the backlight enums
v2:
- Add intro to each enum member (Daniel)
Except backlight type as line lenght was too long.
The generated HTML is the same.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 72 +++++++++++++++++++++++++++++++++++++++
1 file changed, 72 insertions(+)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 7d6cb61e10f5..0f425b32e6be 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -14,26 +14,98 @@
#include <linux/mutex.h>
#include <linux/notifier.h>
+/**
+ * enum backlight_update_reason - what method was used to update backlight
+ *
+ * A driver indicates the method (reason) used for updating the backlight
+ * when calling backlight_force_update().
+ */
enum backlight_update_reason {
+ /**
+ * @BACKLIGHT_UPDATE_HOTKEY: The backlight was updated using a hot-key.
+ */
BACKLIGHT_UPDATE_HOTKEY,
+
+ /**
+ * @BACKLIGHT_UPDATE_SYSFS: The backlight was updated using sysfs.
+ */
BACKLIGHT_UPDATE_SYSFS,
};
+/**
+ * enum backlight_type - the type of backlight control
+ *
+ * The type of interface used to control the backlight.
+ */
enum backlight_type {
+ /**
+ * @BACKLIGHT_RAW:
+ *
+ * The backlight is controlled using hardware registers.
+ */
BACKLIGHT_RAW = 1,
+
+ /**
+ * @BACKLIGHT_PLATFORM:
+ *
+ * The backlight is controlled using a platform-specific interface.
+ */
BACKLIGHT_PLATFORM,
+
+ /**
+ * @BACKLIGHT_FIRMWARE:
+ *
+ * The backlight is controlled using a standard firmware interface.
+ */
BACKLIGHT_FIRMWARE,
+
+ /**
+ * @BACKLIGHT_TYPE_MAX: Number of entries.
+ */
BACKLIGHT_TYPE_MAX,
};
+/**
+ * enum backlight_notification - the type of notification
+ *
+ * The notifications that is used for notification sent to the receiver
+ * that registered notifications using backlight_register_notifier().
+ */
enum backlight_notification {
+ /**
+ * @BACKLIGHT_REGISTERED: The backlight device is registered.
+ */
BACKLIGHT_REGISTERED,
+
+ /**
+ * @BACKLIGHT_UNREGISTERED: The backlight revice is unregistered.
+ */
BACKLIGHT_UNREGISTERED,
};
+/** enum backlight_scale - the type of scale used for brightness values
+ *
+ * The type of scale used for brightness values.
+ */
enum backlight_scale {
+ /**
+ * @BACKLIGHT_SCALE_UNKNOWN: The scale is unknown.
+ */
BACKLIGHT_SCALE_UNKNOWN = 0,
+
+ /**
+ * @BACKLIGHT_SCALE_LINEAR: The scale is linear.
+ *
+ * The linear scale will increase brightness the same for each step.
+ */
BACKLIGHT_SCALE_LINEAR,
+
+ /**
+ * @BACKLIGHT_SCALE_NON_LINEAR: The scale is not linear.
+ *
+ * This is often used when the brightness values tries to adjust to
+ * the relative perception of the eye demanding a non-linear scale.
+ */
BACKLIGHT_SCALE_NON_LINEAR,
};
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 08/20] backlight: remove the unused backlight_bl driver
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (6 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 07/20] backlight: document enums " Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 09/20] backlight: drop extern from prototypes Sam Ravnborg
` (11 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
The backlight_bl driver required initialization using
struct generic_bl_info. As there are no more references
to this struct there is no users left.
So it is safe to delete the driver.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/Kconfig | 8 --
drivers/video/backlight/Makefile | 1 -
drivers/video/backlight/generic_bl.c | 110 ---------------------------
include/linux/backlight.h | 9 ---
4 files changed, 128 deletions(-)
delete mode 100644 drivers/video/backlight/generic_bl.c
diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig
index 7d22d7377606..14abfeee8868 100644
--- a/drivers/video/backlight/Kconfig
+++ b/drivers/video/backlight/Kconfig
@@ -173,14 +173,6 @@ config BACKLIGHT_EP93XX
To compile this driver as a module, choose M here: the module will
be called ep93xx_bl.
-config BACKLIGHT_GENERIC
- tristate "Generic (aka Sharp Corgi) Backlight Driver"
- default y
- help
- Say y to enable the generic platform backlight driver previously
- known as the Corgi backlight driver. If you have a Sharp Zaurus
- SL-C7xx, SL-Cxx00 or SL-6000x say y.
-
config BACKLIGHT_IPAQ_MICRO
tristate "iPAQ microcontroller backlight driver"
depends on MFD_IPAQ_MICRO
diff --git a/drivers/video/backlight/Makefile b/drivers/video/backlight/Makefile
index 0c1a1524627a..9b998cfdc56d 100644
--- a/drivers/video/backlight/Makefile
+++ b/drivers/video/backlight/Makefile
@@ -31,7 +31,6 @@ obj-$(CONFIG_BACKLIGHT_CLASS_DEVICE) += backlight.o
obj-$(CONFIG_BACKLIGHT_DA903X) += da903x_bl.o
obj-$(CONFIG_BACKLIGHT_DA9052) += da9052_bl.o
obj-$(CONFIG_BACKLIGHT_EP93XX) += ep93xx_bl.o
-obj-$(CONFIG_BACKLIGHT_GENERIC) += generic_bl.o
obj-$(CONFIG_BACKLIGHT_GPIO) += gpio_backlight.o
obj-$(CONFIG_BACKLIGHT_HP680) += hp680_bl.o
obj-$(CONFIG_BACKLIGHT_HP700) += jornada720_bl.o
diff --git a/drivers/video/backlight/generic_bl.c b/drivers/video/backlight/generic_bl.c
deleted file mode 100644
index 8fe63dbc8590..000000000000
--- a/drivers/video/backlight/generic_bl.c
+++ /dev/null
@@ -1,110 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/*
- * Generic Backlight Driver
- *
- * Copyright (c) 2004-2008 Richard Purdie
- */
-
-#include <linux/module.h>
-#include <linux/kernel.h>
-#include <linux/init.h>
-#include <linux/platform_device.h>
-#include <linux/mutex.h>
-#include <linux/fb.h>
-#include <linux/backlight.h>
-
-static int genericbl_intensity;
-static struct backlight_device *generic_backlight_device;
-static struct generic_bl_info *bl_machinfo;
-
-static int genericbl_send_intensity(struct backlight_device *bd)
-{
- int intensity = bd->props.brightness;
-
- if (bd->props.power != FB_BLANK_UNBLANK)
- intensity = 0;
- if (bd->props.state & BL_CORE_FBBLANK)
- intensity = 0;
- if (bd->props.state & BL_CORE_SUSPENDED)
- intensity = 0;
-
- bl_machinfo->set_bl_intensity(intensity);
-
- genericbl_intensity = intensity;
-
- if (bl_machinfo->kick_battery)
- bl_machinfo->kick_battery();
-
- return 0;
-}
-
-static int genericbl_get_intensity(struct backlight_device *bd)
-{
- return genericbl_intensity;
-}
-
-static const struct backlight_ops genericbl_ops = {
- .options = BL_CORE_SUSPENDRESUME,
- .get_brightness = genericbl_get_intensity,
- .update_status = genericbl_send_intensity,
-};
-
-static int genericbl_probe(struct platform_device *pdev)
-{
- struct backlight_properties props;
- struct generic_bl_info *machinfo = dev_get_platdata(&pdev->dev);
- const char *name = "generic-bl";
- struct backlight_device *bd;
-
- bl_machinfo = machinfo;
- if (!machinfo->limit_mask)
- machinfo->limit_mask = -1;
-
- if (machinfo->name)
- name = machinfo->name;
-
- memset(&props, 0, sizeof(struct backlight_properties));
- props.type = BACKLIGHT_RAW;
- props.max_brightness = machinfo->max_intensity;
- bd = devm_backlight_device_register(&pdev->dev, name, &pdev->dev,
- NULL, &genericbl_ops, &props);
- if (IS_ERR(bd))
- return PTR_ERR(bd);
-
- platform_set_drvdata(pdev, bd);
-
- bd->props.power = FB_BLANK_UNBLANK;
- bd->props.brightness = machinfo->default_intensity;
- backlight_update_status(bd);
-
- generic_backlight_device = bd;
-
- dev_info(&pdev->dev, "Generic Backlight Driver Initialized.\n");
- return 0;
-}
-
-static int genericbl_remove(struct platform_device *pdev)
-{
- struct backlight_device *bd = platform_get_drvdata(pdev);
-
- bd->props.power = 0;
- bd->props.brightness = 0;
- backlight_update_status(bd);
-
- dev_info(&pdev->dev, "Generic Backlight Driver Unloaded\n");
- return 0;
-}
-
-static struct platform_driver genericbl_driver = {
- .probe = genericbl_probe,
- .remove = genericbl_remove,
- .driver = {
- .name = "generic-bl",
- },
-};
-
-module_platform_driver(genericbl_driver);
-
-MODULE_AUTHOR("Richard Purdie <rpurdie@rpsys.net>");
-MODULE_DESCRIPTION("Generic Backlight Driver");
-MODULE_LICENSE("GPL");
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 0f425b32e6be..8f2005a6f8a9 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -454,15 +454,6 @@ static inline void * bl_get_data(struct backlight_device *bl_dev)
return dev_get_drvdata(&bl_dev->dev);
}
-struct generic_bl_info {
- const char *name;
- int max_intensity;
- int default_intensity;
- int limit_mask;
- void (*set_bl_intensity)(int intensity);
- void (*kick_battery)(void);
-};
-
#ifdef CONFIG_OF
struct backlight_device *of_find_backlight_by_node(struct device_node *node);
#else
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 09/20] backlight: drop extern from prototypes
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (7 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 08/20] backlight: remove the unused backlight_bl driver Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 10/20] backlight: add overview and update existing doc Sam Ravnborg
` (10 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
No need to put "extern" in front of prototypes.
While touching the prototypes adjust indent to follow
the kernel style.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 35 +++++++++++++++++++----------------
1 file changed, 19 insertions(+), 16 deletions(-)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 8f2005a6f8a9..c6ac4cbb9ddb 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -417,23 +417,26 @@ static inline bool backlight_is_blank(const struct backlight_device *bd)
bd->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK);
}
-extern struct backlight_device *backlight_device_register(const char *name,
- struct device *dev, void *devdata, const struct backlight_ops *ops,
- const struct backlight_properties *props);
-extern struct backlight_device *devm_backlight_device_register(
- struct device *dev, const char *name, struct device *parent,
- void *devdata, const struct backlight_ops *ops,
- const struct backlight_properties *props);
-extern void backlight_device_unregister(struct backlight_device *bd);
-extern void devm_backlight_device_unregister(struct device *dev,
- struct backlight_device *bd);
-extern void backlight_force_update(struct backlight_device *bd,
- enum backlight_update_reason reason);
-extern int backlight_register_notifier(struct notifier_block *nb);
-extern int backlight_unregister_notifier(struct notifier_block *nb);
-extern struct backlight_device *backlight_device_get_by_type(enum backlight_type type);
+struct backlight_device *
+backlight_device_register(const char *name, struct device *dev, void *devdata,
+ const struct backlight_ops *ops,
+ const struct backlight_properties *props);
+struct backlight_device *
+devm_backlight_device_register(struct device *dev, const char *name,
+ struct device *parent, void *devdata,
+ const struct backlight_ops *ops,
+ const struct backlight_properties *props);
+void backlight_device_unregister(struct backlight_device *bd);
+void devm_backlight_device_unregister(struct device *dev,
+ struct backlight_device *bd);
+void backlight_force_update(struct backlight_device *bd,
+ enum backlight_update_reason reason);
+int backlight_register_notifier(struct notifier_block *nb);
+int backlight_unregister_notifier(struct notifier_block *nb);
struct backlight_device *backlight_device_get_by_name(const char *name);
-extern int backlight_device_set_brightness(struct backlight_device *bd, unsigned long brightness);
+struct backlight_device *backlight_device_get_by_type(enum backlight_type type);
+int backlight_device_set_brightness(struct backlight_device *bd,
+ unsigned long brightness);
#define to_backlight_device(obj) container_of(obj, struct backlight_device, dev)
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 10/20] backlight: add overview and update existing doc
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (8 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 09/20] backlight: drop extern from prototypes Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 11/20] backlight: wire up kernel-doc documentation Sam Ravnborg
` (9 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Add overview chapter to backlight.c.
Update existing kernel-doc to follow a more consistent
style and drop kernel-doc for deprecated functions.
v4:
- Include updated devm_of_find_backlight doc
(was accidently included in a later patch)
v3:
- Updated a few editorial details (Daniel)
v2:
- Sevaral editorial corrections that makes reading
much easier (Daniel)
- Spelling fixes (Daniel)
- updated intro chapter with a little more info
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/backlight.c | 143 +++++++++++++++++++---------
1 file changed, 100 insertions(+), 43 deletions(-)
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index 7e249aa90b0e..db8717581ec5 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -22,6 +22,47 @@
#include <asm/backlight.h>
#endif
+/**
+ * DOC: overview
+ *
+ * The backlight core supports implementing backlight drivers.
+ *
+ * A backlight driver registers a driver using
+ * devm_backlight_device_register(). The properties of the backlight
+ * driver such as type and max_brightness must be specified.
+ * When the core detect changes in for example brightness or power state
+ * the update_status() operation is called. The backlight driver shall
+ * implement this operation and use it to adjust backlight.
+ *
+ * Several sysfs attributes are provided by the backlight core::
+ *
+ * - brightness R/W, set the requested brightness level
+ * - actual_brighness RO, the brightness level used by the HW
+ * - max_brightness RO, the maximum brightness level supported
+ *
+ * See Documentation/ABI/stable/sysfs-class-backlight for the full list.
+ *
+ * The backlight can be adjusted using the sysfs interface, and
+ * the backlight driver may also support adjusting backlight using
+ * a hot-key or some other platfrom or firmware specific way.
+ *
+ * The driver must implement the get_brightness() operation if
+ * the HW do not support all the levels that can be specified in
+ * brightness, thus providing user-space access to the actual level
+ * via the actual_brightness attribute.
+ *
+ * When the backlight changes this is reported to user-space using
+ * an uevent connected to the actual_brightness attribute.
+ * When brightness is set by platform specific means, for example
+ * a hot-key to adjust backlight, the driver must notify the backlight
+ * core that brightness has changed using backlight_force_update().
+ *
+ * The backlight driver core receives notifications from fbdev and
+ * if the event is FB_EVENT_BLANK and if the value of blank, from the
+ * FBIOBLANK ioclt, results in a change in the backlight state the
+ * update_status() operation is called.
+ */
+
static struct list_head backlight_dev_list;
static struct mutex backlight_dev_list_mutex;
static struct blocking_notifier_head backlight_notifier;
@@ -40,9 +81,17 @@ static const char *const backlight_scale_types[] = {
#if defined(CONFIG_FB) || (defined(CONFIG_FB_MODULE) && \
defined(CONFIG_BACKLIGHT_CLASS_DEVICE_MODULE))
-/* This callback gets called when something important happens inside a
- * framebuffer driver. We're looking if that important event is blanking,
- * and if it is and necessary, we're switching backlight power as well ...
+/*
+ * fb_notifier_callback
+ *
+ * This callback gets called when something important happens inside a
+ * framebuffer driver. The backlight core only cares about FB_BLANK_UNBLANK
+ * which is reported to the driver using backlight_update_status()
+ * as a state change.
+ *
+ * There may be several fbdev's connected to the backlight device,
+ * in which case they are kept track of. A state change is only reported
+ * if there is a change in backlight for the specified fbdev.
*/
static int fb_notifier_callback(struct notifier_block *self,
unsigned long event, void *data)
@@ -318,12 +367,15 @@ static struct attribute *bl_device_attrs[] = {
ATTRIBUTE_GROUPS(bl_device);
/**
- * backlight_force_update - tell the backlight subsystem that hardware state
- * has changed
+ * backlight_force_update - force an update due to a hardware change
* @bd: the backlight device to update
+ * @reason: the method used for the backlight update
*
* Updates the internal state of the backlight in response to a hardware event,
- * and generate a uevent to notify userspace
+ * and generates an uevent to notify userspace. A backlight driver shall call
+ * backlight_force_update() when the backlight is changed using, for example,
+ * a hot-key. The updated brightness is read using get_brightness() and the
+ * brightness value is reported using an uevent.
*/
void backlight_force_update(struct backlight_device *bd,
enum backlight_update_reason reason)
@@ -336,19 +388,7 @@ void backlight_force_update(struct backlight_device *bd,
}
EXPORT_SYMBOL(backlight_force_update);
-/**
- * backlight_device_register - create and register a new object of
- * backlight_device class.
- * @name: the name of the new object(must be the same as the name of the
- * respective framebuffer device).
- * @parent: a pointer to the parent device
- * @devdata: an optional pointer to be stored for private driver use. The
- * methods may retrieve it by using bl_get_data(bd).
- * @ops: the backlight operations structure.
- *
- * Creates and registers new backlight device. Returns either an
- * ERR_PTR() or a pointer to the newly allocated device.
- */
+/* deprecated - use devm_backlight_device_register() */
struct backlight_device *backlight_device_register(const char *name,
struct device *parent, void *devdata, const struct backlight_ops *ops,
const struct backlight_properties *props)
@@ -415,6 +455,15 @@ struct backlight_device *backlight_device_register(const char *name,
}
EXPORT_SYMBOL(backlight_device_register);
+/** backlight_device_get_by_type - find first backlight device of a type
+ * @type: the type of backlight device
+ *
+ * Look up the first backlight device of the specified type
+ *
+ * RETURNS:
+ *
+ * Pointer to backlight device if any was found. Otherwise NULL.
+ */
struct backlight_device *backlight_device_get_by_type(enum backlight_type type)
{
bool found = false;
@@ -454,12 +503,7 @@ struct backlight_device *backlight_device_get_by_name(const char *name)
}
EXPORT_SYMBOL(backlight_device_get_by_name);
-/**
- * backlight_device_unregister - unregisters a backlight device object.
- * @bd: the backlight device object to be unregistered and freed.
- *
- * Unregisters a previously registered via backlight_device_register object.
- */
+/* deprecated - use devm_backlight_device_unregister() */
void backlight_device_unregister(struct backlight_device *bd)
{
if (!bd)
@@ -507,10 +551,12 @@ static int devm_backlight_device_match(struct device *dev, void *res,
* backlight_register_notifier - get notified of backlight (un)registration
* @nb: notifier block with the notifier to call on backlight (un)registration
*
- * @return 0 on success, otherwise a negative error code
- *
* Register a notifier to get notified when backlight devices get registered
* or unregistered.
+ *
+ * RETURNS:
+ *
+ * 0 on success, otherwise a negative error code
*/
int backlight_register_notifier(struct notifier_block *nb)
{
@@ -522,10 +568,12 @@ EXPORT_SYMBOL(backlight_register_notifier);
* backlight_unregister_notifier - unregister a backlight notifier
* @nb: notifier block to unregister
*
- * @return 0 on success, otherwise a negative error code
- *
* Register a notifier to get notified when backlight devices get registered
* or unregistered.
+ *
+ * RETURNS:
+ *
+ * 0 on success, otherwise a negative error code
*/
int backlight_unregister_notifier(struct notifier_block *nb)
{
@@ -534,20 +582,22 @@ int backlight_unregister_notifier(struct notifier_block *nb)
EXPORT_SYMBOL(backlight_unregister_notifier);
/**
- * devm_backlight_device_register - resource managed backlight_device_register()
+ * devm_backlight_device_register - register a new backlight device
* @dev: the device to register
* @name: the name of the device
- * @parent: a pointer to the parent device
+ * @parent: a pointer to the parent device (often the same as @dev)
* @devdata: an optional pointer to be stored for private driver use
* @ops: the backlight operations structure
* @props: the backlight properties
*
- * @return a struct backlight on success, or an ERR_PTR on error
+ * Creates and registers new backlight device. When a backlight device
+ * is registered the configuration must be specified in the @props
+ * parameter. See description of &backlight_properties.
*
- * Managed backlight_device_register(). The backlight_device returned
- * from this function are automatically freed on driver detach.
- * See backlight_device_register() for more information.
- */
+ * RETURNS:
+ *
+ * struct backlight on success, or an ERR_PTR on error
+*/
struct backlight_device *devm_backlight_device_register(struct device *dev,
const char *name, struct device *parent, void *devdata,
const struct backlight_ops *ops,
@@ -574,13 +624,13 @@ struct backlight_device *devm_backlight_device_register(struct device *dev,
EXPORT_SYMBOL(devm_backlight_device_register);
/**
- * devm_backlight_device_unregister - resource managed backlight_device_unregister()
+ * devm_backlight_device_unregister - unregister backlight device
* @dev: the device to unregister
* @bd: the backlight device to unregister
*
- * Deallocated a backlight allocated with devm_backlight_device_register().
+ * Deallocates a backlight allocated with devm_backlight_device_register().
* Normally this function will not need to be called and the resource management
- * code will ensure that the resource is freed.
+ * code will ensure that the resources are freed.
*/
void devm_backlight_device_unregister(struct device *dev,
struct backlight_device *bd)
@@ -671,12 +721,19 @@ static void devm_backlight_release(void *data)
}
/**
- * devm_of_find_backlight - Resource-managed of_find_backlight()
- * @dev: Device
+ * devm_of_find_backlight - find backlight for a device
+ * @dev: the device
*
- * Device managed version of of_find_backlight().
- * The reference on the backlight device is automatically
+ * This function looks for a property named 'backlight' on the DT node
+ * connected to @dev and looks up the backlight device. The lookup is
+ * device managed so the reference to the backlight device is automatically
* dropped on driver detach.
+ *
+ * RETURNS:
+ *
+ * A pointer to the backlight device if found.
+ * Error pointer -EPROBE_DEFER if the DT property is set, but no backlight
+ * device is found. NULL if there's no backlight property.
*/
struct backlight_device *devm_of_find_backlight(struct device *dev)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 11/20] backlight: wire up kernel-doc documentation
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (9 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 10/20] backlight: add overview and update existing doc Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 12/20] backlight: introduce backlight_get_brightness() Sam Ravnborg
` (8 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Include backlight so the documentation is now generated
with make htmldocs and friends.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Maxime Ripard <mripard@kernel.org>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Jonathan Corbet <corbet@lwn.net>
---
Documentation/gpu/backlight.rst | 12 ++++++++++++
Documentation/gpu/index.rst | 1 +
2 files changed, 13 insertions(+)
create mode 100644 Documentation/gpu/backlight.rst
diff --git a/Documentation/gpu/backlight.rst b/Documentation/gpu/backlight.rst
new file mode 100644
index 000000000000..9ebfc9d0aced
--- /dev/null
+++ b/Documentation/gpu/backlight.rst
@@ -0,0 +1,12 @@
+=================
+Backlight support
+=================
+
+.. kernel-doc:: drivers/video/backlight/backlight.c
+ :doc: overview
+
+.. kernel-doc:: include/linux/backlight.h
+ :internal:
+
+.. kernel-doc:: drivers/video/backlight/backlight.c
+ :export:
diff --git a/Documentation/gpu/index.rst b/Documentation/gpu/index.rst
index 1fcf8e851e15..c9a51e3bfb5a 100644
--- a/Documentation/gpu/index.rst
+++ b/Documentation/gpu/index.rst
@@ -12,6 +12,7 @@ Linux GPU Driver Developer's Guide
drm-uapi
drm-client
drivers
+ backlight
vga-switcheroo
vgaarbiter
todo
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 12/20] backlight: introduce backlight_get_brightness()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (10 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 11/20] backlight: wire up kernel-doc documentation Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 10:27 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 13/20] backlight: as3711_bl: simplify update_status Sam Ravnborg
` (7 subsequent siblings)
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Based on an idea from Emil Velikov <emil.l.velikov@gmail.com>
add a helper that checks backlight_is_blank() and return 0 as brightness
if display is blank or the property value if not.
This allows us to simplify the update_status() implementation
in most of the backlight drivers.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
include/linux/backlight.h | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index c6ac4cbb9ddb..38db67588b16 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -417,6 +417,25 @@ static inline bool backlight_is_blank(const struct backlight_device *bd)
bd->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK);
}
+/**
+ * backlight_get_brightness - Returns the current brightness value
+ * @bd: the backlight device
+ *
+ * Returns the current brightness value, taking in consideration the current
+ * state. If backlight_is_blank() returns true then return 0 as brightness
+ * otherwise return the current brightness property value.
+ *
+ * Backlight drivers are expected to use this function in their update_status()
+ * operation to get the brightness value.
+ */
+static inline int backlight_get_brightness(const struct backlight_device *bd)
+{
+ if (backlight_is_blank(bd))
+ return 0;
+ else
+ return bd->props.brightness;
+}
+
struct backlight_device *
backlight_device_register(const char *name, struct device *dev, void *devdata,
const struct backlight_ops *ops,
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 12/20] backlight: introduce backlight_get_brightness()
2020-07-03 18:45 ` [PATCH v4 12/20] backlight: introduce backlight_get_brightness() Sam Ravnborg
@ 2020-07-08 10:27 ` Daniel Thompson
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 10:27 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:38PM +0200, Sam Ravnborg wrote:
> Based on an idea from Emil Velikov <emil.l.velikov@gmail.com>
> add a helper that checks backlight_is_blank() and return 0 as brightness
> if display is blank or the property value if not.
>
> This allows us to simplify the update_status() implementation
> in most of the backlight drivers.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Emil Velikov <emil.l.velikov@gmail.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 13/20] backlight: as3711_bl: simplify update_status
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (11 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 12/20] backlight: introduce backlight_get_brightness() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 14/20] backlight: cr_bllcd: introduce backlight_is_blank() Sam Ravnborg
` (6 subsequent siblings)
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Replaces the open-coded checks of the state, with the
backlight_get_brightness() helper. This increases readability
of the code and align the functionality across the drivers.
Futhermore drop the debug prints in update_status().
If we need debug printing then we can add it to the backlight core.
v2:
- Use backlight_get_brightness()
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
Cc: Emil Velikov <emil.l.velikov@gmail.com>
---
drivers/video/backlight/as3711_bl.c | 11 ++---------
1 file changed, 2 insertions(+), 9 deletions(-)
diff --git a/drivers/video/backlight/as3711_bl.c b/drivers/video/backlight/as3711_bl.c
index 33f0f0f2e8b3..3b60019cdc2b 100644
--- a/drivers/video/backlight/as3711_bl.c
+++ b/drivers/video/backlight/as3711_bl.c
@@ -104,17 +104,10 @@ static int as3711_bl_update_status(struct backlight_device *bl)
struct as3711_bl_data *data = bl_get_data(bl);
struct as3711_bl_supply *supply = to_supply(data);
struct as3711 *as3711 = supply->as3711;
- int brightness = bl->props.brightness;
+ int brightness;
int ret = 0;
- dev_dbg(&bl->dev, "%s(): brightness %u, pwr %x, blank %x, state %x\n",
- __func__, bl->props.brightness, bl->props.power,
- bl->props.fb_blank, bl->props.state);
-
- if (bl->props.power != FB_BLANK_UNBLANK ||
- bl->props.fb_blank != FB_BLANK_UNBLANK ||
- bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK))
- brightness = 0;
+ brightness = backlight_get_brightness(bl);
if (data->type == AS3711_BL_SU1) {
ret = as3711_set_brightness_v(as3711, brightness,
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 14/20] backlight: cr_bllcd: introduce backlight_is_blank()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (12 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 13/20] backlight: as3711_bl: simplify update_status Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 11:06 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 15/20] backlight: gpio_backlight: simplify update_status() Sam Ravnborg
` (5 subsequent siblings)
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
The cr_bllcd uses the FB_BLANK states as brightness.
This results in brightness value equals 0 that turn on
the display and 4 that turn off the display.
Simplify the logic but keep current behaviour
as userspace may expect brightness set to 0 to turn on the display.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/cr_bllcd.c | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/video/backlight/cr_bllcd.c b/drivers/video/backlight/cr_bllcd.c
index 4624b7b7c6a6..edca5fee9689 100644
--- a/drivers/video/backlight/cr_bllcd.c
+++ b/drivers/video/backlight/cr_bllcd.c
@@ -63,22 +63,16 @@ static int cr_backlight_set_intensity(struct backlight_device *bd)
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)
+ if (backlight_is_blank(bd))
intensity = FB_BLANK_POWERDOWN;
- if (intensity == FB_BLANK_UNBLANK) { /* FULL ON */
+ if (intensity != FB_BLANK_POWERDOWN) { /* FULL ON */
cur &= ~CRVML_BACKLIGHT_OFF;
outl(cur, addr);
- } else if (intensity == FB_BLANK_POWERDOWN) { /* OFF */
+ } else { /* OFF */
cur |= CRVML_BACKLIGHT_OFF;
outl(cur, addr);
- } /* anything else, don't bother */
+ }
return 0;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 14/20] backlight: cr_bllcd: introduce backlight_is_blank()
2020-07-03 18:45 ` [PATCH v4 14/20] backlight: cr_bllcd: introduce backlight_is_blank() Sam Ravnborg
@ 2020-07-08 11:06 ` Daniel Thompson
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 11:06 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:40PM +0200, Sam Ravnborg wrote:
> The cr_bllcd uses the FB_BLANK states as brightness.
> This results in brightness value equals 0 that turn on
> the display and 4 that turn off the display.
max_brightness is 0 which makes it impossible to set the brightness to
4.
> Simplify the logic but keep current behaviour
> as userspace may expect brightness set to 0 to turn on the display.
I don't deny the possibility but we'd be talking about a userspace
custom enough to explicitly configure the module loader (otherwise
the module wouldn't load and the backlight would be inherited from
the BIOS) and that also contains an explicit write to the brightness
property that sets it to what is already the default (and only
possible) value.
In other words I'm very sceptical such a userspace exists and would be
happy to see this driver modified to use the new helpers and behave
like gpio-backlight (which can be regarded as our reference 1-bit
backlight driver).
Daniel.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> ---
> drivers/video/backlight/cr_bllcd.c | 14 ++++----------
> 1 file changed, 4 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/video/backlight/cr_bllcd.c b/drivers/video/backlight/cr_bllcd.c
> index 4624b7b7c6a6..edca5fee9689 100644
> --- a/drivers/video/backlight/cr_bllcd.c
> +++ b/drivers/video/backlight/cr_bllcd.c
> @@ -63,22 +63,16 @@ static int cr_backlight_set_intensity(struct backlight_device *bd)
> 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)
> + if (backlight_is_blank(bd))
> intensity = FB_BLANK_POWERDOWN;
>
> - if (intensity == FB_BLANK_UNBLANK) { /* FULL ON */
> + if (intensity != FB_BLANK_POWERDOWN) { /* FULL ON */
> cur &= ~CRVML_BACKLIGHT_OFF;
> outl(cur, addr);
> - } else if (intensity == FB_BLANK_POWERDOWN) { /* OFF */
> + } else { /* OFF */
> cur |= CRVML_BACKLIGHT_OFF;
> outl(cur, addr);
> - } /* anything else, don't bother */
> + }
>
> return 0;
> }
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 15/20] backlight: gpio_backlight: simplify update_status()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (13 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 14/20] backlight: cr_bllcd: introduce backlight_is_blank() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 11:07 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 16/20] backlight: jornada720_bl: introduce backlight_is_blank() Sam Ravnborg
` (4 subsequent siblings)
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Introduce the use of backlight_get_brightness() to simplify
the update_status() operation.
With the simpler implementation drop the gpio_backlight_get_next_brightness()
helper as it was now a one-liner.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/gpio_backlight.c | 17 ++---------------
1 file changed, 2 insertions(+), 15 deletions(-)
diff --git a/drivers/video/backlight/gpio_backlight.c b/drivers/video/backlight/gpio_backlight.c
index 75409ddfba3e..6f78d928f054 100644
--- a/drivers/video/backlight/gpio_backlight.c
+++ b/drivers/video/backlight/gpio_backlight.c
@@ -21,24 +21,11 @@ struct gpio_backlight {
struct gpio_desc *gpiod;
};
-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))
- brightness = 0;
-
- return brightness;
-}
-
static int gpio_backlight_update_status(struct backlight_device *bl)
{
struct gpio_backlight *gbl = bl_get_data(bl);
- int brightness = gpio_backlight_get_next_brightness(bl);
- gpiod_set_value_cansleep(gbl->gpiod, brightness);
+ gpiod_set_value_cansleep(gbl->gpiod, backlight_get_brightness(bl));
return 0;
}
@@ -108,7 +95,7 @@ static int gpio_backlight_probe(struct platform_device *pdev)
bl->props.brightness = 1;
- init_brightness = gpio_backlight_get_next_brightness(bl);
+ init_brightness = backlight_get_brightness(bl);
ret = gpiod_direction_output(gbl->gpiod, init_brightness);
if (ret) {
dev_err(dev, "failed to set initial brightness\n");
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 15/20] backlight: gpio_backlight: simplify update_status()
2020-07-03 18:45 ` [PATCH v4 15/20] backlight: gpio_backlight: simplify update_status() Sam Ravnborg
@ 2020-07-08 11:07 ` Daniel Thompson
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 11:07 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:41PM +0200, Sam Ravnborg wrote:
> Introduce the use of backlight_get_brightness() to simplify
> the update_status() operation.
> With the simpler implementation drop the gpio_backlight_get_next_brightness()
> helper as it was now a one-liner.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 16/20] backlight: jornada720_bl: introduce backlight_is_blank()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (14 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 15/20] backlight: gpio_backlight: simplify update_status() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 11:12 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 17/20] backlight: use backligt_get_brightness() Sam Ravnborg
` (3 subsequent siblings)
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Use the backlight_is_blank() helper to simplify the code a bit.
This add support for fb_blank as a side-effect.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/jornada720_bl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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");
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 16/20] backlight: jornada720_bl: introduce backlight_is_blank()
2020-07-03 18:45 ` [PATCH v4 16/20] backlight: jornada720_bl: introduce backlight_is_blank() Sam Ravnborg
@ 2020-07-08 11:12 ` Daniel Thompson
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 11:12 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:42PM +0200, Sam Ravnborg wrote:
> Use the backlight_is_blank() helper to simplify the code a bit.
> This add support for fb_blank as a side-effect.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
This driver exhibits the "how does userspace know what writing brightness zero to
sysfs actually does" problem so can't use the backlight_get_brightness() helper.
If you respin it would be good to see that in the patch description but
that's such an extreme nitpick It think, either way, this can be:
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (15 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 16/20] backlight: jornada720_bl: introduce backlight_is_blank() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-07 5:48 ` Daniel Vetter
2020-07-08 11:14 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 18/20] backlight: drop backlight_put() Sam Ravnborg
` (2 subsequent siblings)
19 siblings, 2 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
Introduce the backlight_get_brightness() helper in all
video/backlight/* drivers.
This simplifies the code and align the implementation of the
update_status() operation across the different backlight drivers.
Some of the drivers gains a little extra functionality by the change
as they now respect the fb_blank() ioctl.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
---
drivers/video/backlight/88pm860x_bl.c | 13 +------------
drivers/video/backlight/adp5520_bl.c | 10 +---------
drivers/video/backlight/adp8860_bl.c | 10 +---------
drivers/video/backlight/adp8870_bl.c | 10 +---------
drivers/video/backlight/bd6107.c | 7 +------
drivers/video/backlight/corgi_lcd.c | 8 +-------
drivers/video/backlight/da903x_bl.c | 13 +------------
drivers/video/backlight/ep93xx_bl.c | 8 +-------
drivers/video/backlight/hp680_bl.c | 6 +-----
drivers/video/backlight/kb3886_bl.c | 6 +-----
drivers/video/backlight/led_bl.c | 7 +------
drivers/video/backlight/lm3533_bl.c | 8 +-------
drivers/video/backlight/locomolcd.c | 6 +-----
drivers/video/backlight/lv5207lp.c | 7 +------
drivers/video/backlight/max8925_bl.c | 13 +------------
drivers/video/backlight/pwm_bl.c | 7 +------
drivers/video/backlight/qcom-wled.c | 7 +------
drivers/video/backlight/tps65217_bl.c | 10 +---------
drivers/video/backlight/wm831x_bl.c | 13 +------------
19 files changed, 19 insertions(+), 150 deletions(-)
diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
index 20d96a5ac384..25e409bbb1a2 100644
--- a/drivers/video/backlight/88pm860x_bl.c
+++ b/drivers/video/backlight/88pm860x_bl.c
@@ -121,18 +121,7 @@ static int pm860x_backlight_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return pm860x_backlight_set(bl, brightness);
+ return pm860x_backlight_set(bl, backlight_get_brightness(bl));
}
static int pm860x_backlight_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
index 0f63f76723a5..686988c3df3a 100644
--- a/drivers/video/backlight/adp5520_bl.c
+++ b/drivers/video/backlight/adp5520_bl.c
@@ -65,15 +65,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return adp5520_bl_set(bl, brightness);
+ return adp5520_bl_set(bl, backlight_get_brightness(bl));
}
static int adp5520_bl_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
index 19968104fc47..ddc7f5f0401f 100644
--- a/drivers/video/backlight/adp8860_bl.c
+++ b/drivers/video/backlight/adp8860_bl.c
@@ -361,15 +361,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return adp8860_bl_set(bl, brightness);
+ return adp8860_bl_set(bl, backlight_get_brightness(bl));
}
static int adp8860_bl_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
index 4c0032010cfe..8b5213a39527 100644
--- a/drivers/video/backlight/adp8870_bl.c
+++ b/drivers/video/backlight/adp8870_bl.c
@@ -399,15 +399,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return adp8870_bl_set(bl, brightness);
+ return adp8870_bl_set(bl, backlight_get_brightness(bl));
}
static int adp8870_bl_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c
index d5d5fb457e78..515184fbe33a 100644
--- a/drivers/video/backlight/bd6107.c
+++ b/drivers/video/backlight/bd6107.c
@@ -82,12 +82,7 @@ static int bd6107_write(struct bd6107 *bd, u8 reg, u8 data)
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))
- brightness = 0;
+ int brightness = backlight_get_brightness(backlight);
if (brightness) {
bd6107_write(bd, BD6107_PORTSEL, BD6107_PORTSEL_LEDM(2) |
diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
index 25ef0cbd7583..33f5d80495e6 100644
--- a/drivers/video/backlight/corgi_lcd.c
+++ b/drivers/video/backlight/corgi_lcd.c
@@ -420,13 +420,7 @@ static int corgi_bl_set_intensity(struct corgi_lcd *lcd, int intensity)
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)
- intensity = 0;
+ int intensity = backlight_get_brightness(bd);
if (corgibl_flags & CORGIBL_SUSPENDED)
intensity = 0;
diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c
index 62540e4bdedb..71f21bbc7a9f 100644
--- a/drivers/video/backlight/da903x_bl.c
+++ b/drivers/video/backlight/da903x_bl.c
@@ -77,18 +77,7 @@ static int da903x_backlight_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return da903x_backlight_set(bl, brightness);
+ return da903x_backlight_set(bl, backlight_get_brightness(bl));
}
static int da903x_backlight_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c
index 4149e0b2f83c..2387009d452d 100644
--- a/drivers/video/backlight/ep93xx_bl.c
+++ b/drivers/video/backlight/ep93xx_bl.c
@@ -36,13 +36,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return ep93xxbl_set(bl, brightness);
+ return ep93xxbl_set(bl, backlight_get_brightness(bl));
}
static int ep93xxbl_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c
index 8ea42b8d9bc8..9123c33def05 100644
--- a/drivers/video/backlight/hp680_bl.c
+++ b/drivers/video/backlight/hp680_bl.c
@@ -33,12 +33,8 @@ static void hp680bl_send_intensity(struct backlight_device *bd)
{
unsigned long flags;
u16 v;
- int intensity = bd->props.brightness;
+ int intensity = backlight_get_brightness(bd);
- if (bd->props.power != FB_BLANK_UNBLANK)
- intensity = 0;
- if (bd->props.fb_blank != FB_BLANK_UNBLANK)
- intensity = 0;
if (hp680bl_suspended)
intensity = 0;
diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c
index 1dfe13c18925..55794b239cff 100644
--- a/drivers/video/backlight/kb3886_bl.c
+++ b/drivers/video/backlight/kb3886_bl.c
@@ -87,12 +87,8 @@ static const struct dmi_system_id kb3886bl_device_table[] __initconst = {
static int kb3886bl_send_intensity(struct backlight_device *bd)
{
- int intensity = bd->props.brightness;
+ int intensity = backlight_get_brightness(bd);
- if (bd->props.power != FB_BLANK_UNBLANK)
- intensity = 0;
- if (bd->props.fb_blank != FB_BLANK_UNBLANK)
- 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..f54d256e2d54 100644
--- a/drivers/video/backlight/led_bl.c
+++ b/drivers/video/backlight/led_bl.c
@@ -54,12 +54,7 @@ static void led_bl_power_off(struct led_bl_data *priv)
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)
- brightness = 0;
+ int brightness = backlight_get_brightness(bl);
if (brightness > 0)
led_bl_set_brightness(priv, brightness);
diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c
index ee09d1bd02b9..d07a2bd805fc 100644
--- a/drivers/video/backlight/lm3533_bl.c
+++ b/drivers/video/backlight/lm3533_bl.c
@@ -39,14 +39,8 @@ static inline int lm3533_bl_get_ctrlbank_id(struct lm3533_bl *bl)
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)
- brightness = 0;
-
- return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness);
+ return lm3533_ctrlbank_set_brightness(&bl->cb, backlight_get_brightness(bd));
}
static int lm3533_bl_get_brightness(struct backlight_device *bd)
diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c
index cdc02e04f89d..297ee2e1ab0b 100644
--- a/drivers/video/backlight/locomolcd.c
+++ b/drivers/video/backlight/locomolcd.c
@@ -111,12 +111,8 @@ static int current_intensity;
static int locomolcd_set_intensity(struct backlight_device *bd)
{
- int intensity = bd->props.brightness;
+ int intensity = backlight_get_brightness(bd);
- if (bd->props.power != FB_BLANK_UNBLANK)
- intensity = 0;
- if (bd->props.fb_blank != FB_BLANK_UNBLANK)
- 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..1842ae9a55f8 100644
--- a/drivers/video/backlight/lv5207lp.c
+++ b/drivers/video/backlight/lv5207lp.c
@@ -46,12 +46,7 @@ static int lv5207lp_write(struct lv5207lp *lv, u8 reg, u8 data)
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))
- brightness = 0;
+ int brightness = backlight_get_brightness(backlight);
if (brightness) {
lv5207lp_write(lv, LV5207LP_CTRL1,
diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c
index 97cc260ff9d1..e607ec6fd4bf 100644
--- a/drivers/video/backlight/max8925_bl.c
+++ b/drivers/video/backlight/max8925_bl.c
@@ -64,18 +64,7 @@ static int max8925_backlight_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return max8925_backlight_set(bl, brightness);
+ return max8925_backlight_set(bl, backlight_get_brightness(bl));
}
static int max8925_backlight_get_brightness(struct backlight_device *bl)
diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
index 82b8d7594701..eff64db2e02e 100644
--- a/drivers/video/backlight/pwm_bl.c
+++ b/drivers/video/backlight/pwm_bl.c
@@ -108,14 +108,9 @@ static int compute_duty_cycle(struct pwm_bl_data *pb, int brightness)
static int pwm_backlight_update_status(struct backlight_device *bl)
{
struct pwm_bl_data *pb = bl_get_data(bl);
- int brightness = bl->props.brightness;
+ int brightness = backlight_get_brightness(bl);
struct pwm_state state;
- if (bl->props.power != FB_BLANK_UNBLANK ||
- bl->props.fb_blank != FB_BLANK_UNBLANK ||
- bl->props.state & BL_CORE_FBBLANK)
- brightness = 0;
-
if (pb->notify)
brightness = pb->notify(pb->dev, brightness);
diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c
index 4c8c34b99441..94d3b96244f0 100644
--- a/drivers/video/backlight/qcom-wled.c
+++ b/drivers/video/backlight/qcom-wled.c
@@ -433,14 +433,9 @@ static int wled5_ovp_delay(struct wled *wled)
static int wled_update_status(struct backlight_device *bl)
{
struct wled *wled = bl_get_data(bl);
- u16 brightness = bl->props.brightness;
+ u16 brightness = backlight_get_brightness(bl);
int rc = 0;
- if (bl->props.power != FB_BLANK_UNBLANK ||
- bl->props.fb_blank != FB_BLANK_UNBLANK ||
- bl->props.state & BL_CORE_FBBLANK)
- brightness = 0;
-
mutex_lock(&wled->lock);
if (brightness) {
rc = wled->wled_set_brightness(wled, brightness);
diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c
index 762e3feed097..8457166f357f 100644
--- a/drivers/video/backlight/tps65217_bl.c
+++ b/drivers/video/backlight/tps65217_bl.c
@@ -77,15 +77,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl)
{
struct tps65217_bl *tps65217_bl = bl_get_data(bl);
int rc;
- int brightness = bl->props.brightness;
-
- 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 */
- brightness = 0;
+ int brightness = backlight_get_brightness(bl);
if (brightness > 0) {
rc = tps65217_reg_write(tps65217_bl->tps,
diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
index e55977d54c15..c5aaee205bdf 100644
--- a/drivers/video/backlight/wm831x_bl.c
+++ b/drivers/video/backlight/wm831x_bl.c
@@ -91,18 +91,7 @@ static int wm831x_backlight_set(struct backlight_device *bl, int brightness)
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)
- brightness = 0;
-
- return wm831x_backlight_set(bl, brightness);
+ return wm831x_backlight_set(bl, backlight_get_brightness(bl));
}
static int wm831x_backlight_get_brightness(struct backlight_device *bl)
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-03 18:45 ` [PATCH v4 17/20] backlight: use backligt_get_brightness() Sam Ravnborg
@ 2020-07-07 5:48 ` Daniel Vetter
2020-07-07 19:59 ` Sam Ravnborg
2020-07-08 11:14 ` Daniel Thompson
1 sibling, 1 reply; 39+ messages in thread
From: Daniel Vetter @ 2020-07-07 5:48 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas
On Fri, Jul 03, 2020 at 08:45:43PM +0200, Sam Ravnborg wrote:
> Introduce the backlight_get_brightness() helper in all
> video/backlight/* drivers.
> This simplifies the code and align the implementation of the
> update_status() operation across the different backlight drivers.
>
> Some of the drivers gains a little extra functionality by the change
> as they now respect the fb_blank() ioctl.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
This looks really nice!
Since there's a pile more backlight drivers all over the tree, care to
update the gpu/todo.rst entry with a note that conversions should use
backlight_get_brightness?
-Daniel
> ---
> drivers/video/backlight/88pm860x_bl.c | 13 +------------
> drivers/video/backlight/adp5520_bl.c | 10 +---------
> drivers/video/backlight/adp8860_bl.c | 10 +---------
> drivers/video/backlight/adp8870_bl.c | 10 +---------
> drivers/video/backlight/bd6107.c | 7 +------
> drivers/video/backlight/corgi_lcd.c | 8 +-------
> drivers/video/backlight/da903x_bl.c | 13 +------------
> drivers/video/backlight/ep93xx_bl.c | 8 +-------
> drivers/video/backlight/hp680_bl.c | 6 +-----
> drivers/video/backlight/kb3886_bl.c | 6 +-----
> drivers/video/backlight/led_bl.c | 7 +------
> drivers/video/backlight/lm3533_bl.c | 8 +-------
> drivers/video/backlight/locomolcd.c | 6 +-----
> drivers/video/backlight/lv5207lp.c | 7 +------
> drivers/video/backlight/max8925_bl.c | 13 +------------
> drivers/video/backlight/pwm_bl.c | 7 +------
> drivers/video/backlight/qcom-wled.c | 7 +------
> drivers/video/backlight/tps65217_bl.c | 10 +---------
> drivers/video/backlight/wm831x_bl.c | 13 +------------
> 19 files changed, 19 insertions(+), 150 deletions(-)
>
> diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
> index 20d96a5ac384..25e409bbb1a2 100644
> --- a/drivers/video/backlight/88pm860x_bl.c
> +++ b/drivers/video/backlight/88pm860x_bl.c
> @@ -121,18 +121,7 @@ static int pm860x_backlight_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return pm860x_backlight_set(bl, brightness);
> + return pm860x_backlight_set(bl, backlight_get_brightness(bl));
> }
>
> static int pm860x_backlight_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
> index 0f63f76723a5..686988c3df3a 100644
> --- a/drivers/video/backlight/adp5520_bl.c
> +++ b/drivers/video/backlight/adp5520_bl.c
> @@ -65,15 +65,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return adp5520_bl_set(bl, brightness);
> + return adp5520_bl_set(bl, backlight_get_brightness(bl));
> }
>
> static int adp5520_bl_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
> index 19968104fc47..ddc7f5f0401f 100644
> --- a/drivers/video/backlight/adp8860_bl.c
> +++ b/drivers/video/backlight/adp8860_bl.c
> @@ -361,15 +361,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return adp8860_bl_set(bl, brightness);
> + return adp8860_bl_set(bl, backlight_get_brightness(bl));
> }
>
> static int adp8860_bl_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
> index 4c0032010cfe..8b5213a39527 100644
> --- a/drivers/video/backlight/adp8870_bl.c
> +++ b/drivers/video/backlight/adp8870_bl.c
> @@ -399,15 +399,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return adp8870_bl_set(bl, brightness);
> + return adp8870_bl_set(bl, backlight_get_brightness(bl));
> }
>
> static int adp8870_bl_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c
> index d5d5fb457e78..515184fbe33a 100644
> --- a/drivers/video/backlight/bd6107.c
> +++ b/drivers/video/backlight/bd6107.c
> @@ -82,12 +82,7 @@ static int bd6107_write(struct bd6107 *bd, u8 reg, u8 data)
> 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))
> - brightness = 0;
> + int brightness = backlight_get_brightness(backlight);
>
> if (brightness) {
> bd6107_write(bd, BD6107_PORTSEL, BD6107_PORTSEL_LEDM(2) |
> diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
> index 25ef0cbd7583..33f5d80495e6 100644
> --- a/drivers/video/backlight/corgi_lcd.c
> +++ b/drivers/video/backlight/corgi_lcd.c
> @@ -420,13 +420,7 @@ static int corgi_bl_set_intensity(struct corgi_lcd *lcd, int intensity)
> 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)
> - intensity = 0;
> + int intensity = backlight_get_brightness(bd);
>
> if (corgibl_flags & CORGIBL_SUSPENDED)
> intensity = 0;
> diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c
> index 62540e4bdedb..71f21bbc7a9f 100644
> --- a/drivers/video/backlight/da903x_bl.c
> +++ b/drivers/video/backlight/da903x_bl.c
> @@ -77,18 +77,7 @@ static int da903x_backlight_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return da903x_backlight_set(bl, brightness);
> + return da903x_backlight_set(bl, backlight_get_brightness(bl));
> }
>
> static int da903x_backlight_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c
> index 4149e0b2f83c..2387009d452d 100644
> --- a/drivers/video/backlight/ep93xx_bl.c
> +++ b/drivers/video/backlight/ep93xx_bl.c
> @@ -36,13 +36,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return ep93xxbl_set(bl, brightness);
> + return ep93xxbl_set(bl, backlight_get_brightness(bl));
> }
>
> static int ep93xxbl_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c
> index 8ea42b8d9bc8..9123c33def05 100644
> --- a/drivers/video/backlight/hp680_bl.c
> +++ b/drivers/video/backlight/hp680_bl.c
> @@ -33,12 +33,8 @@ static void hp680bl_send_intensity(struct backlight_device *bd)
> {
> unsigned long flags;
> u16 v;
> - int intensity = bd->props.brightness;
> + int intensity = backlight_get_brightness(bd);
>
> - if (bd->props.power != FB_BLANK_UNBLANK)
> - intensity = 0;
> - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> - intensity = 0;
> if (hp680bl_suspended)
> intensity = 0;
>
> diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c
> index 1dfe13c18925..55794b239cff 100644
> --- a/drivers/video/backlight/kb3886_bl.c
> +++ b/drivers/video/backlight/kb3886_bl.c
> @@ -87,12 +87,8 @@ static const struct dmi_system_id kb3886bl_device_table[] __initconst = {
>
> static int kb3886bl_send_intensity(struct backlight_device *bd)
> {
> - int intensity = bd->props.brightness;
> + int intensity = backlight_get_brightness(bd);
>
> - if (bd->props.power != FB_BLANK_UNBLANK)
> - intensity = 0;
> - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> - 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..f54d256e2d54 100644
> --- a/drivers/video/backlight/led_bl.c
> +++ b/drivers/video/backlight/led_bl.c
> @@ -54,12 +54,7 @@ static void led_bl_power_off(struct led_bl_data *priv)
> 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)
> - brightness = 0;
> + int brightness = backlight_get_brightness(bl);
>
> if (brightness > 0)
> led_bl_set_brightness(priv, brightness);
> diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c
> index ee09d1bd02b9..d07a2bd805fc 100644
> --- a/drivers/video/backlight/lm3533_bl.c
> +++ b/drivers/video/backlight/lm3533_bl.c
> @@ -39,14 +39,8 @@ static inline int lm3533_bl_get_ctrlbank_id(struct lm3533_bl *bl)
> 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)
> - brightness = 0;
> -
> - return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness);
> + return lm3533_ctrlbank_set_brightness(&bl->cb, backlight_get_brightness(bd));
> }
>
> static int lm3533_bl_get_brightness(struct backlight_device *bd)
> diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c
> index cdc02e04f89d..297ee2e1ab0b 100644
> --- a/drivers/video/backlight/locomolcd.c
> +++ b/drivers/video/backlight/locomolcd.c
> @@ -111,12 +111,8 @@ static int current_intensity;
>
> static int locomolcd_set_intensity(struct backlight_device *bd)
> {
> - int intensity = bd->props.brightness;
> + int intensity = backlight_get_brightness(bd);
>
> - if (bd->props.power != FB_BLANK_UNBLANK)
> - intensity = 0;
> - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> - 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..1842ae9a55f8 100644
> --- a/drivers/video/backlight/lv5207lp.c
> +++ b/drivers/video/backlight/lv5207lp.c
> @@ -46,12 +46,7 @@ static int lv5207lp_write(struct lv5207lp *lv, u8 reg, u8 data)
> 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))
> - brightness = 0;
> + int brightness = backlight_get_brightness(backlight);
>
> if (brightness) {
> lv5207lp_write(lv, LV5207LP_CTRL1,
> diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c
> index 97cc260ff9d1..e607ec6fd4bf 100644
> --- a/drivers/video/backlight/max8925_bl.c
> +++ b/drivers/video/backlight/max8925_bl.c
> @@ -64,18 +64,7 @@ static int max8925_backlight_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return max8925_backlight_set(bl, brightness);
> + return max8925_backlight_set(bl, backlight_get_brightness(bl));
> }
>
> static int max8925_backlight_get_brightness(struct backlight_device *bl)
> diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
> index 82b8d7594701..eff64db2e02e 100644
> --- a/drivers/video/backlight/pwm_bl.c
> +++ b/drivers/video/backlight/pwm_bl.c
> @@ -108,14 +108,9 @@ static int compute_duty_cycle(struct pwm_bl_data *pb, int brightness)
> static int pwm_backlight_update_status(struct backlight_device *bl)
> {
> struct pwm_bl_data *pb = bl_get_data(bl);
> - int brightness = bl->props.brightness;
> + int brightness = backlight_get_brightness(bl);
> struct pwm_state state;
>
> - if (bl->props.power != FB_BLANK_UNBLANK ||
> - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> - bl->props.state & BL_CORE_FBBLANK)
> - brightness = 0;
> -
> if (pb->notify)
> brightness = pb->notify(pb->dev, brightness);
>
> diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c
> index 4c8c34b99441..94d3b96244f0 100644
> --- a/drivers/video/backlight/qcom-wled.c
> +++ b/drivers/video/backlight/qcom-wled.c
> @@ -433,14 +433,9 @@ static int wled5_ovp_delay(struct wled *wled)
> static int wled_update_status(struct backlight_device *bl)
> {
> struct wled *wled = bl_get_data(bl);
> - u16 brightness = bl->props.brightness;
> + u16 brightness = backlight_get_brightness(bl);
> int rc = 0;
>
> - if (bl->props.power != FB_BLANK_UNBLANK ||
> - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> - bl->props.state & BL_CORE_FBBLANK)
> - brightness = 0;
> -
> mutex_lock(&wled->lock);
> if (brightness) {
> rc = wled->wled_set_brightness(wled, brightness);
> diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c
> index 762e3feed097..8457166f357f 100644
> --- a/drivers/video/backlight/tps65217_bl.c
> +++ b/drivers/video/backlight/tps65217_bl.c
> @@ -77,15 +77,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl)
> {
> struct tps65217_bl *tps65217_bl = bl_get_data(bl);
> int rc;
> - int brightness = bl->props.brightness;
> -
> - 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 */
> - brightness = 0;
> + int brightness = backlight_get_brightness(bl);
>
> if (brightness > 0) {
> rc = tps65217_reg_write(tps65217_bl->tps,
> diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
> index e55977d54c15..c5aaee205bdf 100644
> --- a/drivers/video/backlight/wm831x_bl.c
> +++ b/drivers/video/backlight/wm831x_bl.c
> @@ -91,18 +91,7 @@ static int wm831x_backlight_set(struct backlight_device *bl, int brightness)
>
> 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)
> - brightness = 0;
> -
> - return wm831x_backlight_set(bl, brightness);
> + return wm831x_backlight_set(bl, backlight_get_brightness(bl));
> }
>
> static int wm831x_backlight_get_brightness(struct backlight_device *bl)
> --
> 2.25.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 39+ messages in thread* Re: [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-07 5:48 ` Daniel Vetter
@ 2020-07-07 19:59 ` Sam Ravnborg
2020-07-07 20:15 ` Daniel Vetter
0 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-07 19:59 UTC (permalink / raw)
To: Daniel Vetter
Cc: David Airlie, Daniel Vetter, dri-devel, Bjorn Andersson,
Peter Ujfalusi, Thierry Reding, Lee Jones, Daniel Thompson,
Jonathan Corbet, Tomi Valkeinen, Andy Gross, Uwe Kleine-Konig,
linux-pwm, Michael Hennerich, Bartlomiej Zolnierkiewicz,
linux-arm-msm, Support Opensource, Jingoo Han, Emil Velikov
On Tue, Jul 07, 2020 at 07:48:20AM +0200, Daniel Vetter wrote:
> On Fri, Jul 03, 2020 at 08:45:43PM +0200, Sam Ravnborg wrote:
> > Introduce the backlight_get_brightness() helper in all
> > video/backlight/* drivers.
> > This simplifies the code and align the implementation of the
> > update_status() operation across the different backlight drivers.
> >
> > Some of the drivers gains a little extra functionality by the change
> > as they now respect the fb_blank() ioctl.
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
>
> This looks really nice!
>
> Since there's a pile more backlight drivers all over the tree, care to
> update the gpu/todo.rst entry with a note that conversions should use
> backlight_get_brightness?
When I stop doing backlight stuff I will compose a proper
todo entry. Right now I hope to get first batch included in upstream.
I hope this set can hit upstream.
Next it to convert backlight stuff in gpu/drm/ - let me see how that
all turns out. One idea is to support embedding struct backlight_device,
but I am not sure if this really will fly.
Sam
> -Daniel
>
> > ---
> > drivers/video/backlight/88pm860x_bl.c | 13 +------------
> > drivers/video/backlight/adp5520_bl.c | 10 +---------
> > drivers/video/backlight/adp8860_bl.c | 10 +---------
> > drivers/video/backlight/adp8870_bl.c | 10 +---------
> > drivers/video/backlight/bd6107.c | 7 +------
> > drivers/video/backlight/corgi_lcd.c | 8 +-------
> > drivers/video/backlight/da903x_bl.c | 13 +------------
> > drivers/video/backlight/ep93xx_bl.c | 8 +-------
> > drivers/video/backlight/hp680_bl.c | 6 +-----
> > drivers/video/backlight/kb3886_bl.c | 6 +-----
> > drivers/video/backlight/led_bl.c | 7 +------
> > drivers/video/backlight/lm3533_bl.c | 8 +-------
> > drivers/video/backlight/locomolcd.c | 6 +-----
> > drivers/video/backlight/lv5207lp.c | 7 +------
> > drivers/video/backlight/max8925_bl.c | 13 +------------
> > drivers/video/backlight/pwm_bl.c | 7 +------
> > drivers/video/backlight/qcom-wled.c | 7 +------
> > drivers/video/backlight/tps65217_bl.c | 10 +---------
> > drivers/video/backlight/wm831x_bl.c | 13 +------------
> > 19 files changed, 19 insertions(+), 150 deletions(-)
> >
> > diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
> > index 20d96a5ac384..25e409bbb1a2 100644
> > --- a/drivers/video/backlight/88pm860x_bl.c
> > +++ b/drivers/video/backlight/88pm860x_bl.c
> > @@ -121,18 +121,7 @@ static int pm860x_backlight_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return pm860x_backlight_set(bl, brightness);
> > + return pm860x_backlight_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int pm860x_backlight_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
> > index 0f63f76723a5..686988c3df3a 100644
> > --- a/drivers/video/backlight/adp5520_bl.c
> > +++ b/drivers/video/backlight/adp5520_bl.c
> > @@ -65,15 +65,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return adp5520_bl_set(bl, brightness);
> > + return adp5520_bl_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int adp5520_bl_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
> > index 19968104fc47..ddc7f5f0401f 100644
> > --- a/drivers/video/backlight/adp8860_bl.c
> > +++ b/drivers/video/backlight/adp8860_bl.c
> > @@ -361,15 +361,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return adp8860_bl_set(bl, brightness);
> > + return adp8860_bl_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int adp8860_bl_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
> > index 4c0032010cfe..8b5213a39527 100644
> > --- a/drivers/video/backlight/adp8870_bl.c
> > +++ b/drivers/video/backlight/adp8870_bl.c
> > @@ -399,15 +399,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return adp8870_bl_set(bl, brightness);
> > + return adp8870_bl_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int adp8870_bl_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c
> > index d5d5fb457e78..515184fbe33a 100644
> > --- a/drivers/video/backlight/bd6107.c
> > +++ b/drivers/video/backlight/bd6107.c
> > @@ -82,12 +82,7 @@ static int bd6107_write(struct bd6107 *bd, u8 reg, u8 data)
> > 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))
> > - brightness = 0;
> > + int brightness = backlight_get_brightness(backlight);
> >
> > if (brightness) {
> > bd6107_write(bd, BD6107_PORTSEL, BD6107_PORTSEL_LEDM(2) |
> > diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
> > index 25ef0cbd7583..33f5d80495e6 100644
> > --- a/drivers/video/backlight/corgi_lcd.c
> > +++ b/drivers/video/backlight/corgi_lcd.c
> > @@ -420,13 +420,7 @@ static int corgi_bl_set_intensity(struct corgi_lcd *lcd, int intensity)
> > 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)
> > - intensity = 0;
> > + int intensity = backlight_get_brightness(bd);
> >
> > if (corgibl_flags & CORGIBL_SUSPENDED)
> > intensity = 0;
> > diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c
> > index 62540e4bdedb..71f21bbc7a9f 100644
> > --- a/drivers/video/backlight/da903x_bl.c
> > +++ b/drivers/video/backlight/da903x_bl.c
> > @@ -77,18 +77,7 @@ static int da903x_backlight_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return da903x_backlight_set(bl, brightness);
> > + return da903x_backlight_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int da903x_backlight_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c
> > index 4149e0b2f83c..2387009d452d 100644
> > --- a/drivers/video/backlight/ep93xx_bl.c
> > +++ b/drivers/video/backlight/ep93xx_bl.c
> > @@ -36,13 +36,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return ep93xxbl_set(bl, brightness);
> > + return ep93xxbl_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int ep93xxbl_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c
> > index 8ea42b8d9bc8..9123c33def05 100644
> > --- a/drivers/video/backlight/hp680_bl.c
> > +++ b/drivers/video/backlight/hp680_bl.c
> > @@ -33,12 +33,8 @@ static void hp680bl_send_intensity(struct backlight_device *bd)
> > {
> > unsigned long flags;
> > u16 v;
> > - int intensity = bd->props.brightness;
> > + int intensity = backlight_get_brightness(bd);
> >
> > - if (bd->props.power != FB_BLANK_UNBLANK)
> > - intensity = 0;
> > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > - intensity = 0;
> > if (hp680bl_suspended)
> > intensity = 0;
> >
> > diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c
> > index 1dfe13c18925..55794b239cff 100644
> > --- a/drivers/video/backlight/kb3886_bl.c
> > +++ b/drivers/video/backlight/kb3886_bl.c
> > @@ -87,12 +87,8 @@ static const struct dmi_system_id kb3886bl_device_table[] __initconst = {
> >
> > static int kb3886bl_send_intensity(struct backlight_device *bd)
> > {
> > - int intensity = bd->props.brightness;
> > + int intensity = backlight_get_brightness(bd);
> >
> > - if (bd->props.power != FB_BLANK_UNBLANK)
> > - intensity = 0;
> > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > - 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..f54d256e2d54 100644
> > --- a/drivers/video/backlight/led_bl.c
> > +++ b/drivers/video/backlight/led_bl.c
> > @@ -54,12 +54,7 @@ static void led_bl_power_off(struct led_bl_data *priv)
> > 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)
> > - brightness = 0;
> > + int brightness = backlight_get_brightness(bl);
> >
> > if (brightness > 0)
> > led_bl_set_brightness(priv, brightness);
> > diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c
> > index ee09d1bd02b9..d07a2bd805fc 100644
> > --- a/drivers/video/backlight/lm3533_bl.c
> > +++ b/drivers/video/backlight/lm3533_bl.c
> > @@ -39,14 +39,8 @@ static inline int lm3533_bl_get_ctrlbank_id(struct lm3533_bl *bl)
> > 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)
> > - brightness = 0;
> > -
> > - return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness);
> > + return lm3533_ctrlbank_set_brightness(&bl->cb, backlight_get_brightness(bd));
> > }
> >
> > static int lm3533_bl_get_brightness(struct backlight_device *bd)
> > diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c
> > index cdc02e04f89d..297ee2e1ab0b 100644
> > --- a/drivers/video/backlight/locomolcd.c
> > +++ b/drivers/video/backlight/locomolcd.c
> > @@ -111,12 +111,8 @@ static int current_intensity;
> >
> > static int locomolcd_set_intensity(struct backlight_device *bd)
> > {
> > - int intensity = bd->props.brightness;
> > + int intensity = backlight_get_brightness(bd);
> >
> > - if (bd->props.power != FB_BLANK_UNBLANK)
> > - intensity = 0;
> > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > - 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..1842ae9a55f8 100644
> > --- a/drivers/video/backlight/lv5207lp.c
> > +++ b/drivers/video/backlight/lv5207lp.c
> > @@ -46,12 +46,7 @@ static int lv5207lp_write(struct lv5207lp *lv, u8 reg, u8 data)
> > 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))
> > - brightness = 0;
> > + int brightness = backlight_get_brightness(backlight);
> >
> > if (brightness) {
> > lv5207lp_write(lv, LV5207LP_CTRL1,
> > diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c
> > index 97cc260ff9d1..e607ec6fd4bf 100644
> > --- a/drivers/video/backlight/max8925_bl.c
> > +++ b/drivers/video/backlight/max8925_bl.c
> > @@ -64,18 +64,7 @@ static int max8925_backlight_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return max8925_backlight_set(bl, brightness);
> > + return max8925_backlight_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int max8925_backlight_get_brightness(struct backlight_device *bl)
> > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
> > index 82b8d7594701..eff64db2e02e 100644
> > --- a/drivers/video/backlight/pwm_bl.c
> > +++ b/drivers/video/backlight/pwm_bl.c
> > @@ -108,14 +108,9 @@ static int compute_duty_cycle(struct pwm_bl_data *pb, int brightness)
> > static int pwm_backlight_update_status(struct backlight_device *bl)
> > {
> > struct pwm_bl_data *pb = bl_get_data(bl);
> > - int brightness = bl->props.brightness;
> > + int brightness = backlight_get_brightness(bl);
> > struct pwm_state state;
> >
> > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > - bl->props.state & BL_CORE_FBBLANK)
> > - brightness = 0;
> > -
> > if (pb->notify)
> > brightness = pb->notify(pb->dev, brightness);
> >
> > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c
> > index 4c8c34b99441..94d3b96244f0 100644
> > --- a/drivers/video/backlight/qcom-wled.c
> > +++ b/drivers/video/backlight/qcom-wled.c
> > @@ -433,14 +433,9 @@ static int wled5_ovp_delay(struct wled *wled)
> > static int wled_update_status(struct backlight_device *bl)
> > {
> > struct wled *wled = bl_get_data(bl);
> > - u16 brightness = bl->props.brightness;
> > + u16 brightness = backlight_get_brightness(bl);
> > int rc = 0;
> >
> > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > - bl->props.state & BL_CORE_FBBLANK)
> > - brightness = 0;
> > -
> > mutex_lock(&wled->lock);
> > if (brightness) {
> > rc = wled->wled_set_brightness(wled, brightness);
> > diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c
> > index 762e3feed097..8457166f357f 100644
> > --- a/drivers/video/backlight/tps65217_bl.c
> > +++ b/drivers/video/backlight/tps65217_bl.c
> > @@ -77,15 +77,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl)
> > {
> > struct tps65217_bl *tps65217_bl = bl_get_data(bl);
> > int rc;
> > - int brightness = bl->props.brightness;
> > -
> > - 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 */
> > - brightness = 0;
> > + int brightness = backlight_get_brightness(bl);
> >
> > if (brightness > 0) {
> > rc = tps65217_reg_write(tps65217_bl->tps,
> > diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
> > index e55977d54c15..c5aaee205bdf 100644
> > --- a/drivers/video/backlight/wm831x_bl.c
> > +++ b/drivers/video/backlight/wm831x_bl.c
> > @@ -91,18 +91,7 @@ static int wm831x_backlight_set(struct backlight_device *bl, int brightness)
> >
> > 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)
> > - brightness = 0;
> > -
> > - return wm831x_backlight_set(bl, brightness);
> > + return wm831x_backlight_set(bl, backlight_get_brightness(bl));
> > }
> >
> > static int wm831x_backlight_get_brightness(struct backlight_device *bl)
> > --
> > 2.25.1
> >
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 39+ messages in thread* Re: [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-07 19:59 ` Sam Ravnborg
@ 2020-07-07 20:15 ` Daniel Vetter
2020-07-07 20:23 ` Sam Ravnborg
0 siblings, 1 reply; 39+ messages in thread
From: Daniel Vetter @ 2020-07-07 20:15 UTC (permalink / raw)
To: Sam Ravnborg
Cc: David Airlie, dri-devel, Bjorn Andersson, Peter Ujfalusi,
Thierry Reding, Lee Jones, Daniel Thompson, Jonathan Corbet,
Tomi Valkeinen, Andy Gross, Uwe Kleine-Konig, linux-pwm,
Michael Hennerich, Bartlomiej Zolnierkiewicz, linux-arm-msm,
Support Opensource, Jingoo Han, Emil Velikov
On Tue, Jul 7, 2020 at 9:59 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>
> On Tue, Jul 07, 2020 at 07:48:20AM +0200, Daniel Vetter wrote:
> > On Fri, Jul 03, 2020 at 08:45:43PM +0200, Sam Ravnborg wrote:
> > > Introduce the backlight_get_brightness() helper in all
> > > video/backlight/* drivers.
> > > This simplifies the code and align the implementation of the
> > > update_status() operation across the different backlight drivers.
> > >
> > > Some of the drivers gains a little extra functionality by the change
> > > as they now respect the fb_blank() ioctl.
> > >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> >
> > This looks really nice!
> >
> > Since there's a pile more backlight drivers all over the tree, care to
> > update the gpu/todo.rst entry with a note that conversions should use
> > backlight_get_brightness?
>
> When I stop doing backlight stuff I will compose a proper
> todo entry. Right now I hope to get first batch included in upstream.
Sounds all good to me.
> I hope this set can hit upstream.
> Next it to convert backlight stuff in gpu/drm/ - let me see how that
> all turns out. One idea is to support embedding struct backlight_device,
> but I am not sure if this really will fly.
Embedding means you tie the lifetime of the backlight device to the
lifetime of whatever structure it's embedded in. This might be a
complete pain, but it might also work out since backlight
register/unregistering is fairly simple. But I'd be vary, maybe before
we do that we'll need stuff like device_link between backlight and
drm_device (to make sure the drm_device never outlives the backlight
at a device level), and maybe a few other tricks.
Does sound like a lot of head-scratcher fun though to work through all this :-)
Cheers, Daniel
>
> Sam
>
> > -Daniel
> >
> > > ---
> > > drivers/video/backlight/88pm860x_bl.c | 13 +------------
> > > drivers/video/backlight/adp5520_bl.c | 10 +---------
> > > drivers/video/backlight/adp8860_bl.c | 10 +---------
> > > drivers/video/backlight/adp8870_bl.c | 10 +---------
> > > drivers/video/backlight/bd6107.c | 7 +------
> > > drivers/video/backlight/corgi_lcd.c | 8 +-------
> > > drivers/video/backlight/da903x_bl.c | 13 +------------
> > > drivers/video/backlight/ep93xx_bl.c | 8 +-------
> > > drivers/video/backlight/hp680_bl.c | 6 +-----
> > > drivers/video/backlight/kb3886_bl.c | 6 +-----
> > > drivers/video/backlight/led_bl.c | 7 +------
> > > drivers/video/backlight/lm3533_bl.c | 8 +-------
> > > drivers/video/backlight/locomolcd.c | 6 +-----
> > > drivers/video/backlight/lv5207lp.c | 7 +------
> > > drivers/video/backlight/max8925_bl.c | 13 +------------
> > > drivers/video/backlight/pwm_bl.c | 7 +------
> > > drivers/video/backlight/qcom-wled.c | 7 +------
> > > drivers/video/backlight/tps65217_bl.c | 10 +---------
> > > drivers/video/backlight/wm831x_bl.c | 13 +------------
> > > 19 files changed, 19 insertions(+), 150 deletions(-)
> > >
> > > diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
> > > index 20d96a5ac384..25e409bbb1a2 100644
> > > --- a/drivers/video/backlight/88pm860x_bl.c
> > > +++ b/drivers/video/backlight/88pm860x_bl.c
> > > @@ -121,18 +121,7 @@ static int pm860x_backlight_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return pm860x_backlight_set(bl, brightness);
> > > + return pm860x_backlight_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int pm860x_backlight_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
> > > index 0f63f76723a5..686988c3df3a 100644
> > > --- a/drivers/video/backlight/adp5520_bl.c
> > > +++ b/drivers/video/backlight/adp5520_bl.c
> > > @@ -65,15 +65,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return adp5520_bl_set(bl, brightness);
> > > + return adp5520_bl_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int adp5520_bl_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
> > > index 19968104fc47..ddc7f5f0401f 100644
> > > --- a/drivers/video/backlight/adp8860_bl.c
> > > +++ b/drivers/video/backlight/adp8860_bl.c
> > > @@ -361,15 +361,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return adp8860_bl_set(bl, brightness);
> > > + return adp8860_bl_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int adp8860_bl_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
> > > index 4c0032010cfe..8b5213a39527 100644
> > > --- a/drivers/video/backlight/adp8870_bl.c
> > > +++ b/drivers/video/backlight/adp8870_bl.c
> > > @@ -399,15 +399,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return adp8870_bl_set(bl, brightness);
> > > + return adp8870_bl_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int adp8870_bl_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c
> > > index d5d5fb457e78..515184fbe33a 100644
> > > --- a/drivers/video/backlight/bd6107.c
> > > +++ b/drivers/video/backlight/bd6107.c
> > > @@ -82,12 +82,7 @@ static int bd6107_write(struct bd6107 *bd, u8 reg, u8 data)
> > > 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))
> > > - brightness = 0;
> > > + int brightness = backlight_get_brightness(backlight);
> > >
> > > if (brightness) {
> > > bd6107_write(bd, BD6107_PORTSEL, BD6107_PORTSEL_LEDM(2) |
> > > diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
> > > index 25ef0cbd7583..33f5d80495e6 100644
> > > --- a/drivers/video/backlight/corgi_lcd.c
> > > +++ b/drivers/video/backlight/corgi_lcd.c
> > > @@ -420,13 +420,7 @@ static int corgi_bl_set_intensity(struct corgi_lcd *lcd, int intensity)
> > > 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)
> > > - intensity = 0;
> > > + int intensity = backlight_get_brightness(bd);
> > >
> > > if (corgibl_flags & CORGIBL_SUSPENDED)
> > > intensity = 0;
> > > diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c
> > > index 62540e4bdedb..71f21bbc7a9f 100644
> > > --- a/drivers/video/backlight/da903x_bl.c
> > > +++ b/drivers/video/backlight/da903x_bl.c
> > > @@ -77,18 +77,7 @@ static int da903x_backlight_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return da903x_backlight_set(bl, brightness);
> > > + return da903x_backlight_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int da903x_backlight_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c
> > > index 4149e0b2f83c..2387009d452d 100644
> > > --- a/drivers/video/backlight/ep93xx_bl.c
> > > +++ b/drivers/video/backlight/ep93xx_bl.c
> > > @@ -36,13 +36,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return ep93xxbl_set(bl, brightness);
> > > + return ep93xxbl_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int ep93xxbl_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c
> > > index 8ea42b8d9bc8..9123c33def05 100644
> > > --- a/drivers/video/backlight/hp680_bl.c
> > > +++ b/drivers/video/backlight/hp680_bl.c
> > > @@ -33,12 +33,8 @@ static void hp680bl_send_intensity(struct backlight_device *bd)
> > > {
> > > unsigned long flags;
> > > u16 v;
> > > - int intensity = bd->props.brightness;
> > > + int intensity = backlight_get_brightness(bd);
> > >
> > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > - intensity = 0;
> > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > - intensity = 0;
> > > if (hp680bl_suspended)
> > > intensity = 0;
> > >
> > > diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c
> > > index 1dfe13c18925..55794b239cff 100644
> > > --- a/drivers/video/backlight/kb3886_bl.c
> > > +++ b/drivers/video/backlight/kb3886_bl.c
> > > @@ -87,12 +87,8 @@ static const struct dmi_system_id kb3886bl_device_table[] __initconst = {
> > >
> > > static int kb3886bl_send_intensity(struct backlight_device *bd)
> > > {
> > > - int intensity = bd->props.brightness;
> > > + int intensity = backlight_get_brightness(bd);
> > >
> > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > - intensity = 0;
> > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > - 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..f54d256e2d54 100644
> > > --- a/drivers/video/backlight/led_bl.c
> > > +++ b/drivers/video/backlight/led_bl.c
> > > @@ -54,12 +54,7 @@ static void led_bl_power_off(struct led_bl_data *priv)
> > > 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)
> > > - brightness = 0;
> > > + int brightness = backlight_get_brightness(bl);
> > >
> > > if (brightness > 0)
> > > led_bl_set_brightness(priv, brightness);
> > > diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c
> > > index ee09d1bd02b9..d07a2bd805fc 100644
> > > --- a/drivers/video/backlight/lm3533_bl.c
> > > +++ b/drivers/video/backlight/lm3533_bl.c
> > > @@ -39,14 +39,8 @@ static inline int lm3533_bl_get_ctrlbank_id(struct lm3533_bl *bl)
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness);
> > > + return lm3533_ctrlbank_set_brightness(&bl->cb, backlight_get_brightness(bd));
> > > }
> > >
> > > static int lm3533_bl_get_brightness(struct backlight_device *bd)
> > > diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c
> > > index cdc02e04f89d..297ee2e1ab0b 100644
> > > --- a/drivers/video/backlight/locomolcd.c
> > > +++ b/drivers/video/backlight/locomolcd.c
> > > @@ -111,12 +111,8 @@ static int current_intensity;
> > >
> > > static int locomolcd_set_intensity(struct backlight_device *bd)
> > > {
> > > - int intensity = bd->props.brightness;
> > > + int intensity = backlight_get_brightness(bd);
> > >
> > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > - intensity = 0;
> > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > - 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..1842ae9a55f8 100644
> > > --- a/drivers/video/backlight/lv5207lp.c
> > > +++ b/drivers/video/backlight/lv5207lp.c
> > > @@ -46,12 +46,7 @@ static int lv5207lp_write(struct lv5207lp *lv, u8 reg, u8 data)
> > > 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))
> > > - brightness = 0;
> > > + int brightness = backlight_get_brightness(backlight);
> > >
> > > if (brightness) {
> > > lv5207lp_write(lv, LV5207LP_CTRL1,
> > > diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c
> > > index 97cc260ff9d1..e607ec6fd4bf 100644
> > > --- a/drivers/video/backlight/max8925_bl.c
> > > +++ b/drivers/video/backlight/max8925_bl.c
> > > @@ -64,18 +64,7 @@ static int max8925_backlight_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return max8925_backlight_set(bl, brightness);
> > > + return max8925_backlight_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int max8925_backlight_get_brightness(struct backlight_device *bl)
> > > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
> > > index 82b8d7594701..eff64db2e02e 100644
> > > --- a/drivers/video/backlight/pwm_bl.c
> > > +++ b/drivers/video/backlight/pwm_bl.c
> > > @@ -108,14 +108,9 @@ static int compute_duty_cycle(struct pwm_bl_data *pb, int brightness)
> > > static int pwm_backlight_update_status(struct backlight_device *bl)
> > > {
> > > struct pwm_bl_data *pb = bl_get_data(bl);
> > > - int brightness = bl->props.brightness;
> > > + int brightness = backlight_get_brightness(bl);
> > > struct pwm_state state;
> > >
> > > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > > - bl->props.state & BL_CORE_FBBLANK)
> > > - brightness = 0;
> > > -
> > > if (pb->notify)
> > > brightness = pb->notify(pb->dev, brightness);
> > >
> > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c
> > > index 4c8c34b99441..94d3b96244f0 100644
> > > --- a/drivers/video/backlight/qcom-wled.c
> > > +++ b/drivers/video/backlight/qcom-wled.c
> > > @@ -433,14 +433,9 @@ static int wled5_ovp_delay(struct wled *wled)
> > > static int wled_update_status(struct backlight_device *bl)
> > > {
> > > struct wled *wled = bl_get_data(bl);
> > > - u16 brightness = bl->props.brightness;
> > > + u16 brightness = backlight_get_brightness(bl);
> > > int rc = 0;
> > >
> > > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > > - bl->props.state & BL_CORE_FBBLANK)
> > > - brightness = 0;
> > > -
> > > mutex_lock(&wled->lock);
> > > if (brightness) {
> > > rc = wled->wled_set_brightness(wled, brightness);
> > > diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c
> > > index 762e3feed097..8457166f357f 100644
> > > --- a/drivers/video/backlight/tps65217_bl.c
> > > +++ b/drivers/video/backlight/tps65217_bl.c
> > > @@ -77,15 +77,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl)
> > > {
> > > struct tps65217_bl *tps65217_bl = bl_get_data(bl);
> > > int rc;
> > > - int brightness = bl->props.brightness;
> > > -
> > > - 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 */
> > > - brightness = 0;
> > > + int brightness = backlight_get_brightness(bl);
> > >
> > > if (brightness > 0) {
> > > rc = tps65217_reg_write(tps65217_bl->tps,
> > > diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
> > > index e55977d54c15..c5aaee205bdf 100644
> > > --- a/drivers/video/backlight/wm831x_bl.c
> > > +++ b/drivers/video/backlight/wm831x_bl.c
> > > @@ -91,18 +91,7 @@ static int wm831x_backlight_set(struct backlight_device *bl, int brightness)
> > >
> > > 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)
> > > - brightness = 0;
> > > -
> > > - return wm831x_backlight_set(bl, brightness);
> > > + return wm831x_backlight_set(bl, backlight_get_brightness(bl));
> > > }
> > >
> > > static int wm831x_backlight_get_brightness(struct backlight_device *bl)
> > > --
> > > 2.25.1
> > >
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 39+ messages in thread* Re: [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-07 20:15 ` Daniel Vetter
@ 2020-07-07 20:23 ` Sam Ravnborg
0 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-07 20:23 UTC (permalink / raw)
To: Daniel Vetter
Cc: linux-pwm, Daniel Thompson, Support Opensource,
Michael Hennerich, Jonathan Corbet, David Airlie, Tomi Valkeinen,
linux-arm-msm, Bartlomiej Zolnierkiewicz, Emil Velikov,
Andy Gross, dri-devel, Bjorn Andersson, Peter Ujfalusi,
Thierry Reding, Thomas Zimmermann, Uwe Kleine-Konig, Jingoo Han
Hi Daniel
> >
> > When I stop doing backlight stuff I will compose a proper
> > todo entry. Right now I hope to get first batch included in upstream.
>
> Sounds all good to me.
>
> > I hope this set can hit upstream.
> > Next it to convert backlight stuff in gpu/drm/ - let me see how that
> > all turns out. One idea is to support embedding struct backlight_device,
> > but I am not sure if this really will fly.
>
> Embedding means you tie the lifetime of the backlight device to the
> lifetime of whatever structure it's embedded in. This might be a
> complete pain, but it might also work out since backlight
> register/unregistering is fairly simple. But I'd be vary, maybe before
> we do that we'll need stuff like device_link between backlight and
> drm_device (to make sure the drm_device never outlives the backlight
> at a device level), and maybe a few other tricks.
>
> Does sound like a lot of head-scratcher fun though to work through all this :-)
If I see there are anything other than trivial lifetime issues
to consider I will not go down this route.
Thanks for the warning so I can keep an open eye on it.
Sam
>
> Cheers, Daniel
>
> >
> > Sam
> >
> > > -Daniel
> > >
> > > > ---
> > > > drivers/video/backlight/88pm860x_bl.c | 13 +------------
> > > > drivers/video/backlight/adp5520_bl.c | 10 +---------
> > > > drivers/video/backlight/adp8860_bl.c | 10 +---------
> > > > drivers/video/backlight/adp8870_bl.c | 10 +---------
> > > > drivers/video/backlight/bd6107.c | 7 +------
> > > > drivers/video/backlight/corgi_lcd.c | 8 +-------
> > > > drivers/video/backlight/da903x_bl.c | 13 +------------
> > > > drivers/video/backlight/ep93xx_bl.c | 8 +-------
> > > > drivers/video/backlight/hp680_bl.c | 6 +-----
> > > > drivers/video/backlight/kb3886_bl.c | 6 +-----
> > > > drivers/video/backlight/led_bl.c | 7 +------
> > > > drivers/video/backlight/lm3533_bl.c | 8 +-------
> > > > drivers/video/backlight/locomolcd.c | 6 +-----
> > > > drivers/video/backlight/lv5207lp.c | 7 +------
> > > > drivers/video/backlight/max8925_bl.c | 13 +------------
> > > > drivers/video/backlight/pwm_bl.c | 7 +------
> > > > drivers/video/backlight/qcom-wled.c | 7 +------
> > > > drivers/video/backlight/tps65217_bl.c | 10 +---------
> > > > drivers/video/backlight/wm831x_bl.c | 13 +------------
> > > > 19 files changed, 19 insertions(+), 150 deletions(-)
> > > >
> > > > diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
> > > > index 20d96a5ac384..25e409bbb1a2 100644
> > > > --- a/drivers/video/backlight/88pm860x_bl.c
> > > > +++ b/drivers/video/backlight/88pm860x_bl.c
> > > > @@ -121,18 +121,7 @@ static int pm860x_backlight_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return pm860x_backlight_set(bl, brightness);
> > > > + return pm860x_backlight_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int pm860x_backlight_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/adp5520_bl.c b/drivers/video/backlight/adp5520_bl.c
> > > > index 0f63f76723a5..686988c3df3a 100644
> > > > --- a/drivers/video/backlight/adp5520_bl.c
> > > > +++ b/drivers/video/backlight/adp5520_bl.c
> > > > @@ -65,15 +65,7 @@ static int adp5520_bl_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return adp5520_bl_set(bl, brightness);
> > > > + return adp5520_bl_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int adp5520_bl_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
> > > > index 19968104fc47..ddc7f5f0401f 100644
> > > > --- a/drivers/video/backlight/adp8860_bl.c
> > > > +++ b/drivers/video/backlight/adp8860_bl.c
> > > > @@ -361,15 +361,7 @@ static int adp8860_bl_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return adp8860_bl_set(bl, brightness);
> > > > + return adp8860_bl_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int adp8860_bl_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
> > > > index 4c0032010cfe..8b5213a39527 100644
> > > > --- a/drivers/video/backlight/adp8870_bl.c
> > > > +++ b/drivers/video/backlight/adp8870_bl.c
> > > > @@ -399,15 +399,7 @@ static int adp8870_bl_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return adp8870_bl_set(bl, brightness);
> > > > + return adp8870_bl_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int adp8870_bl_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/bd6107.c b/drivers/video/backlight/bd6107.c
> > > > index d5d5fb457e78..515184fbe33a 100644
> > > > --- a/drivers/video/backlight/bd6107.c
> > > > +++ b/drivers/video/backlight/bd6107.c
> > > > @@ -82,12 +82,7 @@ static int bd6107_write(struct bd6107 *bd, u8 reg, u8 data)
> > > > 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))
> > > > - brightness = 0;
> > > > + int brightness = backlight_get_brightness(backlight);
> > > >
> > > > if (brightness) {
> > > > bd6107_write(bd, BD6107_PORTSEL, BD6107_PORTSEL_LEDM(2) |
> > > > diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
> > > > index 25ef0cbd7583..33f5d80495e6 100644
> > > > --- a/drivers/video/backlight/corgi_lcd.c
> > > > +++ b/drivers/video/backlight/corgi_lcd.c
> > > > @@ -420,13 +420,7 @@ static int corgi_bl_set_intensity(struct corgi_lcd *lcd, int intensity)
> > > > 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)
> > > > - intensity = 0;
> > > > + int intensity = backlight_get_brightness(bd);
> > > >
> > > > if (corgibl_flags & CORGIBL_SUSPENDED)
> > > > intensity = 0;
> > > > diff --git a/drivers/video/backlight/da903x_bl.c b/drivers/video/backlight/da903x_bl.c
> > > > index 62540e4bdedb..71f21bbc7a9f 100644
> > > > --- a/drivers/video/backlight/da903x_bl.c
> > > > +++ b/drivers/video/backlight/da903x_bl.c
> > > > @@ -77,18 +77,7 @@ static int da903x_backlight_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return da903x_backlight_set(bl, brightness);
> > > > + return da903x_backlight_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int da903x_backlight_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/ep93xx_bl.c b/drivers/video/backlight/ep93xx_bl.c
> > > > index 4149e0b2f83c..2387009d452d 100644
> > > > --- a/drivers/video/backlight/ep93xx_bl.c
> > > > +++ b/drivers/video/backlight/ep93xx_bl.c
> > > > @@ -36,13 +36,7 @@ static int ep93xxbl_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return ep93xxbl_set(bl, brightness);
> > > > + return ep93xxbl_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int ep93xxbl_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/hp680_bl.c b/drivers/video/backlight/hp680_bl.c
> > > > index 8ea42b8d9bc8..9123c33def05 100644
> > > > --- a/drivers/video/backlight/hp680_bl.c
> > > > +++ b/drivers/video/backlight/hp680_bl.c
> > > > @@ -33,12 +33,8 @@ static void hp680bl_send_intensity(struct backlight_device *bd)
> > > > {
> > > > unsigned long flags;
> > > > u16 v;
> > > > - int intensity = bd->props.brightness;
> > > > + int intensity = backlight_get_brightness(bd);
> > > >
> > > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > > - intensity = 0;
> > > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > > - intensity = 0;
> > > > if (hp680bl_suspended)
> > > > intensity = 0;
> > > >
> > > > diff --git a/drivers/video/backlight/kb3886_bl.c b/drivers/video/backlight/kb3886_bl.c
> > > > index 1dfe13c18925..55794b239cff 100644
> > > > --- a/drivers/video/backlight/kb3886_bl.c
> > > > +++ b/drivers/video/backlight/kb3886_bl.c
> > > > @@ -87,12 +87,8 @@ static const struct dmi_system_id kb3886bl_device_table[] __initconst = {
> > > >
> > > > static int kb3886bl_send_intensity(struct backlight_device *bd)
> > > > {
> > > > - int intensity = bd->props.brightness;
> > > > + int intensity = backlight_get_brightness(bd);
> > > >
> > > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > > - intensity = 0;
> > > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > > - 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..f54d256e2d54 100644
> > > > --- a/drivers/video/backlight/led_bl.c
> > > > +++ b/drivers/video/backlight/led_bl.c
> > > > @@ -54,12 +54,7 @@ static void led_bl_power_off(struct led_bl_data *priv)
> > > > 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)
> > > > - brightness = 0;
> > > > + int brightness = backlight_get_brightness(bl);
> > > >
> > > > if (brightness > 0)
> > > > led_bl_set_brightness(priv, brightness);
> > > > diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/backlight/lm3533_bl.c
> > > > index ee09d1bd02b9..d07a2bd805fc 100644
> > > > --- a/drivers/video/backlight/lm3533_bl.c
> > > > +++ b/drivers/video/backlight/lm3533_bl.c
> > > > @@ -39,14 +39,8 @@ static inline int lm3533_bl_get_ctrlbank_id(struct lm3533_bl *bl)
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return lm3533_ctrlbank_set_brightness(&bl->cb, (u8)brightness);
> > > > + return lm3533_ctrlbank_set_brightness(&bl->cb, backlight_get_brightness(bd));
> > > > }
> > > >
> > > > static int lm3533_bl_get_brightness(struct backlight_device *bd)
> > > > diff --git a/drivers/video/backlight/locomolcd.c b/drivers/video/backlight/locomolcd.c
> > > > index cdc02e04f89d..297ee2e1ab0b 100644
> > > > --- a/drivers/video/backlight/locomolcd.c
> > > > +++ b/drivers/video/backlight/locomolcd.c
> > > > @@ -111,12 +111,8 @@ static int current_intensity;
> > > >
> > > > static int locomolcd_set_intensity(struct backlight_device *bd)
> > > > {
> > > > - int intensity = bd->props.brightness;
> > > > + int intensity = backlight_get_brightness(bd);
> > > >
> > > > - if (bd->props.power != FB_BLANK_UNBLANK)
> > > > - intensity = 0;
> > > > - if (bd->props.fb_blank != FB_BLANK_UNBLANK)
> > > > - 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..1842ae9a55f8 100644
> > > > --- a/drivers/video/backlight/lv5207lp.c
> > > > +++ b/drivers/video/backlight/lv5207lp.c
> > > > @@ -46,12 +46,7 @@ static int lv5207lp_write(struct lv5207lp *lv, u8 reg, u8 data)
> > > > 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))
> > > > - brightness = 0;
> > > > + int brightness = backlight_get_brightness(backlight);
> > > >
> > > > if (brightness) {
> > > > lv5207lp_write(lv, LV5207LP_CTRL1,
> > > > diff --git a/drivers/video/backlight/max8925_bl.c b/drivers/video/backlight/max8925_bl.c
> > > > index 97cc260ff9d1..e607ec6fd4bf 100644
> > > > --- a/drivers/video/backlight/max8925_bl.c
> > > > +++ b/drivers/video/backlight/max8925_bl.c
> > > > @@ -64,18 +64,7 @@ static int max8925_backlight_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return max8925_backlight_set(bl, brightness);
> > > > + return max8925_backlight_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int max8925_backlight_get_brightness(struct backlight_device *bl)
> > > > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
> > > > index 82b8d7594701..eff64db2e02e 100644
> > > > --- a/drivers/video/backlight/pwm_bl.c
> > > > +++ b/drivers/video/backlight/pwm_bl.c
> > > > @@ -108,14 +108,9 @@ static int compute_duty_cycle(struct pwm_bl_data *pb, int brightness)
> > > > static int pwm_backlight_update_status(struct backlight_device *bl)
> > > > {
> > > > struct pwm_bl_data *pb = bl_get_data(bl);
> > > > - int brightness = bl->props.brightness;
> > > > + int brightness = backlight_get_brightness(bl);
> > > > struct pwm_state state;
> > > >
> > > > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > > > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > > > - bl->props.state & BL_CORE_FBBLANK)
> > > > - brightness = 0;
> > > > -
> > > > if (pb->notify)
> > > > brightness = pb->notify(pb->dev, brightness);
> > > >
> > > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c
> > > > index 4c8c34b99441..94d3b96244f0 100644
> > > > --- a/drivers/video/backlight/qcom-wled.c
> > > > +++ b/drivers/video/backlight/qcom-wled.c
> > > > @@ -433,14 +433,9 @@ static int wled5_ovp_delay(struct wled *wled)
> > > > static int wled_update_status(struct backlight_device *bl)
> > > > {
> > > > struct wled *wled = bl_get_data(bl);
> > > > - u16 brightness = bl->props.brightness;
> > > > + u16 brightness = backlight_get_brightness(bl);
> > > > int rc = 0;
> > > >
> > > > - if (bl->props.power != FB_BLANK_UNBLANK ||
> > > > - bl->props.fb_blank != FB_BLANK_UNBLANK ||
> > > > - bl->props.state & BL_CORE_FBBLANK)
> > > > - brightness = 0;
> > > > -
> > > > mutex_lock(&wled->lock);
> > > > if (brightness) {
> > > > rc = wled->wled_set_brightness(wled, brightness);
> > > > diff --git a/drivers/video/backlight/tps65217_bl.c b/drivers/video/backlight/tps65217_bl.c
> > > > index 762e3feed097..8457166f357f 100644
> > > > --- a/drivers/video/backlight/tps65217_bl.c
> > > > +++ b/drivers/video/backlight/tps65217_bl.c
> > > > @@ -77,15 +77,7 @@ static int tps65217_bl_update_status(struct backlight_device *bl)
> > > > {
> > > > struct tps65217_bl *tps65217_bl = bl_get_data(bl);
> > > > int rc;
> > > > - int brightness = bl->props.brightness;
> > > > -
> > > > - 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 */
> > > > - brightness = 0;
> > > > + int brightness = backlight_get_brightness(bl);
> > > >
> > > > if (brightness > 0) {
> > > > rc = tps65217_reg_write(tps65217_bl->tps,
> > > > diff --git a/drivers/video/backlight/wm831x_bl.c b/drivers/video/backlight/wm831x_bl.c
> > > > index e55977d54c15..c5aaee205bdf 100644
> > > > --- a/drivers/video/backlight/wm831x_bl.c
> > > > +++ b/drivers/video/backlight/wm831x_bl.c
> > > > @@ -91,18 +91,7 @@ static int wm831x_backlight_set(struct backlight_device *bl, int brightness)
> > > >
> > > > 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)
> > > > - brightness = 0;
> > > > -
> > > > - return wm831x_backlight_set(bl, brightness);
> > > > + return wm831x_backlight_set(bl, backlight_get_brightness(bl));
> > > > }
> > > >
> > > > static int wm831x_backlight_get_brightness(struct backlight_device *bl)
> > > > --
> > > > 2.25.1
> > > >
> > >
> > > --
> > > Daniel Vetter
> > > Software Engineer, Intel Corporation
> > > http://blog.ffwll.ch
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 39+ messages in thread
* Re: [PATCH v4 17/20] backlight: use backligt_get_brightness()
2020-07-03 18:45 ` [PATCH v4 17/20] backlight: use backligt_get_brightness() Sam Ravnborg
2020-07-07 5:48 ` Daniel Vetter
@ 2020-07-08 11:14 ` Daniel Thompson
1 sibling, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 11:14 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:43PM +0200, Sam Ravnborg wrote:
> Introduce the backlight_get_brightness() helper in all
> video/backlight/* drivers.
> This simplifies the code and align the implementation of the
> update_status() operation across the different backlight drivers.
>
> Some of the drivers gains a little extra functionality by the change
> as they now respect the fb_blank() ioctl.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 18/20] backlight: drop backlight_put()
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (16 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 17/20] backlight: use backligt_get_brightness() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 19/20] backlight: make of_find_backlight static Sam Ravnborg
2020-07-03 18:45 ` [PATCH v4 20/20] backlight: make of_find_backlight_by_node() static Sam Ravnborg
19 siblings, 0 replies; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
There are no external users of backlight_put().
Drop it and open code the two users in backlight.c.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/backlight.c | 7 +++++--
include/linux/backlight.h | 10 ----------
2 files changed, 5 insertions(+), 12 deletions(-)
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index db8717581ec5..099023771ab1 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -717,7 +717,10 @@ EXPORT_SYMBOL(of_find_backlight);
static void devm_backlight_release(void *data)
{
- backlight_put(data);
+ struct backlight_device *bd = data;
+
+ if (bd)
+ put_device(&bd->dev);
}
/**
@@ -745,7 +748,7 @@ struct backlight_device *devm_of_find_backlight(struct device *dev)
return bd;
ret = devm_add_action(dev, devm_backlight_release, bd);
if (ret) {
- backlight_put(bd);
+ put_device(&bd->dev);
return ERR_PTR(ret);
}
return bd;
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 38db67588b16..f3b484c99789 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -388,16 +388,6 @@ static inline int backlight_disable(struct backlight_device *bd)
return backlight_update_status(bd);
}
-/**
- * backlight_put - Drop backlight reference
- * @bd: the backlight device to put
- */
-static inline void backlight_put(struct backlight_device *bd)
-{
- if (bd)
- put_device(&bd->dev);
-}
-
/**
* backlight_is_blank - Return true if display is expected to be blank
* @bd: the backlight device
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* [PATCH v4 19/20] backlight: make of_find_backlight static
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (17 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 18/20] backlight: drop backlight_put() Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-08 11:17 ` Daniel Thompson
2020-07-03 18:45 ` [PATCH v4 20/20] backlight: make of_find_backlight_by_node() static Sam Ravnborg
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
There are no external users of of_find_backlight,
as they have all changed to use the managed version.
Make of_find_backlight static to prevent new external users.
v3:
- Move doc for devm_of_find_backlight out of this patch
v2:
- Editorial corrections (Daniel)
- Returns => RETURNS (Daniel)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/backlight.c | 18 +-----------------
include/linux/backlight.h | 6 ------
2 files changed, 1 insertion(+), 23 deletions(-)
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index 099023771ab1..06f4da3c58e1 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -672,22 +672,7 @@ struct backlight_device *of_find_backlight_by_node(struct device_node *node)
EXPORT_SYMBOL(of_find_backlight_by_node);
#endif
-/**
- * of_find_backlight - Get backlight device
- * @dev: Device
- *
- * This function looks for a property named 'backlight' on the DT node
- * connected to @dev and looks up the backlight device.
- *
- * Call backlight_put() to drop the reference on the backlight device.
- *
- * Returns:
- * A pointer to the backlight device if found.
- * Error pointer -EPROBE_DEFER if the DT property is set, but no backlight
- * device is found.
- * NULL if there's no backlight property.
- */
-struct backlight_device *of_find_backlight(struct device *dev)
+static struct backlight_device *of_find_backlight(struct device *dev)
{
struct backlight_device *bd = NULL;
struct device_node *np;
@@ -713,7 +698,6 @@ struct backlight_device *of_find_backlight(struct device *dev)
return bd;
}
-EXPORT_SYMBOL(of_find_backlight);
static void devm_backlight_release(void *data)
{
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index f3b484c99789..8b43fd90d84a 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -477,14 +477,8 @@ of_find_backlight_by_node(struct device_node *node)
#endif
#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
-struct backlight_device *of_find_backlight(struct device *dev);
struct backlight_device *devm_of_find_backlight(struct device *dev);
#else
-static inline struct backlight_device *of_find_backlight(struct device *dev)
-{
- return NULL;
-}
-
static inline struct backlight_device *
devm_of_find_backlight(struct device *dev)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 19/20] backlight: make of_find_backlight static
2020-07-03 18:45 ` [PATCH v4 19/20] backlight: make of_find_backlight static Sam Ravnborg
@ 2020-07-08 11:17 ` Daniel Thompson
0 siblings, 0 replies; 39+ messages in thread
From: Daniel Thompson @ 2020-07-08 11:17 UTC (permalink / raw)
To: Sam Ravnborg
Cc: dri-devel, Jingoo Han, Lee Jones, Andy Gross,
Bartlomiej Zolnierkiewicz, Bjorn Andersson, Daniel Vetter,
David Airlie, Emil Velikov, Jonathan Corbet, linux-arm-msm,
linux-pwm, Maarten Lankhorst, Maxime Ripard, Michael Hennerich,
patches, Support Opensource, Thierry Reding, Thomas Zimmermann,
Uwe
On Fri, Jul 03, 2020 at 08:45:45PM +0200, Sam Ravnborg wrote:
> There are no external users of of_find_backlight,
> as they have all changed to use the managed version.
> Make of_find_backlight static to prevent new external users.
>
> v3:
> - Move doc for devm_of_find_backlight out of this patch
>
> v2:
> - Editorial corrections (Daniel)
> - Returns => RETURNS (Daniel)
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 39+ messages in thread
* [PATCH v4 20/20] backlight: make of_find_backlight_by_node() static
2020-07-03 18:45 [PATCH v4 0/20] backlight: backlight updates Sam Ravnborg
` (18 preceding siblings ...)
2020-07-03 18:45 ` [PATCH v4 19/20] backlight: make of_find_backlight static Sam Ravnborg
@ 2020-07-03 18:45 ` Sam Ravnborg
2020-07-07 21:46 ` kernel test robot
19 siblings, 1 reply; 39+ messages in thread
From: Sam Ravnborg @ 2020-07-03 18:45 UTC (permalink / raw)
To: dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: Andy Gross, Bartlomiej Zolnierkiewicz, Bjorn Andersson,
Daniel Vetter, David Airlie, Emil Velikov, Jonathan Corbet,
linux-arm-msm, linux-pwm, Maarten Lankhorst, Maxime Ripard,
Michael Hennerich, patches, Support Opensource, Thierry Reding,
Thomas Zimmermann, Uwe Kleine-Konig, Peter Ujfalusi,
Tomi Valkeinen
There are no external users of of_find_backlight_by_node().
Make it static so we keep it that way.
v2:
- drop EXPORT of of_find_backlight_by_node
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
---
drivers/video/backlight/backlight.c | 23 +++++++++--------------
include/linux/backlight.h | 10 ----------
2 files changed, 9 insertions(+), 24 deletions(-)
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index 06f4da3c58e1..ff84e6607486 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -649,19 +649,9 @@ static int of_parent_match(struct device *dev, const void *data)
return dev->parent && dev->parent->of_node == data;
}
-/**
- * of_find_backlight_by_node() - find backlight device by device-tree node
- * @node: device-tree node of the backlight device
- *
- * Returns a pointer to the backlight device corresponding to the given DT
- * node or NULL if no such backlight device exists or if the device hasn't
- * been probed yet.
- *
- * This function obtains a reference on the backlight device and it is the
- * caller's responsibility to drop the reference by calling put_device() on
- * the backlight device's .dev field.
- */
-struct backlight_device *of_find_backlight_by_node(struct device_node *node)
+/* Find backlight device by device-tree node */
+static struct backlight_device *
+of_find_backlight_by_node(struct device_node *node)
{
struct device *dev;
@@ -669,7 +659,12 @@ struct backlight_device *of_find_backlight_by_node(struct device_node *node)
return dev ? to_backlight_device(dev) : NULL;
}
-EXPORT_SYMBOL(of_find_backlight_by_node);
+#else
+static struct backlight_device *
+of_find_backlight_by_node(struct device_node *node)
+{
+ return NULL;
+}
#endif
static struct backlight_device *of_find_backlight(struct device *dev)
diff --git a/include/linux/backlight.h b/include/linux/backlight.h
index 8b43fd90d84a..fa443899b4ee 100644
--- a/include/linux/backlight.h
+++ b/include/linux/backlight.h
@@ -466,16 +466,6 @@ static inline void * bl_get_data(struct backlight_device *bl_dev)
return dev_get_drvdata(&bl_dev->dev);
}
-#ifdef CONFIG_OF
-struct backlight_device *of_find_backlight_by_node(struct device_node *node);
-#else
-static inline struct backlight_device *
-of_find_backlight_by_node(struct device_node *node)
-{
- return NULL;
-}
-#endif
-
#if IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)
struct backlight_device *devm_of_find_backlight(struct device *dev);
#else
--
2.25.1
^ permalink raw reply related [flat|nested] 39+ messages in thread* Re: [PATCH v4 20/20] backlight: make of_find_backlight_by_node() static
2020-07-03 18:45 ` [PATCH v4 20/20] backlight: make of_find_backlight_by_node() static Sam Ravnborg
@ 2020-07-07 21:46 ` kernel test robot
0 siblings, 0 replies; 39+ messages in thread
From: kernel test robot @ 2020-07-07 21:46 UTC (permalink / raw)
To: Sam Ravnborg, dri-devel, Jingoo Han, Lee Jones, Daniel Thompson
Cc: kbuild-all, linux-pwm, Support Opensource, Michael Hennerich,
Bartlomiej Zolnierkiewicz, David Airlie, Daniel Vetter
[-- Attachment #1: Type: text/plain, Size: 7798 bytes --]
Hi Sam,
I love your patch! Yet something to improve:
[auto build test ERROR on backlight/for-backlight-next]
[also build test ERROR on tegra-drm/drm/tegra/for-next pwm/for-next linus/master v5.8-rc4]
[cannot apply to next-20200707]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sam-Ravnborg/backlight-backlight-updates/20200704-024949
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git for-backlight-next
config: arm-randconfig-r034-20200707 (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/gpu/drm/tilcdc/tilcdc_panel.c: In function 'panel_probe':
>> drivers/gpu/drm/tilcdc/tilcdc_panel.c:320:26: error: implicit declaration of function 'of_find_backlight_by_node'; did you mean 'of_find_i2c_adapter_by_node'? [-Werror=implicit-function-declaration]
320 | panel_mod->backlight = of_find_backlight_by_node(bl_node);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
| of_find_i2c_adapter_by_node
>> drivers/gpu/drm/tilcdc/tilcdc_panel.c:320:24: error: assignment to 'struct backlight_device *' from 'int' makes pointer from integer without a cast [-Werror=int-conversion]
320 | panel_mod->backlight = of_find_backlight_by_node(bl_node);
| ^
cc1: all warnings being treated as errors
vim +320 drivers/gpu/drm/tilcdc/tilcdc_panel.c
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 299
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 300 static int panel_probe(struct platform_device *pdev)
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 301 {
18c44db8cafe9b Ezequiel Garcia 2014-09-02 302 struct device_node *bl_node, *node = pdev->dev.of_node;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 303 struct panel_module *panel_mod;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 304 struct tilcdc_module *mod;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 305 struct pinctrl *pinctrl;
12778fc14301cf Ezequiel Garcia 2014-09-02 306 int ret;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 307
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 308 /* bail out early if no DT data: */
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 309 if (!node) {
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 310 dev_err(&pdev->dev, "device-tree data is missing\n");
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 311 return -ENXIO;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 312 }
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 313
971645d1fd734b Ezequiel Garcia 2014-09-02 314 panel_mod = devm_kzalloc(&pdev->dev, sizeof(*panel_mod), GFP_KERNEL);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 315 if (!panel_mod)
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 316 return -ENOMEM;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 317
18c44db8cafe9b Ezequiel Garcia 2014-09-02 318 bl_node = of_parse_phandle(node, "backlight", 0);
18c44db8cafe9b Ezequiel Garcia 2014-09-02 319 if (bl_node) {
18c44db8cafe9b Ezequiel Garcia 2014-09-02 @320 panel_mod->backlight = of_find_backlight_by_node(bl_node);
18c44db8cafe9b Ezequiel Garcia 2014-09-02 321 of_node_put(bl_node);
18c44db8cafe9b Ezequiel Garcia 2014-09-02 322
18c44db8cafe9b Ezequiel Garcia 2014-09-02 323 if (!panel_mod->backlight)
18c44db8cafe9b Ezequiel Garcia 2014-09-02 324 return -EPROBE_DEFER;
18c44db8cafe9b Ezequiel Garcia 2014-09-02 325
18c44db8cafe9b Ezequiel Garcia 2014-09-02 326 dev_info(&pdev->dev, "found backlight\n");
18c44db8cafe9b Ezequiel Garcia 2014-09-02 327 }
18c44db8cafe9b Ezequiel Garcia 2014-09-02 328
26a5bd26499fba Uwe Kleine-König 2015-02-11 329 panel_mod->enable_gpio = devm_gpiod_get_optional(&pdev->dev, "enable",
26a5bd26499fba Uwe Kleine-König 2015-02-11 330 GPIOD_OUT_LOW);
d898ce03675fc0 Ezequiel Garcia 2014-09-02 331 if (IS_ERR(panel_mod->enable_gpio)) {
d898ce03675fc0 Ezequiel Garcia 2014-09-02 332 ret = PTR_ERR(panel_mod->enable_gpio);
d898ce03675fc0 Ezequiel Garcia 2014-09-02 333 dev_err(&pdev->dev, "failed to request enable GPIO\n");
d898ce03675fc0 Ezequiel Garcia 2014-09-02 334 goto fail_backlight;
d898ce03675fc0 Ezequiel Garcia 2014-09-02 335 }
d898ce03675fc0 Ezequiel Garcia 2014-09-02 336
26a5bd26499fba Uwe Kleine-König 2015-02-11 337 if (panel_mod->enable_gpio)
d898ce03675fc0 Ezequiel Garcia 2014-09-02 338 dev_info(&pdev->dev, "found enable GPIO\n");
d898ce03675fc0 Ezequiel Garcia 2014-09-02 339
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 340 mod = &panel_mod->base;
7cdcce9f8b4c15 Guido Martínez 2014-06-17 341 pdev->dev.platform_data = mod;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 342
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 343 tilcdc_module_init(mod, "panel", &panel_module_ops);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 344
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 345 pinctrl = devm_pinctrl_get_select_default(&pdev->dev);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 346 if (IS_ERR(pinctrl))
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 347 dev_warn(&pdev->dev, "pins are not configured\n");
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 348
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 349 panel_mod->timings = of_get_display_timings(node);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 350 if (!panel_mod->timings) {
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 351 dev_err(&pdev->dev, "could not get panel timings\n");
12778fc14301cf Ezequiel Garcia 2014-09-02 352 ret = -EINVAL;
7cdcce9f8b4c15 Guido Martínez 2014-06-17 353 goto fail_free;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 354 }
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 355
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 356 panel_mod->info = of_get_panel_info(node);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 357 if (!panel_mod->info) {
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 358 dev_err(&pdev->dev, "could not get panel info\n");
12778fc14301cf Ezequiel Garcia 2014-09-02 359 ret = -EINVAL;
7cdcce9f8b4c15 Guido Martínez 2014-06-17 360 goto fail_timings;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 361 }
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 362
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 363 return 0;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 364
7cdcce9f8b4c15 Guido Martínez 2014-06-17 365 fail_timings:
7cdcce9f8b4c15 Guido Martínez 2014-06-17 366 display_timings_release(panel_mod->timings);
7cdcce9f8b4c15 Guido Martínez 2014-06-17 367
7cdcce9f8b4c15 Guido Martínez 2014-06-17 368 fail_free:
7cdcce9f8b4c15 Guido Martínez 2014-06-17 369 tilcdc_module_cleanup(mod);
d898ce03675fc0 Ezequiel Garcia 2014-09-02 370
d898ce03675fc0 Ezequiel Garcia 2014-09-02 371 fail_backlight:
18c44db8cafe9b Ezequiel Garcia 2014-09-02 372 if (panel_mod->backlight)
18c44db8cafe9b Ezequiel Garcia 2014-09-02 373 put_device(&panel_mod->backlight->dev);
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 374 return ret;
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 375 }
0d4bbaf9f3e5b9 Rob Clark 2012-12-18 376
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 30913 bytes --]
^ permalink raw reply [flat|nested] 39+ messages in thread