linux.git/drivers/video/backlight/backlight.c, branch v3.11

backlight: convert from legacy pm ops to dev_pm_ops

2013-07-03T23:07:43+00:00

Convert drivers/video/backlight/class to use dev_pm_ops for power
management and remove Legacy PM ops hooks.

With this change, backlight class registers suspend/resume callbacks via
class->pm (dev_pm_ops) instead of Legacy class->suspend/resume.  When
__device_suspend() runs call-backs, it will find class->pm ops for the
backlight class.

[jg1.han@samsung.com: add CONFIG_PM_SLEEP to suspend/resume functions]
Signed-off-by: Shuah Khan 
Signed-off-by: Jingoo Han 
Cc: Shuah Khan 
Cc: Jingoo Han 
Cc: Shuah Khan 
Cc: Arnd Bergmann 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: add devm_backlight_device_{register,unregister}()

2013-07-03T23:07:43+00:00

These functions allow the driver core to automatically clean up any
allocation made by backlight drivers.  Thus it simplifies the error
paths.

Signed-off-by: Jingoo Han 
Cc: Tejun Heo 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

drivers: avoid format string in dev_set_name

2013-07-03T23:07:41+00:00

Calling dev_set_name with a single paramter causes it to be handled as a
format string.  Many callers are passing potentially dynamic string
content, so use "%s" in those cases to avoid any potential accidents,
including wrappers like device_create*() and bdi_register().

Signed-off-by: Kees Cook 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: fix class_find_device() arguments

2013-02-07T00:43:02+00:00

Michał's previous patch missed this backlight check to fix up the
class_find_device() arguments.

Reported-by: kbuild test robot 
Cc: Michał Mirosław 
Signed-off-by: Greg Kroah-Hartman

backlight: add of_find_backlight_by_node()

2012-12-18T01:15:16+00:00

This function finds the struct backlight_device for a given device tree
node.  A dummy function is provided so that it safely compiles out if OF
support is disabled.

[akpm@linux-foundation.org: Don't use IS_ENABLED(CONFIG_OF)]
Signed-off-by: Thierry Reding 
Acked-by: Jingoo Han 
Reviewed-by: Grant Likely 
Cc: Thierry Reding 
Reviewed-by: Grant Likely 
Acked-by: Jingoo Han 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: use pr_warn() and pr_debug() instead of printk()

2012-05-29T23:22:29+00:00

Use pr_warn() and pr_debug() instead of printk to allow dynamic debugging.
The pr_fmt prefix for pr_ macros is used.  Also fix checkpatch warnings
as below:

WARNING: Prefer pr_warn(... to printk(KERN_WARNING, ...

[akpm@linux-foundation.org: use KBUILD_MODNAME, per Joe]
Signed-off-by: Jingoo Han 
Cc: Richard Purdie 
Cc: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: use kstrtoul()

2012-01-11T00:30:48+00:00

The usage of simple_strtoul() or strict_strtoul() is not preferred.  Thus,
kstrtoul should be used.

This patch also fixes checkpatch error as follows:
ERROR: space required after that ',' (ctx:VxV)

Signed-off-by: Jingoo Han 
Cc: Richard Purdie 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: Declare backlight_types[] const

2011-09-10T21:00:02+00:00

Since backlight_types[] isn't modified, let's declare it const.  That
was probably the intention of the author of commit bb7ca747f8d6
("backlight: add backlight type"), via which the "const char const *"
construct was introduced.  The duplicate const was detected by sparse.

Signed-off-by: Bart Van Assche 
Cc: Matthew Garrett 
Cc: Richard Purdie 
Cc: Florian Tobias Schandinat 
Cc: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: add backlight type

2011-03-23T00:43:59+00:00

There may be multiple ways of controlling the backlight on a given
machine.  Allow drivers to expose the type of interface they are
providing, making it possible for userspace to make appropriate policy
decisions.

Signed-off-by: Matthew Garrett 
Cc: Richard Purdie 
Cc: Chris Wilson 
Cc: David Airlie 
Cc: Alex Deucher 
Cc: Ben Skeggs 
Cc: Zhang Rui 
Cc: Len Brown 
Cc: Jesse Barnes 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: grab ops_lock before testing bd->ops

2010-11-24T21:50:47+00:00

According to the comment describing ops_lock in the definition of struct
backlight_device and when comparing with other functions in backlight.c
the mutex must be hold when checking ops to be non-NULL.

Fixes a problem added by c835ee7f4154992e6 ("backlight: Add suspend/resume
support to the backlight core") in Jan 2009.

Signed-off-by: Uwe Kleine-König 
Acked-by: Richard Purdie 
Cc: 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds