<p style="white-space: pre-wrap; word-wrap: break-word;">(posting forgotten Draft comments, may be hoplessly outdated now. Easiest to get rid of them is to just "Reply", so bear with me, thx)</p><p><a href="https://gerrit.osmocom.org/2484">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/2484/1/src/osmo_gsm_tester/ofono_client.py">File src/osmo_gsm_tester/ofono_client.py:</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/2484/1/src/osmo_gsm_tester/ofono_client.py@225">Patch Set #1, Line 225:</a> <code style="font-family:monospace,monospace">    def connect(self, mcc=None, mnc=None):</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">it would be nice here to have a choice: if I pass explicit MCC+MNC, then we ask the NetworkRegistration interface to do things. If not, just put the modem online and let it figure things out on its own. That way we could also merge the patch and have the current mo_mt_sms.py still use the automatic/implicit registration.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Input validation: We need to make sure either both MCC and MNC are passed or neither.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/2484">change 2484</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/2484"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I3092edc3dd76131da9aeeef6ad5082d1a9009342 </div>
<div style="display:none"> Gerrit-Change-Number: 2484 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 11 Dec 2018 12:10:34 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>