<div dir="ltr">Thanks Joel and Gedare for the comments.<br><div class="gmail_extra"><br></div><div class="gmail_extra">For "<span style="font-size:14px">Did you "clone and own" anything from another BSP? </span>"</div><div class="gmail_extra">If you are asking me to add the copyright back, I'll check to avoid discarding the copyrights.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Peng.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">2016-04-19 23:47 GMT+08:00 Joel Sherrill <span dir="ltr"><<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 19, 2016 at 10:41 AM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>On Tue, Apr 19, 2016 at 9:59 AM, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
> A few quick things.<br>
><br>
> Add a bspimpl.h which is private to the bsp. Do not put anything in bsp.h<br>
> which<br>
> does not need to be public to the application. We have been exceedingly<br>
> uncaring about this in the past and the software tax to correct it is going<br>
> to<br>
> be painful. In general, the bsp.h probably doesn't need much except to<br>
> provide the BSP defaults for confdefs.h if it needs to change them.<br>
> And no.. there aren't any great examples yet.<br>
><br>
<br>
</span>Another point is to check the current .cfg files for examples of how<br>
to add the per-function and per-sections LTO<br>
</blockquote></div><br></div></div></div><div class="gmail_extra">+1 that would be a good think for the script to check. </div><div class="gmail_extra"><br></div><div class="gmail_extra">As well as for the console table structure Sebastian recommended not using.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also the new code should be warning free. It should also not result in</div><div class="gmail_extra">warnings in formerly warning free shared code. </div><div class="gmail_extra"><br></div><div class="gmail_extra">The script isn't great but if we can use grep to help us review for </div><div class="gmail_extra">repeat offenders, then we can focus on harder issues.<span class=""><font color="#888888"><br><br>--joel</font></span></div></div>
</blockquote></div><br></div></div>