* [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*()
2012-03-15 16:20 arm: omap3: pm34xx.c: Some minor fixups Mark A. Greer
@ 2012-03-15 16:20 ` Mark A. Greer
0 siblings, 0 replies; 6+ messages in thread
From: Mark A. Greer @ 2012-03-15 16:20 UTC (permalink / raw)
To: linux-omap; +Cc: Mark A. Greer
Currently, pm34xx.c has a mix of printk() and pr_*() statements
so replace the printk() statements with the equivalent pr_*()
statements.
Signed-off-by: Mark A. Greer <mgreer@animalcreek.com>
---
arch/arm/mach-omap2/pm34xx.c | 17 ++++++++---------
1 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
index af8d741..495e509 100644
--- a/arch/arm/mach-omap2/pm34xx.c
+++ b/arch/arm/mach-omap2/pm34xx.c
@@ -166,8 +166,7 @@ static void omap3_save_secure_ram_context(void)
pwrdm_set_next_pwrst(mpu_pwrdm, mpu_next_state);
/* Following is for error tracking, it should not happen */
if (ret) {
- printk(KERN_ERR "save_secure_sram() returns %08x\n",
- ret);
+ pr_err("save_secure_sram() returns %08x\n", ret);
while (1)
;
}
@@ -307,7 +306,7 @@ void omap_sram_idle(void)
break;
default:
/* Invalid state */
- printk(KERN_ERR "Invalid mpu state in sram_idle\n");
+ pr_err("Invalid mpu state in sram_idle\n");
return;
}
@@ -463,7 +462,7 @@ restore:
list_for_each_entry(pwrst, &pwrst_list, node) {
state = pwrdm_read_prev_pwrst(pwrst->pwrdm);
if (state > pwrst->next_state) {
- printk(KERN_INFO "Powerdomain (%s) didn't enter "
+ pr_info("Powerdomain (%s) didn't enter "
"target state %d\n",
pwrst->pwrdm->name, pwrst->next_state);
ret = -1;
@@ -471,9 +470,9 @@ restore:
omap_set_pwrdm_state(pwrst->pwrdm, pwrst->saved_state);
}
if (ret)
- printk(KERN_ERR "Could not enter target state in pm_suspend\n");
+ pr_err("Could not enter target state in pm_suspend\n");
else
- printk(KERN_INFO "Successfully put all powerdomains "
+ pr_info("Successfully put all powerdomains "
"to target state\n");
return ret;
@@ -822,7 +821,7 @@ static int __init omap3_pm_init(void)
ret = pwrdm_for_each(pwrdms_setup, NULL);
if (ret) {
- printk(KERN_ERR "Failed to setup powerdomains\n");
+ pr_err("Failed to setup powerdomains\n");
goto err3;
}
@@ -830,7 +829,7 @@ static int __init omap3_pm_init(void)
mpu_pwrdm = pwrdm_lookup("mpu_pwrdm");
if (mpu_pwrdm == NULL) {
- printk(KERN_ERR "Failed to get mpu_pwrdm\n");
+ pr_err("Failed to get mpu_pwrdm\n");
ret = -EINVAL;
goto err3;
}
@@ -865,7 +864,7 @@ static int __init omap3_pm_init(void)
omap3_secure_ram_storage =
kmalloc(0x803F, GFP_KERNEL);
if (!omap3_secure_ram_storage)
- printk(KERN_ERR "Memory allocation failed when"
+ pr_err("Memory allocation failed when"
"allocating for secure sram context\n");
local_irq_disable();
--
1.7.0.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* arm: omap3: pm34xx.c: Some minor fixups
@ 2012-03-15 23:07 Mark A. Greer
2012-03-15 23:07 ` [PATCH 1/2] arm: omap3: pm34xx.c: Fix omap3_pm_init() error out paths Mark A. Greer
2012-03-15 23:07 ` [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*() Mark A. Greer
0 siblings, 2 replies; 6+ messages in thread
From: Mark A. Greer @ 2012-03-15 23:07 UTC (permalink / raw)
To: linux-omap; +Cc: linux-arm-kernel
I found some minor issues when looking through pm34xx.c recently
so these patches try to address them. My apologies if they are
already fixed in another branch somewhere. Based on latest k.o.
master branch.
Mark
--
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm: omap3: pm34xx.c: Fix omap3_pm_init() error out paths
2012-03-15 23:07 arm: omap3: pm34xx.c: Some minor fixups Mark A. Greer
@ 2012-03-15 23:07 ` Mark A. Greer
2012-03-15 23:07 ` [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*() Mark A. Greer
1 sibling, 0 replies; 6+ messages in thread
From: Mark A. Greer @ 2012-03-15 23:07 UTC (permalink / raw)
To: linux-omap; +Cc: linux-arm-kernel, Mark A. Greer, Tero Kristo
From: "Mark A. Greer" <mgreer@animalcreek.com>
It appears that the error paths were overlooked when the
omap3_pm_init() routine had the prcm chain handler code
added. Fix this by adding a goto target and reordering
the error handling code. Also fix how the irq argument
for free_irq() is determined.
CC: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Mark A. Greer <mgreer@animalcreek.com>
---
arch/arm/mach-omap2/pm34xx.c | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
index 27fc7a2..af8d741 100644
--- a/arch/arm/mach-omap2/pm34xx.c
+++ b/arch/arm/mach-omap2/pm34xx.c
@@ -817,13 +817,13 @@ static int __init omap3_pm_init(void)
if (ret) {
pr_err("pm: Failed to request pm_io irq\n");
- goto err1;
+ goto err2;
}
ret = pwrdm_for_each(pwrdms_setup, NULL);
if (ret) {
printk(KERN_ERR "Failed to setup powerdomains\n");
- goto err2;
+ goto err3;
}
(void) clkdm_for_each(clkdms_setup, NULL);
@@ -831,7 +831,8 @@ static int __init omap3_pm_init(void)
mpu_pwrdm = pwrdm_lookup("mpu_pwrdm");
if (mpu_pwrdm == NULL) {
printk(KERN_ERR "Failed to get mpu_pwrdm\n");
- goto err2;
+ ret = -EINVAL;
+ goto err3;
}
neon_pwrdm = pwrdm_lookup("neon_pwrdm");
@@ -879,14 +880,17 @@ static int __init omap3_pm_init(void)
}
omap3_save_scratchpad_contents();
-err1:
return ret;
-err2:
- free_irq(INT_34XX_PRCM_MPU_IRQ, NULL);
+
+err3:
list_for_each_entry_safe(pwrst, tmp, &pwrst_list, node) {
list_del(&pwrst->node);
kfree(pwrst);
}
+ free_irq(omap_prcm_event_to_irq("io"), omap3_pm_init);
+err2:
+ free_irq(omap_prcm_event_to_irq("wkup"), NULL);
+err1:
return ret;
}
--
1.7.9.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*()
2012-03-15 23:07 arm: omap3: pm34xx.c: Some minor fixups Mark A. Greer
2012-03-15 23:07 ` [PATCH 1/2] arm: omap3: pm34xx.c: Fix omap3_pm_init() error out paths Mark A. Greer
@ 2012-03-15 23:07 ` Mark A. Greer
2012-03-16 12:42 ` Sergei Shtylyov
1 sibling, 1 reply; 6+ messages in thread
From: Mark A. Greer @ 2012-03-15 23:07 UTC (permalink / raw)
To: linux-omap; +Cc: linux-arm-kernel, Mark A. Greer
From: "Mark A. Greer" <mgreer@animalcreek.com>
Currently, pm34xx.c has a mix of printk() and pr_*() statements
so replace the printk() statements with the equivalent pr_*()
statements.
Signed-off-by: Mark A. Greer <mgreer@animalcreek.com>
---
arch/arm/mach-omap2/pm34xx.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
index af8d741..495e509 100644
--- a/arch/arm/mach-omap2/pm34xx.c
+++ b/arch/arm/mach-omap2/pm34xx.c
@@ -166,8 +166,7 @@ static void omap3_save_secure_ram_context(void)
pwrdm_set_next_pwrst(mpu_pwrdm, mpu_next_state);
/* Following is for error tracking, it should not happen */
if (ret) {
- printk(KERN_ERR "save_secure_sram() returns %08x\n",
- ret);
+ pr_err("save_secure_sram() returns %08x\n", ret);
while (1)
;
}
@@ -307,7 +306,7 @@ void omap_sram_idle(void)
break;
default:
/* Invalid state */
- printk(KERN_ERR "Invalid mpu state in sram_idle\n");
+ pr_err("Invalid mpu state in sram_idle\n");
return;
}
@@ -463,7 +462,7 @@ restore:
list_for_each_entry(pwrst, &pwrst_list, node) {
state = pwrdm_read_prev_pwrst(pwrst->pwrdm);
if (state > pwrst->next_state) {
- printk(KERN_INFO "Powerdomain (%s) didn't enter "
+ pr_info("Powerdomain (%s) didn't enter "
"target state %d\n",
pwrst->pwrdm->name, pwrst->next_state);
ret = -1;
@@ -471,9 +470,9 @@ restore:
omap_set_pwrdm_state(pwrst->pwrdm, pwrst->saved_state);
}
if (ret)
- printk(KERN_ERR "Could not enter target state in pm_suspend\n");
+ pr_err("Could not enter target state in pm_suspend\n");
else
- printk(KERN_INFO "Successfully put all powerdomains "
+ pr_info("Successfully put all powerdomains "
"to target state\n");
return ret;
@@ -822,7 +821,7 @@ static int __init omap3_pm_init(void)
ret = pwrdm_for_each(pwrdms_setup, NULL);
if (ret) {
- printk(KERN_ERR "Failed to setup powerdomains\n");
+ pr_err("Failed to setup powerdomains\n");
goto err3;
}
@@ -830,7 +829,7 @@ static int __init omap3_pm_init(void)
mpu_pwrdm = pwrdm_lookup("mpu_pwrdm");
if (mpu_pwrdm == NULL) {
- printk(KERN_ERR "Failed to get mpu_pwrdm\n");
+ pr_err("Failed to get mpu_pwrdm\n");
ret = -EINVAL;
goto err3;
}
@@ -865,7 +864,7 @@ static int __init omap3_pm_init(void)
omap3_secure_ram_storage =
kmalloc(0x803F, GFP_KERNEL);
if (!omap3_secure_ram_storage)
- printk(KERN_ERR "Memory allocation failed when"
+ pr_err("Memory allocation failed when"
"allocating for secure sram context\n");
local_irq_disable();
--
1.7.9.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*()
2012-03-15 23:07 ` [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*() Mark A. Greer
@ 2012-03-16 12:42 ` Sergei Shtylyov
2012-03-16 18:02 ` Mark A. Greer
0 siblings, 1 reply; 6+ messages in thread
From: Sergei Shtylyov @ 2012-03-16 12:42 UTC (permalink / raw)
To: Mark A. Greer; +Cc: linux-omap, linux-arm-kernel
Hello.
On 16-03-2012 3:07, Mark A. Greer wrote:
> From: "Mark A. Greer" <mgreer@animalcreek.com>
> Currently, pm34xx.c has a mix of printk() and pr_*() statements
> so replace the printk() statements with the equivalent pr_*()
> statements.
> Signed-off-by: Mark A. Greer <mgreer@animalcreek.com>
> ---
> arch/arm/mach-omap2/pm34xx.c | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
> diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
> index af8d741..495e509 100644
> --- a/arch/arm/mach-omap2/pm34xx.c
> +++ b/arch/arm/mach-omap2/pm34xx.c
[...]
> @@ -463,7 +462,7 @@ restore:
> list_for_each_entry(pwrst,&pwrst_list, node) {
> state = pwrdm_read_prev_pwrst(pwrst->pwrdm);
> if (state> pwrst->next_state) {
> - printk(KERN_INFO "Powerdomain (%s) didn't enter "
> + pr_info("Powerdomain (%s) didn't enter "
> "target state %d\n",
Last line should have been realigned I think.
> pwrst->pwrdm->name, pwrst->next_state);
> ret = -1;
> @@ -471,9 +470,9 @@ restore:
> omap_set_pwrdm_state(pwrst->pwrdm, pwrst->saved_state);
> }
> if (ret)
> - printk(KERN_ERR "Could not enter target state in pm_suspend\n");
> + pr_err("Could not enter target state in pm_suspend\n");
> else
> - printk(KERN_INFO "Successfully put all powerdomains "
> + pr_info("Successfully put all powerdomains "
> "to target state\n");
Last line should have been realigned I think.
>
> return ret;
> @@ -865,7 +864,7 @@ static int __init omap3_pm_init(void)
> omap3_secure_ram_storage =
> kmalloc(0x803F, GFP_KERNEL);
> if (!omap3_secure_ram_storage)
> - printk(KERN_ERR "Memory allocation failed when"
> + pr_err("Memory allocation failed when"
BTW, space after "when" is missing here.
> "allocating for secure sram context\n");
Should have been realigned I think.
WBR, Sergei
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*()
2012-03-16 12:42 ` Sergei Shtylyov
@ 2012-03-16 18:02 ` Mark A. Greer
0 siblings, 0 replies; 6+ messages in thread
From: Mark A. Greer @ 2012-03-16 18:02 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: linux-omap, linux-arm-kernel
On Fri, Mar 16, 2012 at 04:42:16PM +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 16-03-2012 3:07, Mark A. Greer wrote:
>
> >From: "Mark A. Greer" <mgreer@animalcreek.com>
> >@@ -463,7 +462,7 @@ restore:
> > list_for_each_entry(pwrst,&pwrst_list, node) {
> > state = pwrdm_read_prev_pwrst(pwrst->pwrdm);
> > if (state> pwrst->next_state) {
> >- printk(KERN_INFO "Powerdomain (%s) didn't enter "
> >+ pr_info("Powerdomain (%s) didn't enter "
> > "target state %d\n",
>
> Last line should have been realigned I think.
Okay, I'll fix this and the others in a bit.
Mark
--
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-03-16 18:02 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-15 23:07 arm: omap3: pm34xx.c: Some minor fixups Mark A. Greer
2012-03-15 23:07 ` [PATCH 1/2] arm: omap3: pm34xx.c: Fix omap3_pm_init() error out paths Mark A. Greer
2012-03-15 23:07 ` [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*() Mark A. Greer
2012-03-16 12:42 ` Sergei Shtylyov
2012-03-16 18:02 ` Mark A. Greer
-- strict thread matches above, loose matches on Subject: below --
2012-03-15 16:20 arm: omap3: pm34xx.c: Some minor fixups Mark A. Greer
2012-03-15 16:20 ` [PATCH 2/2] arm: omap3: pm34xx.c: Replace printk() with appropriate pr_*() Mark A. Greer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox