Re: [PATCH v2] arm64: dts: qcom: hamoa: Add PSCI SYSTEM_RESET2 types

[Xin Liu <xin.liu@oss.qualcomm.com>]

On 3/18/2026 9:42 PM, Bjorn Andersson wrote:
> On Wed, Feb 25, 2026 at 09:41:13PM -0800, Xin Liu wrote:
>> Add support for SYSTEM_RESET2 vendor-specific resets as
>> reboot-modes in the psci node.  Describe the resets: "bootloader"
>> will cause device to reboot and stop in the bootloader's fastboot
>> mode.  "edl" will cause device to reboot into "emergency download
>> mode", which permits loading images via the Firehose protocol.
>>
>> Signed-off-by: Xin Liu <xin.liu@oss.qualcomm.com>
> How did you test this? I tried to apply this, but I'm getting 52 counts
> of:
>
> xyz.dtb: psci (arm,psci-1.0) 'reboot-mode' does not match any of the regexes: '^pinctrl-[0-9]+$', '^power-domain-'
>
> Am I doing something wrong?
>
> Regards,
> Bjorn

Hi Bjorn,

Thanks for testing!

The warning is expected without the companion binding patch. This DTS 
change depends on:
change-id: 20250709-arm-psci-system_reset2-vendor-reboots-46c80044afcf
from Shivendra Pratap's series, which adds reboot-mode as a valid
child node in Documentation/devicetree/bindings/arm/psci.yaml.

Without that patch applied, the schema enforces additionalProperties:
false with no reboot-mode pattern, causing the check failures you see.


Regards,
Xin


>> ---
>> Changes in v2:
>> - Fix mode-bootloader and mode-edl reset_type
>> - Link to v1 : https://lore.kernel.org/all/20260209042700.1186392-1-xin.liu@oss.qualcomm.com/
>>
>>   arch/arm64/boot/dts/qcom/hamoa.dtsi | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/hamoa.dtsi b/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> index db65c392e618..a1bd8c3e4061 100644
>> --- a/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> @@ -450,6 +450,11 @@ system_pd: power-domain-system {
>>   			#power-domain-cells = <0>;
>>   			/* TODO: system-wide idle states */
>>   		};
>> +
>> +		reboot-mode {
>> +			mode-bootloader = <0x80010001 0x2>;
>> +			mode-edl = <0x80000000 0x1>;
>> +		};
>>   	};
>>   
>>   	reserved-memory {
>> -- 
>> 2.43.0
>>