2010/10/10 Sylvain 'ythier' Hitier <span dir="ltr"><<a href="mailto:sylvain.hitier@gmail.com">sylvain.hitier@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hello,<br>
<br>
r5929 seems quite mechanical, especially when removing the inclusions of the<br>
lib/debug.c file.<br>
<br>
I noticed that:<br>
    Inclusion is kept for these mainboards:<br>
        p2b-d<br>
        p2b-ls<br>
    whereas removed from all these ones:<br>
        p2b<br>
        p2b-ds<br>
        p2b-f<br>
        p3b-f<br>
and<br>
    Inclusion is kept for this mainboard:<br>
        ga-6bxe<br>
    whereas removed from this one:<br>
        ga-6bxc<br>
<br>
I feel the mainboards look like quite similar (but I diff'ed only SVN'ed files<br>
in these dirs).<br>
<br>
However I couldn't find any reason why #include is sometimes kept...<br>
<br>
Hence I suggest the following _untested_ patch.<br>
<br>
Regards,<br>
Sylvain "ythier" Hitier<br>
<br>
========<br>
<br>
Remove more inclusions of lib/debug.c after r5929.<br></blockquote><div><br>Does your patch survive abuild for each board ?<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
Signed-off-by: Sylvain Hitier <<a href="mailto:sylvain.hitier@gmail.com">sylvain.hitier@gmail.com</a>><br></blockquote><div><br>If it does:<br>Acked-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com">vidwer@gmail.com</a>><br>
 <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Index: mainboard/gigabyte/ga-6bxe/romstage.c<br>
===================================================================<br>
--- mainboard/gigabyte/ga-6bxe/romstage.c       (revision 5929)<br>
+++ mainboard/gigabyte/ga-6bxe/romstage.c       (working copy)<br>
@@ -28,7 +28,6 @@<br>
 #include <console/console.h><br>
 #include "southbridge/intel/i82371eb/i82371eb.h"<br>
 #include "northbridge/intel/i440bx/raminit.h"<br>
-#include "lib/debug.c"<br>
 #include "pc80/udelay_io.c"<br>
 #include "lib/delay.c"<br>
 #include "cpu/x86/bist.h"<br>
Index: mainboard/asus/p2b-ls/romstage.c<br>
===================================================================<br>
--- mainboard/asus/p2b-ls/romstage.c    (revision 5929)<br>
+++ mainboard/asus/p2b-ls/romstage.c    (working copy)<br>
@@ -28,7 +28,6 @@<br>
 #include <console/console.h><br>
 #include "southbridge/intel/i82371eb/i82371eb.h"<br>
 #include "northbridge/intel/i440bx/raminit.h"<br>
-#include "lib/debug.c"<br>
 #include "pc80/udelay_io.c"<br>
 #include "lib/delay.c"<br>
 #include "cpu/x86/bist.h"<br>
Index: mainboard/asus/p2b-d/romstage.c<br>
===================================================================<br>
--- mainboard/asus/p2b-d/romstage.c     (revision 5929)<br>
+++ mainboard/asus/p2b-d/romstage.c     (working copy)<br>
@@ -29,7 +29,6 @@<br>
 #include <console/console.h><br>
 #include "southbridge/intel/i82371eb/i82371eb.h"<br>
 #include "northbridge/intel/i440bx/raminit.h"<br>
-#include "lib/debug.c"<br>
 #include "pc80/udelay_io.c"<br>
 #include "lib/delay.c"<br>
 #include "cpu/x86/bist.h"<br>
<font color="#888888"><br>
<br>
<br>
--<br>
Free hugs for Free Software Fellows!<br>
Living in a Debian package => <a href="http://en.wikipedia.org/wiki/Apt,_Vaucluse" target="_blank">http://en.wikipedia.org/wiki/Apt,_Vaucluse</a><br>
</font></blockquote></div><br>