<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div id="gmail-:166" class="gmail-a3s gmail-aXjCH gmail-m15bd95a0f57935e2"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​​</div>I don't see any clear benefit in doing this; we're moving the match<br>
with the PID from the udev rules file to the actual source code, which<br>
means that if we ever get any other Dell-branded modem we'll need to<br>
update the source code instead of adding a new udev rules file. But...<br></div></blockquote></div><br><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">This follows the first part of the patch, where the ID_MM_TELIT_PORTS_TAGGED was removed, and then I though to remove it also from here and reduce the dependency from udev, but I understand its not a big change and can be dropped.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:16px">Looking at the actual purpose of the code; it looks like it's for<br></span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:16px">minimizing probing time because Telit modems always reply well to<br></span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:16px">AT+GMI , and that is actually the first command we try... so do we<br></span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:16px">even need that logic there for anything? What happens if you remove<br></span><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:16px">the block setting the cgmi_retries and at_retries to 0?</span></blockquote><br></div><div class="gmail_extra"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Only AT ports reply to AT+GMI, while for the others (5, if I remember correctly) we need to try all the commands and the retries and wait for timeout to go on.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><br></div></div>