<p style="white-space: pre-wrap; word-wrap: break-word;">>>> I'm getting some errors. some of them looks "odd" to me:<br>>><br>>> Looks like the errors to expect when an #include is missing. How<br>>> did<br>>> you detect which files don't require the #include?<br>>><br>>> Some of the errors look like they are actually about missing<br>>> #includes<br>>> in other places that were hidden because of #includes you remove<br>>> here.<br>> <br>> it was my mistake ...<br>> if I remove an "include io.h" for exmple from "src/southbridge/amd/cs5536/cs5536.c",<br>> the compiler will complain about unknown type here : "cpu/amd/vr.h"</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's not your mistake then. It means `cpu/amd/vr.h` is missing<br>an #include (it just wasn't visible before).</p><p style="white-space: pre-wrap; word-wrap: break-word;">The same can happen if you remove #includes and just test if it<br>compiles (some other included file can hide the problem). That's<br>why I ask again: How do you decide which #include statements are<br>unneeded? Did you use some static analysis tool? did you read<br>all the code?</p><p><a href="https://review.coreboot.org/29307">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/29307">change 29307</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/29307"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia18c77876121594a272a07d56acfaa863d0ccb25 </div>
<div style="display:none"> Gerrit-Change-Number: 29307 </div>
<div style="display:none"> Gerrit-PatchSet: 13 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 28 Oct 2018 13:31:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>