<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 1:43 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br>

<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"><div><div class="h5"><br>
</div></div>I already thought about these, and my idea was to completely avoid the<br>
name duplications in the service and command ids, and really rewrite<br>
them ourselves; so something like:<br>
<br>
MBIM_SERVICE_MS_FIRMWARE_ID<br>
MBIM_CID_MS_FIRMWARE_ID_GET<br>
<br>
MBIM_SERVICE_MS_HOST_SHUTDOWN<br>
MBIM_CID_MS_HOST_SHUTDOWN_NOTIFY<br>
<br>
The "Get" and "Notify" names are really made up, but given that these<br>
services only have one command each, it's probably not a big deal.<br>
<br>
And then for the mbimcli actions:<br>
--ms-firmware-id-get<br>
--ms-host-shutdown-notify<br>
<br>
BTW, do you have any device supporting these? None of the ones I know of<br>
have them implemented.<br></blockquote><div><br></div><div>That's not a bad idea. The duplication does look a bit odd (and differ from the spec).</div><div><br></div><div>My only concern is that whether more CIDs will be added to the service, and whether the CIDs will be extended to support 'Query', 'Set', or 'Notification'. However, the second case seems unlikely for these two services. Also, we may want to avoid using '_NOTIFY' for MBIM_CID_MS_HOST_SHUTDOWN_NOTIFY as that seems to imply that the CID is related to 'Notification' while it's a 'Set'.</div>

<div><br></div><div>A more dramatic approach, which I'm not sure if it's worth the effort, is to change to the code generator not to add a service name to CID names. Then we can pick whatever CID names we want as long as they are unique.</div>

<div><br></div><div>As for the mbimcli actions, would you like to following a similar naming convention (e.g. query instead of get for 'Get') like basic-connect? </div></div></div></div>