Skip to content

L
linux
Commit 0978e012 authored by Joe Perches's avatar Joe Perches Committed by Matthew Garrett
Browse files
Options

thinkpad_acpi: Convert printks to pr_<level> 

Add pr_fmt.
Removed local TPACPI_<level> #defines, convert to pr_<level>.
Neaten dbg_<foo> macros.
Added a few missing newlines to logging messages.
Added static inline str_supported for !CONFIG_THINKPAD_ACPI_DEBUG vdbg_printk
defect reported by Sedat Dilek <sedat.dilek@googlemail.com>.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarMatthew Garrett <mjg@redhat.com>
parent 112a6ee0
Hide whitespace changes
Inline Side-by-side
Showing with 188 additions and 284 deletions
drivers/platform/x86/thinkpad_acpi.c
View file @ 0978e012
...@@ -21,6 +21,8 @@ ...@@ -21,6 +21,8 @@
* 02110-1301, USA. * 02110-1301, USA.
*/ */
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#define TPACPI_VERSION "0.24" #define TPACPI_VERSION "0.24"
#define TPACPI_SYSFS_VERSION 0x020700 #define TPACPI_SYSFS_VERSION 0x020700
...@@ -224,17 +226,6 @@ enum tpacpi_hkey_event_t { ...@@ -224,17 +226,6 @@ enum tpacpi_hkey_event_t {
#define TPACPI_MAX_ACPI_ARGS 3 #define TPACPI_MAX_ACPI_ARGS 3
/* printk headers */
#define TPACPI_LOG TPACPI_FILE ": "
#define TPACPI_EMERG KERN_EMERG TPACPI_LOG
#define TPACPI_ALERT KERN_ALERT TPACPI_LOG
#define TPACPI_CRIT KERN_CRIT TPACPI_LOG
#define TPACPI_ERR KERN_ERR TPACPI_LOG
#define TPACPI_WARN KERN_WARNING TPACPI_LOG
#define TPACPI_NOTICE KERN_NOTICE TPACPI_LOG
#define TPACPI_INFO KERN_INFO TPACPI_LOG
#define TPACPI_DEBUG KERN_DEBUG TPACPI_LOG
/* Debugging printk groups */ /* Debugging printk groups */
#define TPACPI_DBG_ALL 0xffff #define TPACPI_DBG_ALL 0xffff
#define TPACPI_DBG_DISCLOSETASK 0x8000 #define TPACPI_DBG_DISCLOSETASK 0x8000
...@@ -389,34 +380,36 @@ static int tpacpi_uwb_emulstate; ...@@ -389,34 +380,36 @@ static int tpacpi_uwb_emulstate;
* Debugging helpers * Debugging helpers
*/ */
#define dbg_printk(a_dbg_level, format, arg...) \ #define dbg_printk(a_dbg_level, format, arg...) \
do { if (dbg_level & (a_dbg_level)) \ do { \
printk(TPACPI_DEBUG "%s: " format, __func__ , ## arg); \ if (dbg_level & (a_dbg_level)) \
} while (0) printk(KERN_DEBUG pr_fmt("%s: " format), \
__func__, ##arg); \
} while (0)
#ifdef CONFIG_THINKPAD_ACPI_DEBUG #ifdef CONFIG_THINKPAD_ACPI_DEBUG
#define vdbg_printk dbg_printk #define vdbg_printk dbg_printk
static const char *str_supported(int is_supported); static const char *str_supported(int is_supported);
#else #else
#define vdbg_printk(a_dbg_level, format, arg...) \ static inline const char *str_supported(int is_supported) { return ""; }
do { } while (0) #define vdbg_printk(a_dbg_level, format, arg...) \
no_printk(format, ##arg)
#endif #endif
static void tpacpi_log_usertask(const char * const what) static void tpacpi_log_usertask(const char * const what)
{ {
printk(TPACPI_DEBUG "%s: access by process with PID %d\n", printk(KERN_DEBUG pr_fmt("%s: access by process with PID %d\n"),
what, task_tgid_vnr(current)); what, task_tgid_vnr(current));
} }
#define tpacpi_disclose_usertask(what, format, arg...) \ #define tpacpi_disclose_usertask(what, format, arg...) \
do { \ do { \
if (unlikely( \ if (unlikely((dbg_level & TPACPI_DBG_DISCLOSETASK) && \
(dbg_level & TPACPI_DBG_DISCLOSETASK) && \ (tpacpi_lifecycle == TPACPI_LIFE_RUNNING))) { \
(tpacpi_lifecycle == TPACPI_LIFE_RUNNING))) { \ printk(KERN_DEBUG pr_fmt("%s: PID %d: " format), \
printk(TPACPI_DEBUG "%s: PID %d: " format, \ what, task_tgid_vnr(current), ## arg); \
what, task_tgid_vnr(current), ## arg); \ } \
} \ } while (0)
} while (0)
/* /*
* Quirk handling helpers * Quirk handling helpers
...@@ -554,7 +547,7 @@ static int acpi_evalf(acpi_handle handle, ...@@ -554,7 +547,7 @@ static int acpi_evalf(acpi_handle handle,
int quiet; int quiet;
if (!*fmt) { if (!*fmt) {
printk(TPACPI_ERR "acpi_evalf() called with empty format\n"); pr_err("acpi_evalf() called with empty format\n");
return 0; return 0;
} }
...@@ -579,7 +572,7 @@ static int acpi_evalf(acpi_handle handle, ...@@ -579,7 +572,7 @@ static int acpi_evalf(acpi_handle handle,
break; break;
/* add more types as needed */ /* add more types as needed */
default: default:
printk(TPACPI_ERR "acpi_evalf() called " pr_err("acpi_evalf() called "
"with invalid format character '%c'\n", c); "with invalid format character '%c'\n", c);
va_end(ap); va_end(ap);
return 0; return 0;
...@@ -608,13 +601,13 @@ static int acpi_evalf(acpi_handle handle, ...@@ -608,13 +601,13 @@ static int acpi_evalf(acpi_handle handle,
break; break;
/* add more types as needed */ /* add more types as needed */
default: default:
printk(TPACPI_ERR "acpi_evalf() called " pr_err("acpi_evalf() called "
"with invalid format character '%c'\n", res_type); "with invalid format character '%c'\n", res_type);
return 0; return 0;
} }
if (!success && !quiet) if (!success && !quiet)
printk(TPACPI_ERR "acpi_evalf(%s, %s, ...) failed: %s\n", pr_err("acpi_evalf(%s, %s, ...) failed: %s\n",
method, fmt0, acpi_format_exception(status)); method, fmt0, acpi_format_exception(status));
return success; return success;
...@@ -758,8 +751,7 @@ static int __init setup_acpi_notify(struct ibm_struct *ibm) ...@@ -758,8 +751,7 @@ static int __init setup_acpi_notify(struct ibm_struct *ibm)
rc = acpi_bus_get_device(*ibm->acpi->handle, &ibm->acpi->device); rc = acpi_bus_get_device(*ibm->acpi->handle, &ibm->acpi->device);
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR "acpi_bus_get_device(%s) failed: %d\n", pr_err("acpi_bus_get_device(%s) failed: %d\n", ibm->name, rc);
ibm->name, rc);
return -ENODEV; return -ENODEV;
} }
...@@ -772,12 +764,10 @@ static int __init setup_acpi_notify(struct ibm_struct *ibm) ...@@ -772,12 +764,10 @@ static int __init setup_acpi_notify(struct ibm_struct *ibm)
ibm->acpi->type, dispatch_acpi_notify, ibm); ibm->acpi->type, dispatch_acpi_notify, ibm);
if (ACPI_FAILURE(status)) { if (ACPI_FAILURE(status)) {
if (status == AE_ALREADY_EXISTS) { if (status == AE_ALREADY_EXISTS) {
printk(TPACPI_NOTICE pr_notice("another device driver is already "
"another device driver is already " "handling %s events\n", ibm->name);
"handling %s events\n", ibm->name);
} else { } else {
printk(TPACPI_ERR pr_err("acpi_install_notify_handler(%s) failed: %s\n",
"acpi_install_notify_handler(%s) failed: %s\n",
ibm->name, acpi_format_exception(status)); ibm->name, acpi_format_exception(status));
} }
return -ENODEV; return -ENODEV;
...@@ -802,8 +792,7 @@ static int __init register_tpacpi_subdriver(struct ibm_struct *ibm) ...@@ -802,8 +792,7 @@ static int __init register_tpacpi_subdriver(struct ibm_struct *ibm)
ibm->acpi->driver = kzalloc(sizeof(struct acpi_driver), GFP_KERNEL); ibm->acpi->driver = kzalloc(sizeof(struct acpi_driver), GFP_KERNEL);
if (!ibm->acpi->driver) { if (!ibm->acpi->driver) {
printk(TPACPI_ERR pr_err("failed to allocate memory for ibm->acpi->driver\n");
"failed to allocate memory for ibm->acpi->driver\n");
return -ENOMEM; return -ENOMEM;
} }
...@@ -814,7 +803,7 @@ static int __init register_tpacpi_subdriver(struct ibm_struct *ibm) ...@@ -814,7 +803,7 @@ static int __init register_tpacpi_subdriver(struct ibm_struct *ibm)
rc = acpi_bus_register_driver(ibm->acpi->driver); rc = acpi_bus_register_driver(ibm->acpi->driver);
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR "acpi_bus_register_driver(%s) failed: %d\n", pr_err("acpi_bus_register_driver(%s) failed: %d\n",
ibm->name, rc); ibm->name, rc);
kfree(ibm->acpi->driver); kfree(ibm->acpi->driver);
ibm->acpi->driver = NULL; ibm->acpi->driver = NULL;
...@@ -1072,15 +1061,14 @@ static int parse_strtoul(const char *buf, ...@@ -1072,15 +1061,14 @@ static int parse_strtoul(const char *buf,
static void tpacpi_disable_brightness_delay(void) static void tpacpi_disable_brightness_delay(void)
{ {
if (acpi_evalf(hkey_handle, NULL, "PWMS", "qvd", 0)) if (acpi_evalf(hkey_handle, NULL, "PWMS", "qvd", 0))
printk(TPACPI_NOTICE pr_notice("ACPI backlight control delay disabled\n");
"ACPI backlight control delay disabled\n");
} }
static void printk_deprecated_attribute(const char * const what, static void printk_deprecated_attribute(const char * const what,
const char * const details) const char * const details)
{ {
tpacpi_log_usertask("deprecated sysfs attribute"); tpacpi_log_usertask("deprecated sysfs attribute");
printk(TPACPI_WARN "WARNING: sysfs attribute %s is deprecated and " pr_warn("WARNING: sysfs attribute %s is deprecated and "
"will be removed. %s\n", "will be removed. %s\n",
what, details); what, details);
} }
...@@ -1255,8 +1243,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id, ...@@ -1255,8 +1243,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id,
&tpacpi_rfk_rfkill_ops, &tpacpi_rfk_rfkill_ops,
atp_rfk); atp_rfk);
if (!atp_rfk || !atp_rfk->rfkill) { if (!atp_rfk || !atp_rfk->rfkill) {
printk(TPACPI_ERR pr_err("failed to allocate memory for rfkill class\n");
"failed to allocate memory for rfkill class\n");
kfree(atp_rfk); kfree(atp_rfk);
return -ENOMEM; return -ENOMEM;
} }
...@@ -1266,9 +1253,8 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id, ...@@ -1266,9 +1253,8 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id,
sw_status = (tp_rfkops->get_status)(); sw_status = (tp_rfkops->get_status)();
if (sw_status < 0) { if (sw_status < 0) {
printk(TPACPI_ERR pr_err("failed to read initial state for %s, error %d\n",
"failed to read initial state for %s, error %d\n", name, sw_status);
name, sw_status);
} else { } else {
sw_state = (sw_status == TPACPI_RFK_RADIO_OFF); sw_state = (sw_status == TPACPI_RFK_RADIO_OFF);
if (set_default) { if (set_default) {
...@@ -1282,9 +1268,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id, ...@@ -1282,9 +1268,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id,
res = rfkill_register(atp_rfk->rfkill); res = rfkill_register(atp_rfk->rfkill);
if (res < 0) { if (res < 0) {
printk(TPACPI_ERR pr_err("failed to register %s rfkill switch: %d\n", name, res);
"failed to register %s rfkill switch: %d\n",
name, res);
rfkill_destroy(atp_rfk->rfkill); rfkill_destroy(atp_rfk->rfkill);
kfree(atp_rfk); kfree(atp_rfk);
return res; return res;
...@@ -1292,7 +1276,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id, ...@@ -1292,7 +1276,7 @@ static int __init tpacpi_new_rfkill(const enum tpacpi_rfk_id id,
tpacpi_rfkill_switches[id] = atp_rfk; tpacpi_rfkill_switches[id] = atp_rfk;
printk(TPACPI_INFO "rfkill switch %s: radio is %sblocked\n", pr_info("rfkill switch %s: radio is %sblocked\n",
name, (sw_state || hw_state) ? "" : "un"); name, (sw_state || hw_state) ? "" : "un");
return 0; return 0;
} }
...@@ -1816,10 +1800,8 @@ static void __init tpacpi_check_outdated_fw(void) ...@@ -1816,10 +1800,8 @@ static void __init tpacpi_check_outdated_fw(void)
* broken, or really stable to begin with, so it is * broken, or really stable to begin with, so it is
* best if the user upgrades the firmware anyway. * best if the user upgrades the firmware anyway.
*/ */
printk(TPACPI_WARN pr_warn("WARNING: Outdated ThinkPad BIOS/EC firmware\n");
"WARNING: Outdated ThinkPad BIOS/EC firmware\n"); pr_warn("WARNING: This firmware may be missing critical bug "
printk(TPACPI_WARN
"WARNING: This firmware may be missing critical bug "
"fixes and/or important features\n"); "fixes and/or important features\n");
} }
} }
...@@ -2108,9 +2090,7 @@ void static hotkey_mask_warn_incomplete_mask(void) ...@@ -2108,9 +2090,7 @@ void static hotkey_mask_warn_incomplete_mask(void)
(hotkey_all_mask | TPACPI_HKEY_NVRAM_KNOWN_MASK); (hotkey_all_mask | TPACPI_HKEY_NVRAM_KNOWN_MASK);
if (wantedmask) if (wantedmask)
printk(TPACPI_NOTICE pr_notice("required events 0x%08x not enabled!\n", wantedmask);
"required events 0x%08x not enabled!\n",
wantedmask);
} }
/* /*
...@@ -2148,10 +2128,9 @@ static int hotkey_mask_set(u32 mask) ...@@ -2148,10 +2128,9 @@ static int hotkey_mask_set(u32 mask)
* a given event. * a given event.
*/ */
if (!hotkey_mask_get() && !rc && (fwmask & ~hotkey_acpi_mask)) { if (!hotkey_mask_get() && !rc && (fwmask & ~hotkey_acpi_mask)) {
printk(TPACPI_NOTICE pr_notice("asked for hotkey mask 0x%08x, but "
"asked for hotkey mask 0x%08x, but " "firmware forced it to 0x%08x\n",
"firmware forced it to 0x%08x\n", fwmask, hotkey_acpi_mask);
fwmask, hotkey_acpi_mask);
} }
if (tpacpi_lifecycle != TPACPI_LIFE_EXITING) if (tpacpi_lifecycle != TPACPI_LIFE_EXITING)
...@@ -2175,13 +2154,11 @@ static int hotkey_user_mask_set(const u32 mask) ...@@ -2175,13 +2154,11 @@ static int hotkey_user_mask_set(const u32 mask)
(mask == 0xffff || mask == 0xffffff || (mask == 0xffff || mask == 0xffffff ||
mask == 0xffffffff)) { mask == 0xffffffff)) {
tp_warned.hotkey_mask_ff = 1; tp_warned.hotkey_mask_ff = 1;
printk(TPACPI_NOTICE pr_notice("setting the hotkey mask to 0x%08x is likely "
"setting the hotkey mask to 0x%08x is likely " "not the best way to go about it\n", mask);
"not the best way to go about it\n", mask); pr_notice("please consider using the driver defaults, "
printk(TPACPI_NOTICE "and refer to up-to-date thinkpad-acpi "
"please consider using the driver defaults, " "documentation\n");
"and refer to up-to-date thinkpad-acpi "
"documentation\n");
} }
/* Try to enable what the user asked for, plus whatever we need. /* Try to enable what the user asked for, plus whatever we need.
...@@ -2565,8 +2542,7 @@ static void hotkey_poll_setup(const bool may_warn) ...@@ -2565,8 +2542,7 @@ static void hotkey_poll_setup(const bool may_warn)
NULL, TPACPI_NVRAM_KTHREAD_NAME); NULL, TPACPI_NVRAM_KTHREAD_NAME);
if (IS_ERR(tpacpi_hotkey_task)) { if (IS_ERR(tpacpi_hotkey_task)) {
tpacpi_hotkey_task = NULL; tpacpi_hotkey_task = NULL;
printk(TPACPI_ERR pr_err("could not create kernel thread "
"could not create kernel thread "
"for hotkey polling\n"); "for hotkey polling\n");
} }
} }
...@@ -2574,11 +2550,10 @@ static void hotkey_poll_setup(const bool may_warn) ...@@ -2574,11 +2550,10 @@ static void hotkey_poll_setup(const bool may_warn)
hotkey_poll_stop_sync(); hotkey_poll_stop_sync();
if (may_warn && (poll_driver_mask || poll_user_mask) && if (may_warn && (poll_driver_mask || poll_user_mask) &&
hotkey_poll_freq == 0) { hotkey_poll_freq == 0) {
printk(TPACPI_NOTICE pr_notice("hot keys 0x%08x and/or events 0x%08x "
"hot keys 0x%08x and/or events 0x%08x " "require polling, which is currently "
"require polling, which is currently " "disabled\n",
"disabled\n", poll_user_mask, poll_driver_mask);
poll_user_mask, poll_driver_mask);
} }
} }
} }
...@@ -2802,13 +2777,13 @@ static ssize_t hotkey_source_mask_store(struct device *dev, ...@@ -2802,13 +2777,13 @@ static ssize_t hotkey_source_mask_store(struct device *dev,
mutex_unlock(&hotkey_mutex); mutex_unlock(&hotkey_mutex);
if (rc < 0) if (rc < 0)
printk(TPACPI_ERR "hotkey_source_mask: failed to update the" pr_err("hotkey_source_mask: "
"firmware event mask!\n"); "failed to update the firmware event mask!\n");
if (r_ev) if (r_ev)
printk(TPACPI_NOTICE "hotkey_source_mask: " pr_notice("hotkey_source_mask: "
"some important events were disabled: " "some important events were disabled: 0x%04x\n",
"0x%04x\n", r_ev); r_ev);
tpacpi_disclose_usertask("hotkey_source_mask", "set to 0x%08lx\n", t); tpacpi_disclose_usertask("hotkey_source_mask", "set to 0x%08lx\n", t);
...@@ -3039,8 +3014,7 @@ static void hotkey_exit(void) ...@@ -3039,8 +3014,7 @@ static void hotkey_exit(void)
if (((tp_features.hotkey_mask && if (((tp_features.hotkey_mask &&
hotkey_mask_set(hotkey_orig_mask)) | hotkey_mask_set(hotkey_orig_mask)) |
hotkey_status_set(false)) != 0) hotkey_status_set(false)) != 0)
printk(TPACPI_ERR pr_err("failed to restore hot key mask "
"failed to restore hot key mask "
"to BIOS defaults\n"); "to BIOS defaults\n");
} }
...@@ -3279,10 +3253,9 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3279,10 +3253,9 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
for HKEY interface version 0x100 */ for HKEY interface version 0x100 */
if (acpi_evalf(hkey_handle, &hkeyv, "MHKV", "qd")) { if (acpi_evalf(hkey_handle, &hkeyv, "MHKV", "qd")) {
if ((hkeyv >> 8) != 1) { if ((hkeyv >> 8) != 1) {
printk(TPACPI_ERR "unknown version of the " pr_err("unknown version of the HKEY interface: 0x%x\n",
"HKEY interface: 0x%x\n", hkeyv); hkeyv);
printk(TPACPI_ERR "please report this to %s\n", pr_err("please report this to %s\n", TPACPI_MAIL);
TPACPI_MAIL);
} else { } else {
/* /*
* MHKV 0x100 in A31, R40, R40e, * MHKV 0x100 in A31, R40, R40e,
...@@ -3295,8 +3268,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3295,8 +3268,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
/* Paranoia check AND init hotkey_all_mask */ /* Paranoia check AND init hotkey_all_mask */
if (!acpi_evalf(hkey_handle, &hotkey_all_mask, if (!acpi_evalf(hkey_handle, &hotkey_all_mask,
"MHKA", "qd")) { "MHKA", "qd")) {
printk(TPACPI_ERR pr_err("missing MHKA handler, "
"missing MHKA handler, "
"please report this to %s\n", "please report this to %s\n",
TPACPI_MAIL); TPACPI_MAIL);
/* Fallback: pre-init for FN+F3,F4,F12 */ /* Fallback: pre-init for FN+F3,F4,F12 */
...@@ -3334,16 +3306,14 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3334,16 +3306,14 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
if (dbg_wlswemul) { if (dbg_wlswemul) {
tp_features.hotkey_wlsw = 1; tp_features.hotkey_wlsw = 1;
radiosw_state = !!tpacpi_wlsw_emulstate; radiosw_state = !!tpacpi_wlsw_emulstate;
printk(TPACPI_INFO pr_info("radio switch emulation enabled\n");
"radio switch emulation enabled\n");
} else } else
#endif #endif
/* Not all thinkpads have a hardware radio switch */ /* Not all thinkpads have a hardware radio switch */
if (acpi_evalf(hkey_handle, &status, "WLSW", "qd")) { if (acpi_evalf(hkey_handle, &status, "WLSW", "qd")) {
tp_features.hotkey_wlsw = 1; tp_features.hotkey_wlsw = 1;
radiosw_state = !!status; radiosw_state = !!status;
printk(TPACPI_INFO pr_info("radio switch found; radios are %s\n",
"radio switch found; radios are %s\n",
enabled(status, 0)); enabled(status, 0));
} }
if (tp_features.hotkey_wlsw) if (tp_features.hotkey_wlsw)
...@@ -3354,8 +3324,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3354,8 +3324,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
if (!res && acpi_evalf(hkey_handle, &status, "MHKG", "qd")) { if (!res && acpi_evalf(hkey_handle, &status, "MHKG", "qd")) {
tp_features.hotkey_tablet = 1; tp_features.hotkey_tablet = 1;
tabletsw_state = !!(status & TP_HOTKEY_TABLET_MASK); tabletsw_state = !!(status & TP_HOTKEY_TABLET_MASK);
printk(TPACPI_INFO pr_info("possible tablet mode switch found; "
"possible tablet mode switch found; "
"ThinkPad in %s mode\n", "ThinkPad in %s mode\n",
(tabletsw_state) ? "tablet" : "laptop"); (tabletsw_state) ? "tablet" : "laptop");
res = add_to_attr_set(hotkey_dev_attributes, res = add_to_attr_set(hotkey_dev_attributes,
...@@ -3373,8 +3342,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3373,8 +3342,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
hotkey_keycode_map = kmalloc(TPACPI_HOTKEY_MAP_SIZE, hotkey_keycode_map = kmalloc(TPACPI_HOTKEY_MAP_SIZE,
GFP_KERNEL); GFP_KERNEL);
if (!hotkey_keycode_map) { if (!hotkey_keycode_map) {
printk(TPACPI_ERR pr_err("failed to allocate memory for key map\n");
"failed to allocate memory for key map\n");
res = -ENOMEM; res = -ENOMEM;
goto err_exit; goto err_exit;
} }
...@@ -3417,13 +3385,11 @@ static int __init hotkey_init(struct ibm_init_struct *iibm) ...@@ -3417,13 +3385,11 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
* userspace. tpacpi_detect_brightness_capabilities() must have * userspace. tpacpi_detect_brightness_capabilities() must have
* been called before this point */ * been called before this point */
if (tp_features.bright_acpimode && acpi_video_backlight_support()) { if (tp_features.bright_acpimode && acpi_video_backlight_support()) {
printk(TPACPI_INFO pr_info("This ThinkPad has standard ACPI backlight "
"This ThinkPad has standard ACPI backlight " "brightness control, supported by the ACPI "
"brightness control, supported by the ACPI " "video driver\n");
"video driver\n"); pr_notice("Disabling thinkpad-acpi brightness events "
printk(TPACPI_NOTICE "by default...\n");
"Disabling thinkpad-acpi brightness events "
"by default...\n");
/* Disable brightness up/down on Lenovo thinkpads when /* Disable brightness up/down on Lenovo thinkpads when
* ACPI is handling them, otherwise it is plain impossible * ACPI is handling them, otherwise it is plain impossible
...@@ -3530,8 +3496,7 @@ static bool hotkey_notify_wakeup(const u32 hkey, ...@@ -3530,8 +3496,7 @@ static bool hotkey_notify_wakeup(const u32 hkey,
case TP_HKEY_EV_WKUP_S3_BATLOW: /* Battery on critical low level/S3 */ case TP_HKEY_EV_WKUP_S3_BATLOW: /* Battery on critical low level/S3 */
case TP_HKEY_EV_WKUP_S4_BATLOW: /* Battery on critical low level/S4 */ case TP_HKEY_EV_WKUP_S4_BATLOW: /* Battery on critical low level/S4 */
printk(TPACPI_ALERT pr_alert("EMERGENCY WAKEUP: battery almost empty\n");
"EMERGENCY WAKEUP: battery almost empty\n");
/* how to auto-heal: */ /* how to auto-heal: */
/* 2313: woke up from S3, go to S4/S5 */ /* 2313: woke up from S3, go to S4/S5 */
/* 2413: woke up from S4, go to S5 */ /* 2413: woke up from S4, go to S5 */
...@@ -3542,9 +3507,7 @@ static bool hotkey_notify_wakeup(const u32 hkey, ...@@ -3542,9 +3507,7 @@ static bool hotkey_notify_wakeup(const u32 hkey,
} }
if (hotkey_wakeup_reason != TP_ACPI_WAKEUP_NONE) { if (hotkey_wakeup_reason != TP_ACPI_WAKEUP_NONE) {
printk(TPACPI_INFO pr_info("woke up due to a hot-unplug request...\n");
"woke up due to a hot-unplug "
"request...\n");
hotkey_wakeup_reason_notify_change(); hotkey_wakeup_reason_notify_change();
} }
return true; return true;
...@@ -3596,37 +3559,31 @@ static bool hotkey_notify_thermal(const u32 hkey, ...@@ -3596,37 +3559,31 @@ static bool hotkey_notify_thermal(const u32 hkey,
switch (hkey) { switch (hkey) {
case TP_HKEY_EV_THM_TABLE_CHANGED: case TP_HKEY_EV_THM_TABLE_CHANGED:
printk(TPACPI_INFO pr_info("EC reports that Thermal Table has changed\n");
"EC reports that Thermal Table has changed\n");
/* recommended action: do nothing, we don't have /* recommended action: do nothing, we don't have
* Lenovo ATM information */ * Lenovo ATM information */
return true; return true;
case TP_HKEY_EV_ALARM_BAT_HOT: case TP_HKEY_EV_ALARM_BAT_HOT:
printk(TPACPI_CRIT pr_crit("THERMAL ALARM: battery is too hot!\n");
"THERMAL ALARM: battery is too hot!\n");
/* recommended action: warn user through gui */ /* recommended action: warn user through gui */
break; break;
case TP_HKEY_EV_ALARM_BAT_XHOT: case TP_HKEY_EV_ALARM_BAT_XHOT:
printk(TPACPI_ALERT pr_alert("THERMAL EMERGENCY: battery is extremely hot!\n");
"THERMAL EMERGENCY: battery is extremely hot!\n");
/* recommended action: immediate sleep/hibernate */ /* recommended action: immediate sleep/hibernate */
break; break;
case TP_HKEY_EV_ALARM_SENSOR_HOT: case TP_HKEY_EV_ALARM_SENSOR_HOT:
printk(TPACPI_CRIT pr_crit("THERMAL ALARM: "
"THERMAL ALARM: "
"a sensor reports something is too hot!\n"); "a sensor reports something is too hot!\n");
/* recommended action: warn user through gui, that */ /* recommended action: warn user through gui, that */
/* some internal component is too hot */ /* some internal component is too hot */
break; break;
case TP_HKEY_EV_ALARM_SENSOR_XHOT: case TP_HKEY_EV_ALARM_SENSOR_XHOT:
printk(TPACPI_ALERT pr_alert("THERMAL EMERGENCY: "
"THERMAL EMERGENCY: " "a sensor reports something is extremely hot!\n");
"a sensor reports something is extremely hot!\n");
/* recommended action: immediate sleep/hibernate */ /* recommended action: immediate sleep/hibernate */
break; break;
default: default:
printk(TPACPI_ALERT pr_alert("THERMAL ALERT: unknown thermal alarm received\n");
"THERMAL ALERT: unknown thermal alarm received\n");
known = false; known = false;
} }
...@@ -3643,8 +3600,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) ...@@ -3643,8 +3600,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event)
bool known_ev; bool known_ev;
if (event != 0x80) { if (event != 0x80) {
printk(TPACPI_ERR pr_err("unknown HKEY notification event %d\n", event);
"unknown HKEY notification event %d\n", event);
/* forward it to userspace, maybe it knows how to handle it */ /* forward it to userspace, maybe it knows how to handle it */
acpi_bus_generate_netlink_event( acpi_bus_generate_netlink_event(
ibm->acpi->device->pnp.device_class, ibm->acpi->device->pnp.device_class,
...@@ -3655,7 +3611,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) ...@@ -3655,7 +3611,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event)
while (1) { while (1) {
if (!acpi_evalf(hkey_handle, &hkey, "MHKP", "d")) { if (!acpi_evalf(hkey_handle, &hkey, "MHKP", "d")) {
printk(TPACPI_ERR "failed to retrieve HKEY event\n"); pr_err("failed to retrieve HKEY event\n");
return; return;
} }
...@@ -3683,8 +3639,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) ...@@ -3683,8 +3639,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event)
switch (hkey) { switch (hkey) {
case TP_HKEY_EV_BAYEJ_ACK: case TP_HKEY_EV_BAYEJ_ACK:
hotkey_autosleep_ack = 1; hotkey_autosleep_ack = 1;
printk(TPACPI_INFO pr_info("bay ejected\n");
"bay ejected\n");
hotkey_wakeup_hotunplug_complete_notify_change(); hotkey_wakeup_hotunplug_complete_notify_change();
known_ev = true; known_ev = true;
break; break;
...@@ -3700,8 +3655,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) ...@@ -3700,8 +3655,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event)
/* 0x4000-0x4FFF: dock-related wakeups */ /* 0x4000-0x4FFF: dock-related wakeups */
if (hkey == TP_HKEY_EV_UNDOCK_ACK) { if (hkey == TP_HKEY_EV_UNDOCK_ACK) {
hotkey_autosleep_ack = 1; hotkey_autosleep_ack = 1;
printk(TPACPI_INFO pr_info("undocked\n");
"undocked\n");
hotkey_wakeup_hotunplug_complete_notify_change(); hotkey_wakeup_hotunplug_complete_notify_change();
known_ev = true; known_ev = true;
} else { } else {
...@@ -3732,11 +3686,9 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) ...@@ -3732,11 +3686,9 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event)
known_ev = false; known_ev = false;
} }
if (!known_ev) { if (!known_ev) {
printk(TPACPI_NOTICE pr_notice("unhandled HKEY event 0x%04x\n", hkey);
"unhandled HKEY event 0x%04x\n", hkey); pr_notice("please report the conditions when this "
printk(TPACPI_NOTICE "event happened to %s\n", TPACPI_MAIL);
"please report the conditions when this "
"event happened to %s\n", TPACPI_MAIL);
} }
/* Legacy events */ /* Legacy events */
...@@ -3769,8 +3721,7 @@ static void hotkey_resume(void) ...@@ -3769,8 +3721,7 @@ static void hotkey_resume(void)
if (hotkey_status_set(true) < 0 || if (hotkey_status_set(true) < 0 ||
hotkey_mask_set(hotkey_acpi_mask) < 0) hotkey_mask_set(hotkey_acpi_mask) < 0)
printk(TPACPI_ERR pr_err("error while attempting to reset the event "
"error while attempting to reset the event "
"firmware interface\n"); "firmware interface\n");
tpacpi_send_radiosw_update(); tpacpi_send_radiosw_update();
...@@ -3815,14 +3766,12 @@ static void hotkey_enabledisable_warn(bool enable) ...@@ -3815,14 +3766,12 @@ static void hotkey_enabledisable_warn(bool enable)
{ {
tpacpi_log_usertask("procfs hotkey enable/disable"); tpacpi_log_usertask("procfs hotkey enable/disable");
if (!WARN((tpacpi_lifecycle == TPACPI_LIFE_RUNNING || !enable), if (!WARN((tpacpi_lifecycle == TPACPI_LIFE_RUNNING || !enable),
TPACPI_WARN pr_fmt("hotkey enable/disable functionality has been "
"hotkey enable/disable functionality has been " "removed from the driver. "
"removed from the driver. Hotkeys are always " "Hotkeys are always enabled.\n")))
"enabled\n")) pr_err("Please remove the hotkey=enable module "
printk(TPACPI_ERR "parameter, it is deprecated. "
"Please remove the hotkey=enable module " "Hotkeys are always enabled.\n");
"parameter, it is deprecated. Hotkeys are always "
"enabled\n");
} }
static int hotkey_write(char *buf) static int hotkey_write(char *buf)
...@@ -4002,8 +3951,7 @@ static void bluetooth_shutdown(void) ...@@ -4002,8 +3951,7 @@ static void bluetooth_shutdown(void)
/* Order firmware to save current state to NVRAM */ /* Order firmware to save current state to NVRAM */
if (!acpi_evalf(NULL, NULL, "\\BLTH", "vd", if (!acpi_evalf(NULL, NULL, "\\BLTH", "vd",
TP_ACPI_BLTH_SAVE_STATE)) TP_ACPI_BLTH_SAVE_STATE))
printk(TPACPI_NOTICE pr_notice("failed to save bluetooth state to NVRAM\n");
"failed to save bluetooth state to NVRAM\n");
else else
vdbg_printk(TPACPI_DBG_RFKILL, vdbg_printk(TPACPI_DBG_RFKILL,
"bluestooth state saved to NVRAM\n"); "bluestooth state saved to NVRAM\n");
...@@ -4042,8 +3990,7 @@ static int __init bluetooth_init(struct ibm_init_struct *iibm) ...@@ -4042,8 +3990,7 @@ static int __init bluetooth_init(struct ibm_init_struct *iibm)
#ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES #ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES
if (dbg_bluetoothemul) { if (dbg_bluetoothemul) {
tp_features.bluetooth = 1; tp_features.bluetooth = 1;
printk(TPACPI_INFO pr_info("bluetooth switch emulation enabled\n");
"bluetooth switch emulation enabled\n");
} else } else
#endif #endif
if (tp_features.bluetooth && if (tp_features.bluetooth &&
...@@ -4194,8 +4141,7 @@ static void wan_shutdown(void) ...@@ -4194,8 +4141,7 @@ static void wan_shutdown(void)
/* Order firmware to save current state to NVRAM */ /* Order firmware to save current state to NVRAM */
if (!acpi_evalf(NULL, NULL, "\\WGSV", "vd", if (!acpi_evalf(NULL, NULL, "\\WGSV", "vd",
TP_ACPI_WGSV_SAVE_STATE)) TP_ACPI_WGSV_SAVE_STATE))
printk(TPACPI_NOTICE pr_notice("failed to save WWAN state to NVRAM\n");
"failed to save WWAN state to NVRAM\n");
else else
vdbg_printk(TPACPI_DBG_RFKILL, vdbg_printk(TPACPI_DBG_RFKILL,
"WWAN state saved to NVRAM\n"); "WWAN state saved to NVRAM\n");
...@@ -4232,8 +4178,7 @@ static int __init wan_init(struct ibm_init_struct *iibm) ...@@ -4232,8 +4178,7 @@ static int __init wan_init(struct ibm_init_struct *iibm)
#ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES #ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES
if (dbg_wwanemul) { if (dbg_wwanemul) {
tp_features.wan = 1; tp_features.wan = 1;
printk(TPACPI_INFO pr_info("wwan switch emulation enabled\n");
"wwan switch emulation enabled\n");
} else } else
#endif #endif
if (tp_features.wan && if (tp_features.wan &&
...@@ -4373,8 +4318,7 @@ static int __init uwb_init(struct ibm_init_struct *iibm) ...@@ -4373,8 +4318,7 @@ static int __init uwb_init(struct ibm_init_struct *iibm)
#ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES #ifdef CONFIG_THINKPAD_ACPI_DEBUGFACILITIES
if (dbg_uwbemul) { if (dbg_uwbemul) {
tp_features.uwb = 1; tp_features.uwb = 1;
printk(TPACPI_INFO pr_info("uwb switch emulation enabled\n");
"uwb switch emulation enabled\n");
} else } else
#endif #endif
if (tp_features.uwb && if (tp_features.uwb &&
...@@ -4487,7 +4431,7 @@ static void video_exit(void) ...@@ -4487,7 +4431,7 @@ static void video_exit(void)
dbg_printk(TPACPI_DBG_EXIT, dbg_printk(TPACPI_DBG_EXIT,
"restoring original video autoswitch mode\n"); "restoring original video autoswitch mode\n");
if (video_autosw_set(video_orig_autosw)) if (video_autosw_set(video_orig_autosw))
printk(TPACPI_ERR "error while trying to restore original " pr_err("error while trying to restore original "
"video autoswitch mode\n"); "video autoswitch mode\n");
} }
...@@ -4560,8 +4504,7 @@ static int video_outputsw_set(int status) ...@@ -4560,8 +4504,7 @@ static int video_outputsw_set(int status)
res = acpi_evalf(vid_handle, NULL, res = acpi_evalf(vid_handle, NULL,
"ASWT", "vdd", status * 0x100, 0); "ASWT", "vdd", status * 0x100, 0);
if (!autosw && video_autosw_set(autosw)) { if (!autosw && video_autosw_set(autosw)) {
printk(TPACPI_ERR pr_err("video auto-switch left enabled due to error\n");
"video auto-switch left enabled due to error\n");
return -EIO; return -EIO;
} }
break; break;
...@@ -4630,8 +4573,7 @@ static int video_outputsw_cycle(void) ...@@ -4630,8 +4573,7 @@ static int video_outputsw_cycle(void)
return -ENOSYS; return -ENOSYS;
} }
if (!autosw && video_autosw_set(autosw)) { if (!autosw && video_autosw_set(autosw)) {
printk(TPACPI_ERR pr_err("video auto-switch left enabled due to error\n");
"video auto-switch left enabled due to error\n");
return -EIO; return -EIO;
} }
...@@ -5348,7 +5290,7 @@ static int __init led_init(struct ibm_init_struct *iibm) ...@@ -5348,7 +5290,7 @@ static int __init led_init(struct ibm_init_struct *iibm)
tpacpi_leds = kzalloc(sizeof(*tpacpi_leds) * TPACPI_LED_NUMLEDS, tpacpi_leds = kzalloc(sizeof(*tpacpi_leds) * TPACPI_LED_NUMLEDS,
GFP_KERNEL); GFP_KERNEL);
if (!tpacpi_leds) { if (!tpacpi_leds) {
printk(TPACPI_ERR "Out of memory for LED data\n"); pr_err("Out of memory for LED data\n");
return -ENOMEM; return -ENOMEM;
} }
...@@ -5367,9 +5309,8 @@ static int __init led_init(struct ibm_init_struct *iibm) ...@@ -5367,9 +5309,8 @@ static int __init led_init(struct ibm_init_struct *iibm)
} }
#ifdef CONFIG_THINKPAD_ACPI_UNSAFE_LEDS #ifdef CONFIG_THINKPAD_ACPI_UNSAFE_LEDS
printk(TPACPI_NOTICE pr_notice("warning: userspace override of important "
"warning: userspace override of important " "firmware LEDs is enabled\n");
"firmware LEDs is enabled\n");
#endif #endif
return 0; return 0;
} }
...@@ -5639,17 +5580,16 @@ static void thermal_dump_all_sensors(void) ...@@ -5639,17 +5580,16 @@ static void thermal_dump_all_sensors(void)
if (n <= 0) if (n <= 0)
return; return;
printk(TPACPI_NOTICE pr_notice("temperatures (Celsius):");
"temperatures (Celsius):");
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
if (t.temp[i] != TPACPI_THERMAL_SENSOR_NA) if (t.temp[i] != TPACPI_THERMAL_SENSOR_NA)
printk(KERN_CONT " %d", (int)(t.temp[i] / 1000)); pr_cont(" %d", (int)(t.temp[i] / 1000));
else else
printk(KERN_CONT " N/A"); pr_cont(" N/A");
} }
printk(KERN_CONT "\n"); pr_cont("\n");
} }
/* sysfs temp##_input -------------------------------------------------- */ /* sysfs temp##_input -------------------------------------------------- */
...@@ -5769,14 +5709,12 @@ static int __init thermal_init(struct ibm_init_struct *iibm) ...@@ -5769,14 +5709,12 @@ static int __init thermal_init(struct ibm_init_struct *iibm)
if (ta1 == 0) { if (ta1 == 0) {
/* This is sheer paranoia, but we handle it anyway */ /* This is sheer paranoia, but we handle it anyway */
if (acpi_tmp7) { if (acpi_tmp7) {
printk(TPACPI_ERR pr_err("ThinkPad ACPI EC access misbehaving, "
"ThinkPad ACPI EC access misbehaving, "
"falling back to ACPI TMPx access " "falling back to ACPI TMPx access "
"mode\n"); "mode\n");
thermal_read_mode = TPACPI_THERMAL_ACPI_TMP07; thermal_read_mode = TPACPI_THERMAL_ACPI_TMP07;
} else { } else {
printk(TPACPI_ERR pr_err("ThinkPad ACPI EC access misbehaving, "
"ThinkPad ACPI EC access misbehaving, "
"disabling thermal sensors access\n"); "disabling thermal sensors access\n");
thermal_read_mode = TPACPI_THERMAL_NONE; thermal_read_mode = TPACPI_THERMAL_NONE;
} }
...@@ -6129,8 +6067,8 @@ static int __init tpacpi_query_bcl_levels(acpi_handle handle) ...@@ -6129,8 +6067,8 @@ static int __init tpacpi_query_bcl_levels(acpi_handle handle)
if (ACPI_SUCCESS(acpi_evaluate_object(handle, "_BCL", NULL, &buffer))) { if (ACPI_SUCCESS(acpi_evaluate_object(handle, "_BCL", NULL, &buffer))) {
obj = (union acpi_object *)buffer.pointer; obj = (union acpi_object *)buffer.pointer;
if (!obj || (obj->type != ACPI_TYPE_PACKAGE)) { if (!obj || (obj->type != ACPI_TYPE_PACKAGE)) {
printk(TPACPI_ERR "Unknown _BCL data, " pr_err("Unknown _BCL data, please report this to %s\n",
"please report this to %s\n", TPACPI_MAIL); TPACPI_MAIL);
rc = 0; rc = 0;
} else { } else {
rc = obj->package.count; rc = obj->package.count;
...@@ -6214,18 +6152,15 @@ static void __init tpacpi_detect_brightness_capabilities(void) ...@@ -6214,18 +6152,15 @@ static void __init tpacpi_detect_brightness_capabilities(void)
switch (b) { switch (b) {
case 16: case 16:
bright_maxlvl = 15; bright_maxlvl = 15;
printk(TPACPI_INFO pr_info("detected a 16-level brightness capable ThinkPad\n");
"detected a 16-level brightness capable ThinkPad\n");
break; break;
case 8: case 8:
case 0: case 0:
bright_maxlvl = 7; bright_maxlvl = 7;
printk(TPACPI_INFO pr_info("detected a 8-level brightness capable ThinkPad\n");
"detected a 8-level brightness capable ThinkPad\n");
break; break;
default: default:
printk(TPACPI_ERR pr_err("Unsupported brightness interface, "
"Unsupported brightness interface, "
"please contact %s\n", TPACPI_MAIL); "please contact %s\n", TPACPI_MAIL);
tp_features.bright_unkfw = 1; tp_features.bright_unkfw = 1;
bright_maxlvl = b - 1; bright_maxlvl = b - 1;
...@@ -6260,22 +6195,19 @@ static int __init brightness_init(struct ibm_init_struct *iibm) ...@@ -6260,22 +6195,19 @@ static int __init brightness_init(struct ibm_init_struct *iibm)
if (acpi_video_backlight_support()) { if (acpi_video_backlight_support()) {
if (brightness_enable > 1) { if (brightness_enable > 1) {
printk(TPACPI_INFO pr_info("Standard ACPI backlight interface "
"Standard ACPI backlight interface " "available, not loading native one\n");
"available, not loading native one.\n");
return 1; return 1;
} else if (brightness_enable == 1) { } else if (brightness_enable == 1) {
printk(TPACPI_WARN pr_warn("Cannot enable backlight brightness support, "
"Cannot enable backlight brightness support, "
"ACPI is already handling it. Refer to the " "ACPI is already handling it. Refer to the "
"acpi_backlight kernel parameter\n"); "acpi_backlight kernel parameter.\n");
return 1; return 1;
} }
} else if (tp_features.bright_acpimode && brightness_enable > 1) { } else if (tp_features.bright_acpimode && brightness_enable > 1) {
printk(TPACPI_NOTICE pr_notice("Standard ACPI backlight interface not "
"Standard ACPI backlight interface not " "available, thinkpad_acpi native "
"available, thinkpad_acpi native " "brightness control enabled\n");
"brightness control enabled\n");
} }
/* /*
...@@ -6319,19 +6251,17 @@ static int __init brightness_init(struct ibm_init_struct *iibm) ...@@ -6319,19 +6251,17 @@ static int __init brightness_init(struct ibm_init_struct *iibm)
if (IS_ERR(ibm_backlight_device)) { if (IS_ERR(ibm_backlight_device)) {
int rc = PTR_ERR(ibm_backlight_device); int rc = PTR_ERR(ibm_backlight_device);
ibm_backlight_device = NULL; ibm_backlight_device = NULL;
printk(TPACPI_ERR "Could not register backlight device\n"); pr_err("Could not register backlight device\n");
return rc; return rc;
} }
vdbg_printk(TPACPI_DBG_INIT | TPACPI_DBG_BRGHT, vdbg_printk(TPACPI_DBG_INIT | TPACPI_DBG_BRGHT,
"brightness is supported\n"); "brightness is supported\n");
if (quirks & TPACPI_BRGHT_Q_ASK) { if (quirks & TPACPI_BRGHT_Q_ASK) {
printk(TPACPI_NOTICE pr_notice("brightness: will use unverified default: "
"brightness: will use unverified default: " "brightness_mode=%d\n", brightness_mode);
"brightness_mode=%d\n", brightness_mode); pr_notice("brightness: please report to %s whether it works well "
printk(TPACPI_NOTICE "or not on your ThinkPad\n", TPACPI_MAIL);
"brightness: please report to %s whether it works well "
"or not on your ThinkPad\n", TPACPI_MAIL);
} }
/* Added by mistake in early 2007. Probably useless, but it could /* Added by mistake in early 2007. Probably useless, but it could
...@@ -6804,8 +6734,7 @@ static int __init volume_create_alsa_mixer(void) ...@@ -6804,8 +6734,7 @@ static int __init volume_create_alsa_mixer(void)
rc = snd_card_create(alsa_index, alsa_id, THIS_MODULE, rc = snd_card_create(alsa_index, alsa_id, THIS_MODULE,
sizeof(struct tpacpi_alsa_data), &card); sizeof(struct tpacpi_alsa_data), &card);
if (rc < 0 || !card) { if (rc < 0 || !card) {
printk(TPACPI_ERR pr_err("Failed to create ALSA card structures: %d\n", rc);
"Failed to create ALSA card structures: %d\n", rc);
return 1; return 1;
} }
...@@ -6839,9 +6768,8 @@ static int __init volume_create_alsa_mixer(void) ...@@ -6839,9 +6768,8 @@ static int __init volume_create_alsa_mixer(void)
ctl_vol = snd_ctl_new1(&volume_alsa_control_vol, NULL); ctl_vol = snd_ctl_new1(&volume_alsa_control_vol, NULL);
rc = snd_ctl_add(card, ctl_vol); rc = snd_ctl_add(card, ctl_vol);
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR pr_err("Failed to create ALSA volume control: %d\n",
"Failed to create ALSA volume control: %d\n", rc);
rc);
goto err_exit; goto err_exit;
} }
data->ctl_vol_id = &ctl_vol->id; data->ctl_vol_id = &ctl_vol->id;
...@@ -6850,8 +6778,7 @@ static int __init volume_create_alsa_mixer(void) ...@@ -6850,8 +6778,7 @@ static int __init volume_create_alsa_mixer(void)
ctl_mute = snd_ctl_new1(&volume_alsa_control_mute, NULL); ctl_mute = snd_ctl_new1(&volume_alsa_control_mute, NULL);
rc = snd_ctl_add(card, ctl_mute); rc = snd_ctl_add(card, ctl_mute);
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR "Failed to create ALSA mute control: %d\n", pr_err("Failed to create ALSA mute control: %d\n", rc);
rc);
goto err_exit; goto err_exit;
} }
data->ctl_mute_id = &ctl_mute->id; data->ctl_mute_id = &ctl_mute->id;
...@@ -6859,7 +6786,7 @@ static int __init volume_create_alsa_mixer(void) ...@@ -6859,7 +6786,7 @@ static int __init volume_create_alsa_mixer(void)
snd_card_set_dev(card, &tpacpi_pdev->dev); snd_card_set_dev(card, &tpacpi_pdev->dev);
rc = snd_card_register(card); rc = snd_card_register(card);
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR "Failed to register ALSA card: %d\n", rc); pr_err("Failed to register ALSA card: %d\n", rc);
goto err_exit; goto err_exit;
} }
...@@ -6915,9 +6842,8 @@ static int __init volume_init(struct ibm_init_struct *iibm) ...@@ -6915,9 +6842,8 @@ static int __init volume_init(struct ibm_init_struct *iibm)
return -EINVAL; return -EINVAL;
if (volume_mode == TPACPI_VOL_MODE_UCMS_STEP) { if (volume_mode == TPACPI_VOL_MODE_UCMS_STEP) {
printk(TPACPI_ERR pr_err("UCMS step volume mode not implemented, "
"UCMS step volume mode not implemented, " "please contact %s\n", TPACPI_MAIL);
"please contact %s\n", TPACPI_MAIL);
return 1; return 1;
} }
...@@ -6981,13 +6907,11 @@ static int __init volume_init(struct ibm_init_struct *iibm) ...@@ -6981,13 +6907,11 @@ static int __init volume_init(struct ibm_init_struct *iibm)
rc = volume_create_alsa_mixer(); rc = volume_create_alsa_mixer();
if (rc) { if (rc) {
printk(TPACPI_ERR pr_err("Could not create the ALSA mixer interface\n");
"Could not create the ALSA mixer interface\n");
return rc; return rc;
} }
printk(TPACPI_INFO pr_info("Console audio control enabled, mode: %s\n",
"Console audio control enabled, mode: %s\n",
(volume_control_allowed) ? (volume_control_allowed) ?
"override (read/write)" : "override (read/write)" :
"monitor (read only)"); "monitor (read only)");
...@@ -7049,12 +6973,10 @@ static int volume_write(char *buf) ...@@ -7049,12 +6973,10 @@ static int volume_write(char *buf)
if (!volume_control_allowed && tpacpi_lifecycle != TPACPI_LIFE_INIT) { if (!volume_control_allowed && tpacpi_lifecycle != TPACPI_LIFE_INIT) {
if (unlikely(!tp_warned.volume_ctrl_forbidden)) { if (unlikely(!tp_warned.volume_ctrl_forbidden)) {
tp_warned.volume_ctrl_forbidden = 1; tp_warned.volume_ctrl_forbidden = 1;
printk(TPACPI_NOTICE pr_notice("Console audio control in monitor mode, "
"Console audio control in monitor mode, " "changes are not allowed\n");
"changes are not allowed.\n"); pr_notice("Use the volume_control=1 module parameter "
printk(TPACPI_NOTICE "to enable volume control\n");
"Use the volume_control=1 module parameter "
"to enable volume control\n");
} }
return -EPERM; return -EPERM;
} }
...@@ -7129,8 +7051,7 @@ static void inline volume_alsa_notify_change(void) ...@@ -7129,8 +7051,7 @@ static void inline volume_alsa_notify_change(void)
static int __init volume_init(struct ibm_init_struct *iibm) static int __init volume_init(struct ibm_init_struct *iibm)
{ {
printk(TPACPI_INFO pr_info("volume: disabled as there is no ALSA support in this kernel\n");
"volume: disabled as there is no ALSA support in this kernel\n");
return 1; return 1;
} }
...@@ -7337,9 +7258,8 @@ TPACPI_HANDLE(sfan, ec, "SFAN", /* 570 */ ...@@ -7337,9 +7258,8 @@ TPACPI_HANDLE(sfan, ec, "SFAN", /* 570 */
static void fan_quirk1_setup(void) static void fan_quirk1_setup(void)
{ {
if (fan_control_initial_status == 0x07) { if (fan_control_initial_status == 0x07) {
printk(TPACPI_NOTICE pr_notice("fan_init: initial fan status is unknown, "
"fan_init: initial fan status is unknown, " "assuming it is in auto mode\n");
"assuming it is in auto mode\n");
tp_features.fan_ctrl_status_undef = 1; tp_features.fan_ctrl_status_undef = 1;
} }
} }
...@@ -7726,8 +7646,7 @@ static void fan_watchdog_reset(void) ...@@ -7726,8 +7646,7 @@ static void fan_watchdog_reset(void)
if (!queue_delayed_work(tpacpi_wq, &fan_watchdog_task, if (!queue_delayed_work(tpacpi_wq, &fan_watchdog_task,
msecs_to_jiffies(fan_watchdog_maxinterval msecs_to_jiffies(fan_watchdog_maxinterval
* 1000))) { * 1000))) {
printk(TPACPI_ERR pr_err("failed to queue the fan watchdog, "
"failed to queue the fan watchdog, "
"watchdog will not trigger\n"); "watchdog will not trigger\n");
} }
} else } else
...@@ -7741,11 +7660,11 @@ static void fan_watchdog_fire(struct work_struct *ignored) ...@@ -7741,11 +7660,11 @@ static void fan_watchdog_fire(struct work_struct *ignored)
if (tpacpi_lifecycle != TPACPI_LIFE_RUNNING) if (tpacpi_lifecycle != TPACPI_LIFE_RUNNING)
return; return;
printk(TPACPI_NOTICE "fan watchdog: enabling fan\n"); pr_notice("fan watchdog: enabling fan\n");
rc = fan_set_enable(); rc = fan_set_enable();
if (rc < 0) { if (rc < 0) {
printk(TPACPI_ERR "fan watchdog: error %d while enabling fan, " pr_err("fan watchdog: error %d while enabling fan, "
"will try again later...\n", -rc); "will try again later...\n", -rc);
/* reschedule for later */ /* reschedule for later */
fan_watchdog_reset(); fan_watchdog_reset();
} }
...@@ -8049,8 +7968,7 @@ static int __init fan_init(struct ibm_init_struct *iibm) ...@@ -8049,8 +7968,7 @@ static int __init fan_init(struct ibm_init_struct *iibm)
"secondary fan support enabled\n"); "secondary fan support enabled\n");
} }
} else { } else {
printk(TPACPI_ERR pr_err("ThinkPad ACPI EC access misbehaving, "
"ThinkPad ACPI EC access misbehaving, "
"fan status and control unavailable\n"); "fan status and control unavailable\n");
return 1; return 1;
} }
...@@ -8150,9 +8068,8 @@ static void fan_suspend(pm_message_t state) ...@@ -8150,9 +8068,8 @@ static void fan_suspend(pm_message_t state)
fan_control_resume_level = 0; fan_control_resume_level = 0;
rc = fan_get_status_safe(&fan_control_resume_level); rc = fan_get_status_safe(&fan_control_resume_level);
if (rc < 0) if (rc < 0)
printk(TPACPI_NOTICE pr_notice("failed to read fan level for later "
"failed to read fan level for later " "restore during resume: %d\n", rc);
"restore during resume: %d\n", rc);
/* if it is undefined, don't attempt to restore it. /* if it is undefined, don't attempt to restore it.
* KEEP THIS LAST */ * KEEP THIS LAST */
...@@ -8207,13 +8124,11 @@ static void fan_resume(void) ...@@ -8207,13 +8124,11 @@ static void fan_resume(void)
return; return;
} }
if (do_set) { if (do_set) {
printk(TPACPI_NOTICE pr_notice("restoring fan level to 0x%02x\n",
"restoring fan level to 0x%02x\n", fan_control_resume_level);
fan_control_resume_level);
rc = fan_set_level_safe(fan_control_resume_level); rc = fan_set_level_safe(fan_control_resume_level);
if (rc < 0) if (rc < 0)
printk(TPACPI_NOTICE pr_notice("failed to restore fan level: %d\n", rc);
"failed to restore fan level: %d\n", rc);
} }
} }
...@@ -8305,8 +8220,8 @@ static int fan_write_cmd_level(const char *cmd, int *rc) ...@@ -8305,8 +8220,8 @@ static int fan_write_cmd_level(const char *cmd, int *rc)
*rc = fan_set_level_safe(level); *rc = fan_set_level_safe(level);
if (*rc == -ENXIO) if (*rc == -ENXIO)
printk(TPACPI_ERR "level command accepted for unsupported " pr_err("level command accepted for unsupported access mode %d\n",
"access mode %d", fan_control_access_mode); fan_control_access_mode);
else if (!*rc) else if (!*rc)
tpacpi_disclose_usertask("procfs fan", tpacpi_disclose_usertask("procfs fan",
"set level to %d\n", level); "set level to %d\n", level);
...@@ -8321,8 +8236,8 @@ static int fan_write_cmd_enable(const char *cmd, int *rc) ...@@ -8321,8 +8236,8 @@ static int fan_write_cmd_enable(const char *cmd, int *rc)
*rc = fan_set_enable(); *rc = fan_set_enable();
if (*rc == -ENXIO) if (*rc == -ENXIO)
printk(TPACPI_ERR "enable command accepted for unsupported " pr_err("enable command accepted for unsupported access mode %d\n",
"access mode %d", fan_control_access_mode); fan_control_access_mode);
else if (!*rc) else if (!*rc)
tpacpi_disclose_usertask("procfs fan", "enable\n"); tpacpi_disclose_usertask("procfs fan", "enable\n");
...@@ -8336,8 +8251,8 @@ static int fan_write_cmd_disable(const char *cmd, int *rc) ...@@ -8336,8 +8251,8 @@ static int fan_write_cmd_disable(const char *cmd, int *rc)
*rc = fan_set_disable(); *rc = fan_set_disable();
if (*rc == -ENXIO) if (*rc == -ENXIO)
printk(TPACPI_ERR "disable command accepted for unsupported " pr_err("disable command accepted for unsupported access mode %d\n",
"access mode %d", fan_control_access_mode); fan_control_access_mode);
else if (!*rc) else if (!*rc)
tpacpi_disclose_usertask("procfs fan", "disable\n"); tpacpi_disclose_usertask("procfs fan", "disable\n");
...@@ -8356,8 +8271,8 @@ static int fan_write_cmd_speed(const char *cmd, int *rc) ...@@ -8356,8 +8271,8 @@ static int fan_write_cmd_speed(const char *cmd, int *rc)
*rc = fan_set_speed(speed); *rc = fan_set_speed(speed);
if (*rc == -ENXIO) if (*rc == -ENXIO)
printk(TPACPI_ERR "speed command accepted for unsupported " pr_err("speed command accepted for unsupported access mode %d\n",
"access mode %d", fan_control_access_mode); fan_control_access_mode);
else if (!*rc) else if (!*rc)
tpacpi_disclose_usertask("procfs fan", tpacpi_disclose_usertask("procfs fan",
"set speed to %d\n", speed); "set speed to %d\n", speed);
...@@ -8560,8 +8475,8 @@ static int __init ibm_init(struct ibm_init_struct *iibm) ...@@ -8560,8 +8475,8 @@ static int __init ibm_init(struct ibm_init_struct *iibm)
if (ibm->acpi->notify) { if (ibm->acpi->notify) {
ret = setup_acpi_notify(ibm); ret = setup_acpi_notify(ibm);
if (ret == -ENODEV) { if (ret == -ENODEV) {
printk(TPACPI_NOTICE "disabling subdriver %s\n", pr_notice("disabling subdriver %s\n",
ibm->name); ibm->name);
ret = 0; ret = 0;
goto err_out; goto err_out;
} }
...@@ -8583,8 +8498,7 @@ static int __init ibm_init(struct ibm_init_struct *iibm) ...@@ -8583,8 +8498,7 @@ static int __init ibm_init(struct ibm_init_struct *iibm)
entry = proc_create_data(ibm->name, mode, proc_dir, entry = proc_create_data(ibm->name, mode, proc_dir,
&dispatch_proc_fops, ibm); &dispatch_proc_fops, ibm);
if (!entry) { if (!entry) {
printk(TPACPI_ERR "unable to create proc entry %s\n", pr_err("unable to create proc entry %s\n", ibm->name);
ibm->name);
ret = -ENODEV; ret = -ENODEV;
goto err_out; goto err_out;
} }
...@@ -8683,13 +8597,11 @@ static int __must_check __init get_thinkpad_model_data( ...@@ -8683,13 +8597,11 @@ static int __must_check __init get_thinkpad_model_data(
tp->ec_release = (ec_fw_string[4] << 8) tp->ec_release = (ec_fw_string[4] << 8)
| ec_fw_string[5]; | ec_fw_string[5];
} else { } else {
printk(TPACPI_NOTICE pr_notice("ThinkPad firmware release %s "
"ThinkPad firmware release %s " "doesn't match the known patterns\n",
"doesn't match the known patterns\n", ec_fw_string);
ec_fw_string); pr_notice("please report this to %s\n",
printk(TPACPI_NOTICE TPACPI_MAIL);
"please report this to %s\n",
TPACPI_MAIL);
} }
break; break;
} }
...@@ -8733,8 +8645,7 @@ static int __init probe_for_thinkpad(void) ...@@ -8733,8 +8645,7 @@ static int __init probe_for_thinkpad(void)
tpacpi_acpi_handle_locate("ec", TPACPI_ACPI_EC_HID, &ec_handle); tpacpi_acpi_handle_locate("ec", TPACPI_ACPI_EC_HID, &ec_handle);
if (!ec_handle) { if (!ec_handle) {
if (is_thinkpad) if (is_thinkpad)
printk(TPACPI_ERR pr_err("Not yet supported ThinkPad detected!\n");
"Not yet supported ThinkPad detected!\n");
return -ENODEV; return -ENODEV;
} }
...@@ -8746,10 +8657,10 @@ static int __init probe_for_thinkpad(void) ...@@ -8746,10 +8657,10 @@ static int __init probe_for_thinkpad(void)
static void __init thinkpad_acpi_init_banner(void) static void __init thinkpad_acpi_init_banner(void)
{ {
printk(TPACPI_INFO "%s v%s\n", TPACPI_DESC, TPACPI_VERSION); pr_info("%s v%s\n", TPACPI_DESC, TPACPI_VERSION);
printk(TPACPI_INFO "%s\n", TPACPI_URL); pr_info("%s\n", TPACPI_URL);
printk(TPACPI_INFO "ThinkPad BIOS %s, EC %s\n", pr_info("ThinkPad BIOS %s, EC %s\n",
(thinkpad_id.bios_version_str) ? (thinkpad_id.bios_version_str) ?
thinkpad_id.bios_version_str : "unknown", thinkpad_id.bios_version_str : "unknown",
(thinkpad_id.ec_version_str) ? (thinkpad_id.ec_version_str) ?
...@@ -8758,7 +8669,7 @@ static void __init thinkpad_acpi_init_banner(void) ...@@ -8758,7 +8669,7 @@ static void __init thinkpad_acpi_init_banner(void)
BUG_ON(!thinkpad_id.vendor); BUG_ON(!thinkpad_id.vendor);
if (thinkpad_id.model_str) if (thinkpad_id.model_str)
printk(TPACPI_INFO "%s %s, model %s\n", pr_info("%s %s, model %s\n",
(thinkpad_id.vendor == PCI_VENDOR_ID_IBM) ? (thinkpad_id.vendor == PCI_VENDOR_ID_IBM) ?
"IBM" : ((thinkpad_id.vendor == "IBM" : ((thinkpad_id.vendor ==
PCI_VENDOR_ID_LENOVO) ? PCI_VENDOR_ID_LENOVO) ?
...@@ -9024,8 +8935,7 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9024,8 +8935,7 @@ static int __init thinkpad_acpi_module_init(void)
ret = get_thinkpad_model_data(&thinkpad_id); ret = get_thinkpad_model_data(&thinkpad_id);
if (ret) { if (ret) {
printk(TPACPI_ERR pr_err("unable to get DMI data: %d\n", ret);
"unable to get DMI data: %d\n", ret);
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9051,16 +8961,14 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9051,16 +8961,14 @@ static int __init thinkpad_acpi_module_init(void)
proc_dir = proc_mkdir(TPACPI_PROC_DIR, acpi_root_dir); proc_dir = proc_mkdir(TPACPI_PROC_DIR, acpi_root_dir);
if (!proc_dir) { if (!proc_dir) {
printk(TPACPI_ERR pr_err("unable to create proc dir " TPACPI_PROC_DIR "\n");
"unable to create proc dir " TPACPI_PROC_DIR);
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return -ENODEV; return -ENODEV;
} }
ret = platform_driver_register(&tpacpi_pdriver); ret = platform_driver_register(&tpacpi_pdriver);
if (ret) { if (ret) {
printk(TPACPI_ERR pr_err("unable to register main platform driver\n");
"unable to register main platform driver\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9068,8 +8976,7 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9068,8 +8976,7 @@ static int __init thinkpad_acpi_module_init(void)
ret = platform_driver_register(&tpacpi_hwmon_pdriver); ret = platform_driver_register(&tpacpi_hwmon_pdriver);
if (ret) { if (ret) {
printk(TPACPI_ERR pr_err("unable to register hwmon platform driver\n");
"unable to register hwmon platform driver\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9082,8 +8989,7 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9082,8 +8989,7 @@ static int __init thinkpad_acpi_module_init(void)
&tpacpi_hwmon_pdriver.driver); &tpacpi_hwmon_pdriver.driver);
} }
if (ret) { if (ret) {
printk(TPACPI_ERR pr_err("unable to create sysfs driver attributes\n");
"unable to create sysfs driver attributes\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9096,7 +9002,7 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9096,7 +9002,7 @@ static int __init thinkpad_acpi_module_init(void)
if (IS_ERR(tpacpi_pdev)) { if (IS_ERR(tpacpi_pdev)) {
ret = PTR_ERR(tpacpi_pdev); ret = PTR_ERR(tpacpi_pdev);
tpacpi_pdev = NULL; tpacpi_pdev = NULL;
printk(TPACPI_ERR "unable to register platform device\n"); pr_err("unable to register platform device\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9106,16 +9012,14 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9106,16 +9012,14 @@ static int __init thinkpad_acpi_module_init(void)
if (IS_ERR(tpacpi_sensors_pdev)) { if (IS_ERR(tpacpi_sensors_pdev)) {
ret = PTR_ERR(tpacpi_sensors_pdev); ret = PTR_ERR(tpacpi_sensors_pdev);
tpacpi_sensors_pdev = NULL; tpacpi_sensors_pdev = NULL;
printk(TPACPI_ERR pr_err("unable to register hwmon platform device\n");
"unable to register hwmon platform device\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
ret = device_create_file(&tpacpi_sensors_pdev->dev, ret = device_create_file(&tpacpi_sensors_pdev->dev,
&dev_attr_thinkpad_acpi_pdev_name); &dev_attr_thinkpad_acpi_pdev_name);
if (ret) { if (ret) {
printk(TPACPI_ERR pr_err("unable to create sysfs hwmon device attributes\n");
"unable to create sysfs hwmon device attributes\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
...@@ -9124,14 +9028,14 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9124,14 +9028,14 @@ static int __init thinkpad_acpi_module_init(void)
if (IS_ERR(tpacpi_hwmon)) { if (IS_ERR(tpacpi_hwmon)) {
ret = PTR_ERR(tpacpi_hwmon); ret = PTR_ERR(tpacpi_hwmon);
tpacpi_hwmon = NULL; tpacpi_hwmon = NULL;
printk(TPACPI_ERR "unable to register hwmon device\n"); pr_err("unable to register hwmon device\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} }
mutex_init(&tpacpi_inputdev_send_mutex); mutex_init(&tpacpi_inputdev_send_mutex);
tpacpi_inputdev = input_allocate_device(); tpacpi_inputdev = input_allocate_device();
if (!tpacpi_inputdev) { if (!tpacpi_inputdev) {
printk(TPACPI_ERR "unable to allocate input device\n"); pr_err("unable to allocate input device\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return -ENOMEM; return -ENOMEM;
} else { } else {
...@@ -9163,7 +9067,7 @@ static int __init thinkpad_acpi_module_init(void) ...@@ -9163,7 +9067,7 @@ static int __init thinkpad_acpi_module_init(void)
ret = input_register_device(tpacpi_inputdev); ret = input_register_device(tpacpi_inputdev);
if (ret < 0) { if (ret < 0) {
printk(TPACPI_ERR "unable to register input device\n"); pr_err("unable to register input device\n");
thinkpad_acpi_module_exit(); thinkpad_acpi_module_exit();
return ret; return ret;
} else { } else {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7