| Message ID | 20231206212140.7458-1-mario.limonciello@amd.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/2] Input: i8042: Avoid probing if no keyboard and mouse are set in quirks | expand |
On Wed, 06 Dec, 2023 15:21:39 -0600 Mario Limonciello <mario.limonciello@amd.com> wrote: > Some laptops have an i8042 controller in the SOC, nothing mentioned in > ACPI PNP and nothing connected to the controller. Add the ability to > skip probing in this case. > > Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> > --- Thanks. I think this is a good choice for handling the issue you presented with the Framework 16. Reviewed-by: Rahul Rameshbabu <rrameshbabu@nvidia.com>
On 12/6/2023 15:55, Rahul Rameshbabu wrote: > On Wed, 06 Dec, 2023 15:21:39 -0600 Mario Limonciello <mario.limonciello@amd.com> wrote: >> Some laptops have an i8042 controller in the SOC, nothing mentioned in >> ACPI PNP and nothing connected to the controller. Add the ability to >> skip probing in this case. >> >> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> >> --- > > Thanks. I think this is a good choice for handling the issue you > presented with the Framework 16. > > Reviewed-by: Rahul Rameshbabu <rrameshbabu@nvidia.com> Thanks! Dmitry, Are you OK with this? The other direction I considered was to add a DMI BIOS year check and "only continue to probe" non PNP devices on systems older than 2023. That could let you cut and run without needing to continue to add quirks like this but it could be riskier.
diff --git a/drivers/input/serio/i8042-acpipnpio.h b/drivers/input/serio/i8042-acpipnpio.h index 9c39553d30fa..0fd88bbfaee1 100644 --- a/drivers/input/serio/i8042-acpipnpio.h +++ b/drivers/input/serio/i8042-acpipnpio.h @@ -1707,6 +1707,9 @@ static int __init i8042_platform_init(void) ""); #endif + if (i8042_nokbd && i8042_noaux) + return -ENODEV; + retval = i8042_pnp_init(); if (retval) return retval;
Some laptops have an i8042 controller in the SOC, nothing mentioned in ACPI PNP and nothing connected to the controller. Add the ability to skip probing in this case. Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> --- drivers/input/serio/i8042-acpipnpio.h | 3 +++ 1 file changed, 3 insertions(+)