<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.6944.0">
<TITLE>Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function only if CONFIG_SKIP_LOWLEVEL_INIT is not defined</TITLE>
</HEAD>
<BODY>
<DIV id=idOWAReplyText8658 dir=ltr>
<DIV dir=ltr><FONT face=Verdana color=#000000 size=2>Hi,</FONT></DIV>
<DIV dir=ltr><FONT face=Verdana size=2></FONT> </DIV>
<DIV dir=ltr><FONT face=Verdana size=2>At present in "start.S" inside
cpu\arm926ejs, cpu_init_crit is called only if
"CONFIG_SKIP_LOWLEVEL_INIT" is not defined </FONT></DIV>
<DIV dir=ltr><FONT face=Verdana size=2></FONT> </DIV></DIV>
<DIV dir=ltr>#ifndef
CONFIG_SKIP_LOWLEVEL_INIT<BR> bl cpu_init_crit<BR>#endif</DIV>
<DIV dir=ltr> </DIV>
<DIV dir=ltr>But "cpu_init_crit" function doesnt have any if defs which
means cpu_init_crit function will compile irrespective of <FONT
face=Verdana size=2>"CONFIG_SKIP_LOWLEVEL_INIT". Hence the patch. The same is
done in other processor files. Kindly check "cpu\arm920t\start.S" </FONT></DIV>
<P><FONT face=Verdana size=2>Regards</FONT></P>
<P><FONT face=Verdana size=2>Gururaja</FONT></P>
<DIV dir=ltr><BR></DIV>
<DIV dir=ltr>
<HR tabIndex=-1>
</DIV>
<DIV dir=ltr><FONT face=Tahoma size=2><B>From:</B> Dirk Behme
[mailto:dirk.behme@googlemail.com]<BR><B>Sent:</B> Wed 19-Dec-07 3:27
PM<BR><B>To:</B> Gururaja Hebbar K R<BR><B>Cc:</B>
u-boot-users@lists.sourceforge.net<BR><B>Subject:</B> Re: [U-Boot-Users] [PATCH]
ARM926: compile cpu_init_crit function only if CONFIG_SKIP_LOWLEVEL_INIT is not
defined<BR></FONT><BR></DIV>
<DIV>
<P><FONT size=2>Hebbar wrote:<BR>> This patches allows cpu_init_crit function
to be compilled only if<BR>> CONFIG_SKIP_LOWLEVEL_INIT is not defined. At
present irrespective of<BR>> CONFIG_SKIP_LOWLEVEL_INIT, cpu_init_crit is
always compilled. This is for<BR>> arm926ejs module.<BR>><BR>>
Signed-off-by: K R Gururaja Hebbar
<gururajakr@sanyo.co.in><BR>><BR>><BR>> ---
u-boot-1.3.1/cpu/arm926ejs/start.S
2007-12-06 01:21:19.000000000 -0800<BR>> +++
uboot/cpu/arm926ejs/start.S 2007-12-19
08:40:37.296875000 -0800<BR>> @@ -187,7 +187,7 @@
clbss_l:str r2,
[r0]
/* clear loop... <BR>>
_start_armboot:<BR>> .word
start_armboot<BR>> <BR>> -<BR>> +#ifndef
CONFIG_SKIP_LOWLEVEL_INIT<BR>> /*<BR>>
*************************************************************************<BR>>
*<BR>> @@ -225,6 +225,9 @@ cpu_init_crit:<BR>>
bl
lowlevel_init /* go setup pll,mux,memory */<BR>>
mov lr, ip
/* restore link */<BR>>
mov pc, lr
/* back to my caller */<BR>>
+ <BR>> +#endif /* CONFIG_SKIP_LOWLEVEL_INIT
*/<BR>> + <BR>> /*<BR>>
*************************************************************************<BR>>
*<BR>> I welcome comments, complaints, suggestions and advices.<BR><BR>The
reason for this is to decrease resulting binary size? If so,
all<BR>lowlevel_init() should be encapsulated as well to save even more
space?<BR><BR>Dirk<BR><BR></FONT></P></DIV>
</BODY>
</HTML>