Hi Andrew, Thanks for the check. > -----Original Message----- > From: Andrew Jeffery <andrew@xxxxxxxxxxxxxxxxxxxx> > 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: > > 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
