Commit 9222ba68 authored by Takashi Iwai's avatar Takashi Iwai Committed by Dmitry Torokhov

Input: i8042 - add deferred probe support

We've got a bug report about the non-working keyboard on ASUS ZenBook
UX425UA.  It seems that the PS/2 device isn't ready immediately at
boot but takes some seconds to get ready.  Until now, the only
workaround is to defer the probe, but it's available only when the
driver is a module.  However, many distros, including openSUSE as in
the original report, build the PS/2 input drivers into kernel, hence
it won't work easily.

This patch adds the support for the deferred probe for i8042 stuff as
a workaround of the problem above.  When the deferred probe mode is
enabled and the device couldn't be probed, it'll be repeated with the
standard deferred probe mechanism.

The deferred probe mode is enabled either via the new option
i8042.probe_defer or via the quirk table entry.  As of this patch, the
quirk table contains only ASUS ZenBook UX425UA.

The deferred probe part is based on Fabio's initial work.

BugLink: https://bugzilla.suse.com/show_bug.cgi?id=1190256Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
Tested-by: default avatarSamuel Čavoj <samuel@cavoj.net>
Link: https://lore.kernel.org/r/20211117063757.11380-1-tiwai@suse.deSigned-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent efe6f16c
...@@ -1627,6 +1627,8 @@ ...@@ -1627,6 +1627,8 @@
architectures force reset to be always executed architectures force reset to be always executed
i8042.unlock [HW] Unlock (ignore) the keylock i8042.unlock [HW] Unlock (ignore) the keylock
i8042.kbdreset [HW] Reset device connected to KBD port i8042.kbdreset [HW] Reset device connected to KBD port
i8042.probe_defer
[HW] Allow deferred probing upon i8042 probe errors
i810= [HW,DRM] i810= [HW,DRM]
......
...@@ -995,6 +995,17 @@ static const struct dmi_system_id __initconst i8042_dmi_kbdreset_table[] = { ...@@ -995,6 +995,17 @@ static const struct dmi_system_id __initconst i8042_dmi_kbdreset_table[] = {
{ } { }
}; };
static const struct dmi_system_id i8042_dmi_probe_defer_table[] __initconst = {
{
/* ASUS ZenBook UX425UA */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
DMI_MATCH(DMI_PRODUCT_NAME, "ZenBook UX425UA"),
},
},
{ }
};
#endif /* CONFIG_X86 */ #endif /* CONFIG_X86 */
#ifdef CONFIG_PNP #ifdef CONFIG_PNP
...@@ -1315,6 +1326,9 @@ static int __init i8042_platform_init(void) ...@@ -1315,6 +1326,9 @@ static int __init i8042_platform_init(void)
if (dmi_check_system(i8042_dmi_kbdreset_table)) if (dmi_check_system(i8042_dmi_kbdreset_table))
i8042_kbdreset = true; i8042_kbdreset = true;
if (dmi_check_system(i8042_dmi_probe_defer_table))
i8042_probe_defer = true;
/* /*
* A20 was already enabled during early kernel init. But some buggy * A20 was already enabled during early kernel init. But some buggy
* BIOSes (in MSI Laptops) require A20 to be enabled using 8042 to * BIOSes (in MSI Laptops) require A20 to be enabled using 8042 to
......
...@@ -45,6 +45,10 @@ static bool i8042_unlock; ...@@ -45,6 +45,10 @@ static bool i8042_unlock;
module_param_named(unlock, i8042_unlock, bool, 0); module_param_named(unlock, i8042_unlock, bool, 0);
MODULE_PARM_DESC(unlock, "Ignore keyboard lock."); MODULE_PARM_DESC(unlock, "Ignore keyboard lock.");
static bool i8042_probe_defer;
module_param_named(probe_defer, i8042_probe_defer, bool, 0);
MODULE_PARM_DESC(probe_defer, "Allow deferred probing.");
enum i8042_controller_reset_mode { enum i8042_controller_reset_mode {
I8042_RESET_NEVER, I8042_RESET_NEVER,
I8042_RESET_ALWAYS, I8042_RESET_ALWAYS,
...@@ -711,7 +715,7 @@ static int i8042_set_mux_mode(bool multiplex, unsigned char *mux_version) ...@@ -711,7 +715,7 @@ static int i8042_set_mux_mode(bool multiplex, unsigned char *mux_version)
* LCS/Telegraphics. * LCS/Telegraphics.
*/ */
static int __init i8042_check_mux(void) static int i8042_check_mux(void)
{ {
unsigned char mux_version; unsigned char mux_version;
...@@ -740,10 +744,10 @@ static int __init i8042_check_mux(void) ...@@ -740,10 +744,10 @@ static int __init i8042_check_mux(void)
/* /*
* The following is used to test AUX IRQ delivery. * The following is used to test AUX IRQ delivery.
*/ */
static struct completion i8042_aux_irq_delivered __initdata; static struct completion i8042_aux_irq_delivered;
static bool i8042_irq_being_tested __initdata; static bool i8042_irq_being_tested;
static irqreturn_t __init i8042_aux_test_irq(int irq, void *dev_id) static irqreturn_t i8042_aux_test_irq(int irq, void *dev_id)
{ {
unsigned long flags; unsigned long flags;
unsigned char str, data; unsigned char str, data;
...@@ -770,7 +774,7 @@ static irqreturn_t __init i8042_aux_test_irq(int irq, void *dev_id) ...@@ -770,7 +774,7 @@ static irqreturn_t __init i8042_aux_test_irq(int irq, void *dev_id)
* verifies success by readinng CTR. Used when testing for presence of AUX * verifies success by readinng CTR. Used when testing for presence of AUX
* port. * port.
*/ */
static int __init i8042_toggle_aux(bool on) static int i8042_toggle_aux(bool on)
{ {
unsigned char param; unsigned char param;
int i; int i;
...@@ -798,7 +802,7 @@ static int __init i8042_toggle_aux(bool on) ...@@ -798,7 +802,7 @@ static int __init i8042_toggle_aux(bool on)
* the presence of an AUX interface. * the presence of an AUX interface.
*/ */
static int __init i8042_check_aux(void) static int i8042_check_aux(void)
{ {
int retval = -1; int retval = -1;
bool irq_registered = false; bool irq_registered = false;
...@@ -1005,7 +1009,7 @@ static int i8042_controller_init(void) ...@@ -1005,7 +1009,7 @@ static int i8042_controller_init(void)
if (i8042_command(&ctr[n++ % 2], I8042_CMD_CTL_RCTR)) { if (i8042_command(&ctr[n++ % 2], I8042_CMD_CTL_RCTR)) {
pr_err("Can't read CTR while initializing i8042\n"); pr_err("Can't read CTR while initializing i8042\n");
return -EIO; return i8042_probe_defer ? -EPROBE_DEFER : -EIO;
} }
} while (n < 2 || ctr[0] != ctr[1]); } while (n < 2 || ctr[0] != ctr[1]);
...@@ -1320,7 +1324,7 @@ static void i8042_shutdown(struct platform_device *dev) ...@@ -1320,7 +1324,7 @@ static void i8042_shutdown(struct platform_device *dev)
i8042_controller_reset(false); i8042_controller_reset(false);
} }
static int __init i8042_create_kbd_port(void) static int i8042_create_kbd_port(void)
{ {
struct serio *serio; struct serio *serio;
struct i8042_port *port = &i8042_ports[I8042_KBD_PORT_NO]; struct i8042_port *port = &i8042_ports[I8042_KBD_PORT_NO];
...@@ -1349,7 +1353,7 @@ static int __init i8042_create_kbd_port(void) ...@@ -1349,7 +1353,7 @@ static int __init i8042_create_kbd_port(void)
return 0; return 0;
} }
static int __init i8042_create_aux_port(int idx) static int i8042_create_aux_port(int idx)
{ {
struct serio *serio; struct serio *serio;
int port_no = idx < 0 ? I8042_AUX_PORT_NO : I8042_MUX_PORT_NO + idx; int port_no = idx < 0 ? I8042_AUX_PORT_NO : I8042_MUX_PORT_NO + idx;
...@@ -1386,13 +1390,13 @@ static int __init i8042_create_aux_port(int idx) ...@@ -1386,13 +1390,13 @@ static int __init i8042_create_aux_port(int idx)
return 0; return 0;
} }
static void __init i8042_free_kbd_port(void) static void i8042_free_kbd_port(void)
{ {
kfree(i8042_ports[I8042_KBD_PORT_NO].serio); kfree(i8042_ports[I8042_KBD_PORT_NO].serio);
i8042_ports[I8042_KBD_PORT_NO].serio = NULL; i8042_ports[I8042_KBD_PORT_NO].serio = NULL;
} }
static void __init i8042_free_aux_ports(void) static void i8042_free_aux_ports(void)
{ {
int i; int i;
...@@ -1402,7 +1406,7 @@ static void __init i8042_free_aux_ports(void) ...@@ -1402,7 +1406,7 @@ static void __init i8042_free_aux_ports(void)
} }
} }
static void __init i8042_register_ports(void) static void i8042_register_ports(void)
{ {
int i; int i;
...@@ -1443,7 +1447,7 @@ static void i8042_free_irqs(void) ...@@ -1443,7 +1447,7 @@ static void i8042_free_irqs(void)
i8042_aux_irq_registered = i8042_kbd_irq_registered = false; i8042_aux_irq_registered = i8042_kbd_irq_registered = false;
} }
static int __init i8042_setup_aux(void) static int i8042_setup_aux(void)
{ {
int (*aux_enable)(void); int (*aux_enable)(void);
int error; int error;
...@@ -1485,7 +1489,7 @@ static int __init i8042_setup_aux(void) ...@@ -1485,7 +1489,7 @@ static int __init i8042_setup_aux(void)
return error; return error;
} }
static int __init i8042_setup_kbd(void) static int i8042_setup_kbd(void)
{ {
int error; int error;
...@@ -1535,7 +1539,7 @@ static int i8042_kbd_bind_notifier(struct notifier_block *nb, ...@@ -1535,7 +1539,7 @@ static int i8042_kbd_bind_notifier(struct notifier_block *nb,
return 0; return 0;
} }
static int __init i8042_probe(struct platform_device *dev) static int i8042_probe(struct platform_device *dev)
{ {
int error; int error;
...@@ -1600,6 +1604,7 @@ static struct platform_driver i8042_driver = { ...@@ -1600,6 +1604,7 @@ static struct platform_driver i8042_driver = {
.pm = &i8042_pm_ops, .pm = &i8042_pm_ops,
#endif #endif
}, },
.probe = i8042_probe,
.remove = i8042_remove, .remove = i8042_remove,
.shutdown = i8042_shutdown, .shutdown = i8042_shutdown,
}; };
...@@ -1610,7 +1615,6 @@ static struct notifier_block i8042_kbd_bind_notifier_block = { ...@@ -1610,7 +1615,6 @@ static struct notifier_block i8042_kbd_bind_notifier_block = {
static int __init i8042_init(void) static int __init i8042_init(void)
{ {
struct platform_device *pdev;
int err; int err;
dbg_init(); dbg_init();
...@@ -1626,17 +1630,29 @@ static int __init i8042_init(void) ...@@ -1626,17 +1630,29 @@ static int __init i8042_init(void)
/* Set this before creating the dev to allow i8042_command to work right away */ /* Set this before creating the dev to allow i8042_command to work right away */
i8042_present = true; i8042_present = true;
pdev = platform_create_bundle(&i8042_driver, i8042_probe, NULL, 0, NULL, 0); err = platform_driver_register(&i8042_driver);
if (IS_ERR(pdev)) { if (err)
err = PTR_ERR(pdev);
goto err_platform_exit; goto err_platform_exit;
i8042_platform_device = platform_device_alloc("i8042", -1);
if (!i8042_platform_device) {
err = -ENOMEM;
goto err_unregister_driver;
} }
err = platform_device_add(i8042_platform_device);
if (err)
goto err_free_device;
bus_register_notifier(&serio_bus, &i8042_kbd_bind_notifier_block); bus_register_notifier(&serio_bus, &i8042_kbd_bind_notifier_block);
panic_blink = i8042_panic_blink; panic_blink = i8042_panic_blink;
return 0; return 0;
err_free_device:
platform_device_put(i8042_platform_device);
err_unregister_driver:
platform_driver_unregister(&i8042_driver);
err_platform_exit: err_platform_exit:
i8042_platform_exit(); i8042_platform_exit();
return err; return err;
......
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