<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Is there any way to avoid needing the<br>
"mm_broadband_modem_huawei_ignore_ndisstat_unsolicited_msg()" method?<br>
Why not always listen to NDISSTAT messages in the modem object and check<br>
whether the bearers are being connected/disconnected and avoid doing<br>
anything in those cases? Ideally, a bearer shouldn't ask to modify the<br>
modem behaviour; but the modem can check its own bearers and do<br>
different things based on what it sees. Think of modems with multiple<br>
bearers.<br>
<br></blockquote><div><br></div><div>Done. I've uploaded a revised patch to do what you suggested.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


This whole thing of needing to ignore the unsolicited messages during<br>
the connection and disconnection attempts is the one complicating the<br>
logic a lot. How sure/unsure are you that we cannot use the unsolicited<br>
messages to complete the connection/disconnection attempt with these?<br>
Are really not reliable? Or are they and we can just use them (that<br>
would simplify the logic quite a lot)?<br></blockquote><div><br></div><div>Yes, that would be ideal. But I don't have enough Huawei modems to confirm whether the NDISSTAT unsolicited message can fully replace the NDISSTATQRY poll. Perhaps Franko can provide more details. We can have a follow-up patch to simplify the code.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Also, I would just modify the regex to accept either NDISSTATQRY or<br>
NDISSTAT in the message; instead of creating a new regex with the<br>
specific string each time.</blockquote><div><br></div><div>Done. The revised patch now extends the ndisstatqry parser to handle both NDISSTAT and NDISSTATQRY response.</div><div> </div></div></div></div>