RE: [PATCH V7 1/3] watchdog: imx_sc: Add i.MX system controller watchdog support

From: Aisheng Dong
Date: Thu Mar 07 2019 - 21:27:32 EST


[...]

> >
> > > > 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.
>

So far the only thing that watchdog depends on SCU driver is that it needs
SCU driver to create a platform devices for it.
We need wait Rob's feedback, if he agree moved the watchdog node
Out of SCU, we can remove this dependency.

Regards
Dong Aisheng

> 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.
> > > > > +