RE: [PATCH v2 1/3] ARM: dts: aspeed: Add WDT controller into alias field

Hi Andrew,

Thanks for the check.

-----Original Message-----
From: Andrew Jeffery <andrew@codeconstruct.com.au>
Sent: Tuesday, October 29, 2024 7:55 AM
Subject: Re: [PATCH v2 1/3] ARM: dts: aspeed: Add WDT controller into alias
field

On Mon, 2024-10-28 at 10:48 +0800, Chin-Ting Kuo wrote:

quoted

Add WDT controller into alias field. After that, WDT index, used to
distinguish different WDT controllers in the driver, can be gotten by
using of_alias_get_id dts API.

I feel it would be less brittle if we encode the mapping in the driver?
Based on reg the driver can derive the watchdog index. That way there's no
constraint on how the platform architect arranges the aliases for the
watchdogs (if they define them at all).

It is also a feasible approach. It will be changed in the next patch version.

Andrew

Chin-Ting