>> -----Original Message----- >> From: Karicheri, Muralidharan >> Sent: Thursday, August 30, 2012 2:29 PM >> To: linux-kernel@xxxxxxxxxxxxxxx; mturquette@xxxxxxxxxx; linux- >> watchdog@xxxxxxxxxxxxxxx; wim@xxxxxxxxx >> Cc: Karicheri, Muralidharan >> Subject: [PATCH] wdt:clk: preparation for switch to common clock framework >> >> As a first step towards migrating davinci platforms to use common clock >> framework, replace all instances of clk_enable() with clk_prepare_enable() >> and clk_disable() with clk_disable_unprepare(). Until the platform is >> switched to use the CONFIG_HAVE_CLK_PREPARE Kconfig variable, this just >> adds a might_sleep() call and would work without any issues. >> >> This will make it easy later to switch to common clk based implementation >> of clk driver from DaVinci specific driver. >> >> Signed-off-by: Murali Karicheri <m-karicheri2@xxxxxx> >> --- >> drivers/watchdog/davinci_wdt.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/watchdog/davinci_wdt.c b/drivers/watchdog/davinci_wdt.c >> index c8c5c80..eb44b03 100644 >> --- a/drivers/watchdog/davinci_wdt.c >> +++ b/drivers/watchdog/davinci_wdt.c >> @@ -208,7 +208,7 @@ static int __devinit davinci_wdt_probe(struct platform_device >> *pdev) >> if (WARN_ON(IS_ERR(wdt_clk))) >> return PTR_ERR(wdt_clk); >> >> - clk_enable(wdt_clk); >> + clk_prepare_enable(wdt_clk); >> >> if (heartbeat < 1 || heartbeat > MAX_HEARTBEAT) >> heartbeat = DEFAULT_HEARTBEAT; >> @@ -256,7 +256,7 @@ static int __devexit davinci_wdt_remove(struct platform_device >> *pdev) >> wdt_mem = NULL; >> } >> >> - clk_disable(wdt_clk); >> + clk_disable_unprepare(wdt_clk); >> clk_put(wdt_clk); >> >> return 0; >> -- >> 1.7.9.5 subsystem owner, Could you please review and apply this patch as needed? We need to get this upstream before submitting the patch for Common clock framework support for mach-davinci. If I need to follow a specific procedure to get this merged, please direct me to a process page. Thanks. Murali Karicheri Software Design Engineer -- 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
