> -----Original Message----- > From: Xie Shaohui-B21989 > Sent: Wednesday, May 09, 2012 8:50 AM > To: Bhushan Bharat-R65777; linux-watchdog@xxxxxxxxxxxxxxx; linuxppc- > dev@xxxxxxxxxxxxxxxx > Subject: RE: [PATCH 1/2] powerpc/watchdog: move booke watchdog param related > code to prom.c > > >> -----Original Message----- > >> From: > >> linuxppc-dev-bounces+bharat.bhushan=freescale.com@xxxxxxxxxxxxxxxx > >> [mailto:linuxppc-dev-bounces+bharat.bhushan=freescale.com@lists.ozlab > >> s > >> .org] On Behalf Of Shaohui Xie > >> Sent: Tuesday, May 08, 2012 11:37 AM > >> To: linux-watchdog@xxxxxxxxxxxxxxx; linuxppc-dev@xxxxxxxxxxxxxxxx > >> Cc: Xie Shaohui-B21989 > >> Subject: [PATCH 1/2] powerpc/watchdog: move booke watchdog param > >> related code to prom.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 prom.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> > >> --- > >> arch/powerpc/kernel/prom.c | 27 +++++++++++++++++++++++++++ > >> arch/powerpc/kernel/setup_32.c | 24 ------------------------ > >> 2 files changed, 27 insertions(+), 24 deletions(-) > > > >Is not setup-common.c is better place to move this? > > Move out from setup_32.c does not mean it have to go into setup-common.c, I need > better reason to do this. > What I think that setup_32.c is for 32 bit, setup_64.c is for 64 bit and setup-common.c is for both. I am not saying that you move this to setup-common.c. I am asking why you have not used setup-common.c ? I am ok even with prom.c. Thanks -Bharat -- 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