On Wednesday, November 06, 2013 09:03:15 AM Aaron Lu wrote: > When testing if the firmware's initial value is valid, we should use > the corrected level value instead of the raw value returned from > firmware. > > Signed-off-by: Aaron Lu <aaron.lu@xxxxxxxxx> > Cc: Danny Baumann <dannybaumann@xxxxxx> > Cc: stable <stable@xxxxxxxxxx> Any pointers to bug reports, BZ entries, etc? > --- > drivers/acpi/video.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c > index 38c3a28d6392..bf521b36c2f9 100644 > --- a/drivers/acpi/video.c > +++ b/drivers/acpi/video.c > @@ -856,7 +856,7 @@ acpi_video_init_brightness(struct acpi_video_device *device) > * or an index). Set the backlight to max_level in this case. > */ > for (i = 2; i < br->count; i++) > - if (level_old == br->levels[i]) > + if (level == br->levels[i]) > break; > if (i == br->count) > level = max_level; > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html