Re: OMAP3: enabling CPU idle leads to panic

[Sergey Lapin <slapinid@gmail.com>]

On Tue, Nov 24, 2009 at 11:21 PM, Premi, Sanjeev <premi@ti.com> wrote:
> From: Kevin Hilman [khilman@deeprootsystems.com]
> Sent: Tuesday, November 24, 2009 11:57 PM
> To: Sergey Lapin; Premi, Sanjeev
> Cc: linux-omap@vger.kernel.org
> Subject: Re: OMAP3: enabling CPU idle leads to panic
>
> Sergey Lapin <slapinid@gmail.com> writes:
>
>> I use PM branch merged with linux-omap branch.
>
> Not sure what this means.  PM branch is already based at linux-omap
> master branch.  You'll need to be more specific, ideally with commit
> IDs.

I do git branch tmp origin/master
git checkout -f tmp
git merge origin/pm
git merge origin/mybranch

git log origin/pm^..origin/pm
commit 596f2d2c8153fe24de5c3b6ad78658ce9fff7f28
Author: Kevin Hilman <khilman@deeprootsystems.com>
Date:   Wed Jun 17 14:42:22 2009 -0700

    OMAP3: PM: default defconfig for PM kernel testing

    Supports 3430SDP, OMAP3EVM and 3630SDP by default.

    Also supports, Beagle, Overo, Zoom[23] and RX51 simply by changing

     System Type-->TI OMAP implementations-->Low-level debug console UART

    from UART1 to UART3.

    Signed-off-by: Kevin Hilman <khilman@ti.deeprootsystems.com>


>
> [sp] I will also need the last commit from pm branch - if not the id; the
> commit header to start.
See above

>
> [snip]--[snip]
>
>>
>> addr2line -e vmlinux c003b5c0
>> /home/slapin/linux-omap-my-2.6.git/arch/arm/mach-omap2/powerdomain.c:452
>> It is in function
>> int pwrdm_for_each_clkdm(), where pwrdm is NULL.
>> farther investigations found that "core_pwrdm" is defined only for 24xx,
>> but I use 3525, and it is being searched.
>
> [sp] I am a little confused here. core_pwrdm is defined for 34xx as well.
> Other than sgx, there isn't any difference between 3530 and 3525.
Yes, I see that in source, but that function returns NULL for unknown reason.
How could I debug it farther?


> See the definition at:
> http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=blob;f=arch/arm/mach-omap2/powerdomains34xx.h;hb=867d320b6c30d2478358eafeca0e1a6c60cf06c3#l203
>
Yes, there is.

>>
>> This is where variable allocated. If I add BUG_ON(!core_pd), it crashes here.
>> arch/arm/mach-omap2/cpuidle34xx.c:348:        core_pd = pwrdm_lookup("core_pwrdm");
>> Otherwise it crashes the above way. Any ideas on how to fix this?
>
> [sp] Have you hand-picked the commits from pm branch and applied against
>  master?
See above. I use git merge on fresh branch based on linux-omap master.

Thanks a lot,
S.