Commit 2d5e94d7 authored Apr 26, 2008 by Henrique de Moraes Holschuh Committed by Len Brown Apr 29, 2008

ACPI: thinkpad-acpi: rate-limit CMOS/EC unsynced error messages



If userspace applications mess with the CMOS NVRAM, or something causes
both the ACPI firmware and thinkpad-acpi to try to change the brightness at
the same time, it is possible to have the CMOS and EC registers for the
current brightness go out of sync.

Should that happen, thinkpad-acpi could be really obnoxious when using a
brightness_mode of 3 (both EC and CMOS).  Instead of complaining a massive
number of times, make sure to complain only once until EC and CMOS are back
in sync.

Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Joerg Platte <lists@naasa.net>
Signed-off-by: Len Brown <len.brown@intel.com>

parent 8c74adbc

drivers/misc/thinkpad_acpi.c

+15 −7

Original line number	Diff line number	Diff line
		@@ -239,6 +239,7 @@ static struct {

		static struct {
		u16 hotkey_mask_ff:1;
		u16 bright_cmos_ec_unsync:1;
		} tp_warned;

		struct thinkpad_id_data {
		@@ -4323,14 +4324,21 @@ static int brightness_get(struct backlight_device *bd)
		level = lcmos;
		}

		if (brightness_mode == 3 && lec != lcmos) {
		if (brightness_mode == 3) {
		if (lec == lcmos)
		tp_warned.bright_cmos_ec_unsync = 0;
		else {
		if (!tp_warned.bright_cmos_ec_unsync) {
		printk(TPACPI_ERR
		"CMOS NVRAM (%u) and EC (%u) do not agree "
		"on display brightness level\n",
		"CMOS NVRAM (%u) and EC (%u) do not "
		"agree on display brightness level\n",
		(unsigned int) lcmos,
		(unsigned int) lec);
		tp_warned.bright_cmos_ec_unsync = 1;
		}
		return -EIO;
		}
		}

		return level;
		}