ACK: > -----Original Message----- > From: Linuxppc-dev [mailto:linuxppc-dev- > bounces+bharat.bhushan=freescale.com@xxxxxxxxxxxxxxxx] On Behalf Of Shaohui Xie > Sent: Wednesday, July 11, 2012 3:26 PM > To: linux-watchdog@xxxxxxxxxxxxxxx; linuxppc-dev@xxxxxxxxxxxxxxxx > Cc: Xie Shaohui-B21989 > Subject: [PATCH][v2] powerpc/watchdog: move booke watchdog param related code to > setup-common.c > > Currently, BOOKE watchdog code for checking "wdt" and "wdt_period" is in > setup_32.c, it cannot be used in 64-bit, so move it to a common place setup- > common.c, which will be shared by 32-bit and 64-bit. > > Also, replace the simple_strtoul with kstrtol. > > Signed-off-by: Shaohui Xie <Shaohui.Xie@xxxxxxxxxxxxx> > --- > changes for v2: > use setup-common.c instead of prom.c > > arch/powerpc/kernel/setup-common.c | 27 +++++++++++++++++++++++++++ > arch/powerpc/kernel/setup_32.c | 24 ------------------------ > 2 files changed, 27 insertions(+), 24 deletions(-) > > diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup- > common.c > index afd4f05..bdc499c 100644 > --- a/arch/powerpc/kernel/setup-common.c > +++ b/arch/powerpc/kernel/setup-common.c > @@ -720,6 +720,33 @@ static int powerpc_debugfs_init(void) > arch_initcall(powerpc_debugfs_init); > #endif > > +#ifdef CONFIG_BOOKE_WDT > +extern u32 booke_wdt_enabled; > +extern u32 booke_wdt_period; > + > +/* Checks wdt=x and wdt_period=xx command-line option */ notrace int > +__init early_parse_wdt(char *p) { > + if (p && strncmp(p, "0", 1) != 0) > + booke_wdt_enabled = 1; > + > + return 0; > +} > +early_param("wdt", early_parse_wdt); > + > +int __init early_parse_wdt_period(char *p) { > + unsigned long ret; > + if (p) { > + if (!kstrtol(p, 0, &ret)) > + booke_wdt_period = ret; > + } > + > + return 0; > +} > +early_param("wdt_period", early_parse_wdt_period); > +#endif /* CONFIG_BOOKE_WDT */ > + > void ppc_printk_progress(char *s, unsigned short hex) { > pr_info("%s\n", s); > diff --git a/arch/powerpc/kernel/setup_32.c b/arch/powerpc/kernel/setup_32.c > index ec8a53f..a8f54ec 100644 > --- a/arch/powerpc/kernel/setup_32.c > +++ b/arch/powerpc/kernel/setup_32.c > @@ -149,30 +149,6 @@ notrace void __init machine_init(u64 dt_ptr) > ppc_md.progress("id mach(): done", 0x200); } > > -#ifdef CONFIG_BOOKE_WDT > -extern u32 booke_wdt_enabled; > -extern u32 booke_wdt_period; > - > -/* Checks wdt=x and wdt_period=xx command-line option */ -notrace int __init > early_parse_wdt(char *p) -{ > - if (p && strncmp(p, "0", 1) != 0) > - booke_wdt_enabled = 1; > - > - return 0; > -} > -early_param("wdt", early_parse_wdt); > - > -int __init early_parse_wdt_period (char *p) -{ > - if (p) > - booke_wdt_period = simple_strtoul(p, NULL, 0); > - > - return 0; > -} > -early_param("wdt_period", early_parse_wdt_period); > -#endif /* CONFIG_BOOKE_WDT */ > - > /* Checks "l2cr=xxxx" command-line option */ int __init ppc_setup_l2cr(char > *str) { > -- > 1.6.4 > > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@xxxxxxxxxxxxxxxx > https://lists.ozlabs.org/listinfo/linuxppc-dev -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
