<br><br><div class="gmail_quote">2011/1/3 Sascha Hauer <span dir="ltr">&lt;<a href="mailto:s.hauer@pengutronix.de">s.hauer@pengutronix.de</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Mon, Jan 03, 2011 at 01:45:18PM +0100, Franck JULLIEN wrote:<br>
&gt; 2011/1/3 Sascha Hauer &lt;<a href="mailto:s.hauer@pengutronix.de">s.hauer@pengutronix.de</a>&gt;<br>
&gt;<br>
&gt;<br>
</div><div class="im">&gt; I know CONFIG_BOARDINFO is board specific. However, I think it is more<br>
&gt; conveniant<br>
&gt; to have the possibility to define a banner text outside the Kconfig. For<br>
&gt; example, I use<br>
&gt; this in my config.h:<br>
&gt;<br>
&gt; #define CONFIG_BOARD_BANNER_TEXT &quot;\e[1;32m\<br>
&gt; ***********************************************************************\n\<br>
&gt; *           Communication board bootloader (ODSFT0152)        *\n\<br>
&gt; ************************************************************************\<br>
&gt; \e[0m\n\n&quot;<br>
&gt;<br>
&gt; So may be we could define this kind of banner in the Kconfig.....<br>
<br>
</div>Ok, I can see your problem. I think we should rather define a board<br>
specific function to display a banner than a string, so something like:<br>
<br>
config BOARD_BANNER<br>
        bool<br>
<br>
#ifdef CONFIG_BOARD_BANNER<br>
        display_board_banner();<br>
#else<br>
<div class="im">        printf(RELOC(&quot;Board: &quot; CONFIG_BOARDINFO &quot;\n&quot;));<br>
</div>#endif<br>
<br>
This way we can also show runtime dependent information in the banner<br>
(like baseboard or whatever).<br></blockquote><div><br></div><div><span class="Apple-style-span" style="border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; "><div>Sounds great, I&#39;ll do this. Is there anyone responsible for the list in first place ?</div>
<div>Because if I do this in one way but someone would prefer do it another way and</div><div>finally the git boss wants it a third way.......</div><div><br></div><div>I&#39;ll try to post the new patch in response to this thread</div>
<div>using the right git command...By the way, I there anything more user friendly</div><div>than the git command line to send email in the right format ?</div><div><br></div><div>I told you this is a practice/test patch so I have a lot of questions/comments !</div>
</span></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
&gt;<br>
&gt; Let me know if you don&#39;t like this. This patch was kind of a test for me and<br>
&gt; we could<br>
&gt; forget about it.<br>
&gt;<br>
&gt; Although this patch was very small, we had a lot of discussion around it and<br>
&gt; I don&#39;t want to imagine what it is going to be when I submit the nios2 port :)<br>
<br>
</div>Don&#39;t worry, the amount of comments you receive usually is not<br>
proportional to the patch size you send ;)<br>
<div><div></div><div class="h5"><br>
Sascha<br></div></div></blockquote><div><br></div><div><span class="Apple-style-span" style="border-collapse: collapse; font-family: arial, sans-serif; font-size: 13px; ">Ok, because I was worry :)</span></div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="h5">
<br>
--<br>
Pengutronix e.K.                           |                             |<br>
Industrial Linux Solutions                 | <a href="http://www.pengutronix.de/" target="_blank">http://www.pengutronix.de/</a>  |<br>
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |<br>
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |<br></div></div></blockquote><div><br></div><div>Best regards,</div><div>FJ. </div></div><br>