Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/22286 )
Change subject: winbond/w83667hg-a: Disable mouse controller also during resume ......................................................................
winbond/w83667hg-a: Disable mouse controller also during resume
There is no reason to not disable the controller during resume. That way, no ASL is needed.
Change-Id: I282a03647ee0958abb118fafe306abe5782db71c Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/22286 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M src/superio/winbond/w83667hg-a/superio.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/src/superio/winbond/w83667hg-a/superio.c b/src/superio/winbond/w83667hg-a/superio.c index bc7bd92..6850621 100644 --- a/src/superio/winbond/w83667hg-a/superio.c +++ b/src/superio/winbond/w83667hg-a/superio.c @@ -36,7 +36,7 @@
mouse_detected = pc_keyboard_init(PROBE_AUX_DEVICE);
- if (!mouse_detected && !acpi_is_wakeup_s3()) { + if (!mouse_detected) { printk(BIOS_INFO, "%s: Disable mouse controller.", __func__); pnp_enter_conf_mode(dev);