From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Fri, 29 Apr 2011 20:17:45 -0400 Subject: [U-Boot] [PATCH] README: Clarify difference of CONFIG_WATCHDOG and CONFIG_HW_WATCHDOG In-Reply-To: References: <1303917959-32324-1-git-send-email-dzu@denx.de> Message-ID: <201104292017.47119.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Friday, April 29, 2011 07:06:56 Detlev Zundel wrote: > > On Wed, Apr 27, 2011 at 11:25, Detlev Zundel wrote: > >> Now that we have the documentation, the code should be changed to > >> reflect it ;) > >> > >> Asd far as I can see, these are the places where HW_WATCHDOG is used > > > >> instead of WATCHDOG: > > the trouble is that watchdog.h doesnt seem to match, nor does the > > name. "hardware" means to me "hardware" as in "the cpu hardware". > > Yes, I understand that the situation is not 100% ideal, but I do not > share your pessimism. it isnt pessimism ... it's a fact that the documentation does not match the source, nor is the source clear. > > > the watchdog system is a legacy mess, so probably be easier to just > > gut the whole thing. > > A agree that the naming is not ideal, but this isn't a good enough > argument for removing it. Apart from that, this feature is used in many > commercial systems... i didnt mean remove it and not provide equivalent functionality. there should be a single CONFIG_WATCHDOG and every different watchdog driver should have its own CONFIG_WATCHDOG_FOO define. -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. Url : http://lists.denx.de/pipermail/u-boot/attachments/20110429/f99694a0/attachment.pgp