[U-Boot] [PATCH v2 1/3] watchdog: Add Cadence watchdog driver

[Lukasz Majewski <lukma@denx.de>]

Hi Michal,

> Hi,
> 
> 2018-02-28 9:51 GMT+01:00 Lukasz Majewski <lukma@denx.de>:
> 
> > Hi Michal,
> >  
> > > From: Shreenidhi Shedi <imshedi@gmail.com>
> > >
> > > This IP can be found on Zynq and ZynqMP devices.
> > > The driver was tested with reset-on-timeout; feature.
> > >
> > > Also adding WATCHDOG symbol to Kconfig because it is required.  
> >
> > If I may ask - what is the purpose of adding separate WATCHDOG
> > symbol?
> >
> > Cannot HW_WATCHDOG or CONFIG_WDT be reused?
> >
> > Please look into ULP_WATCHDOG as a reference.
> >
> >  
> ULP_WATCHDOG hasn't been converted to DM that's why you use
> hw_watchdog wiring.
> 
> CONFIG_WDT is used for enabling watchdog uclass.
> CONFIG_WDT_CDNS - for enabling cadence DM watchdog driver
> CONFIG_WATCHDOG - for servicing watchdog by u-boot
	^^^^^^^^^ - I'm just wondering about this one. How does it
	corresponds to CONFIG_HW_WATCHDOG?


> 
> It means there are some dependencies - WDT_CDNS depends on WDT.
> And WATCHDOG doesn't need to be enabled which depends on your setting
> which is described in Kconfig symbol.
> 
> Thanks,
> Michal
> 
> 




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180228/51751c9a/attachment.sig>