From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Wed, 28 Feb 2018 12:06:27 +0100 Subject: [U-Boot] [PATCH v2 1/3] watchdog: Add Cadence watchdog driver In-Reply-To: References: <302235bba3e7e6d051ab5937f36bda627bf3f206.1519636192.git.michal.simek@xilinx.com> <20180228095144.3e973567@jawa> Message-ID: <20180228120627.15457852@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Michal, > Hi, > > 2018-02-28 9:51 GMT+01:00 Lukasz Majewski : > > > Hi Michal, > > > > > From: Shreenidhi Shedi > > > > > > 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: