<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >Hey Sam,</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks for this information. Actually there is no compilation error for the following code snip but the functionality is not correct. We should have PRIx64 outside of the quote with a whitespace before it.</div>
<div dir="ltr" > </div>
<div dir="ltr" >I actually discovered this in another branch during testing but unfortunately I forgot to merge it with my pull request, shame on me! There will be an update soon. </div>
<div dir="ltr" > </div>
<div dir="ltr" >> -        memcpy(response, &buf, *data_len);<br>> +        uint64_t return_value;<br>> +        sscanf(buf, "%PRIx64", &return_value);</div>
<div dir="ltr" > </div>
<div dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >GOU, Peng Fei (苟鹏飞), Ph.D.</div>
<div dir="ltr" >OpenPower Team.</div>
<div dir="ltr" >+86-21-609-28631</div></div></div>
<div dir="ltr" > </div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: Sam Mendoza-Jonas <sam@mendozajonas.com><br>To: OpenBMC Patches <openbmc-patches@stwcx.xyz><br>Cc: openbmc@lists.ozlabs.org, Peng Fei BG Gou/China/IBM@IBMCN<br>Subject: Re: [PATCH phosphor-host-ipmid v7] Add get/set boot option ipmid command support with correct DBUS property handling.<br>Date: Tue, Jan 12, 2016 10:05 AM<br> 
<div><font face="Default Monospace,Courier New,Courier,monospace" size="2" >Hi,<br><br>On Mon, Jan 11, 2016 at 06:50:39PM -0600, OpenBMC Patches wrote:<br>> From: shgoupf <shgoupf@cn.ibm.com><br><br>(snip)<br><br>> -        memcpy(response, &buf, *data_len);<br>> +        uint64_t return_value;<br>> +        sscanf(buf, "%PRIx64", &return_value);<br><br>I just noticed this change from the previous version; have you tested<br>this? I believe you need to use the PRIx64 specifier like this:<br><br>sscanf(buf, "%" PRIx64, &return_value);<br><br>Where PRIx64 is not within the quotes. Do you get an error when you<br>build this?<br><br>Regards,<br>Sam Mendoza-Jonas</font></div></blockquote>
<div dir="ltr" > </div></div></div><BR>