2010/10/18 Warren Turkal <span dir="ltr"><<a href="mailto:wt@penguintechs.org" target="_blank">wt@penguintechs.org</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;">

<div>On Sunday, October 17, 2010 09:37:31 am Idwer Vollering wrote:<br>
> Add support for FreeBSD.<br>
><br>
> Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>><br></div></blockquote><div><br>Updated patch attached.<br>Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">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;"><div>
<br>
</div>In inteltool.h:<br>
* Can you please briefly explain the need for the macros for {IN,OUT}{B,W,L}<br>
when I don't seen them called from anywhere in the code?<br></blockquote><div><br>Dropped, thanks.<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;">


* If you mean to use them, why are they implemented as macros instead of<br>
functions. I think it'd be easier to read if they were function, and gcc could<br>
possibly even inline such a simple function.<br></blockquote><div><br>See above.<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>
In inteltool.c:<br>
* Why not just include unistd.h on all platforms?<br>
* I think the #ifdef __FREEBSD__ just makes the code difficult to read. I think<br>
the platform specific code need to be factored out somehow.<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
* The io_fd variable doesn't appear to be used anywhere after opening the<br>
/dev/io file. Doesn't it need to be closed somewhere? If not, why even bother<br>
creating a variable to hold the value of the open instead of just testing it<br>
directly?<br></blockquote><div><br>Is this a possible memory leak ? If so, it needs to be fixed in flashrom too.<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>
The Makefile change looks ok.<br>
<br>
Thanks,<br>
<font color="#888888">wt<br>
</font></blockquote></div><br>