Re: [PATCH 2/3] arm64: dts: qcom: sm8650: Add watchdog node

[Konrad Dybcio]

On 10/13/26 4:09 PM, Neil Armstrong wrote:
> On 10/13/26 11:21, Konrad Dybcio wrote:
>> On 10/2/26 11:45 AM, Neil Armstrong wrote:
>>> Add a node for the watchdog on the SM8650 SoC, the node
>>> is marked as reserved since on most platforms the Watchdog
>>> is owned by the Gunyah Hypervisor.
>>>
>>> Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
>>> ---
>>
>> Do you have a HDK or something with gunyah at hand? A couple months
>> ago qcom_scm_gunyah_wdt_init() was introduced to handle this -
>> could you enable the node and give it a shot if you have one?
> 
> Why enabling the node ? the qcom_scm_gunyah_wdt_init() will bail out
> if the node is enabled. Here we only want the node to be enabled when
> we are in EL1 without Gunyah.

Right, sorry, I misremembered and then misread

Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>

Konrad