<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this doesn't really solve the problem, sorry.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The point is basically: --mcc and --mnc are *only* used in gen_parameters(), which is only used for ICCID and/or IMSI generation if you don't specify either of those two.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Would you tell me more about it?  IMHO,</p><ul><li>mcc/mnc are extracted from IMSI for CSV mode in read_params_csv()</li><li>gen_parameters() deals with cmdline mode</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">Therefore, we have a design choice here, either,<br>1. Check whether mcc/mnc mismatch with IMSI (current patch)<br>2. Reject -x and -y when IMSI is provided</p><p style="white-space: pre-wrap; word-wrap: break-word;">Which one do you want to choose (or maybe there is a third choice)?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks.</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/17694">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/17694">change 17694</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/pysim/+/17694"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic7e98b827f32f8d59d346ceac0d2d215e0180109 </div>
<div style="display:none"> Gerrit-Change-Number: 17694 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: miaoski <miaoski@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: miaoski <miaoski@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 02 Apr 2020 02:32:16 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>