Re: [U-Boot] [PATCH 2/2] Create the S5PC100 SoC header under the include/asm-arm/arch-s5pc100

2009-06-03 Thread Kim, Heung Jun
Dear Dirk, Ah, I was busy yesterday. Well, I can write the message now. > I wonder if there is any patch missing here? Is it correct that this series > stops after 2 patches? Shouldn't there be at least something added to board > directory? And I miss a config file to be able to compile this stuf

[U-Boot] [PATCH 2/2] Create the S5PC100 SoC header under the include/asm-arm/arch-s5pc100

2009-06-02 Thread Kim, Heung Jun
Signed-off-by: HeungJun, Kim --- This patch is the new processor - S5PC100's SoC code & headers. This patch consists of 2 files. The First file is to create the new SoC code related with S5PC100 application processor. It's located in ther cpu/arm_cortesa8/s5pc100. The Second file is to create

[U-Boot] [PATCH 1/2] Create the S5PC100 SoC code under the cpu/arm_cortesa8/

2009-06-02 Thread Kim, Heung Jun
Signed-off-by: HeungJun, Kim --- This patch is the new processor - S5PC100's SoC code & headers. This patch consists of 2 files. The First file is to create the new SoC code related with S5PC100 application processor. It's located in ther cpu/arm_cortesa8/s5pc100. The Second file is to create

[U-Boot] [PATCH] move L2 cache enable/disable function to cache.c in the omap3 SoC directory

2009-06-02 Thread Kim, Heung Jun
CC: Dirk Behme Signed-off-by: HeungJun, Kim --- The omap3 L2 cache enable/disable function in the cpu/arm_cortexa8/cpu.c moved to cpu/arm_cortexa8/omap3/cache.c. Because, it must be CortexA8 ARCH generic code below the cpu/arm_cortexa8. This patches fixes the First issue in the following http

[U-Boot] I wonder "---" in the patch file generated by the git-format-patch

2009-06-02 Thread Kim, Heung Jun
Dear Wolfgang Denk, I have a question about git-format-patch usage. I generated the patch using git-format-patch. The my usage is the following : 1. pull latest git repo. 2. change the code 3. # git commit -a -s 4. insert commit & any other things. 5. # git-format-patch -1 So, I get patch. But,

Re: [U-Boot] [PATCH] The omap3 L2 cache enable/disable function to omap3 dependent code

2009-06-01 Thread Kim, Heung Jun
/pipermail/u-boot/2009-June/053533.html It's my fault that u don't understand my words :) BTW, Very thanks to review. Best Regards, riverful 2009/6/1 Kim, Heung Jun : > CC: Dirk Behme > Signed-off-by: HeungJun, Kim > > --- > > The L2 cache enable/disable function in the

[U-Boot] [PATCH] The omap3 L2 cache enable/disable function to omap3 dependent code

2009-06-01 Thread Kim, Heung Jun
CC: Dirk Behme Signed-off-by: HeungJun, Kim --- The L2 cache enable/disable function in the cpu/arm_cortexa8/cpu.c moved to cpu/arm_cortexa8/omap3/cache.c. This patches fixes the First issue in the following http://lists.denx.de/pipermail/u-boot/2009-May/053433.html The Second issue is fixed

Re: [U-Boot] [PATCH] The omap3 L2 cache enable/disable function to omap3 dependent code

2009-06-01 Thread Kim, Heung Jun
mments. Is it right or something like? BTW, Thanks one more :) Best Regards, riverful 2009/6/2 Kim Phillips : > On Mon, 1 Jun 2009 19:14:49 +0900 > "Kim, Heung Jun" wrote: > >> CC: Dirk Behme >> Signed-off-by: HeungJun, Kim >> >> --- >> >>

[U-Boot] [PATCH] The omap3 L2 cache enable/disable function to omap3 dependent code

2009-06-01 Thread Kim, Heung Jun
CC: Dirk Behme Signed-off-by: HeungJun, Kim --- The L2 cache enable/disable function in the cpu/arm_cortexa8/cpu.c moved to cpu/arm_cortexa8/omap3/cache.c. This patches fixes the First issue in the following http://lists.denx.de/pipermail/u-boot/2009-May/053433.html The Second issue is fixed

Re: [U-Boot] [PATCH] ARM Cortex A8: Move OMAP3 specific reset handler to OMAP3 code

2009-06-01 Thread Kim, Heung Jun
30 May     , Dirk Behme wrote: >>> Reset is SoC specific and not ARM Cortex A8 generic. Move it from generic >>> code to OMAP3 SoC specific file. >>> >>> CC: "Kim, Heung Jun" >>> Signed-off-by: Dirk Behme >>> >>> --- >>

[U-Boot] [PATCH] omap3 L2-cache enable/disable function to omap3 dependent code

2009-05-31 Thread Kim, Heung Jun
Dear Jean, I miss mail title sorry. :) = Dear Jean, I made new patch about L2 cane enable/disable function. The related thread is here. http://www.mail-archive.com/u-boot@lists.denx.de/msg14744.html I fixed title issue & apart header. Thanks to read :) Best Regards, riverful

[U-Boot] [PATCH]

2009-05-31 Thread Kim, Heung Jun
Dear Jean, I made new patch about L2 cane enable/disable function. The related thread is here. http://www.mail-archive.com/u-boot@lists.denx.de/msg14744.html I fixed title issue & apart header. Thanks to read :) Best Regards, riverful = omap3 L2 cache enable/disable function moved from cpu

[U-Boot] [PATCH] Move L2-cache enable/disable function to soc dependent code

2009-05-29 Thread Kim, Heung Jun
The L2 cache enable/disable function in the cpu/arm_cortexa8/cpu.c moved to cpu/arm_cortexa8/omap3/cache.c. Signed-off-by: HeungJun, Kim --- cpu/arm_cortexa8/cpu.c | 65 + cpu/arm_cortexa8/omap3/Makefile|2 +- cpu/arm_cortexa8/omap3/board.c

Re: [U-Boot] [PATCH] The cache flush using coprocessor must be in lib_arm/cache-cp15.c

2009-05-28 Thread Kim, Heung Jun
Thanks to point. Dirk. And call me "riverful" next time, plz :) > Kim mentioned that he shuffles around the code due to compile errors while > adding his new code. So we have to see if we can stay with the current > implementation or have to change something. Yes. I shuffles around a little now.

[U-Boot] [PATCH] The cache flush using coprocessor must be in lib_arm/cache-cp15.c

2009-05-28 Thread Kim, Heung Jun
Hi? I'm about to suggest one more thing related to "change cpu.c under cpu/arm_cortexa8 dir to common code.". asm ("mcr p15, 0, %0, c7, c5, 0": :"r" (0)); It's the common feature all over the arm core, not only arm cortex A8. The common cache function is defined in the lib_arm/cache-cp15.c as you

Re: [U-Boot] [PATCH] change cpu.c under cpu/arm_cortexa8 dir to common code.

2009-05-27 Thread Kim, Heung Jun
First of all, thanks for reading this to Dirk, Jean-Christophe. >> Anyway, yes, from functionality point of view I agree this patch. OMAP3 was >> the first Cortex A8 device. So it was more or less expected that while >> adding additional ones we have to re-arrange some stuff. Being the first >>

Re: [U-Boot] [PATCH] change cpu.c under cpu/arm_cortexa8 dir to common code.

2009-05-27 Thread Kim, Heung Jun
Sorry about wrong Signed-off-by. Signed-off-by: riverful.kim Regards, Kim, HeungJun 2009/5/27 Kim, Heung Jun : > The cpu.c under cpu/arm_cortexa8 has a dependency of omap3. > The part of cache in cpu.c is moved in the omap3/board.c, > because the functions about controlling cache se

[U-Boot] [PATCH] change cpu.c under cpu/arm_cortexa8 dir to common code.

2009-05-27 Thread Kim, Heung Jun
The cpu.c under cpu/arm_cortexa8 has a dependency of omap3. The part of cache in cpu.c is moved in the omap3/board.c, because the functions about controlling cache seems to be different with a lot of processors. Signed-off-by: root --- cpu/arm_cortexa8/cpu.c | 55 +-