<p>Werner Zeh <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://review.coreboot.org/20471">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">siemens/nc_fpga: Modify macro FPGA_SET_PARAM to avoid hwilib errors<br><br>The macro FPGA_SET_PARAM was introduced to make the setting of different<br>FPGA registers with the appropriate values from hwinfo more<br>transparent. The hwilib takes care about the size of the provided buffer<br>where the requested value should be stored in. The fields in hwinfo have<br>not always the same size as the matching registers in the FPGA. So to<br>avoid errors resulting in a too small buffer when calling hwilib_get_field()<br>the buffer is now fixed to 32 bit and will be casted to the destination<br>type when the value is written into the FPGA register.<br><br>Changing the field size in hwilib would be the wrong way as the defined<br>lengths are specified this way to be expandable in the future.<br><br>In addition the number of maximum supported temperature sensors in<br>increased to 8 as the FPGA now supports more.<br><br>Change-Id: I0c697106783158420708a973c3cff2be90fa4fce<br>Signed-off-by: Werner Zeh <werner.zeh@siemens.com><br>---<br>M src/drivers/siemens/nc_fpga/nc_fpga.c<br>M src/drivers/siemens/nc_fpga/nc_fpga.h<br>2 files changed, 5 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/20471/2</pre><p>To view, visit <a href="https://review.coreboot.org/20471">change 20471</a>. To unsubscribe, 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/20471"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I0c697106783158420708a973c3cff2be90fa4fce </div>
<div style="display:none"> Gerrit-Change-Number: 20471 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Werner Zeh <werner.zeh@siemens.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>