<p><a href="https://gerrit.osmocom.org/13681">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13681/4/sysmoOCTSIM/main.c">File sysmoOCTSIM/main.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13681/4/sysmoOCTSIM/main.c@248">Patch Set #4, Line 248:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">       settings.clkdiv = SIM_CLKDIV_8;<br>       ncn8025_set(slotnr, &settings);<br>   // set card voltage to 3.0 V (the most supported)<br>     // note: according to ISO/IEC 7816-3:2006 no voltage should damage the card, and you should cycle from low to high<br>    settings.vsel = SIM_VOLT_3V0;<br> ncn8025_set(slotnr, &settings);<br>   // provide power (the NCN8025 should perform the activation according to spec)<br>        // note: activation sequence is documented in ISO/IEC 7816-3:2006 section 6.2<br> settings.cmdvcc = true;<br>       ncn8025_set(slotnr, &settings);<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">those could all be part of one ncn8025_set.  I will test and update.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13681/4/sysmoOCTSIM/main.c@259">Patch Set #4, Line 259:</a> <code style="font-family:monospace,monospace">400 * 10000 / 2500</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">can that really safely be expressed in uint16_t arithmetic? delay_us uses a 16bit argument :/</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13681/4/sysmoOCTSIM/main.c@261">Patch Set #4, Line 261:</a> <code style="font-family:monospace,monospace">      settings.rstin = true;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the logic of rstin has changed in master yesterday, I will update + test</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13681">change 13681</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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://gerrit.osmocom.org/13681"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ccid-firmware </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I30b284cab60a50d4cd3080f46f4d332193bbf1ee </div>
<div style="display:none"> Gerrit-Change-Number: 13681 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Kévin Redon <kredon@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 17 Apr 2019 08:31:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>