<p><a href="https://gerrit.osmocom.org/c/osmo-e1-hardware/+/21715">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/osmo-e1-hardware/+/21715/5/firmware/ice40-riscv/icE1usb/e1.c">File firmware/ice40-riscv/icE1usb/e1.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/osmo-e1-hardware/+/21715/5/firmware/ice40-riscv/icE1usb/e1.c@356">Patch Set #5, Line 356:</a> <code style="font-family:monospace,monospace">/* get cumulative error bit mask since last call of this function */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Huh, it's not "since the last call" AFAICT this is never reset. Counters just wrap.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-e1-hardware/+/21715/5/firmware/ice40-riscv/icE1usb/e1.c@380">Patch Set #5, Line 380:</a> <code style="font-family:monospace,monospace">           if (g_e1.rx.last_tick == cur_tick) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Again, as in previous patches, I'd remove any kind of support for LOS until there is actual hw to detect that.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-e1-hardware/+/21715/5/firmware/ice40-riscv/icE1usb/ice1usb_proto.h">File firmware/ice40-riscv/icE1usb/ice1usb_proto.h:</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/osmo-e1-hardware/+/21715/5/firmware/ice40-riscv/icE1usb/ice1usb_proto.h@83">Patch Set #5, Line 83:</a> <code style="font-family:monospace,monospace">#define ICE1USB_ERR_F_TICK_ERR  0x02</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I would call that LOS and not TICK_ERR.<br>However LOS ends up being implemented ...</p><p style="white-space: pre-wrap; word-wrap: break-word;">(I mean, even on a "real" card, there is no "signal level" to detect, the idle state line is 0, so either you detect pulses or your don't, you can't measure any kind of analog voltage change if it's plugged or not).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-e1-hardware/+/21715">change 21715</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/c/osmo-e1-hardware/+/21715"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-e1-hardware </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iba3e00a2b28a2fef6dbd986bfc706c1619c3a3ed </div>
<div style="display:none"> Gerrit-Change-Number: 21715 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 20 Dec 2020 18:21:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>