[U-Boot] [PATCH 3/4] ARM: AM33xx: Move s_init to a common place

From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/4] ARM: AM33xx: Move s_init to a common place
Date: Tue, 25 Jun 2013 07:19:13 +0200	[thread overview]
Message-ID: <51C92851.2030204@denx.de> (raw)
In-Reply-To: <CAD6qDH6_bGLwKqsbGgDRCCJsAKoAeoUtjmAykXsjugYujzjV=w@mail.gmail.com>

Hello Summit,

Am 25.06.2013 07:09, schrieb Sumit Gemini:
> Hi Heiko,
> 
>      I followed the same steps that you mentioned, but still i have same problem.
>      During get uboot-spl.bin from uart, system become hanged and it is not ready to receive u-boot.img.

On which hardware?

>      Still compiling to uboot, i had to comment rtc32k_enable() function. then it is working fine.

Ok ... Hmm.. maybe some missing init the ROM Bootloader does (and some
ROM bootloader versions not, and we miss this in spl code)?
(just speculation ...)

bye,
Heiko
> On Tue, Jun 25, 2013 at 1:14 AM, Heiko Schocher <hs at denx.de <mailto:hs@denx.de>> wrote:
> 
>     Hello Tom,
> 
>     Am 24.06.2013 21 <tel:24.06.2013%2021>:31, schrieb Tom Rini:
>     > On 06/24/2013 03:17 PM, Heiko Schocher wrote:
>     >> Hello Lokesh,
>     > [snip]
>     >>> +   /* Enable RTC32K clock */ +     rtc32k_enable();
>     >
>     >> I tried your patches on my three boards. The board with the
>     >> rtc32k_enable() problem, did not work with them too :-(
>     >
>     >> Maybe we make rtc32k_enable() weak in common code, and so I can
>     >> make a board specific (dummy) function?
>     >
>     > I'm not a big fan of this since without this kicked, you can't do PM.
>     >  And we don't want to do it in the kernel as it takes a few seconds to
>     > settle.  Kicking it in U-Boot means it will be settled by the time
>     > it's needed, if I recall things right.
> 
>     Hmm.. but on my board, the cpu hangs (not accessible through bdi
>     anymore) if I access this registers ... looking still for an
>     explanation ...
> 
>     Clocks are enabled, 24MHz (same as on am335x eval board) used ...
>     I know, this is only a bugfix, without knowing the real reason ...
> 
>     bye,
>     Heiko
>     --
>     DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
>     HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>     _______________________________________________
>     U-Boot mailing list
>     U-Boot at lists.denx.de <mailto:U-Boot@lists.denx.de>
>     http://lists.denx.de/mailman/listinfo/u-boot
> 
> 

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany