Messages in this thread | | | From | Chris Packham <> | Subject | Re: [PATCH 0/3] watchdog: sbsa_gwdt: add support for Marvell ac5 | Date | Fri, 15 Dec 2023 04:21:57 +0000 |
| |
On 15/12/23 04:04, Elad Nachman wrote: > From: Elad Nachman <enachman@marvell.com> > > Add support for Marvell ac5/x variant of the ARM > sbsa global watchdog. This watchdog deviates from > the standard driver by the following items: > > 1. Registers reside in secure register section. > hence access is only possible via SMC calls to ATF. > > 2. There are couple more registers which reside in > other register areas, which needs to be configured > in order for the watchdog to properly generate > reset through the SOC. > > The new Marvell compatibility string differentiates between > the original sbsa mode of operation and the Marvell mode of > operation.
I gave this a quick try on our AC5X based board and it worked well with both action=0/action=1
> Elad Nachman (3): > dt-bindings: watchdog: add Marvell AC5 watchdog > arm64: dts: ac5: add watchdog nodes > watchdog: sbsa_gwdt: add support for Marvell ac5 > > .../bindings/watchdog/arm,sbsa-gwdt.yaml | 52 +++- > arch/arm64/boot/dts/marvell/ac5-98dx25xx.dtsi | 14 + > arch/arm64/boot/dts/marvell/ac5-98dx35xx.dtsi | 8 + > drivers/watchdog/sbsa_gwdt.c | 247 ++++++++++++++++-- > 4 files changed, 298 insertions(+), 23 deletions(-) >
| |