<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 26, 2013 at 12:04 AM, Ben Chan <span dir="ltr"><<a href="mailto:benchan@chromium.org" target="_blank">benchan@chromium.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Wed, Sep 25, 2013 at 11:41 PM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@lanedo.com" target="_blank">aleksander@lanedo.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 09/26/2013 02:33 AM, Ben Chan wrote:<br>
> This patch modifies mm_3gpp_parse_iccid() to auto-detect if an ICCID<br>
> response is character swapped or not by comparsing the major industry<br>
> identifier part of the ICCID response to the known value (89) for<br>
> telecommunication purposes. This addresses the issue where the same AT<br>
> command (e.g. AT^ICCID used by the huawei plugin) does not report ICCID<br>
> in a consistent format.<br>
<br>
</div>This one looks fine to me; but I'm not sure whether Dan was asking for<br>
just warning if an ICCID didn't start with 89 (instead of returning an<br>
error).<br>
<div><div><br></div></div></blockquote><div><br></div></div><div>Another data point: I examined 2 callbox test SIMs and their ICCIDs also start with 89. So testing equipments also seem to follow the standard.</div>
<div><br></div><div><br></div></div></div></div></blockquote><div><br></div><div>I discussed with Huawei engineers. They also suggested checking the leading digits. Dan, if you prefer raising a warning instead of an error, I can revise the patch to address that. Though I think an error would be easier to spot.</div>

<div><br></div><div>Thanks,</div><div>Ben </div></div><br></div></div>