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

[Konrad Dybcio]

On 3/2/26 5:33 PM, Krzysztof Kozlowski wrote:
> 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.

It won't even probe, we talked about that a long time ago on a similar
occasion

https://lore.kernel.org/linux-arm-msm/f6b16d1d-3730-46d1-81aa-bfaf09c20754@xxxxxxxxxxxxxxxx/

Konrad