<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >OK, that's a fair suggestion.</div>
<div dir="ltr" > </div>
<div dir="ltr" >I will fix this.</div>
<div dir="ltr" > </div>
<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >GOU, Peng Fei (苟鹏飞), Ph.D.</div>
<div dir="ltr" >OpenPOWER Enablement.</div>
<div dir="ltr" >+86-21-609-28631</div></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: Stewart Smith <stewart@linux.vnet.ibm.com><br>To: OpenBMC Patches <openbmc-patches@stwcx.xyz>, openbmc@lists.ozlabs.org<br>Cc: Peng Fei BG Gou/China/IBM@IBMCN<br>Subject: Re: [PATCH phosphor-host-ipmid v9] Add get/set boot option ipmid command support with correct DBUS property handling.<br>Date: Fri, Jan 29, 2016 11:56 AM<br>
<div><font face="Default Monospace,Courier New,Courier,monospace" size="2" >OpenBMC Patches <openbmc-patches@stwcx.xyz> writes:<br>> struct get_sys_boot_options_t {<br>> uint8_t parameter;<br>> uint8_t set;<br>> uint8_t block;<br>> } __attribute__ ((packed));<br>> <br>> +struct set_sys_boot_options_t {<br>> + uint8_t parameter;<br>> + union {<br>> + struct {<br>> + uint8_t d1;<br>> + uint8_t d2;<br>> + uint8_t d3;<br>> + uint8_t d4;<br>> + uint8_t d5;<br>> + uint8_t d6;<br>> + uint8_t d7;<br>> + uint8_t d8;<br>> + } data8;<br>> + uint64_t data64;<br>> + } data;<br>> +} __attribute__ ((packed));<br><br>Why not "uint8_t d[8]" ?<br><br><br>--<br>Stewart Smith<br>OPAL Architect, IBM.</font></div></blockquote>
<div dir="ltr" > </div></div><BR>