Re: [PATCH v3 1/6] dt-bindings: sram: Describe the IMEM present in Qualcomm IPQ SoCs

[Krzysztof Kozlowski <krzk@kernel.org>]

On 02/03/2026 15:56, Konrad Dybcio wrote:
> On 3/2/26 3:54 PM, Krzysztof Kozlowski wrote:
>> On 02/03/2026 15:10, Konrad Dybcio wrote:
>>>>
>>>> Also not accurate - "sram" is not the reboot reason, which has node name
>>>> called "reboot-mode".
>>>
>>> What I was referring to is:
>>>
>>> patternProperties:
>>>   "^([a-z0-9]*-)?sram(-section)?@[a-f0-9]+$":
>>>
>>> where the 'sram' (not 'smem' as I typo'd above) is obligatory
>>
>> I know and sram is not part of "reboot-mode" name. It is "reboot-mode"
>> in existing binding and DTS, not "reboot-mode-sram".
> 
> In any case, I believe it'd be good to drop that requirement

Ah, and one more thing, the syscon-reboot-mode is Linux driver with its
own Linux requirements - syscon - so probably not working with sram
bindings. That's another reason full binding should be posted and tested
- I speculate it simply does not work.


> Konrad


Best regards,
Krzysztof