Most MSM8916 devices shipped without PSCI support in their TZ firmware. Instead, they use PSHOLD. These patches enable both PSCI and PSHOLD sysreset drivers in qcom_defconfig.
Caleb had already submitted ([1]) a patch to do the switch to PSCI sysreset driver as part of the UEFI capsule update series, but it seems that the sysreset patch didn't make it upstream.
It's worth noting that sysreset still won't work if booting with a control DT from upstream, as the msm8916.dtsi there includes a /psci node.
As far as I'm aware, only the db410c ever received a firmware update with any PSCI support, so I intend to send a patch to kernel to remove the default /psci node for msm8916 devices, assuming I get the green light ([2]).
I've tested in a local tree that includes a u-boot specific tweak to remove the /psci node, and confirmed that qcom_defconfig can reset both an msm8916 device and an sdm845 one.
[1]: https://lore.kernel.org/all/20240527-b4-qcom-capsule-updates-v2-6-47583d7ad4... [2]: https://github.com/msm8916-mainline/linux/issues/388
Signed-off-by: Sam Day me@samcday.com --- Sam Day (3): mach-snapdragon: use PSCI sysreset driver sysreset: qcom-pshold: remove ARCH_IPQ40XX dependency qcom_defconfig: enable SYSRESET_QCOM_PSHOLD
arch/arm/mach-snapdragon/board.c | 5 ----- configs/qcom_defconfig | 3 +++ drivers/sysreset/Kconfig | 1 - 3 files changed, 3 insertions(+), 6 deletions(-) --- base-commit: 8162f35a108441b8d7a44ac9266c1a64dbf79fd5 change-id: 20250121-msm8916-sysreset-580d9a65a759
Best regards,