Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754053Ab3F0VsO (ORCPT ); Thu, 27 Jun 2013 17:48:14 -0400 Received: from mga01.intel.com ([192.55.52.88]:52508 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752773Ab3F0VsN (ORCPT ); Thu, 27 Jun 2013 17:48:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.87,954,1363158000"; d="scan'208";a="361682827" Date: Thu, 27 Jun 2013 23:47:05 +0200 From: Samuel Ortiz To: Kevin Strasser Cc: Wim Van Sebroeck , Kevin Strasser , Darren Hart , Guenter Roeck , Michael Brunner , Michael Brunner , Chris Healy , Thomas Gleixner , Dirk Hohndel , Wolfram Sang , Ben Dooks , Grant Likely , Linus Walleij , linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 4/4] watchdog: Kontron PLD watchdog timer driver Message-ID: <20130627214705.GA26192@zurbaran> References: <1365441321-21952-1-git-send-email-kevin.strasser@linux.intel.com> <1372046406-3127-1-git-send-email-kevin.strasser@linux.intel.com> <1372046406-3127-5-git-send-email-kevin.strasser@linux.intel.com> <20130627182314.GA16090@wrk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20130627182314.GA16090@wrk> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1103 Lines: 30 Hi Kevin, On Thu, Jun 27, 2013 at 11:23:15AM -0700, Kevin Strasser wrote: > Hi Wim, > > This driver hasn't received any further feedback. Are you ready to take it > into watchdog-next, or do you still need to review it? I applied the watchdog patch to mfd-next and got the following warning: CC [M] drivers/watchdog/kempld_wdt.o drivers/watchdog/kempld_wdt.c: In function ‘kempld_wdt_suspend’: drivers/watchdog/kempld_wdt.c:538:3: warning: passing argument 1 of ‘kempld_wdt_stop’ from incompatible pointer type [enabled by default] drivers/watchdog/kempld_wdt.c:313:12: note: expected ‘struct watchdog_device *’ but argument is of type ‘struct platform_device *’ I fixed it up, but I'd appreciate if you could double check that kempld_wdt_suspend() is ok. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/