Best Regards! Anson Huang
> -----Original Message----- > From: Aisheng Dong > Sent: 2019年3月7日 22:00 > To: Anson Huang <anson.hu...@nxp.com>; catalin.mari...@arm.com; > will.dea...@arm.com; shawn...@kernel.org; s.ha...@pengutronix.de; > ker...@pengutronix.de; feste...@gmail.com; w...@linux-watchdog.org; > li...@roeck-us.net; Andy Gross <andy.gr...@linaro.org>; he...@sntech.de; > horms+rene...@verge.net.au; a...@arndb.de; > maxime.rip...@bootlin.com; ja...@amarulasolutions.com; > bjorn.anders...@linaro.org; enric.balle...@collabora.com; > marc.w.gonza...@free.fr; o...@lixom.net; linux-arm- > ker...@lists.infradead.org; linux-kernel@vger.kernel.org; linux- > watch...@vger.kernel.org > Cc: dl-linux-imx <linux-...@nxp.com> > Subject: RE: [PATCH V7 1/3] watchdog: imx_sc: Add i.MX system controller > watchdog support > > [...] > > > > As I replied in another mail, it actually does not depend on SCU. > > > Let's wait for Rob's comment on whether we could move watchdog Out > > > of SCU node. > > > > Per previous discussion, the dependency here is to prevent enabling > > this module for platform without IMX SCU, although it does NOT use SCU > > IPC call, but the SMC call trap into ARM-Trusted-Firmware will > > eventually call SCU API, so it still depends on IMX SCU hardware, I am > > NOT very sure if it is correct to remove it here. > > > > Then it's ATF depends on SCU, not this virtual watchdog based on ARM SIP, > right? But we don't have ATF driver in Linux kernel..., I am confused, anyway, if Guenter agrees, I can remove this dependency here. Anson. > > Regards > Dong Aisheng > > > > > > > > + depends on HAVE_ARM_SMCCC > > > > + select WATCHDOG_CORE > > > > + help > > > > + This is the driver for the system controller watchdog > > > > + on the NXP i.MX SoCs with system controller inside. > > > > + If you have one of these processors and wish to have > > > > + watchdog support enabled, say Y, otherwise say N. > > > > +