<div dir="ltr">Hi Sebastian,<div><br></div><div style>Thanks for your reviewing the patch.</div><div style>I'll modify the code and add the copyright related header.</div><div style>Because s3c6410 is based on armv6 architecture, I create the linkcmds. I'll </div>
<div style>move the size stack setting to linkcmds.armv4.Thanks again.</div><div style><br></div><div style>Regards,</div><div style>Peng.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/17 Sebastian Huber <span dir="ltr"><<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Peng Fan,<br>
<br>
thanks for your new BSP.  I have some comments.<div><div class="h5"><br>
<br>
On 04/17/2013 07:27 AM, Peng Fan wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
diff --git a/c/src/lib/libbsp/arm/shared/<u></u>startup/linkcmds.armv6 b/c/src/lib/libbsp/arm/shared/<u></u>startup/linkcmds.armv6<br>
new file mode 100644<br>
index 0000000..16fa3ef<br>
--- /dev/null<br>
+++ b/c/src/lib/libbsp/arm/shared/<u></u>startup/linkcmds.armv6<br>
@@ -0,0 +1,26 @@<br>
+/**<br>
+ * @file<br>
+ *<br>
+ * @ingroup bsp_linker<br>
+ *<br>
+ * @brief Linker command file for ARMv6 architecture.<br>
+ */<br>
+<br>
+/*<br>
+ * Copyright (c) 2010 embedded brains GmbH.  All rights reserved.<br>
+ *<br>
+ *  embedded brains GmbH<br>
+ *  Obere Lagerstr. 30<br>
+ *  82178 Puchheim<br>
+ *  Germany<br>
+ *<<a href="mailto:rtems@embedded-brains.de" target="_blank">rtems@embedded-brains.de</a>><br>
+ *<br>
+ * The license and distribution terms for this file may be<br>
+ * found in the file LICENSE in this distribution or at<br></div></div>
+ *<a href="http://www.rtems.com/license/LICENSE" target="_blank">http://www.rtems.com/license/<u></u>LICENSE</a>.<div class="im"><br>
+ */<br>
+<br>
+bsp_stack_svc_size = DEFINED (bsp_stack_svc_size) ? bsp_stack_svc_size : 4096;<br>
+bsp_stack_svc_size = ALIGN (bsp_stack_svc_size, bsp_stack_align);<br>
+<br>
+INCLUDE linkcmds.base<br>
</div></blockquote>
<br>
I think this file is superfluous and you should use the linkcmds.armv4 in your BSP.  If the SVC stack size is too small for your BSP, then change it in the BSP specific linker command file.<br>
<br>
Please make sure that all files have a copyright and license header.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-<u></u>brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<u></u>_________________<br>
rtems-devel mailing list<br>
<a href="mailto:rtems-devel@rtems.org" target="_blank">rtems-devel@rtems.org</a><br>
<a href="http://www.rtems.org/mailman/listinfo/rtems-devel" target="_blank">http://www.rtems.org/mailman/<u></u>listinfo/rtems-devel</a><br>
</div></div></blockquote></div><br></div>