* [PATCH] arm:davinci: clk - migrate to new common clock API [not found] <1343926428-5210-1-git-send-email-m-karicheri2@ti.com> @ 2012-08-02 21:20 ` Mike Turquette 2012-08-13 21:13 ` [linux-keystone] " Karicheri, Muralidharan 2012-08-02 21:28 ` Mike Turquette 2012-08-14 6:30 ` Sekhar Nori 2 siblings, 1 reply; 7+ messages in thread From: Mike Turquette @ 2012-08-02 21:20 UTC (permalink / raw) To: linux-arm-kernel On 20120802-12:53, Murali Karicheri wrote: > 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@ti.com> Murali, The important thing to know about clk_prepare is that it might sleep. All of the functions below look like initialization functions for code that is never run in interrupt context. If you can confirm that this is true then feel free to add my, Reviewed-by: Mike Turquette <mturquette@linaro.org> Regards, Mike > --- > arch/arm/mach-davinci/board-dm355-evm.c | 2 +- > arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- > arch/arm/mach-davinci/board-dm365-evm.c | 4 ++-- > arch/arm/mach-davinci/board-dm644x-evm.c | 2 +- > arch/arm/mach-davinci/board-neuros-osd2.c | 2 +- > arch/arm/mach-davinci/devices-da8xx.c | 6 +++--- > arch/arm/mach-davinci/serial.c | 2 +- > arch/arm/mach-davinci/time.c | 4 ++-- > 8 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach-davinci/board-dm355-evm.c > index 1c7b1f4..4821fba 100644 > --- a/arch/arm/mach-davinci/board-dm355-evm.c > +++ b/arch/arm/mach-davinci/board-dm355-evm.c > @@ -324,7 +324,7 @@ static __init void dm355_evm_init(void) > if (IS_ERR(aemif)) > WARN("%s: unable to get AEMIF clock\n", __func__); > else > - clk_enable(aemif); > + clk_prepare_enable(aemif); > > platform_add_devices(davinci_evm_devices, > ARRAY_SIZE(davinci_evm_devices)); > diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach-davinci/board-dm355-leopard.c > index 8e77032..b754ccf 100644 > --- a/arch/arm/mach-davinci/board-dm355-leopard.c > +++ b/arch/arm/mach-davinci/board-dm355-leopard.c > @@ -246,7 +246,7 @@ static __init void dm355_leopard_init(void) > if (IS_ERR(aemif)) > WARN("%s: unable to get AEMIF clock\n", __func__); > else > - clk_enable(aemif); > + clk_prepare_enable(aemif); > > platform_add_devices(davinci_leopard_devices, > ARRAY_SIZE(davinci_leopard_devices)); > diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c > index 2830692..3a4743b 100644 > --- a/arch/arm/mach-davinci/board-dm365-evm.c > +++ b/arch/arm/mach-davinci/board-dm365-evm.c > @@ -481,7 +481,7 @@ static void __init evm_init_cpld(void) > aemif_clk = clk_get(NULL, "aemif"); > if (IS_ERR(aemif_clk)) > return; > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (request_mem_region(DM365_ASYNC_EMIF_DATA_CE1_BASE, SECTION_SIZE, > "cpld") == NULL) > @@ -492,7 +492,7 @@ static void __init evm_init_cpld(void) > SECTION_SIZE); > fail: > pr_err("ERROR: can't map CPLD\n"); > - clk_disable(aemif_clk); > + clk_disable_unprepare(aemif_clk); > return; > } > > diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach-davinci/board-dm644x-evm.c > index d34ed55..04ef708 100644 > --- a/arch/arm/mach-davinci/board-dm644x-evm.c > +++ b/arch/arm/mach-davinci/board-dm644x-evm.c > @@ -775,7 +775,7 @@ static __init void davinci_evm_init(void) > struct davinci_soc_info *soc_info = &davinci_soc_info; > > aemif_clk = clk_get(NULL, "aemif"); > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (HAS_ATA) { > if (HAS_NAND || HAS_NOR) > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach-davinci/board-neuros-osd2.c > index 5de69f2..7fdd0a7b 100644 > --- a/arch/arm/mach-davinci/board-neuros-osd2.c > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c > @@ -221,7 +221,7 @@ static __init void davinci_ntosd2_init(void) > int status; > > aemif_clk = clk_get(NULL, "aemif"); > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (HAS_ATA) { > if (HAS_NAND) > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices-da8xx.c > index d1624a3..20141f5 100644 > --- a/arch/arm/mach-davinci/devices-da8xx.c > +++ b/arch/arm/mach-davinci/devices-da8xx.c > @@ -905,7 +905,7 @@ static int da850_sata_init(struct device *dev, void __iomem *addr) > if (IS_ERR(da850_sata_clk)) > return PTR_ERR(da850_sata_clk); > > - ret = clk_enable(da850_sata_clk); > + ret = clk_prepare_enable(da850_sata_clk); > if (ret) > goto err0; > > @@ -936,7 +936,7 @@ static int da850_sata_init(struct device *dev, void __iomem *addr) > return 0; > > err1: > - clk_disable(da850_sata_clk); > + clk_disable_unprepare(da850_sata_clk); > err0: > clk_put(da850_sata_clk); > return ret; > @@ -944,7 +944,7 @@ err0: > > static void da850_sata_exit(struct device *dev) > { > - clk_disable(da850_sata_clk); > + clk_disable_unprepare(da850_sata_clk); > clk_put(da850_sata_clk); > } > > diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c > index 1875740..24ddb0d 100644 > --- a/arch/arm/mach-davinci/serial.c > +++ b/arch/arm/mach-davinci/serial.c > @@ -95,7 +95,7 @@ int __init davinci_serial_init(struct davinci_uart_config *info) > continue; > } > > - clk_enable(uart_clk); > + clk_prepare_enable(uart_clk); > p->uartclk = clk_get_rate(uart_clk); > > if (!p->membase && p->mapbase) { > diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c > index 75da315..9847938 100644 > --- a/arch/arm/mach-davinci/time.c > +++ b/arch/arm/mach-davinci/time.c > @@ -379,7 +379,7 @@ static void __init davinci_timer_init(void) > > timer_clk = clk_get(NULL, "timer0"); > BUG_ON(IS_ERR(timer_clk)); > - clk_enable(timer_clk); > + clk_prepare_enable(timer_clk); > > /* init timer hw */ > timer_init(); > @@ -429,7 +429,7 @@ void davinci_watchdog_reset(struct platform_device *pdev) > wd_clk = clk_get(&pdev->dev, NULL); > if (WARN_ON(IS_ERR(wd_clk))) > return; > - clk_enable(wd_clk); > + clk_prepare_enable(wd_clk); > > /* disable, internal clock source */ > __raw_writel(0, base + TCR); > -- > 1.7.9.5 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [linux-keystone] Re: [PATCH] arm:davinci: clk - migrate to new common clock API 2012-08-02 21:20 ` [PATCH] arm:davinci: clk - migrate to new common clock API Mike Turquette @ 2012-08-13 21:13 ` Karicheri, Muralidharan 2012-08-13 21:44 ` Turquette, Mike 0 siblings, 1 reply; 7+ messages in thread From: Karicheri, Muralidharan @ 2012-08-13 21:13 UTC (permalink / raw) To: linux-arm-kernel >> -----Original Message----- >> From: Turquette, Mike >> Sent: Thursday, August 02, 2012 5:20 PM >> To: Karicheri, Muralidharan >> Cc: linux-arm-kernel at lists.infradead.org; davinci-linux-open- >> source at linux.davincidsp.com; linux-keystone at list.ti.com - Linux developers for Keystone >> family of devices (May contain non-TIers); Chemparathy, Cyril; Nori, Sekhar; >> linux at arm.linux.org.uk >> Subject: [linux-keystone] Re: [PATCH] arm:davinci: clk - migrate to new common clock >> API >> >> On 20120802-12:53, Murali Karicheri wrote: >> > 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@ti.com> >> >> Murali, >> >> The important thing to know about clk_prepare is that it might sleep. >> All of the functions below look like initialization functions for code >> that is never run in interrupt context. If you can confirm that this is >> true then feel free to add my, >> Mike, I believe they are. I will double check and then post the patch again. If they are called under interrupt context, basically I need to make two API calls instead as below right? clk_prepare() clk_enable() Murali Karicheri Software Design Engineer >> Reviewed-by: Mike Turquette <mturquette@linaro.org> >> >> Regards, >> Mike >> >> > --- >> > arch/arm/mach-davinci/board-dm355-evm.c | 2 +- >> > arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- >> > arch/arm/mach-davinci/board-dm365-evm.c | 4 ++-- >> > arch/arm/mach-davinci/board-dm644x-evm.c | 2 +- >> > arch/arm/mach-davinci/board-neuros-osd2.c | 2 +- >> > arch/arm/mach-davinci/devices-da8xx.c | 6 +++--- >> > arch/arm/mach-davinci/serial.c | 2 +- >> > arch/arm/mach-davinci/time.c | 4 ++-- >> > 8 files changed, 12 insertions(+), 12 deletions(-) >> > >> > diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach- >> davinci/board-dm355-evm.c >> > index 1c7b1f4..4821fba 100644 >> > --- a/arch/arm/mach-davinci/board-dm355-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm355-evm.c >> > @@ -324,7 +324,7 @@ static __init void dm355_evm_init(void) >> > if (IS_ERR(aemif)) >> > WARN("%s: unable to get AEMIF clock\n", __func__); >> > else >> > - clk_enable(aemif); >> > + clk_prepare_enable(aemif); >> > >> > platform_add_devices(davinci_evm_devices, >> > ARRAY_SIZE(davinci_evm_devices)); >> > diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach- >> davinci/board-dm355-leopard.c >> > index 8e77032..b754ccf 100644 >> > --- a/arch/arm/mach-davinci/board-dm355-leopard.c >> > +++ b/arch/arm/mach-davinci/board-dm355-leopard.c >> > @@ -246,7 +246,7 @@ static __init void dm355_leopard_init(void) >> > if (IS_ERR(aemif)) >> > WARN("%s: unable to get AEMIF clock\n", __func__); >> > else >> > - clk_enable(aemif); >> > + clk_prepare_enable(aemif); >> > >> > platform_add_devices(davinci_leopard_devices, >> > ARRAY_SIZE(davinci_leopard_devices)); >> > diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach- >> davinci/board-dm365-evm.c >> > index 2830692..3a4743b 100644 >> > --- a/arch/arm/mach-davinci/board-dm365-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm365-evm.c >> > @@ -481,7 +481,7 @@ static void __init evm_init_cpld(void) >> > aemif_clk = clk_get(NULL, "aemif"); >> > if (IS_ERR(aemif_clk)) >> > return; >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (request_mem_region(DM365_ASYNC_EMIF_DATA_CE1_BASE, SECTION_SIZE, >> > "cpld") == NULL) >> > @@ -492,7 +492,7 @@ static void __init evm_init_cpld(void) >> > SECTION_SIZE); >> > fail: >> > pr_err("ERROR: can't map CPLD\n"); >> > - clk_disable(aemif_clk); >> > + clk_disable_unprepare(aemif_clk); >> > return; >> > } >> > >> > diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach- >> davinci/board-dm644x-evm.c >> > index d34ed55..04ef708 100644 >> > --- a/arch/arm/mach-davinci/board-dm644x-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm644x-evm.c >> > @@ -775,7 +775,7 @@ static __init void davinci_evm_init(void) >> > struct davinci_soc_info *soc_info = &davinci_soc_info; >> > >> > aemif_clk = clk_get(NULL, "aemif"); >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (HAS_ATA) { >> > if (HAS_NAND || HAS_NOR) >> > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach- >> davinci/board-neuros-osd2.c >> > index 5de69f2..7fdd0a7b 100644 >> > --- a/arch/arm/mach-davinci/board-neuros-osd2.c >> > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c >> > @@ -221,7 +221,7 @@ static __init void davinci_ntosd2_init(void) >> > int status; >> > >> > aemif_clk = clk_get(NULL, "aemif"); >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (HAS_ATA) { >> > if (HAS_NAND) >> > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices- >> da8xx.c >> > index d1624a3..20141f5 100644 >> > --- a/arch/arm/mach-davinci/devices-da8xx.c >> > +++ b/arch/arm/mach-davinci/devices-da8xx.c >> > @@ -905,7 +905,7 @@ static int da850_sata_init(struct device *dev, void __iomem >> *addr) >> > if (IS_ERR(da850_sata_clk)) >> > return PTR_ERR(da850_sata_clk); >> > >> > - ret = clk_enable(da850_sata_clk); >> > + ret = clk_prepare_enable(da850_sata_clk); >> > if (ret) >> > goto err0; >> > >> > @@ -936,7 +936,7 @@ static int da850_sata_init(struct device *dev, void __iomem >> *addr) >> > return 0; >> > >> > err1: >> > - clk_disable(da850_sata_clk); >> > + clk_disable_unprepare(da850_sata_clk); >> > err0: >> > clk_put(da850_sata_clk); >> > return ret; >> > @@ -944,7 +944,7 @@ err0: >> > >> > static void da850_sata_exit(struct device *dev) >> > { >> > - clk_disable(da850_sata_clk); >> > + clk_disable_unprepare(da850_sata_clk); >> > clk_put(da850_sata_clk); >> > } >> > >> > diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c >> > index 1875740..24ddb0d 100644 >> > --- a/arch/arm/mach-davinci/serial.c >> > +++ b/arch/arm/mach-davinci/serial.c >> > @@ -95,7 +95,7 @@ int __init davinci_serial_init(struct davinci_uart_config *info) >> > continue; >> > } >> > >> > - clk_enable(uart_clk); >> > + clk_prepare_enable(uart_clk); >> > p->uartclk = clk_get_rate(uart_clk); >> > >> > if (!p->membase && p->mapbase) { >> > diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c >> > index 75da315..9847938 100644 >> > --- a/arch/arm/mach-davinci/time.c >> > +++ b/arch/arm/mach-davinci/time.c >> > @@ -379,7 +379,7 @@ static void __init davinci_timer_init(void) >> > >> > timer_clk = clk_get(NULL, "timer0"); >> > BUG_ON(IS_ERR(timer_clk)); >> > - clk_enable(timer_clk); >> > + clk_prepare_enable(timer_clk); >> > >> > /* init timer hw */ >> > timer_init(); >> > @@ -429,7 +429,7 @@ void davinci_watchdog_reset(struct platform_device *pdev) >> > wd_clk = clk_get(&pdev->dev, NULL); >> > if (WARN_ON(IS_ERR(wd_clk))) >> > return; >> > - clk_enable(wd_clk); >> > + clk_prepare_enable(wd_clk); >> > >> > /* disable, internal clock source */ >> > __raw_writel(0, base + TCR); >> > -- >> > 1.7.9.5 >> > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [linux-keystone] Re: [PATCH] arm:davinci: clk - migrate to new common clock API 2012-08-13 21:13 ` [linux-keystone] " Karicheri, Muralidharan @ 2012-08-13 21:44 ` Turquette, Mike 0 siblings, 0 replies; 7+ messages in thread From: Turquette, Mike @ 2012-08-13 21:44 UTC (permalink / raw) To: linux-arm-kernel On Mon, Aug 13, 2012 at 2:13 PM, Karicheri, Muralidharan <m-karicheri2@ti.com> wrote: >>> -----Original Message----- >>> From: Turquette, Mike >>> Sent: Thursday, August 02, 2012 5:20 PM >>> To: Karicheri, Muralidharan >>> Cc: linux-arm-kernel at lists.infradead.org; davinci-linux-open- >>> source at linux.davincidsp.com; linux-keystone at list.ti.com - Linux developers for Keystone >>> family of devices (May contain non-TIers); Chemparathy, Cyril; Nori, Sekhar; >>> linux at arm.linux.org.uk >>> Subject: [linux-keystone] Re: [PATCH] arm:davinci: clk - migrate to new common clock >>> API >>> >>> On 20120802-12:53, Murali Karicheri wrote: >>> > 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@ti.com> >>> >>> Murali, >>> >>> The important thing to know about clk_prepare is that it might sleep. >>> All of the functions below look like initialization functions for code >>> that is never run in interrupt context. If you can confirm that this is >>> true then feel free to add my, >>> > > Mike, > > I believe they are. I will double check and then post the patch again. If they are called under > interrupt context, basically I need to make two API calls instead as below right? > > clk_prepare() > clk_enable() > This is correct. You must insure that clk_prepare is called (and will complete) before clk_enable is called. clk_prepare must be run from process context. It is still OK to call clk_enable/clk_disable from interrupt context as needed. Regards, Mike > Murali Karicheri > Software Design Engineer > > >>> Reviewed-by: Mike Turquette <mturquette@linaro.org> >>> >>> Regards, >>> Mike >>> >>> > --- >>> > arch/arm/mach-davinci/board-dm355-evm.c | 2 +- >>> > arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- >>> > arch/arm/mach-davinci/board-dm365-evm.c | 4 ++-- >>> > arch/arm/mach-davinci/board-dm644x-evm.c | 2 +- >>> > arch/arm/mach-davinci/board-neuros-osd2.c | 2 +- >>> > arch/arm/mach-davinci/devices-da8xx.c | 6 +++--- >>> > arch/arm/mach-davinci/serial.c | 2 +- >>> > arch/arm/mach-davinci/time.c | 4 ++-- >>> > 8 files changed, 12 insertions(+), 12 deletions(-) >>> > >>> > diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach- >>> davinci/board-dm355-evm.c >>> > index 1c7b1f4..4821fba 100644 >>> > --- a/arch/arm/mach-davinci/board-dm355-evm.c >>> > +++ b/arch/arm/mach-davinci/board-dm355-evm.c >>> > @@ -324,7 +324,7 @@ static __init void dm355_evm_init(void) >>> > if (IS_ERR(aemif)) >>> > WARN("%s: unable to get AEMIF clock\n", __func__); >>> > else >>> > - clk_enable(aemif); >>> > + clk_prepare_enable(aemif); >>> > >>> > platform_add_devices(davinci_evm_devices, >>> > ARRAY_SIZE(davinci_evm_devices)); >>> > diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach- >>> davinci/board-dm355-leopard.c >>> > index 8e77032..b754ccf 100644 >>> > --- a/arch/arm/mach-davinci/board-dm355-leopard.c >>> > +++ b/arch/arm/mach-davinci/board-dm355-leopard.c >>> > @@ -246,7 +246,7 @@ static __init void dm355_leopard_init(void) >>> > if (IS_ERR(aemif)) >>> > WARN("%s: unable to get AEMIF clock\n", __func__); >>> > else >>> > - clk_enable(aemif); >>> > + clk_prepare_enable(aemif); >>> > >>> > platform_add_devices(davinci_leopard_devices, >>> > ARRAY_SIZE(davinci_leopard_devices)); >>> > diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach- >>> davinci/board-dm365-evm.c >>> > index 2830692..3a4743b 100644 >>> > --- a/arch/arm/mach-davinci/board-dm365-evm.c >>> > +++ b/arch/arm/mach-davinci/board-dm365-evm.c >>> > @@ -481,7 +481,7 @@ static void __init evm_init_cpld(void) >>> > aemif_clk = clk_get(NULL, "aemif"); >>> > if (IS_ERR(aemif_clk)) >>> > return; >>> > - clk_enable(aemif_clk); >>> > + clk_prepare_enable(aemif_clk); >>> > >>> > if (request_mem_region(DM365_ASYNC_EMIF_DATA_CE1_BASE, SECTION_SIZE, >>> > "cpld") == NULL) >>> > @@ -492,7 +492,7 @@ static void __init evm_init_cpld(void) >>> > SECTION_SIZE); >>> > fail: >>> > pr_err("ERROR: can't map CPLD\n"); >>> > - clk_disable(aemif_clk); >>> > + clk_disable_unprepare(aemif_clk); >>> > return; >>> > } >>> > >>> > diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach- >>> davinci/board-dm644x-evm.c >>> > index d34ed55..04ef708 100644 >>> > --- a/arch/arm/mach-davinci/board-dm644x-evm.c >>> > +++ b/arch/arm/mach-davinci/board-dm644x-evm.c >>> > @@ -775,7 +775,7 @@ static __init void davinci_evm_init(void) >>> > struct davinci_soc_info *soc_info = &davinci_soc_info; >>> > >>> > aemif_clk = clk_get(NULL, "aemif"); >>> > - clk_enable(aemif_clk); >>> > + clk_prepare_enable(aemif_clk); >>> > >>> > if (HAS_ATA) { >>> > if (HAS_NAND || HAS_NOR) >>> > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach- >>> davinci/board-neuros-osd2.c >>> > index 5de69f2..7fdd0a7b 100644 >>> > --- a/arch/arm/mach-davinci/board-neuros-osd2.c >>> > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c >>> > @@ -221,7 +221,7 @@ static __init void davinci_ntosd2_init(void) >>> > int status; >>> > >>> > aemif_clk = clk_get(NULL, "aemif"); >>> > - clk_enable(aemif_clk); >>> > + clk_prepare_enable(aemif_clk); >>> > >>> > if (HAS_ATA) { >>> > if (HAS_NAND) >>> > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices- >>> da8xx.c >>> > index d1624a3..20141f5 100644 >>> > --- a/arch/arm/mach-davinci/devices-da8xx.c >>> > +++ b/arch/arm/mach-davinci/devices-da8xx.c >>> > @@ -905,7 +905,7 @@ static int da850_sata_init(struct device *dev, void __iomem >>> *addr) >>> > if (IS_ERR(da850_sata_clk)) >>> > return PTR_ERR(da850_sata_clk); >>> > >>> > - ret = clk_enable(da850_sata_clk); >>> > + ret = clk_prepare_enable(da850_sata_clk); >>> > if (ret) >>> > goto err0; >>> > >>> > @@ -936,7 +936,7 @@ static int da850_sata_init(struct device *dev, void __iomem >>> *addr) >>> > return 0; >>> > >>> > err1: >>> > - clk_disable(da850_sata_clk); >>> > + clk_disable_unprepare(da850_sata_clk); >>> > err0: >>> > clk_put(da850_sata_clk); >>> > return ret; >>> > @@ -944,7 +944,7 @@ err0: >>> > >>> > static void da850_sata_exit(struct device *dev) >>> > { >>> > - clk_disable(da850_sata_clk); >>> > + clk_disable_unprepare(da850_sata_clk); >>> > clk_put(da850_sata_clk); >>> > } >>> > >>> > diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c >>> > index 1875740..24ddb0d 100644 >>> > --- a/arch/arm/mach-davinci/serial.c >>> > +++ b/arch/arm/mach-davinci/serial.c >>> > @@ -95,7 +95,7 @@ int __init davinci_serial_init(struct davinci_uart_config *info) >>> > continue; >>> > } >>> > >>> > - clk_enable(uart_clk); >>> > + clk_prepare_enable(uart_clk); >>> > p->uartclk = clk_get_rate(uart_clk); >>> > >>> > if (!p->membase && p->mapbase) { >>> > diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c >>> > index 75da315..9847938 100644 >>> > --- a/arch/arm/mach-davinci/time.c >>> > +++ b/arch/arm/mach-davinci/time.c >>> > @@ -379,7 +379,7 @@ static void __init davinci_timer_init(void) >>> > >>> > timer_clk = clk_get(NULL, "timer0"); >>> > BUG_ON(IS_ERR(timer_clk)); >>> > - clk_enable(timer_clk); >>> > + clk_prepare_enable(timer_clk); >>> > >>> > /* init timer hw */ >>> > timer_init(); >>> > @@ -429,7 +429,7 @@ void davinci_watchdog_reset(struct platform_device *pdev) >>> > wd_clk = clk_get(&pdev->dev, NULL); >>> > if (WARN_ON(IS_ERR(wd_clk))) >>> > return; >>> > - clk_enable(wd_clk); >>> > + clk_prepare_enable(wd_clk); >>> > >>> > /* disable, internal clock source */ >>> > __raw_writel(0, base + TCR); >>> > -- >>> > 1.7.9.5 >>> > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] arm:davinci: clk - migrate to new common clock API [not found] <1343926428-5210-1-git-send-email-m-karicheri2@ti.com> 2012-08-02 21:20 ` [PATCH] arm:davinci: clk - migrate to new common clock API Mike Turquette @ 2012-08-02 21:28 ` Mike Turquette 2012-08-13 21:15 ` Karicheri, Muralidharan 2012-08-14 6:30 ` Sekhar Nori 2 siblings, 1 reply; 7+ messages in thread From: Mike Turquette @ 2012-08-02 21:28 UTC (permalink / raw) To: linux-arm-kernel On 20120802-12:53, Murali Karicheri wrote: > 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@ti.com> Murali, The patch $subject is a bit misleading. This patch is preparation for converting davinci over to the common clk framework. How about: arm: davinci: clk: call clk_prepare_enable Also, the important thing to know about clk_prepare is that it might sleep. All of the functions below look like initialization functions for code that is never run in interrupt context. If you can confirm that this is true then feel free to add my, Reviewed-by: Mike Turquette <mturquette@linaro.org> Regards, Mike > --- > arch/arm/mach-davinci/board-dm355-evm.c | 2 +- > arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- > arch/arm/mach-davinci/board-dm365-evm.c | 4 ++-- > arch/arm/mach-davinci/board-dm644x-evm.c | 2 +- > arch/arm/mach-davinci/board-neuros-osd2.c | 2 +- > arch/arm/mach-davinci/devices-da8xx.c | 6 +++--- > arch/arm/mach-davinci/serial.c | 2 +- > arch/arm/mach-davinci/time.c | 4 ++-- > 8 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach-davinci/board-dm355-evm.c > index 1c7b1f4..4821fba 100644 > --- a/arch/arm/mach-davinci/board-dm355-evm.c > +++ b/arch/arm/mach-davinci/board-dm355-evm.c > @@ -324,7 +324,7 @@ static __init void dm355_evm_init(void) > if (IS_ERR(aemif)) > WARN("%s: unable to get AEMIF clock\n", __func__); > else > - clk_enable(aemif); > + clk_prepare_enable(aemif); > > platform_add_devices(davinci_evm_devices, > ARRAY_SIZE(davinci_evm_devices)); > diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach-davinci/board-dm355-leopard.c > index 8e77032..b754ccf 100644 > --- a/arch/arm/mach-davinci/board-dm355-leopard.c > +++ b/arch/arm/mach-davinci/board-dm355-leopard.c > @@ -246,7 +246,7 @@ static __init void dm355_leopard_init(void) > if (IS_ERR(aemif)) > WARN("%s: unable to get AEMIF clock\n", __func__); > else > - clk_enable(aemif); > + clk_prepare_enable(aemif); > > platform_add_devices(davinci_leopard_devices, > ARRAY_SIZE(davinci_leopard_devices)); > diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c > index 2830692..3a4743b 100644 > --- a/arch/arm/mach-davinci/board-dm365-evm.c > +++ b/arch/arm/mach-davinci/board-dm365-evm.c > @@ -481,7 +481,7 @@ static void __init evm_init_cpld(void) > aemif_clk = clk_get(NULL, "aemif"); > if (IS_ERR(aemif_clk)) > return; > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (request_mem_region(DM365_ASYNC_EMIF_DATA_CE1_BASE, SECTION_SIZE, > "cpld") == NULL) > @@ -492,7 +492,7 @@ static void __init evm_init_cpld(void) > SECTION_SIZE); > fail: > pr_err("ERROR: can't map CPLD\n"); > - clk_disable(aemif_clk); > + clk_disable_unprepare(aemif_clk); > return; > } > > diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach-davinci/board-dm644x-evm.c > index d34ed55..04ef708 100644 > --- a/arch/arm/mach-davinci/board-dm644x-evm.c > +++ b/arch/arm/mach-davinci/board-dm644x-evm.c > @@ -775,7 +775,7 @@ static __init void davinci_evm_init(void) > struct davinci_soc_info *soc_info = &davinci_soc_info; > > aemif_clk = clk_get(NULL, "aemif"); > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (HAS_ATA) { > if (HAS_NAND || HAS_NOR) > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach-davinci/board-neuros-osd2.c > index 5de69f2..7fdd0a7b 100644 > --- a/arch/arm/mach-davinci/board-neuros-osd2.c > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c > @@ -221,7 +221,7 @@ static __init void davinci_ntosd2_init(void) > int status; > > aemif_clk = clk_get(NULL, "aemif"); > - clk_enable(aemif_clk); > + clk_prepare_enable(aemif_clk); > > if (HAS_ATA) { > if (HAS_NAND) > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices-da8xx.c > index d1624a3..20141f5 100644 > --- a/arch/arm/mach-davinci/devices-da8xx.c > +++ b/arch/arm/mach-davinci/devices-da8xx.c > @@ -905,7 +905,7 @@ static int da850_sata_init(struct device *dev, void __iomem *addr) > if (IS_ERR(da850_sata_clk)) > return PTR_ERR(da850_sata_clk); > > - ret = clk_enable(da850_sata_clk); > + ret = clk_prepare_enable(da850_sata_clk); > if (ret) > goto err0; > > @@ -936,7 +936,7 @@ static int da850_sata_init(struct device *dev, void __iomem *addr) > return 0; > > err1: > - clk_disable(da850_sata_clk); > + clk_disable_unprepare(da850_sata_clk); > err0: > clk_put(da850_sata_clk); > return ret; > @@ -944,7 +944,7 @@ err0: > > static void da850_sata_exit(struct device *dev) > { > - clk_disable(da850_sata_clk); > + clk_disable_unprepare(da850_sata_clk); > clk_put(da850_sata_clk); > } > > diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c > index 1875740..24ddb0d 100644 > --- a/arch/arm/mach-davinci/serial.c > +++ b/arch/arm/mach-davinci/serial.c > @@ -95,7 +95,7 @@ int __init davinci_serial_init(struct davinci_uart_config *info) > continue; > } > > - clk_enable(uart_clk); > + clk_prepare_enable(uart_clk); > p->uartclk = clk_get_rate(uart_clk); > > if (!p->membase && p->mapbase) { > diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c > index 75da315..9847938 100644 > --- a/arch/arm/mach-davinci/time.c > +++ b/arch/arm/mach-davinci/time.c > @@ -379,7 +379,7 @@ static void __init davinci_timer_init(void) > > timer_clk = clk_get(NULL, "timer0"); > BUG_ON(IS_ERR(timer_clk)); > - clk_enable(timer_clk); > + clk_prepare_enable(timer_clk); > > /* init timer hw */ > timer_init(); > @@ -429,7 +429,7 @@ void davinci_watchdog_reset(struct platform_device *pdev) > wd_clk = clk_get(&pdev->dev, NULL); > if (WARN_ON(IS_ERR(wd_clk))) > return; > - clk_enable(wd_clk); > + clk_prepare_enable(wd_clk); > > /* disable, internal clock source */ > __raw_writel(0, base + TCR); > -- > 1.7.9.5 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] arm:davinci: clk - migrate to new common clock API 2012-08-02 21:28 ` Mike Turquette @ 2012-08-13 21:15 ` Karicheri, Muralidharan 0 siblings, 0 replies; 7+ messages in thread From: Karicheri, Muralidharan @ 2012-08-13 21:15 UTC (permalink / raw) To: linux-arm-kernel >> -----Original Message----- >> From: Turquette, Mike >> Sent: Thursday, August 02, 2012 5:28 PM >> To: Karicheri, Muralidharan >> Cc: linux-arm-kernel at lists.infradead.org; davinci-linux-open- >> source at linux.davincidsp.com; linux-keystone at list.ti.com - Linux developers for Keystone >> family of devices (May contain non-TIers); Chemparathy, Cyril; Nori, Sekhar; >> linux at arm.linux.org.uk >> Subject: Re: [PATCH] arm:davinci: clk - migrate to new common clock API >> >> On 20120802-12:53, Murali Karicheri wrote: >> > 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@ti.com> >> >> Murali, >> >> The patch $subject is a bit misleading. This patch is preparation for >> converting davinci over to the common clk framework. How about: >> >> arm: davinci: clk: call clk_prepare_enable >> Ok >> Also, the important thing to know about clk_prepare is that it might >> sleep. All of the functions below look like initialization functions >> for code that is never run in interrupt context. If you can confirm >> that this is true then feel free to add my, >> Will investigate and repost as appropriate. Murali Karicheri Software Design Engineer >> Reviewed-by: Mike Turquette <mturquette@linaro.org> >> >> Regards, >> Mike >> >> > --- >> > arch/arm/mach-davinci/board-dm355-evm.c | 2 +- >> > arch/arm/mach-davinci/board-dm355-leopard.c | 2 +- >> > arch/arm/mach-davinci/board-dm365-evm.c | 4 ++-- >> > arch/arm/mach-davinci/board-dm644x-evm.c | 2 +- >> > arch/arm/mach-davinci/board-neuros-osd2.c | 2 +- >> > arch/arm/mach-davinci/devices-da8xx.c | 6 +++--- >> > arch/arm/mach-davinci/serial.c | 2 +- >> > arch/arm/mach-davinci/time.c | 4 ++-- >> > 8 files changed, 12 insertions(+), 12 deletions(-) >> > >> > diff --git a/arch/arm/mach-davinci/board-dm355-evm.c b/arch/arm/mach- >> davinci/board-dm355-evm.c >> > index 1c7b1f4..4821fba 100644 >> > --- a/arch/arm/mach-davinci/board-dm355-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm355-evm.c >> > @@ -324,7 +324,7 @@ static __init void dm355_evm_init(void) >> > if (IS_ERR(aemif)) >> > WARN("%s: unable to get AEMIF clock\n", __func__); >> > else >> > - clk_enable(aemif); >> > + clk_prepare_enable(aemif); >> > >> > platform_add_devices(davinci_evm_devices, >> > ARRAY_SIZE(davinci_evm_devices)); >> > diff --git a/arch/arm/mach-davinci/board-dm355-leopard.c b/arch/arm/mach- >> davinci/board-dm355-leopard.c >> > index 8e77032..b754ccf 100644 >> > --- a/arch/arm/mach-davinci/board-dm355-leopard.c >> > +++ b/arch/arm/mach-davinci/board-dm355-leopard.c >> > @@ -246,7 +246,7 @@ static __init void dm355_leopard_init(void) >> > if (IS_ERR(aemif)) >> > WARN("%s: unable to get AEMIF clock\n", __func__); >> > else >> > - clk_enable(aemif); >> > + clk_prepare_enable(aemif); >> > >> > platform_add_devices(davinci_leopard_devices, >> > ARRAY_SIZE(davinci_leopard_devices)); >> > diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach- >> davinci/board-dm365-evm.c >> > index 2830692..3a4743b 100644 >> > --- a/arch/arm/mach-davinci/board-dm365-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm365-evm.c >> > @@ -481,7 +481,7 @@ static void __init evm_init_cpld(void) >> > aemif_clk = clk_get(NULL, "aemif"); >> > if (IS_ERR(aemif_clk)) >> > return; >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (request_mem_region(DM365_ASYNC_EMIF_DATA_CE1_BASE, SECTION_SIZE, >> > "cpld") == NULL) >> > @@ -492,7 +492,7 @@ static void __init evm_init_cpld(void) >> > SECTION_SIZE); >> > fail: >> > pr_err("ERROR: can't map CPLD\n"); >> > - clk_disable(aemif_clk); >> > + clk_disable_unprepare(aemif_clk); >> > return; >> > } >> > >> > diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c b/arch/arm/mach- >> davinci/board-dm644x-evm.c >> > index d34ed55..04ef708 100644 >> > --- a/arch/arm/mach-davinci/board-dm644x-evm.c >> > +++ b/arch/arm/mach-davinci/board-dm644x-evm.c >> > @@ -775,7 +775,7 @@ static __init void davinci_evm_init(void) >> > struct davinci_soc_info *soc_info = &davinci_soc_info; >> > >> > aemif_clk = clk_get(NULL, "aemif"); >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (HAS_ATA) { >> > if (HAS_NAND || HAS_NOR) >> > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach- >> davinci/board-neuros-osd2.c >> > index 5de69f2..7fdd0a7b 100644 >> > --- a/arch/arm/mach-davinci/board-neuros-osd2.c >> > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c >> > @@ -221,7 +221,7 @@ static __init void davinci_ntosd2_init(void) >> > int status; >> > >> > aemif_clk = clk_get(NULL, "aemif"); >> > - clk_enable(aemif_clk); >> > + clk_prepare_enable(aemif_clk); >> > >> > if (HAS_ATA) { >> > if (HAS_NAND) >> > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices- >> da8xx.c >> > index d1624a3..20141f5 100644 >> > --- a/arch/arm/mach-davinci/devices-da8xx.c >> > +++ b/arch/arm/mach-davinci/devices-da8xx.c >> > @@ -905,7 +905,7 @@ static int da850_sata_init(struct device *dev, void __iomem >> *addr) >> > if (IS_ERR(da850_sata_clk)) >> > return PTR_ERR(da850_sata_clk); >> > >> > - ret = clk_enable(da850_sata_clk); >> > + ret = clk_prepare_enable(da850_sata_clk); >> > if (ret) >> > goto err0; >> > >> > @@ -936,7 +936,7 @@ static int da850_sata_init(struct device *dev, void __iomem >> *addr) >> > return 0; >> > >> > err1: >> > - clk_disable(da850_sata_clk); >> > + clk_disable_unprepare(da850_sata_clk); >> > err0: >> > clk_put(da850_sata_clk); >> > return ret; >> > @@ -944,7 +944,7 @@ err0: >> > >> > static void da850_sata_exit(struct device *dev) >> > { >> > - clk_disable(da850_sata_clk); >> > + clk_disable_unprepare(da850_sata_clk); >> > clk_put(da850_sata_clk); >> > } >> > >> > diff --git a/arch/arm/mach-davinci/serial.c b/arch/arm/mach-davinci/serial.c >> > index 1875740..24ddb0d 100644 >> > --- a/arch/arm/mach-davinci/serial.c >> > +++ b/arch/arm/mach-davinci/serial.c >> > @@ -95,7 +95,7 @@ int __init davinci_serial_init(struct davinci_uart_config *info) >> > continue; >> > } >> > >> > - clk_enable(uart_clk); >> > + clk_prepare_enable(uart_clk); >> > p->uartclk = clk_get_rate(uart_clk); >> > >> > if (!p->membase && p->mapbase) { >> > diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c >> > index 75da315..9847938 100644 >> > --- a/arch/arm/mach-davinci/time.c >> > +++ b/arch/arm/mach-davinci/time.c >> > @@ -379,7 +379,7 @@ static void __init davinci_timer_init(void) >> > >> > timer_clk = clk_get(NULL, "timer0"); >> > BUG_ON(IS_ERR(timer_clk)); >> > - clk_enable(timer_clk); >> > + clk_prepare_enable(timer_clk); >> > >> > /* init timer hw */ >> > timer_init(); >> > @@ -429,7 +429,7 @@ void davinci_watchdog_reset(struct platform_device *pdev) >> > wd_clk = clk_get(&pdev->dev, NULL); >> > if (WARN_ON(IS_ERR(wd_clk))) >> > return; >> > - clk_enable(wd_clk); >> > + clk_prepare_enable(wd_clk); >> > >> > /* disable, internal clock source */ >> > __raw_writel(0, base + TCR); >> > -- >> > 1.7.9.5 >> > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] arm:davinci: clk - migrate to new common clock API [not found] <1343926428-5210-1-git-send-email-m-karicheri2@ti.com> 2012-08-02 21:20 ` [PATCH] arm:davinci: clk - migrate to new common clock API Mike Turquette 2012-08-02 21:28 ` Mike Turquette @ 2012-08-14 6:30 ` Sekhar Nori 2012-08-30 16:17 ` Karicheri, Muralidharan 2 siblings, 1 reply; 7+ messages in thread From: Sekhar Nori @ 2012-08-14 6:30 UTC (permalink / raw) To: linux-arm-kernel Hi Murali, On 8/2/2012 10:23 PM, Murali Karicheri wrote: > 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@ti.com> Queuing this patch for v3.7. The subject line should have ARM in capitals for consistency. Also, you are not really migrating to new API in this patch - just preparing for migration so I prefer: "ARM: davinci: convert platform code to use clk_prepare/clk_unprepare" I will make this change while committing. Thanks, Sekhar ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] arm:davinci: clk - migrate to new common clock API 2012-08-14 6:30 ` Sekhar Nori @ 2012-08-30 16:17 ` Karicheri, Muralidharan 0 siblings, 0 replies; 7+ messages in thread From: Karicheri, Muralidharan @ 2012-08-30 16:17 UTC (permalink / raw) To: linux-arm-kernel Agreed. So I am not re-sending this patch. Murali Karicheri Software Design Engineer >> -----Original Message----- >> From: Nori, Sekhar >> Sent: Tuesday, August 14, 2012 2:30 AM >> To: Karicheri, Muralidharan >> Cc: linux-arm-kernel at lists.infradead.org; davinci-linux-open- >> source at linux.davincidsp.com; linux-keystone at list.ti.com - Linux developers for Keystone >> family of devices (May contain non-TIers); Turquette, Mike; Chemparathy, Cyril; >> linux at arm.linux.org.uk >> Subject: Re: [PATCH] arm:davinci: clk - migrate to new common clock API >> >> >> Hi Murali, >> >> On 8/2/2012 10:23 PM, Murali Karicheri wrote: >> > 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@ti.com> >> >> Queuing this patch for v3.7. The subject line should have ARM in >> capitals for consistency. Also, you are not really migrating to new API >> in this patch - just preparing for migration so I prefer: >> >> "ARM: davinci: convert platform code to use clk_prepare/clk_unprepare" >> >> I will make this change while committing. >> >> Thanks, >> Sekhar ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-08-30 16:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1343926428-5210-1-git-send-email-m-karicheri2@ti.com>
2012-08-02 21:20 ` [PATCH] arm:davinci: clk - migrate to new common clock API Mike Turquette
2012-08-13 21:13 ` [linux-keystone] " Karicheri, Muralidharan
2012-08-13 21:44 ` Turquette, Mike
2012-08-02 21:28 ` Mike Turquette
2012-08-13 21:15 ` Karicheri, Muralidharan
2012-08-14 6:30 ` Sekhar Nori
2012-08-30 16:17 ` Karicheri, Muralidharan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).