[PATCH 1/9] davinci: pm: fix compiler errors and kernel panics from sram consolidation

[Ben Gardiner]

From: Nori, Sekhar <nsek...@ti.com>

Hi Russell,

On Thu, May 12, 2011 at 23:15:46, Russell King - ARM Linux wrote:

> diff --git a/arch/arm/mach-davinci/pm.c b/arch/arm/mach-davinci/pm.c
> index 1bd73a0..c2f9767 100644
> --- a/arch/arm/mach-davinci/pm.c
> +++ b/arch/arm/mach-davinci/pm.c

[...]

> @@ -123,14 +118,14 @@ static int __init davinci_pm_probe(struct 
> platform_device *pdev)
>               return -ENOENT;
>       }
>
> -     davinci_sram_suspend = sram_alloc(davinci_cpu_suspend_sz, NULL);
> -     if (!davinci_sram_suspend) {
> +     davinci_sram_suspend_mem = pv_pool_alloc(davinci_pv_pool,
> +                             davinci_cpu_suspend_sz, NULL);
> +     if (!davinci_sram_suspend_mem) {
>               dev_err(&pdev->dev, "cannot allocate SRAM memory\n");
>               return -ENOMEM;
>       }
> -
> -     davinci_sram_push(davinci_sram_suspend, davinci_cpu_suspend,
> -                                             davinci_cpu_suspend_sz);
> +     davinci_sram_suspend = fncpy(davinci_sram_suspend_mem,
> +                             davinci_cpu_suspend, davinci_cpu_suspend_sz);

This gave these build errors:

arch/arm/mach-davinci/pm.c: In function 'davinci_pm_probe':
arch/arm/mach-davinci/pm.c:127: error: lvalue required in asm statement
arch/arm/mach-davinci/pm.c:127: error: invalid lvalue in asm output 0
make[1]: *** [arch/arm/mach-davinci/pm.o] Error 1

Replacing davinci_cpu_suspend with &davinci_cpu_suspend fixed the issue.

With that change done, tested suspend-to-RAM on DA850 platform.
Also tested audio driver on DM365 platform with IRAM buffers enabled.

Can you also fold the following patch in? Without this the
kernel panics when suspend-to-RAM is enabled.

Thanks,
Sekhar
---
 arch/arm/mach-davinci/pm.c    |    2 +-
 arch/arm/mach-davinci/sleep.S |    1 +
 2 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-davinci/pm.c b/arch/arm/mach-davinci/pm.c
index c2f9767..5255223 100644
--- a/arch/arm/mach-davinci/pm.c
+++ b/arch/arm/mach-davinci/pm.c
@@ -125,7 +125,7 @@ static int __init davinci_pm_probe(struct platform_device 
*pdev)
                return -ENOMEM;
        }
        davinci_sram_suspend = fncpy(davinci_sram_suspend_mem,
-                               davinci_cpu_suspend, davinci_cpu_suspend_sz);
+                               &davinci_cpu_suspend, davinci_cpu_suspend_sz);
 
        suspend_set_ops(&davinci_pm_ops);
 
diff --git a/arch/arm/mach-davinci/sleep.S b/arch/arm/mach-davinci/sleep.S
index fb5e72b..2cef533 100644
--- a/arch/arm/mach-davinci/sleep.S
+++ b/arch/arm/mach-davinci/sleep.S
@@ -37,6 +37,7 @@
 #define DEEPSLEEP_SLEEPENABLE_BIT      BIT(31)
 
        .text
+       .align 3
 /*
  * Move DaVinci into deep sleep state
  *
-- 
1.7.4.1

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source