回复: [PATCH v1 07/12] arm: imx952: Add watchdog base address definitions

[Alice Guo (OSS)]

> -----邮件原件-----
> 发件人: Fabio Estevam <festevam at gmail.com>
> 发送时间: 2026年2月6日 11:59
> 收件人: Alice Guo (OSS) <alice.guo at oss.nxp.com>
> 抄送: dl-uboot-imx <uboot-imx at nxp.com>; u-boot at lists.denx.de; Stefano
> Babic <sbabic at nabladev.com>; Tom Rini <trini at konsulko.com>; Peng Fan
> <peng.fan at nxp.com>; Ye Li <ye.li at nxp.com>; Jindong Yue
> <jindong.yue at nxp.com>; Jacky Bai <ping.bai at nxp.com>; Marek Vasut
> <marek.vasut+renesas at mailbox.org>; Rasmus Villemoes <ravi at prevas.dk>;
> Ross, Fedor <fedor.ross at ifm.com>; João Paulo Gonçalves
> <joao.goncalves at toradex.com>; Patrice Chotard
> <patrice.chotard at foss.st.com>; Valentin Caron <valentin.caron at foss.st.com>;
> Vinh Nguyen <vinh.nguyen.xz at renesas.com>; Sam Protsenko
> <semen.protsenko at linaro.org>; Ranjani Vaidyanathan
> <ranjani.vaidyanathan at nxp.com>; Ji Luo <ji.luo at nxp.com>; Frank Li
> <frank.li at nxp.com>; Sumit Garg <sumit.garg at oss.qualcomm.com>; Adam
> Ford <aford173 at gmail.com>; Frieder Schrempf
> <frieder.schrempf at kontron.de>; David Zang <davidzangcs at gmail.com>;
> Andrew Goodbody <andrew.goodbody at linaro.org>; Sumit Garg
> <sumit.garg at kernel.org>; Marek Vasut <marex at nabladev.com>; Joseph Guo
> <qijian.guo at nxp.com>; tharvey at gateworks.com; Alice Guo
> <alice.guo at nxp.com>
> 主题: Re: [PATCH v1 07/12] arm: imx952: Add watchdog base address
> definitions
> 
> Hi Alice,
> 
> On Thu, Feb 5, 2026 at 12:16 PM <alice.guo at oss.nxp.com> wrote:
> >
> > From: Alice Guo <alice.guo at nxp.com>
> >
> > Add WDG3, WDG4 and WDG5 base addresses for i.MX952.
> >
> > Signed-off-by: Alice Guo <alice.guo at nxp.com>
> > ---
> >  arch/arm/include/asm/arch-imx9/imx-regs.h | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/arch/arm/include/asm/arch-imx9/imx-regs.h
> b/arch/arm/include/asm/arch-imx9/imx-regs.h
> > index 2d084e5227a..7c7a34f74af 100644
> > --- a/arch/arm/include/asm/arch-imx9/imx-regs.h
> > +++ b/arch/arm/include/asm/arch-imx9/imx-regs.h
> > @@ -17,6 +17,11 @@
> >
> >  #define ANATOP_BASE_ADDR    0x44480000UL
> >
> > +#ifdef CONFIG_IMX952
> > +#define WDG3_BASE_ADDR      0x420b0000UL
> > +#define WDG4_BASE_ADDR      0x420c0000UL
> > +#define WDG5_BASE_ADDR      0x420d0000UL
> > +#else
> >  #ifdef CONFIG_IMX94
> >  #define WDG3_BASE_ADDR      0x49220000UL
> >  #define WDG4_BASE_ADDR      0x49230000UL
> > @@ -25,6 +30,7 @@
> >  #define WDG4_BASE_ADDR      0x424a0000UL
> >  #endif
> >  #define WDG5_BASE_ADDR      0x424b0000UL
> > +#endif
> 
> Can we improve this and retrieve the watchdog base address from the
> devicetree?

Hi Fabio,

Thanks for the suggestion. Using the device tree would be cleaner, but this patch follows the existing i.MX9 imx-regs.h pattern. I suggest we merge this patch to keep consistency with the current codebase. Thoughts?

Best regards,
Alice Guo