* [PATCH] ARM pca-a-l1: call omap3_core_init from board code
@ 2012-01-20 21:25 Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: delete platform.S Juergen Kilb
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Juergen Kilb @ 2012-01-20 21:25 UTC (permalink / raw)
To: barebox; +Cc: Juergen Kilb
since commit ae2ac15105cbf198ebd2b7ac0428641423b83c08
a_init was renamed to omap3_core_init and called from board code.
Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
---
arch/arm/boards/phycard-a-l1/pca-a-l1.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boards/phycard-a-l1/pca-a-l1.c b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
index 0518894..8015087 100644
--- a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
+++ b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
@@ -226,6 +226,8 @@ static int pcaal1_board_init(void)
{
int in_sdram = running_in_sdram();
+ omap3_core_init();
+
pcaal1_mux_config();
/* Dont reconfigure SDRAM while running in SDRAM! */
--
1.7.0.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM pca-a-l1: delete platform.S
2012-01-20 21:25 [PATCH] ARM pca-a-l1: call omap3_core_init from board code Juergen Kilb
@ 2012-01-20 21:25 ` Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: fix board_init Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: should depend on ARCH_OMAP3 Juergen Kilb
2 siblings, 0 replies; 8+ messages in thread
From: Juergen Kilb @ 2012-01-20 21:25 UTC (permalink / raw)
To: barebox; +Cc: Juergen Kilb
since commit 494a12f703c33f80fe96e2a728545c490347eceb
platform lowlevel init was moved to mach-omap.
Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
---
arch/arm/boards/phycard-a-l1/platform.S | 65 -------------------------------
1 files changed, 0 insertions(+), 65 deletions(-)
delete mode 100644 arch/arm/boards/phycard-a-l1/platform.S
diff --git a/arch/arm/boards/phycard-a-l1/platform.S b/arch/arm/boards/phycard-a-l1/platform.S
deleted file mode 100644
index 596d3ab..0000000
--- a/arch/arm/boards/phycard-a-l1/platform.S
+++ /dev/null
@@ -1,65 +0,0 @@
-/**
- * @file
- * @brief Wrapper to call board level initialization routine
- *
- * FileName: arch/arm/boards/phycard-a-l1/platform.S
- *
- * board_init_lowlevel is defined here. This calls board_init which
- * is linked to the binary - the board_init only has a SRAM stack.
- * so it needs to be careful about the usage of global variables
- * and the likes. Enabled only if CONFIG_MACH_DO_LOWLEVEL_INIT is
- * defined
- */
-/*
- * (C) Copyright 2006-2008
- * Texas Instruments, <www.ti.com>
- * Nishanth Menon <x0nishan@ti.com>
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include <config.h>
-#include <mach/silicon.h>
-
-#ifdef CONFIG_MACH_DO_LOWLEVEL_INIT
-/**
- * @fn void board_init_lowlevel(void)
- *
- * @brief This provides a assembly wrapper setting up SRAM before calling
- * board_init
- *
- * @return void
- */
-.globl board_init_lowlevel
-board_init_lowlevel:
- /* Setup a temporary stack so that we can call C functions
- * Yes. this might have been already done by arch code.
- * No harm in being a bit redundant to avoid future complications
- */
- ldr sp, SRAM_STACK
- str ip, [sp] /* stash old link register */
- str lr, [sp] /* stash current link register */
- mov ip, lr /* save link reg across call */
- /* Do the pin muxes, sdram init etc..board-xxx.c */
- bl board_init
- ldr lr, [sp] /* restore current link register */
- ldr ip, [sp] /* restore save ip */
- /* back to arch calling code */
- mov pc, lr
-SRAM_STACK:
- .word OMAP_SRAM_STACK
-
-#endif /* CONFIG_MACH_DO_LOWLEVEL_INIT */
--
1.7.0.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM pca-a-l1: fix board_init
2012-01-20 21:25 [PATCH] ARM pca-a-l1: call omap3_core_init from board code Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: delete platform.S Juergen Kilb
@ 2012-01-20 21:25 ` Juergen Kilb
2012-01-23 8:35 ` Sascha Hauer
2012-01-23 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: should depend on ARCH_OMAP3 Juergen Kilb
2 siblings, 2 replies; 8+ messages in thread
From: Juergen Kilb @ 2012-01-20 21:25 UTC (permalink / raw)
To: barebox; +Cc: Juergen Kilb
board_init was moved to pure_initcall. broken since:
commit 0adce7ec683b4b325f51de4ac8892b32925d7ef8
Author: Sascha Hauer <s.hauer@pengutronix.de>
Date: Sun Jan 15 21:11:17 2012 +0100
ARM omap3: move board_init to pure_initcall
board_init initializes the mux and sdram. For both there is no
need to configure this so early. Move the code to a pure_initcall
and remove the surrounding unneeded code.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Tested-by: Sanjeev Premi <premi@ti.com>
Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
---
arch/arm/boards/phycard-a-l1/pca-a-l1.c | 8 +++++---
arch/arm/mach-omap/Kconfig | 1 -
barebox_default_env.gz | Bin 0 -> 2656 bytes
3 files changed, 5 insertions(+), 4 deletions(-)
create mode 100644 barebox_default_env.gz
diff --git a/arch/arm/boards/phycard-a-l1/pca-a-l1.c b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
index 11e8638..0518894 100644
--- a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
+++ b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
@@ -74,8 +74,6 @@
#include <mach/syslib.h>
#include <mach/xload.h>
-#include "pca-a-l1.h"
-
#define SMC911X_BASE 0x2c000000
/*
@@ -224,15 +222,19 @@ static void pcaal1_mux_config(void)
*
* @return void
*/
-void omap3_board_init(void)
+static int pcaal1_board_init(void)
{
int in_sdram = running_in_sdram();
pcaal1_mux_config();
+
/* Dont reconfigure SDRAM while running in SDRAM! */
if (!in_sdram)
pcaal1_sdrc_init();
+
+ return 0;
}
+pure_initcall(pcaal1_board_init);
/*
* Run-time initialization(s)
diff --git a/arch/arm/mach-omap/Kconfig b/arch/arm/mach-omap/Kconfig
index 970c899..ecdabb0 100644
--- a/arch/arm/mach-omap/Kconfig
+++ b/arch/arm/mach-omap/Kconfig
@@ -150,7 +150,6 @@ config MACH_PCM049
config MACH_PCAAL1
bool "Phytec phyCARD-A-L1"
- select MACH_HAS_LOWLEVEL_INIT
select OMAP_CLOCK_ALL
select HAS_OMAP_NAND
help
diff --git a/barebox_default_env.gz b/barebox_default_env.gz
new file mode 100644
index 0000000000000000000000000000000000000000..bdf466e5d710f54fe8d66db5fb74edb33a788ff3
GIT binary patch
literal 2656
zcmV-m3ZL~KiwFP!000021Em@5ZrjLFxT3(B=%GM?{w%ttg~~oi6eT;?R0)%&L9bU7
zX@Xu3#UTk)id>pJkz9gYSyF60LR<7L`V4)6zQH|DXZC|!%95f>4JYId=X+*nXV&Zg
z{BzxbpMQM+ujid#cIuyBz4$!-`+xs^{kKm$9r*1G1C~e0b?5AKI7{+j6<b#32XT^9
zuLyiU9a(3Kik6;bosr+?oR;Lpv**MQ!;)4NiD^km5|Ny7Qc_+f^oEASUvQTBJPG_X
zU0Y|A$F2kaA(0o-i62g8bV8$v>%uQG1_`7W#zD~|oRJWI;hiKonfWE1u@&+25SMdW
z<}}p=Im>AerLWn7Ed87#aWzj0g-75GRgo4hO3;YqR#JG#9mTSeRFormNpi}UtehwL
zH7Wek&nTEG?NbIMh0x!3W=`OQ3J6r0U(L6m;IA86f}l3|+UIoXuWKl15cpstB{bn6
ziS&5Hi(Vfriz@K|vwNszFHEX=&yr?MQx<sm={`NEQ`7Yom7q`ZgqLB@DjDNZ)o=h7
z5qytQzly=x6$DSBOi1s$D5}Pid@)NvBS3~21b{_YM36beWest^UZg%SNm5#^D^4?#
zCUb}nUGXuh0nbZ(mNWs+AS0y)Ul-I9Vu1Bh=2At#6s3eIu4kw&?K)4Be6b=q#OgwG
zgwXM+fKNku<7HVeOXqHJrM@Sbe@#94KIq=6MBg}FLm_Kh7oAK<+tSDgM=L(4gn+ot
z7`eR$CXyQYBWc6dA<tz97XqVC-m7QnFVXezn5OTUbtciJ0q_5ak3m=IZ<*f~Ti;rQ
z$A5EBsHL`_L{JOUZ%{Eb4$W^4qFJy{-;hC0x>!!gyGQ7R+&XEAu*FAvV0TThU1QZk
z?jh<gfNL7n;2BWBG1H;JF`$40n-Bq>J&ajKhXQM;fJ5yk+oEYjP2|Go@sKmb93sfJ
zqQtVW4$WXS<w-`_f_s<XU0D0&b>-Pmp(>VAkMs4*aep*=Fm_!VQwI|zVrj~jSePKE
ziqv0M1Z*bNq9|C&DVBjEbRW1k>4=<vapjT~xq9ecNbu!@UEq;>RizO*f9N9k3s-_O
z_t)++z0zQbjb$?Pi)iL#ETrCJrNfiSV_ZM!zv(-U^RB@P>1=WBJzn4CPXMTX25@xv
zvy>9?l@w)?b1=vu-$Lb%{TqsvTRdOBc=qbc-zhw0BB6pfZ1|DpFd(wJJD_Rf&#rzI
zT7^{wYDH62-P5kJt+QkCg%6L%m(D#JUwIwEjTKoG51@E38qd#HgOvFpEiX#eAN^+R
z&d*i0U-U=ghxH0_|Du2O;#@l9qCYstyt}X^L#TsEKs^*Z9w5YVb+ycRID_gnpCZ`1
z!i2-~&?k|f2Wtou%L-1;Vdv^dKuHV$2$des;|uH6*CWq<OW%whU5+!Q&Tr*<l)a57
zZ<nxeGh$oVbNu;}PRGKY13L{olHlFwJ6=6|{gpQaYY<}5m5_))%W{Z-%UG*YLdg(p
z8|JKZ$-u|bVs~L7J!m%-299k68Q2Y)*qbeOqHfpxaCMlW#5WM{vqbM2D%MfntE0d-
z5H0KEjC>RK34?#}0*>k^DJ$3)s{(K;vP{x2@Jo2ZJ_$LJiX$Ol&|raE4LLyMGQ@Yt
zUt=G`414U<I;o)j2`JHyAEJP1><4qhlBoHNahuK<h-=h$c6}D)G>Dn_PZ<AI!bQt4
zr)$xxaV9%7+#VB15>*`0SKaN3DK+UnC8r*78_{%?a3KkCA-OQme(|GD=Vv<4ri&tk
z#%mhWv}os>P|IFaQ0^y0VvzwSkL5nPlbhsDW%XUN+3r@jEuWA<NFGBb-C>}fkT-*j
zVE=Zv!orxn13)B?ee6))kwIW<iBc42EUg?3)K6Amms;s1kcoN+-U?zG%*BZil)>d#
zG;X|ZEim$;jf0)Z#JSXB3-!idq@18`qOk!@^*7O#H1?~yYFc-P>HT37K}p3%_e?o}
zgjRcGsXp(Ad9o2xw}~GRqsNGkm80p86$DB0|H?uZ9~37N@uMY4YEwu@?w>z>YSd}m
zSM7b%F2-_SuDVS(OgyrJNu=4MMx%_HHFagx9SLMi-Wq#*MmgCYd!UUoiCMXumX&Q%
zSZ`56>g=SXyiFNZTa;+UQKZ7$Hm#VOcG2R87O?;rzJhTU>2Wk}+UFfThMI8gMzEyB
zdHI8sIcu8QJt$pc9-eet6R%pTNms4ZkUW{-Y9jk#ZBt|ay{y?KUG&)Ukc8mrrn{?N
z`Zt$-4&KR`oOcZmwB@v8a9B`x=V+-)kC<T1>9LkA5SD(z_bQ?Aj#Q)cGnbSh5_!W)
zpR_tsyS>@kRwr9cU#)hl3xoqzFgyYq_ZNn8OStd?NEdeohIrXXyodR3mLRI}=|q7%
zTMkSo$oFKwLjc{K)O_ISfL8Z3v};=qdsgG`pcm%tqdpXU+vuHgqu*?d1}Uo5he27r
zZ;E(bL7zSl<=6omZ)?iUHp{v@Y<Sim8;*vTcS?LDt=#<)#5WVfW}=om)zLIN)A?4v
zZ%8f25M!G~ZtEDq=78M;x9jhL@^~Vy`h<mZybv0SgU~uL`I%P(u)xiIcSo?VB%@~U
zZs}znI)bxZj~9+awfEAtwxvmoeZQi1X0yEv+SSWo&^0cgbY+lz11(EdPQ-L)@PdOM
zj|9BS+I91YPgWWikhW~M!7aMACpLx_?<Q2o7s1RZooLIJ99H5ng~nEXutbmXbLc64
zs(Xxj<mhySccA|_Jo35jOP>C?)A@;}kxh)*mx%2M9k0ySsa}e{*~=|)e<2R!?G9!c
z!$3=}DMuacm`}n<G|47`O<sy_+zOeRK5ivo17R=<n5+Z0FJHZW_Lo0wbl6N}BjBcX
z2WcWB!*%<1m?CH>q|xhLSea3T&?P2I%ot;sGiN6idlzFLLfX_j_7+V+#w(QrZM8jf
ze}VaWQ1;AhZBo7ArkeY0ZpI~@@;pLzG0<K6HV5-=(Q3}6TLCaDX6r&z++t!S*)Iuu
z_C3P%{p9iTfZ~uy*}QK{6HqDWm|%|DMf|qTv&}uu#y7KE-x}$>xnJExppC7p41nrp
z*mRGDSzHYcSX&i?1Jv#6*pLIJPQA?!bq<KZt+K(p2YL8Fg=zuiutnaH`d&hg`gwg#
zD@Sid17rVD((mIKXOmV<>~&%L+9&|xjsQl#t~}TVUy`ocX=pkx7ZfFky88Vmoz7=l
z_a&Y7%%`)~7EEkBl*LM3mK)bG`nDl~gXOJ27FQE*yZTyv+qb89Po~*!@p%Qn1nZX6
zorH{5;%|+>q~i9jKnX8@T#EqNf+tyfqEWQnr?5jPUOd$iRyuwy5J{MX>aU<hx;`b}
z$YA+15qk``))gh~&{rpr=8hYAX2vi0peB+e^2O~e?4iKV1De9(7<CqTQ8G>guF}M~
zxr6S(uhbt#&2ZQXr^G*q$pAIUY0`^!R=8IE8H}|U91fjWDZ*_bGM6EmgPVOQXj|K|
OPUn9!+#d!U9{>R02OW_B
literal 0
HcmV?d00001
--
1.7.0.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM pca-a-l1: should depend on ARCH_OMAP3
2012-01-20 21:25 [PATCH] ARM pca-a-l1: call omap3_core_init from board code Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: delete platform.S Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: fix board_init Juergen Kilb
@ 2012-01-20 21:25 ` Juergen Kilb
2 siblings, 0 replies; 8+ messages in thread
From: Juergen Kilb @ 2012-01-20 21:25 UTC (permalink / raw)
To: barebox; +Cc: Juergen Kilb
Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
---
arch/arm/mach-omap/Kconfig | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap/Kconfig b/arch/arm/mach-omap/Kconfig
index ecdabb0..50f53dd 100644
--- a/arch/arm/mach-omap/Kconfig
+++ b/arch/arm/mach-omap/Kconfig
@@ -152,6 +152,7 @@ config MACH_PCAAL1
bool "Phytec phyCARD-A-L1"
select OMAP_CLOCK_ALL
select HAS_OMAP_NAND
+ depends on ARCH_OMAP3
help
Say Y here if you are using a phyCARD-A-L1 PCA-A-L1
--
1.7.0.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM pca-a-l1: fix board_init
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: fix board_init Juergen Kilb
@ 2012-01-23 8:35 ` Sascha Hauer
2012-01-23 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
1 sibling, 0 replies; 8+ messages in thread
From: Sascha Hauer @ 2012-01-23 8:35 UTC (permalink / raw)
To: Juergen Kilb; +Cc: barebox
On Fri, Jan 20, 2012 at 10:25:02PM +0100, Juergen Kilb wrote:
> board_init was moved to pure_initcall. broken since:
>
> commit 0adce7ec683b4b325f51de4ac8892b32925d7ef8
> Author: Sascha Hauer <s.hauer@pengutronix.de>
> Date: Sun Jan 15 21:11:17 2012 +0100
>
> ARM omap3: move board_init to pure_initcall
>
> board_init initializes the mux and sdram. For both there is no
> need to configure this so early. Move the code to a pure_initcall
> and remove the surrounding unneeded code.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> Tested-by: Sanjeev Premi <premi@ti.com>
>
> Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
> ---
> arch/arm/boards/phycard-a-l1/pca-a-l1.c | 8 +++++---
> arch/arm/mach-omap/Kconfig | 1 -
> barebox_default_env.gz | Bin 0 -> 2656 bytes
> 3 files changed, 5 insertions(+), 4 deletions(-)
> create mode 100644 barebox_default_env.gz
>
> diff --git a/arch/arm/boards/phycard-a-l1/pca-a-l1.c b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
> index 11e8638..0518894 100644
> --- a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
> +++ b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
> @@ -74,8 +74,6 @@
> #include <mach/syslib.h>
> #include <mach/xload.h>
>
> -#include "pca-a-l1.h"
> -
> #define SMC911X_BASE 0x2c000000
>
> /*
> @@ -224,15 +222,19 @@ static void pcaal1_mux_config(void)
> *
> * @return void
> */
> -void omap3_board_init(void)
> +static int pcaal1_board_init(void)
> {
> int in_sdram = running_in_sdram();
>
> pcaal1_mux_config();
> +
> /* Dont reconfigure SDRAM while running in SDRAM! */
> if (!in_sdram)
> pcaal1_sdrc_init();
> +
> + return 0;
> }
> +pure_initcall(pcaal1_board_init);
>
> /*
> * Run-time initialization(s)
> diff --git a/arch/arm/mach-omap/Kconfig b/arch/arm/mach-omap/Kconfig
> index 970c899..ecdabb0 100644
> --- a/arch/arm/mach-omap/Kconfig
> +++ b/arch/arm/mach-omap/Kconfig
> @@ -150,7 +150,6 @@ config MACH_PCM049
>
> config MACH_PCAAL1
> bool "Phytec phyCARD-A-L1"
> - select MACH_HAS_LOWLEVEL_INIT
> select OMAP_CLOCK_ALL
> select HAS_OMAP_NAND
> help
> diff --git a/barebox_default_env.gz b/barebox_default_env.gz
Ups. Applied the series with this removed.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM pca-a-l1: fix board_init
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: fix board_init Juergen Kilb
2012-01-23 8:35 ` Sascha Hauer
@ 2012-01-23 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-23 21:17 ` Jürgen Kilb
1 sibling, 1 reply; 8+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2012-01-23 10:40 UTC (permalink / raw)
To: Juergen Kilb; +Cc: barebox
On 22:25 Fri 20 Jan , Juergen Kilb wrote:
> board_init was moved to pure_initcall. broken since:
>
> commit 0adce7ec683b4b325f51de4ac8892b32925d7ef8
> Author: Sascha Hauer <s.hauer@pengutronix.de>
> Date: Sun Jan 15 21:11:17 2012 +0100
>
> ARM omap3: move board_init to pure_initcall
>
> board_init initializes the mux and sdram. For both there is no
> need to configure this so early. Move the code to a pure_initcall
> and remove the surrounding unneeded code.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> Tested-by: Sanjeev Premi <premi@ti.com>
>
> Signed-off-by: Juergen Kilb <J.Kilb@phytec.de>
can't you fix all in one patch?
> ---
> arch/arm/boards/phycard-a-l1/pca-a-l1.c | 8 +++++---
> arch/arm/mach-omap/Kconfig | 1 -
> barebox_default_env.gz | Bin 0 -> 2656 bytes
> 3 files changed, 5 insertions(+), 4 deletions(-)
> create mode 100644 barebox_default_env.gz
???
Best Regards,
J.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM pca-a-l1: fix board_init
2012-01-23 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2012-01-23 21:17 ` Jürgen Kilb
0 siblings, 0 replies; 8+ messages in thread
From: Jürgen Kilb @ 2012-01-23 21:17 UTC (permalink / raw)
To: Jean-Christophe PLAGNIOL-VILLARD; +Cc: barebox
Hi Jean-Christophe
On 23.01.2012 11:40, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 22:25 Fri 20 Jan , Juergen Kilb wrote:
>> board_init was moved to pure_initcall. broken since:
>>
>> commit 0adce7ec683b4b325f51de4ac8892b32925d7ef8
>> Author: Sascha Hauer<s.hauer@pengutronix.de>
>> Date: Sun Jan 15 21:11:17 2012 +0100
>>
>> ARM omap3: move board_init to pure_initcall
>>
>> board_init initializes the mux and sdram. For both there is no
>> need to configure this so early. Move the code to a pure_initcall
>> and remove the surrounding unneeded code.
>>
>> Signed-off-by: Sascha Hauer<s.hauer@pengutronix.de>
>> Tested-by: Sanjeev Premi<premi@ti.com>
>>
>> Signed-off-by: Juergen Kilb<J.Kilb@phytec.de>
> can't you fix all in one patch?
I'v used separate patches because the break was created by two different
commits and the other two doesn't depend on each other.
>> ---
>> arch/arm/boards/phycard-a-l1/pca-a-l1.c | 8 +++++---
>> arch/arm/mach-omap/Kconfig | 1 -
>> barebox_default_env.gz | Bin 0 -> 2656 bytes
>> 3 files changed, 5 insertions(+), 4 deletions(-)
>> create mode 100644 barebox_default_env.gz
> ???
ups.. have not seen this. Was a result of 'git add .'
> Best Regards,
> J.
greetings,
Jürgen
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM pca-a-l1: fix board_init
@ 2012-01-17 10:23 Sascha Hauer
0 siblings, 0 replies; 8+ messages in thread
From: Sascha Hauer @ 2012-01-17 10:23 UTC (permalink / raw)
To: barebox; +Cc: Juergen Kilb
board_init is now called omap3_board_init. broken since:
commit 494a12f703c33f80fe96e2a728545c490347eceb
Author: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Sat Jan 14 15:51:05 2012 +0100
omap3: move platform lowlevel init to mach-omap
this will allow to switch omap3 to standard organisation
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
arch/arm/boards/phycard-a-l1/Makefile | 1 -
arch/arm/boards/phycard-a-l1/lowlevel.c | 39 -------------------------------
arch/arm/boards/phycard-a-l1/pca-a-l1.c | 2 +-
3 files changed, 1 insertions(+), 41 deletions(-)
delete mode 100644 arch/arm/boards/phycard-a-l1/lowlevel.c
diff --git a/arch/arm/boards/phycard-a-l1/Makefile b/arch/arm/boards/phycard-a-l1/Makefile
index cb0106b..db48b6d 100644
--- a/arch/arm/boards/phycard-a-l1/Makefile
+++ b/arch/arm/boards/phycard-a-l1/Makefile
@@ -18,5 +18,4 @@
# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
# MA 02111-1307 USA
-obj-$(CONFIG_MACH_DO_LOWLEVEL_INIT) += lowlevel.o
obj-y += pca-a-l1.o
diff --git a/arch/arm/boards/phycard-a-l1/lowlevel.c b/arch/arm/boards/phycard-a-l1/lowlevel.c
deleted file mode 100644
index bffbb08..0000000
--- a/arch/arm/boards/phycard-a-l1/lowlevel.c
+++ /dev/null
@@ -1,39 +0,0 @@
-/*
- * (C) Copyright 2011
- * Phytec Messtechnik GmbH <www.phytec.de>
- * Juergen Kilb <j.kilb@phytec.de>
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-#include <common.h>
-#include <asm/barebox-arm.h>
-#include <mach/sdrc.h>
-#include <mach/omap3-silicon.h>
-
-void __naked board_init_lowlevel(void)
-{
- uint32_t r;
-
- /* setup a stack */
- r = OMAP_SRAM_STACK;
- __asm__ __volatile__("mov sp, %0" : : "r"(r));
-
- board_init();
-
- board_init_lowlevel_return();
-}
diff --git a/arch/arm/boards/phycard-a-l1/pca-a-l1.c b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
index 7b127f3..11e8638 100644
--- a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
+++ b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
@@ -224,7 +224,7 @@ static void pcaal1_mux_config(void)
*
* @return void
*/
-void board_init(void)
+void omap3_board_init(void)
{
int in_sdram = running_in_sdram();
--
1.7.8.3
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-01-23 21:18 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-20 21:25 [PATCH] ARM pca-a-l1: call omap3_core_init from board code Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: delete platform.S Juergen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: fix board_init Juergen Kilb
2012-01-23 8:35 ` Sascha Hauer
2012-01-23 10:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-23 21:17 ` Jürgen Kilb
2012-01-20 21:25 ` [PATCH] ARM pca-a-l1: should depend on ARCH_OMAP3 Juergen Kilb
-- strict thread matches above, loose matches on Subject: below --
2012-01-17 10:23 [PATCH] ARM pca-a-l1: fix board_init Sascha Hauer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox