linux-stable.git/drivers/video/backlight, branch linux-2.6.33.y

backlight: grab ops_lock before testing bd->ops

2011-03-21T19:44:07+00:00

commit d1d73578e053b981c3611e5a211534290d24a5eb upstream.

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 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds 
Signed-off-by: Greg Kroah-Hartman

backlight: mbp_nvidia_bl - add five more MacBook variants

2010-04-26T14:47:51+00:00

commit 36bc5ee6a8d13333980fa54e97d3469d3d4cda98 upstream.

This adds the MacBook 1,1 2,1 3,1 4,1 and 4,2 to the DMI tables.

Signed-off-by: Evan McClain 
Signed-off-by: Richard Purdie 
Signed-off-by: Greg Kroah-Hartman

omap1: Fix compile for omap1_bl.c

2010-01-08T18:29:06+00:00

Commit 9905a43b made struct backlight_ops const. Omap was
setting check_fb dynamically, which caused the following
compile error:

drivers/video/backlight/omap1_bl.c: In function 'omapbl_probe':
drivers/video/backlight/omap1_bl.c:142: error: assignment of read-only variable 'omapbl_ops'

Turns out pdata->check_fb is not being used, so just remove
it to fix the compile.

Cc: Emese Revfy 
Cc: Richard Purdie 
Signed-off-by: Tony Lindgren

backlight: mbp_nvidia_bl - add two more MacBookPro variants

2009-12-17T11:50:26+00:00

This adds the MacBookPro 5,3 and 5,4 to the DMI tables.

Signed-off-by: Daniel Ritz 
Signed-off-by: Richard Purdie

backlight: Pass device through notify callback in the pwm driver

2009-12-17T11:46:01+00:00

Add the device to the notify callback's arguments in the PWM backlight 
driver. This brings the notify callback into line with the other 
callbacks defined by this driver.

Signed-off-by: Ben Dooks 
Signed-off-by: Simtec Linux Team 
Signed-off-by: Richard Purdie

backlight: PTR_ERR return of wrong pointer in cr_backlight_probe()

2009-12-17T11:19:07+00:00

Return the PTR_ERR of the correct pointer.

Signed-off-by: Roel Kluin 
Signed-off-by: Richard Purdie

backlight: Constify struct backlight_ops

2009-12-16T11:33:46+00:00

Signed-off-by: Emese Revfy 
Signed-off-by: Richard Purdie

tree-wide: convert open calls to remove spaces to skip_spaces() lib function

2009-12-15T16:53:32+00:00

Makes use of skip_spaces() defined in lib/string.c for removing leading
spaces from strings all over the tree.

It decreases lib.a code size by 47 bytes and reuses the function tree-wide:
   text    data     bss     dec     hex filename
  64688     584     592   65864   10148 (TOTALS-BEFORE)
  64641     584     592   65817   10119 (TOTALS-AFTER)

Also, while at it, if we see (*str && isspace(*str)), we can be sure to
remove the first condition (*str) as the second one (isspace(*str)) also
evaluates to 0 whenever *str == 0, making it redundant. In other words,
"a char equals zero is never a space".

Julia Lawall tried the semantic patch (http://coccinelle.lip6.fr) below,
and found occurrences of this pattern on 3 more files:
    drivers/leds/led-class.c
    drivers/leds/ledtrig-timer.c
    drivers/video/output.c

@@
expression str;
@@

( // ignore skip_spaces cases
while (*str &&  isspace(*str)) { \(str++;\|++str;\) }
|
- *str &&
isspace(*str)
)

Signed-off-by: André Goddard Rosa 
Cc: Julia Lawall 
Cc: Martin Schwidefsky 
Cc: Jeff Dike 
Cc: Ingo Molnar 
Cc: Thomas Gleixner 
Cc: "H. Peter Anvin" 
Cc: Richard Purdie 
Cc: Neil Brown 
Cc: Kyle McMartin 
Cc: Henrique de Moraes Holschuh 
Cc: David Howells 
Cc: 
Cc: Samuel Ortiz 
Cc: Patrick McHardy 
Cc: Takashi Iwai 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

const: constify remaining dev_pm_ops

2009-12-15T16:53:25+00:00

Signed-off-by: Alexey Dobriyan 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

backlight: adp5520: rename common defines and typos

2009-12-13T18:20:50+00:00

The common adp5520 mfd defines were namespaced to avoid collisions, so
update the define used in this driver accordingly.  The structs were also
renamed to fix a spelling typo.

Signed-off-by: Michael Hennerich 
Signed-off-by: Mike Frysinger 
Signed-off-by: Samuel Ortiz