linux-kernel - Re: [PATCH RFT v2] watchdog: Convert orion

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120326141233.GR2484@titan.lakedaemon.net>
Date:	Mon, 26 Mar 2012 10:12:33 -0400
From:	Jason Cooper <jason@...edaemon.net>
To:	Axel Lin <axel.lin@...il.com>
Cc:	linux-kernel@...r.kernel.org,
	Sylver Bruneau <sylver.bruneau@...glemail.com>,
	Wim Van Sebroeck <wim@...ana.be>,
	linux-watchdog@...r.kernel.org,
	Nicolas Pitre <nicolas.pitre@...aro.org>,
	Andrew Lunn <andrew@...n.ch>, Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH RFT v2] watchdog: Convert orion_wdt driver to watchdog
 core

On Mon, Mar 26, 2012 at 07:29:55PM +0800, Axel Lin wrote:
> Convert the orion_wdt driver to the watchdog framework API.
> 

Hi Axel, thanks for the patch.

> Signed-off-by: Axel Lin <axel.lin@...il.com>
> ---
> v2: v1 does not apply to current tree. so I re-generate this patch.
>  drivers/watchdog/Kconfig     |    1 +
>  drivers/watchdog/orion_wdt.c |  198 ++++++++++--------------------------------
>  2 files changed, 48 insertions(+), 151 deletions(-)
> 
...
> diff --git a/drivers/watchdog/orion_wdt.c b/drivers/watchdog/orion_wdt.c
> index 788aa15..8ba7d76 100644
> --- a/drivers/watchdog/orion_wdt.c
> +++ b/drivers/watchdog/orion_wdt.c
...
> @@ -31,7 +30,7 @@
>   * Watchdog timer block registers.
>   */
>  #define TIMER_CTRL		0x0000
> -#define  WDT_EN			0x0010
> +#define WDT_EN			0x0010

nit, whitespace change.

...
>  static int __devinit orion_wdt_probe(struct platform_device *pdev)
> @@ -241,26 +146,28 @@ static int __devinit orion_wdt_probe(struct platform_device *pdev)
>  	struct resource *res;
>  	int ret;
>  
> -	if (pdata) {
> -		wdt_tclk = pdata->tclk;
> -	} else {
> +	if (!pdata) {
>  		pr_err("misses platform data\n");
>  		return -ENODEV;
>  	}

Platform data is changing soon with devicetree support (and converting
to common clock).  Please don't change this if you don't need to.

Also, I'll include Andrew Lunn in the CC.  He's doing the conversion of
orion over to common clock, which affects this driver.

hth,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/