From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH 1/4] watchdog: da9063: Add restart handler support Date: Tue, 9 Dec 2014 08:22:22 -0800 Message-ID: <20141209162222.GA19663@roeck-us.net> References: <1418124169-7123-1-git-send-email-geert+renesas@glider.be> <1418124169-7123-2-git-send-email-geert+renesas@glider.be> <6ED8E3B22081A4459DAC7699F3695FB7D0B4C115@SW-EX-MBX02.diasemi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <6ED8E3B22081A4459DAC7699F3695FB7D0B4C115-68WUHU125fLzLL1Oxlh9IgLouzNaz+3S@public.gmane.org> Sender: linux-watchdog-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Opensource [Steve Twiss]" Cc: Geert Uytterhoeven , Wim Van Sebroeck , Wolfram Sang , Simon Horman , Magnus Damm , Support Opensource , Lee Jones , "linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-i2c@vger.kernel.org On Tue, Dec 09, 2014 at 12:37:15PM +0000, Opensource [Steve Twiss] wrote: > On 09 December 2014 11:23 Geert Uytterhoeven wrote: > [ ... ] > > + ret = register_restart_handler(&wdt->restart_handler); > > + if (ret) > > + dev_err(wdt->da9063->dev, > > + "Failed to register restart handler (err = %d)\n", ret); > > (I guess this is intentional).. if the restart handler doesn't get registered then > this problem is just ignored and not counted as a real error. > This makes sense, since one doesn't want the watchdog driver to fail loading just because the restart handler failed to register. One option might be to make the message dev_warn instead of dev_err, but that is really a nitpick. Reviewed-by: Guenter Roeck Thanks, Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html