Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I400f299c4fef607c39b24ebd33f27c22cde8127f

build: use AM_CPPFLAGS instead of INCLUDES

Fixes recent automake warnings.

cli,nas: use '%u' for printing cell IDs

This patch fixes the following type mismatches in the format string for
printing cell IDs:

qmicli-nas.c:1271:71: error: format specifies type 'unsigned short' but the argument has type 'guint32' (aka 'unsigned int') [-Werror,-Wformat]
                    g_print ("\t\tCell ID: '%" G_GUINT16_FORMAT"'\n", cid);
                                            ~~~                       ^~~
qmicli-nas.c:1391:71: error: format specifies type 'unsigned short' but the argument has type 'guint32' (aka 'unsigned int') [-Werror,-Wformat]
                    g_print ("\t\tCell ID: '%" G_GUINT16_FORMAT"'\n", cid);
                                            ~~~                       ^~~
qmicli-nas.c:1505:71: error: format specifies type 'unsigned short' but the argument has type 'guint32' (aka 'unsigned int') [-Werror,-Wformat]
                    g_print ("\t\tCell ID: '%" G_GUINT16_FORMAT"'\n", cid);
                                            ~~~                       ^~~
qmicli-nas.c:1626:71: error: format specifies type 'unsigned short' but the argument has type 'guint32' (aka 'unsigned int') [-Werror,-Wformat]
                    g_print ("\t\tCell ID: '%" G_GUINT16_FORMAT"'\n", cid);
                                            ~~~                       ^~~

Bug reported on https://code.google.com/p/chromium/issues/detail?id=239736

docs: always generate docs at dist time

The pregenerated docs will be in the dist tarball, and will even
be installed with 'make install' when building the dist tarball.

cli,nas: fix persistence technology preference reporting

dms: DMS v1.1 supports GetOperatingMode/SetOperatingMode

My Gobi 1K with 2008-era firmware and DMS v1.1 supports these, so
whatever docs said they required DMS 1.2 are wrong.

cli: implement Set System Selection Preference

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

cli: Add --dms-activate-manual

Added the --dms-activate-manual option to the client, which triggers
manual activation.

release: bump version to 1.3.0 (development)

release: bump version to 1.2.0

release: update NEWS

build: fix passing arguments to configure from autogen.sh

dms: activation code string needs explicit 1-byte length prefix

Reported by Arman Uguray <armansito@google.com>

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I668282a08373c80bec37e354a40167139945c267

cli: fix call to --dms-activate-automatic

--dms-activate-automatic incorrectly calls
qmi_client_dms_get_activation_state. This patch corrects it
so that it calls qmi_client_dms_activate_automatic.

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: Ifdc9baacffc2ab37edebb0e55b3488ad75313ec6

qmi-network: Fix grabbing output from lines with spaces

The qmi-network script used awk with a field separator of ":". The
output looked like key: 'value'. The second field in this case includes
a space at the beginning. This was making the 'status' command fail.

Now, we account for the space. It is now also not dependent on awk
(this can matter on embedded systems).

cli,wds: make packet data handle formatting consistent

Other values are surrounded by single quotes. The debug test in
utils/qmi-network also shows the output in quotes.

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I56daebd047fa4a58329103b80b30f7d697899aea

cli,nas: fix 2 or 3 digit MNC printing in several actions

https://bugs.freedesktop.org/show_bug.cgi?id=59664

cli: all sources are GPLv2+

cli: allow specifying auth info in the start network command

qmicli: update timeout for the start network command

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: Icd83ba97c5dcbf0aa09d9a364e6f3e582b3dc237

qmi-codegen: clean up compiled python files

These caused problems for deb packaging, as their content depends on the Python
version used.

qmi-codegen python3: change string.lower(str) to str.lower()

The string.lower(str) class method is no longer available.

sed regex:
s/string\.lower(\(.*\))/\1.lower()/

qmi-codegen python3: fix dict.has_key('key') to 'key' in dict

Python 3 no longer supports the has_key() method.

sed regex:
s/\([^ ]\+\)\.has_key(\([^)]*\))/\2 in \1/g

qmi-codegen python3: change string.replace() class method use to str.replace() instance method

Python 3 doesn't support the replace class method; it's now an instance method only.
string.replace(object, old, new) changes to object.replace(old, new)

sed:
s/string\.replace(\([^,]*\), /\1.replace(/

qmi-codegen: fix some python3 compatibility issues

Simple methods are no longer in the string module.

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: Iba10654ab918e7c2e2a07ca400fbfe006c09e5af

gobi-api: add GobiAPI_2012-09-12-0719

release: bump version to 1.0.0

release: update NEWS

libqmi-glib,test: enable tests expecting warnings only if glib >= 2.34

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I1228f9e659c278f64de5d25a160d604cce923804

nas: new 'QMI_NAS_DATA_CAPABILITY_NONE' enum value

qmi-codegen: translate the values of enums/flags in traces

libqmi-glib: make 'QmiCtlDataLinkProtocol' a enum instead of flags

If the values are like flags, but mutually exclusive, just set it as an enum.

cli: log messages from the 'Qmi' domain

docs: add missing documentation for "NAS Get CDMA Position Info"

qmi-codegen: fix computation of expected TLV length when using strings

nas: add Get CDMA Position Info

Not sure what NAS version supports it, but at least NAS 1.3 is required.

libqmi-glib,test: new test for messages with invalid TLVs

qmi-codegen: validate TLV before really reading it

Try to handle buggy firmware, or just make the library more robust, by
validating the read TLV before really reading it. If a TLV is not considered
valid, we just skip it for now.

E.g.: the "Detailed Service Status" TLV (0x21) in the "NAS Get Serving System"
message is supposed to be a sequence of 5 bytes, but some models (e.g. ZTE MF683)
end up sending only the first 4 bytes.

Revert "libqmi-glib,message: include raw data buffer when building printable string"

This reverts commit d89b9064afa991d5cc7737fa9b0b1b55c0e31158.

We already print the raw buffer when sending/receiving the message in QmiDevice,
which is actually better as we end up priting it *before* parsing it, so it's
much better for debugging purposes.

libqmi-glib,test: include printable string in message tests

libqmi-glib: define library G_LOG_DOMAIN

libqmi-glib,utils: minor documentation fix

libqmi-glib,message: include raw data buffer when building printable string

libqmi-glib,device: print raw sent/received message before translating it

This will help us debug issues in the message translation.

libqmi-glib,utils: no need to pass endianness when reading/writing single bytes

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I35274a07e48ea4d14608faaf67244eb4174463ec

cli,wds: print connection status string instead of integer

wds: annotate GetCurrentSettings with correct byte orders

IPv6 addresses are always sent in network byte order.  Oddly,
IPv4 addresses appear to be sent as LE uint32s rather than
BE/network byte order.

libqmi-glib,qmi-codegen: add endian-ness annotation capability

Some values are sent by the firmware in big endian byte order,
specifically IP addresses, which are sent in network byte order
(ie, big endian).  Add the ability to specify the byte order
the firmware handles the value as, and convert that to host
byte order when reading/writing QMI buffers.

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: I8e317fda3afe7cbdc8224414bf66f6963c2b64d5

libqmi-glib,qmi-codegen: cancelled operations will issue an ABORT message

We allow passing a GCancellable to every async operation with the clients. Now,
if the cancellable gets cancelled and the operation can be ABORT-ed, then we
do it.

libqmi-glib,device: avoid invalid reads in the sync indication callback

If we don't disconnect the signal handler when we clear the private
QmiClientCtl, we may end up in the situation where we try to use the
already disposed QmiDevice object.

wds: "Start Network" may be aborted

nas: "Network Scan" may be aborted

libqmi-glib,qmi-codegen: timed out operations will issue an ABORT message

Messages can now be tagged with a special 'abort' keyword, so that whenever the
message times out we issue a new ABORT command to cancel the specific timed out
request.

This support is currently only available for the NAS and WDS services, which are
the ones supporting ABORT for their long-running operations.

libqmi-glib,device: include missing `qmi_device_command()' documentation

docs: hide class structures of `QmiDevice' and `QmiClient'

dms: activation state related commands already in DMS 1.3

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: Iba3f9a2ca7a39a09e9e1c847d7909f7b4302df82

build: fix documentation build

cli: traces enabled if running in verbose mode

libqmi-glib: message traces compiled always

Message traces have been very useful when debugging issues in the protocol, and
we should avoid requiring a full recompilation in order to get them enabled.

Instead, we provide two new API methods, `qmi_utils_(get|set)_traces_enabled()',
which allow specifying whether traces should be dumped with g_debug() or not.

libqmi-glib,utils: improve documentation

libqmi-glib,utils: make qmi_utils_str_hex() private to the library

device: remove transaction ID related traces, not useful any more

qmi-codegen: make internal get_printable() and get_version_introduced()

These methods are (should only be) used only by the library.

docs: improve documentation of enums, flags and errors

qmi-codegen: request creator and response/indication parsers are private

libqmi-glib: completely hide the implicit CTL Client in the API

build,libqmi-glib: only allow including `libqmi-glib.h' directly

build,libqmi-glib: include missing headers in SOURCES

libqmi-glib: new header file for private enum/flag types

build: include missing files in dist

docs: improve generated `libqmi-glib' documentation

Among the tons of fixes done here, we now generate some per-service .sections
file which we then concatenate to build the final libqmi-glib-sections.txt file.

build: compile all generated code into a non-installable library

Also simplify the build by using BUILT_SOURCES to avoid needing to specify
custom dependency rules.

libmm-glib: fix multiple documentation issues

Merge remote-tracking branch 'cros/upstream' into 'cros/master'

Change-Id: Ib60b16de8638a40c9c694c014a707e5d6f333ba3

cli: format fix

nas: Ec/Io given always as a signed integer

cli: skip printing network info for SINR and IO

Seems that SINR and IO, even if they are supposedly only meaningful in EV-DO,
may still be given for other networks.

[27 Sep 2012, 11:17:36] [Debug] [/dev/cdc-wdm0] Received message...
>>>>>> QMUX:
>>>>>>   length  = 49
>>>>>>   flags   = 0x80
>>>>>>   service = "nas"
>>>>>>   client  = 6
>>>>>> QMI:
>>>>>>   flags       = "response"
>>>>>>   transaction = 1
>>>>>>   tlv_length  = 37
>>>>>>   message     = "Get Signal Strength" (0x0020)
>>>>>> TLV:
>>>>>>   type       = "Result" (0x02)
>>>>>>   length     = 4
>>>>>>   value      = 00:00:00:00
>>>>>>   translated = SUCCESS
>>>>>> TLV:
>>>>>>   type       = "Signal Strength" (0x01)
>>>>>>   length     = 2
>>>>>>   value      = A8:05
>>>>>>   translated = [ strength = '-88' radio_interface = '5' ]
>>>>>> TLV:
>>>>>>   type       = "RSSI List" (0x11)
>>>>>>   length     = 4
>>>>>>   value      = 01:00:58:05
>>>>>>   translated = { [0] = '[ rssi = '88' radio_interface = '5' ] '}
>>>>>> TLV:
>>>>>>   type       = "ECIO List" (0x12)
>>>>>>   length     = 4
>>>>>>   value      = 01:00:FC:05
>>>>>>   translated = { [0] = '[ ecio = '252' radio_interface = '5' ] '}
>>>>>> TLV:
>>>>>>   type       = "IO" (0x13)
>>>>>>   length     = 4
>>>>>>   value      = 96:FF:FF:FF
>>>>>>   translated = -106
>>>>>> TLV:
>>>>>>   type       = "SINR" (0x14)
>>>>>>   length     = 1
>>>>>>   value      = 08
>>>>>>   translated = 8

cli: avoid shadowing the global `index' variable

libqmi-glib,utils: handle alignment issues when reading integers from the buffer

Some architectures require that the value of a pointer is aligned in memory.
Given that we're reading from a raw buffer, the integers in it may not end up
aligned so we cannot safely cast any address to a valid 16/32/64 bit integer
value. Handle this by copying the integer from the raw buffer directly into the
output variable, which is of course properly aligned. Also added new test cases
to check this.

Thanks to:
    Shawn J. Goff <shawnjgoff@gmail.com>
for reporting the issue and his endless tests.

cli: new `--dms-select-stored-image' action

Can be run e.g. like:
$> sudo qmicli -d /dev/cdc-wdm0 --dms-select-stored-image="modem1,pri2"

cli: new `--dms-delete-stored-image' action

Can be run e.g. like:
$> sudo qmicli -d /dev/cdc-wdm0 --dms-delete-stored-image="pri2"

The passed index is the one given by a previous `--dms-list-stored-images'
action. Please note that if you delete a given image, the listed indexes will
change, so if you want to delete multiple images you'll need to first list
stored images and grab the index and then use that bew index to delete the new
image.

Also, don't try this command many times or you'll run out of images to delete!

cli: new `--dms-list-stored-images' action

dms: implement "Set Firmware Preference" request/response

dms: implement "Delete Stored Image" request/response

dms: implement "Get Stored Image Info" request/response

dms: implement "List Stored Images" request/response

dms: implement "Get Firmware Preference" request/response

qmi-codegen: don't issue the array element clear function on 'Input' arrays

When an array is required to be passed in an input TLV, the user who created it
is responsible for freeing it. Therefore, we should not dump the static array
element clear function in these cases, or these unused methods will end up
breaking the compilation.

libqmi-glib,tests: new tests to check parsing valid/invalid QMI messages

message: document the section