Commit da23a95e authored by Nícolas F. R. A. Prado's avatar Nícolas F. R. A. Prado Committed by Wojciech Macek
Browse files

FROMLIST: arm64: dts: mediatek: mt8186: Move wakeup to MTU3 to get working suspend



The current DT has the wakeup-source and mediatek,syscon-wakeup
properties in the XHCI nodes, which configures USB wakeup after powering
down the XHCI hardware block. However, since the XHCI controller is
behind an MTU3 (USB3 DRD controller), the MTU3 only gets powered down
after USB wakeup has been configured, causing the system to detect a
wakeup, and results in broken suspend support as the system resumes
immediately.

Move the wakeup properties to the MTU3 nodes so that USB wakeup is only
enabled after the MTU3 has powered down.

With this change in place, it is possible to suspend and resume, and
also to wakeup through USB, as tested on the Google Steelix (Lenovo 300e
Yoga Chromebook Gen 4).

Fixes: f6c3e61c ("arm64: dts: mediatek: mt8186: Add MTU3 nodes")
Suggested-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: default avatarNícolas F. R. A. Prado <nfraprado@collabora.com>
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
(am from https://patchwork.kernel.org/patch/13865526/)
(also found at https://lore.kernel.org/r/20241106-mt8186-suspend-with-usb-wakeup-v1-1-07734a4c8236@collabora.com)

Link: https://lore.kernel.org/r/20241106-mt8186-suspend-with-usb-wakeup-v1-1-07734a4c8236@collabora.com


Bug: 351926349
Test: suspend and resume Corsola device

Change-Id: I4c870dd36e8ee5c57dfc8b0f739596ae6cd5fd46
Signed-off-by: default avatarWojciech Macek <wmacek@google.com>
parent 22c0197d
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment