diff options
| author | Lv Zheng <lv.zheng@intel.com> | 2017-06-14 13:59:18 +0800 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-06-28 02:28:22 +0200 |
| commit | 4625d752e600c116e6327f9d7fc16e8f5be107e9 (patch) | |
| tree | 5472e436016c950da31f7817b7584a73977cf974 /drivers/acpi/ec.c | |
| parent | 1ab69f27cbb57e0f970bcd852d9f22ed34259b57 (diff) | |
| download | op-kernel-dev-4625d752e600c116e6327f9d7fc16e8f5be107e9.zip op-kernel-dev-4625d752e600c116e6327f9d7fc16e8f5be107e9.tar.gz | |
ACPI: EC: Fix EC command visibility for dynamic debug
acpi_ec_cmd_string() currently is only enabled for "DEBUG" macro, but users
trend to use CONFIG_DYNAMIC_DEBUG and enable ec.c pr_debug() print-outs by
"dyndbg='file ec.c +p'". In this use case, all command names are turned
into UNDEF and the log is confusing. This affects bugzilla triage work.
This patch fixes this issue by enabling acpi_ec_cmd_string() for
CONFIG_DYNAMIC_DEBUG.
Tested-by: Wang Wendy <wendy.wang@intel.com>
Tested-by: Feng Chenzhou <chenzhoux.feng@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/ec.c')
| -rw-r--r-- | drivers/acpi/ec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c index 30d7f82..f3ff591 100644 --- a/drivers/acpi/ec.c +++ b/drivers/acpi/ec.c @@ -316,7 +316,7 @@ static inline void acpi_ec_write_data(struct acpi_ec *ec, u8 data) ec->timestamp = jiffies; } -#ifdef DEBUG +#if defined(DEBUG) || defined(CONFIG_DYNAMIC_DEBUG) static const char *acpi_ec_cmd_string(u8 cmd) { switch (cmd) { |
