<div dir="ltr">Agreed a BSP or user space code should not use the same name as <div>anything that is included in POSIX. :)</div><div><br></div><div>How did my build sweep miss this?</div><div><br></div><div>--joel</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 20, 2017 at 1:01 PM, Kirspel, Kevin <span dir="ltr"><<a href="mailto:Kevin-Kirspel@idexx.com" target="_blank">Kevin-Kirspel@idexx.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div class="m_8188735712682587194WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Actually the best thing to do is change the structure member name and fix up any access to that member.<u></u><u></u></span></p><span class="">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p>
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Kevin Kirspel<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Electrical Engineer - Sr. Staff<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Idexx Roswell<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">235 Hembree Park Drive<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Roswell GA 30076<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Tel: <a href="tel:(770)%20510-4444" value="+17705104444" target="_blank">(770)-510-4444 ext. 81642</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Direct: <a href="tel:(770)%20688-1642" value="+17706881642" target="_blank">(770)-688-1642</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Fax: <a href="tel:(770)%20510-4445" value="+17705104445" target="_blank">(770)-510-4445</a><u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p>
</span><div>
<div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> devel [mailto:<a href="mailto:devel-bounces@rtems.org" target="_blank">devel-bounces@rtems.<wbr>org</a>]
<b>On Behalf Of </b>Kirspel, Kevin<br>
<b>Sent:</b> Monday, March 20, 2017 1:50 PM<br>
<b>To:</b> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<b>Subject:</b> RE: [PATCH 1/5] Adding modified FREEBSD headers to sync RTEMS termios with FREEBSD<u></u><u></u></span></p>
</div>
</div><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">I tracked the arm failure down to a header file (s3c2410.h) that contains a structure that has a member with the same name as a termios #define. Re-ordering the include file
 declarations in each affected C file will fix the problem (I’m not sure that’s the best solution but it will do for now).  I am going to build all the tool chains so I can check the rest of the BSPs.  Sebastian found another issue with a SH BSP.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Kevin Kirspel<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Electrical Engineer - Sr. Staff<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Idexx Roswell<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">235 Hembree Park Drive<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Roswell GA 30076<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Tel: <a href="tel:(770)%20510-4444" value="+17705104444" target="_blank">(770)-510-4444 ext. 81642</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Direct: <a href="tel:(770)%20688-1642" value="+17706881642" target="_blank">(770)-688-1642</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Fax: <a href="tel:(770)%20510-4445" value="+17705104445" target="_blank">(770)-510-4445</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p>
<p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Joel Sherrill [<a href="mailto:joel@rtems.org" target="_blank">mailto:joel@rtems.org</a>]
<br>
<b>Sent:</b> Monday, March 20, 2017 1:33 PM<br>
<b>To:</b> Kirspel, Kevin <<a href="mailto:Kevin-Kirspel@idexx.com" target="_blank">Kevin-Kirspel@idexx.com</a>><br>
<b>Cc:</b> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<b>Subject:</b> Re: [PATCH 1/5] Adding modified FREEBSD headers to sync RTEMS termios with FREEBSD<u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">I just built all BSPs and didn't see any failures. <u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Sebastian.. what was going on with the build you saw fail?<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">--joel<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Mon, Mar 20, 2017 at 9:19 AM, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Tue, Mar 14, 2017 at 8:54 AM, Kirspel, Kevin <<a href="mailto:Kevin-Kirspel@idexx.com" target="_blank">Kevin-Kirspel@idexx.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<p class="MsoNormal" style="margin-bottom:12.0pt">I tested it against xilinx_zynq_a9_qemu.  I'm not sure it exists, but if there is a way to compile the patch against every (or most) BSP then we could ensure no BSP is broken.<u></u><u></u></p>
</blockquote>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">There are scripts in rtems-testing/rtems to ease testing of a single configuation<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">on all BSPs. Or a variety of configurations on a single BSP. I use them to <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">generate the warnings reports that I periodically announce.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I am seeing if all BSPs build now. Give me a few hours.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="color:#888888"><u></u> <u></u></span></p>
</div>
<div>
<p class="MsoNormal"><span style="color:#888888">--joel<u></u><u></u></span></p>
</div>
<div>
<div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<p class="MsoNormal"><span class="m_8188735712682587194m73650082701939426hoenzb"><span style="color:#888888">Kevin Kirspel</span></span><span style="color:#888888"><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Electrical Engineer - Sr. Staff</span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Idexx Roswell</span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">235 Hembree Park Drive</span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Roswell GA 30076</span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Tel: <a href="tel:%28770%29-510-4444%20ext.%2081642" target="_blank">
(770)-510-4444 ext. 81642</a></span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Direct: <a href="tel:%28770%29-688-1642" target="_blank">
(770)-688-1642</a></span><br>
<span class="m_8188735712682587194m73650082701939426hoenzb">Fax: <a href="tel:%28770%29-510-4445" target="_blank">
(770)-510-4445</a></span></span><u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"><br>
-----Original Message-----<br>
From: Sebastian Huber [mailto:<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@<wbr>embedded-brains.de</a>]<br>
Sent: Tuesday, March 14, 2017 9:42 AM<br>
To: Kirspel, Kevin <<a href="mailto:Kevin-Kirspel@idexx.com" target="_blank">Kevin-Kirspel@idexx.com</a>>;
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
Subject: Re: [PATCH 1/5] Adding modified FREEBSD headers to sync RTEMS termios with FREEBSD<br>
<br>
The patch set looks good after a rough review. I will check this in in two or three days if nobody objects.<br>
<br>
--<br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : <a href="tel:%2B49%2089%20189%2047%2041-16" target="_blank">+49 89 189 47 41-16</a><br>
Fax     : <a href="tel:%2B49%2089%20189%2047%2041-09" target="_blank">+49 89 189 47 41-09</a><br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-<wbr>brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
______________________________<wbr>_________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.rtems.org_mailman_listinfo_devel&d=DwMFaQ&c=2do6VJGs3LvEOe4OFFM1bA&r=HDiJ93ANMEQ32G5JGdpyUxbdebuwKHBbeiHMr3RbR74&m=Rh9qB1wyRTbNNWuecoD2ZnxQG4zzUt_g-GM295-SibA&s=DM2kkkY3t3lj7LgOPBCejJ85dfSKMbJxO7971XSRMJI&e=" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><u></u><u></u></p>
</div>
</div>
</blockquote>
</div>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div>
</div></div></div>
</div>

<br>______________________________<wbr>_________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div>