SapImpl: log error description

Test: pass
Bug: 35937742
Change-Id: I9784a15d8de5dc8a0bff849d5d3ef8c162bcb1a0

Merge "Move RILC logs under verbose flag to reduce the RILC log spam"

Move RILC logs under verbose flag to reduce the RILC log spam

This change moves most of RLOGD prints under VDGB flag

Test: Verified that the RILC log spam is reduced
Bug: 36613198
Change-Id: Ife7042b523a445605b297ed244d0f884d0b39179

Merge "Removing GENERIC_FAILURE from API documentation in ril.h"

Removing GENERIC_FAILURE from API documentation in ril.h

GENERIC_FAILURE should not be returned in any case.

Bug: 36385785
Test: No change to behavior
Change-Id: I5d3da76aadf9f2643d1e2ca5ca1d8f741e7a25c5

Merge "Remove rild sockets."

Remove rild sockets.

Test: Basic telephony sanity
Bug: 32020264
Merged-in: Icff46cadab085ca7882ed32ee0c6f9247a028817
Change-Id: Icff46cadab085ca7882ed32ee0c6f9247a028817

Merge "Convert empty response to -1 for some SIM APIs"

Convert empty response to -1 for some SIM APIs

Earlier RILs did not send a response for some cases although the
interface expected an integer as response. Do not return error if
response is empty. Instead Return -1 in those cases to maintain backward
compatibility.

Bug: 36209518
Test: Enable fixed dialing number and call barring by dialing *35*PASSWORD#.
      Check logs to make sure RILC does not fail with invalid response
      on devices where RIL doesn't send a response to these requests
      on success.

Change-Id: I1d91bacd60bd19985710a111de57e86f03e6ce4a

Merge "Add new precise call fail causes and OEM causes"

Merge "Fix for race condition between radio and oemhook services."

Fix for race condition between radio and oemhook services.

The race condition was this:
1. Phone process crashes and restarts
2. Phone process calls IRadio.setResponseFunctions()
3. oemHookInd is sent from vendor RIL. At this point oemHook
callbacks are stale, and due to that the callback fails and
sets callback objects to NULL, including the IRadio ones which is
not correct.

Test: Basic telephony sanity. The condition was easily reproducible
on angler; verified that it works fine now.
Bug: 32020264

Change-Id: I33bbdf01b19d009530c78baae90188acb4986d15

Add new precise call fail causes and OEM causes

Bug: 35609963
Change-Id: I46ffc8280a25286aedf311c5e63d9c7748849f3b

Merge "Remove unused using statement."

Remove unused using statement.

Test: pass
Bug: 36099713
Change-Id: Iaacfa2d2ce8555e191850a620f75111cb2d108a5

Merge "Return meaningful error code for RIL_REQUEST_QUERY_AVAILABLE_NETWORKS"

Return meaningful error code for RIL_REQUEST_QUERY_AVAILABLE_NETWORKS

Updated documentation for INTERNAL_ERR & MODEM_ERR

Bug: 34102797
Change-Id: If1ac0f50eb3e3a21056d137e060a8fccb24304c2

Merge "Return Void() from oneway APIs."

Return Void() from oneway APIs.

Test: none
Bug: 36162904
Change-Id: I323633c398085f96d265666136d451a33d683e51

Merge "Use cid and lac to update ci and tac for LTE"

Merge "Fix bug in getCallWaiting api."

Fix bug in getCallWaiting api.

Test: Build and local VTS.
Bug: 32020264
Change-Id: I9933b2d45a124e4f0704b987af49bab3bdcb3418

Merge "Fix signed-unsigned comparison"

Merge "Delete Parcel references and other unused code."

Use cid and lac to update ci and tac for LTE

As per older RIL interface LTE cid and lac were filled
using ci and tac. Hence use the same while converting
voice registration state.

Test: ran unit testing
      verified on device that registration state is
      reported correctly
Bug: 32223344
Change-Id: I74ea670ff51ceaf0387cdf3b9cafc5601ab54d98

Merge "Removed the deprecated screen state API"

Delete Parcel references and other unused code.

Test: Basic telephony sanity
Bug: 32020264
Change-Id: I410988c7f347fcca2770d9b099f489bd6d625d0b

Merge "Remove unnecessary prefix from log lines since LOG_TAG is added."

Merge "Remove unused response and dispatch functions."

Remove unnecessary prefix from log lines since LOG_TAG is added.

Test: Basic telephony sanity
Bug: 35869745
Change-Id: I38aafc9a699e6a21fa5b2dbf52c405b524ec0d3a

Remove unused response and dispatch functions.

Test: Basic telephony sanity
Bug: 32020264
Merged-in: I732234a22328a1bfcb603bb020547f543b6fd766
Change-Id: I732234a22328a1bfcb603bb020547f543b6fd766

Merge "Cleanup for radioStateChanged."

Cleanup for radioStateChanged.

Test: Basic telephony sanity
Bug: 32020264
Change-Id: I7b7675406409dbb92c3559499665fbe59030573b

Merge "Fix in getAvailableNetworksResponse() to use correct index"

Merge "DO NOT MERGE ANYWHERE Emulator: adjust reference-ril to the refactored qemu_pipe"

Removed the deprecated screen state API

The screen state API was deprecated on O and should
not be added into the radio interace.

Test: Telephony sanity tests
bug: 32223897
Change-Id: I587cb104cdfd17d8f3c0e0d6e354b051d3892eaa

Merge "Added two new RIL commands support"

Added two new RIL commands support

Added RIL_REQUEST_SEND_DEVICE_STATE and
RIL_REQUEST_SET_UNSOLICITED_RESPONSE_FILTER RIL requests support.

Test: telephony sanity tests
bug: 32223897, 20125037, 28294587

Merged-In: I55adc30b11eac269331c54ff08947dd5eefc463b
Change-Id: I55adc30b11eac269331c54ff08947dd5eefc463b

Fix signed-unsigned comparison

Bug: 35901901
Test: compilation yields no warning
Change-Id: I74e60def5205782e291c7bc24846804be89097b3

Fix in getAvailableNetworksResponse() to use correct index

Test: Manual
Bug: 35787804
Change-Id: I03e17ee3e2de449c6b38c3dda39d2eb8b2ea7576

DO NOT MERGE ANYWHERE Emulator: adjust reference-ril to the refactored qemu_pipe

qemu_pipe is refactored into a library in system/core/qemu_pipe,
and reference-ril needs to adjust accordingly.

Change-Id: I13160ce6a07a721501fd893dd9c5a9931927e767

Merge "Add log tag for IRadio and ISap services."

Merge "Fix in setAllowedCarriers() to use correct length for excludedCarriers"

Add log tag for IRadio and ISap services.

Test: verified logs
Bug: 35869745
Change-Id: I35c79c9d638e4328086def1b550f0aafcf9e9e5b

Fix in setAllowedCarriers() to use correct length for excludedCarriers

Test: make -j64 gts
gts-tradefed run gts -m GtsGmscoreHostTestCases -t
com.google.android.gts.telephony.TelephonyHostTest
Bug: 35374455

Change-Id: I05ccc58e71f4b7aa645543fb3c9ffc71f9817435

Merge "Cleanup DATA_REG and VOICE_REG Location Info"

Cleanup DATA_REG and VOICE_REG Location Info

The data registration and voice registration states contain
large swaths of location information. While it seemed at first
blush we could remove that information, having synchronized
knowledge of when a registration has resulted in a cell change
provides valuable infomation.  So cleaning up the same by using
the same cell location structure that is being used elsewhere
in the interface to ensure that this method
always keeps sufficiently up-to-date information, without the
need to increasingly bloat it. Note this also fixes subtle bugs
that existed due to the information previously being incomplete.

Bug: 32223344
Test: Tested updated qcril

Captured RILJ logs on different rats as follows

02-27 10:22:48.489 D/RILJ    ( 1483): [4169]< DATA_REGISTRATION_STATE
{.regState = REG_HOME, .rat = 14, .reasonDataDenied = -1, .maxDataCalls
= 20, .cellIdentity = {.cellInfoType = LTE, .cellIdentityGsm = [],
.cellIdentityWcdma = [], .cellIdentityCdma = [], .cellIdentityLte =
[{.mcc = -1, .mnc = -1, .ci = 56, .pci = 135, .tac = 13826, .earfcn =
-1}], .cellIdentityTdscdma = []}} [SUB0]

02-27 10:22:48.463 D/RILJ    ( 1483): [4166]< VOICE_REGISTRATION_STATE
{.regState = REG_HOME, .rat = 14, .cssSupported = true,
.roamingIndicator = 1, .systemIsInPrl = -1, .defaultRoamingIndicator =
-1, .reasonForDenial = 0, .cellIdentity = {.cellInfoType = LTE,
.cellIdentityGsm = [], .cellIdentityWcdma = [], .cellIdentityCdma = [],
.cellIdentityLte = [{.mcc = -1, .mnc = -1, .ci = 56, .pci = -1, .tac =
-1, .earfcn = -1}], .cellIdentityTdscdma = []}} [SUB

02-27 11:13:16.138 D/RILJ    ( 2110): [3992]< DATA_REGISTRATION_STATE
{.regState = REG_ROAMING, .rat = 13, .reasonDataDenied = -1,
.maxDataCalls = 20, .cellIdentity = {.cellInfoType = CDMA,
.cellIdentityGsm = [], .cellIdentityWcdma = [], .cellIdentityCdma =
[{.networkId = -1, .systemId = -1, .baseStationId = -1, .longitude = -1,
.latitude = -1}], .cellIdentityLte = [], .cellIdentityTdscdma = []}}
[SUB0]

02-27 11:13:16.136 D/RILJ    ( 2110): [3993]< VOICE_REGISTRATION_STATE
{.regState = NOT_REG_MT_SEARCHING_OP, .rat = 0, .cssSupported = false,
.roamingIndicator = 64, .systemIsInPrl = 1, .defaultRoamingIndicator =
-1, .reasonForDenial = 0, .cellIdentity = {.cellInfoType = 0x0,
.cellIdentityGsm = [], .cellIdentityWcdma = [], .cellIdentityCdma = [],
.cellIdentityLte = [], .cellIdentityTdscdma = []}} [SUB0]

02-27 11:23:21.875 D/RILJ    ( 2104): [4058]< VOICE_REGISTRATION_STATE
{.regState = REG_ROAMING, .rat = 6, .cssSupported = false,
.roamingIndicator = 64, .systemIsInPrl = 1, .defaultRoamingIndicator =
-1, .reasonForDenial = 0, .cellIdentity = {.cellInfoType = CDMA,
.cellIdentityGsm = [], .cellIdentityWcdma = [], .cellIdentityCdma =
[{.networkId = 54, .systemId = 4, .baseStationId = 7155, .longitude =
2147483647, .latitude = 0}], .cellIdentityLte = [], .cellIdentityTdscdma
= []}} [SUB0]

02-27 11:29:36.774 D/RILJ    ( 2157): [3925]< VOICE_REGISTRATION_STATE
{.regState = REG_HOME, .rat = 16, .cssSupported = false,
.roamingIndicator = 1, .systemIsInPrl = -1, .defaultRoamingIndicator =
-1, .reasonForDenial = 0, .cellIdentity = {.cellInfoType = GSM,
.cellIdentityGsm = [{.mcc = -1, .mnc = -1, .lac = 8, .cid = 0, .arfcn =
-1, .bsic = -1}], .cellIdentityWcdma = [], .cellIdentityCdma = [],
.cellIdentityLte = [], .cellIdentityTdscdma = []}} [SUB0]

02-27 11:29:36.768 D/RILJ    ( 2157): [3924]< DATA_REGISTRATION_STATE
{.regState = REG_HOME, .rat = 2, .reasonDataDenied = -1, .maxDataCalls =
20, .cellIdentity = {.cellInfoType = GSM, .cellIdentityGsm = [{.mcc =
-1, .mnc = -1, .lac = 8, .cid = 0, .arfcn = -1, .bsic = -1}],
.cellIdentityWcdma = [], .cellIdentityCdma = [], .cellIdentityLte = [],
.cellIdentityTdscdma = []}} [SUB0]

Merged-In: I62243cd0497bc742abf2fb58a24087698b25769d
Change-Id: I62243cd0497bc742abf2fb58a24087698b25769d

Merge "Fixed that type of data call response status"

Fixed that type of data call response status

Fixed the incorrect type for SetupDataCallResult.status.
Changed the type from int_32 to DataCallFailCause

Test: Telephony sanity tests
bug: 32224135
Change-Id: I55436f835318a0a60d2b53daabc493d8349af3b3

Merge "Data call refactoring and hidlization"

Data call refactoring and hidlization

Support the new HIDL interface for data call APIs.
The affacted APIs are RIL_REQUEST_SETUP_DATA_CALL,
RIL_REQUEST_SET_DATA_PROFILE, RIL_REQUEST_SET_INITIAL_ATTACH_APN
More parameters are passing down to the modem to address
some issues we had earlier.

This is also backward compatible with the old vendor RIL.

bug: 32224135, 30173958, 33561503, 30282096, 32020264
Test: Telephony sanity tests, unit tests

Merged-In: I5ed76321ef52d6bc7a00bce918fde7bb316f8c93
Change-Id: I5ed76321ef52d6bc7a00bce918fde7bb316f8c93

Merge "Fix for RadioCapability."

Fix for RadioCapability.

sizeof was called on incorrect struct.

Test: verified from the logs that RadioCapability shows on error
Bug: 35754369
Change-Id: I42bcb1f15cb5600e940550e51e109f1b2bce37a8

Merge "Move rild to vendor partition."

Move rild to vendor partition.

Test: Basic telephony sanity
Bug: 35672432
Change-Id: I7aac54db10fe190596378cabbbc634b862d13acb

Merge "OemHook service implementation."

OemHook service implementation.

Test: Basic telephony sanity
Bug: 32020264
Bug: 34344851
Merged-in: Ie165921c02b301b378f8f05eeec24a2f273691f4
Change-Id: Ie165921c02b301b378f8f05eeec24a2f273691f4

Merge "Check return status for response callbacks."

Merge "Added the new RIL request to set SIM power state"

Added the new RIL request to set SIM power state

The new RIL request for powering up/down the SIM card.

Test: Telephony sanity tests
bug:32224755
Change-Id: I43245ed8178235e45e592d06c2d1bcb71cd69225

Merge "Fix to handle hidl_string correctly in error scenarios."

Fix to handle hidl_string correctly in error scenarios.

Test: Basic telephony sanity
Bug: 32020264
Change-Id: If638839043923975b047cbd086146e64a8c73d99

Check return status for response callbacks.

Test: Basic telephony sanity
Bug: 32020264
Change-Id: I2b55b83834d43bf1803e2af05d409d0c8ea6186e

Merge "Service side HAL changes for RIL solicited responses - 2"

Merge "Service side HAL changes for RIL solicited responses."

Service side HAL changes for RIL solicited responses - 2

Test: Basic telephony sanity
Bug: 32020264
Change-Id: I7fda74cfb926752adc00f2b84af9c1e761e3433e

Service side HAL changes for RIL solicited responses.

Bug: 32020264
Merged-in: I8bbbf650770b5da13085ed85a760af6b45d6d3db
Change-Id: I8bbbf650770b5da13085ed85a760af6b45d6d3db

Merge "Fix build warnings in libril"

Merge "Fix bug for Cellbroadcast activation Radio hal commands."

Merge "Replaced strcpy, strlen, strncpy with safer string functions"

Fix bug for Cellbroadcast activation Radio hal commands.

+ Vendor expects activate ? 0 : 1

Test: Manual test run. Command succeeds.
Bug: 32020264
Change-Id: Ic1e8faeaa8dce1b5956117dcae3c23e9abe7c8b7

Replaced strcpy, strlen, strncpy with safer string functions

Using strcpy() or strlen() followed by strncpy() could be dangerous.
We should replace them.

strcpy -> strncpy
strlen -> strnlen
strcmp -> strncmp

Also fixed uninitialized structure uses.

Test: Telephony sanity tests
bug: 35108861
Change-Id: I323e2947b153d9214cfc1827168346200916431f

Merge "Provide HIDL support for RIL commands in ril.cpp (patch 2)."

Fix build warnings in libril

Bug:none
Test: on marlin
Change-Id: I561f68b00732b2e0bda22a769198e13a53698908

Provide HIDL support for RIL commands in ril.cpp (patch 2).

Also has revert of
https://googleplex-android-review.git.corp.google.com/#/c/1880364/

Bug: 32020264
Test: Basic telephony sanity. Phone calls/sms/data. No assert for FI sim.
Change-Id: I0617cdeca367962cc345a0d501e7754f40dc20e0
Merged-in: I0617cdeca367962cc345a0d501e7754f40dc20e0

Merge "Resend NITZ when the upstream client is not ready"

Resend NITZ when the upstream client is not ready

When the upstream client android RILJ is not ready, NITZ time data
must be saved and resend when the connection is established.

b/34856813

Change-Id: I468918ac807674b3ae0cbfbb1757a65ac79b20e4
Signed-off-by: Weilun Du <wdu@google.com>

Merge "RIL header update for O"

RIL header update for O

RIL_REQUEST_DEVICE_STATE and
RIL_REQUEST_SET_UNSOLICITED_RESPONSE_FILTER were added to replace
RIL_REQUEST_SCREEN_STATE, which will not be invoked by Android O
or above.

Data call refactoring works including adding more APN parameters for
RIL_REQUEST_SETUP_DATA_CALL, RIL_REQUEST_SET_INITIAL_ATTACH_APN,
and RIL_REQUEST_SET_DATA_PROFILE

bug: 20125037, 32224135, 32223897, 32249926, 30282096, 30173958,
33653220

Test: Build
Merged-In: I2e9a8a3a6cdc7fbd12489e476d2043440e72260a
Change-Id: I999f2ac9fed4cea17f2fad282503ba5e5f00c5b9

Merge "Add synchronization between IRadio callbacks and service creation."

Add synchronization between IRadio callbacks and service creation.

Radio indications are sent by vendor ril on a separate thread and
can be received even before radio service is created and registered.
This is to avoid that race and make sure indication callbacks are
accessed only after service is created.

Test: Basic telephony sanity
Bug: 34363807
Merged-in: Iab11add5ca458476575c8b52d5e5878ff2b1d175
Change-Id: Iab11add5ca458476575c8b52d5e5878ff2b1d175

Merge "Check return status for response and indication callbacks."

Merge "Service side HAL changes for RIL unsolicited responses - 3"

Check return status for response and indication callbacks.

Test: Basic telephony sanity
Bug: 32020264
Bug: 34472671
Merged-in: Ie74972845ee0a8d60a8887f8daff164a3abcb255
Change-Id: Ie74972845ee0a8d60a8887f8daff164a3abcb255

Service side HAL changes for RIL unsolicited responses - 3

Test: Basic telephony sanity (voice call, sms, data, APM)
Bug: 32020264
Merged-in: I916e0d5bc8cff8e52b3f7b55390e146adccc43bf
Change-Id: I916e0d5bc8cff8e52b3f7b55390e146adccc43bf

Merge "Service side HAL changes for RIL unsolicited responses - 2"

Merge changes from topics 'RADIO HAL 4', 'RADIO HAL 3'

* changes:
  Service side HAL changes for RIL unsolicited responses.
  Convert ril cpp to HIDL service.

Service side HAL changes for RIL unsolicited responses - 2

Test: Basic telephony sanity (voice call, sms, data, APM)
Bug: 32020264
Merged-in: I5d97084510adfa79d4911bd556cfd616526492c2
Change-Id: I5d97084510adfa79d4911bd556cfd616526492c2

Service side HAL changes for RIL unsolicited responses.

Test: Basic telephony sanity (voice call, sms, data, APM)
Bug: 32020264
Merged-in: I902500d589940e2af27be1d1961729dab409b4ae
Change-Id: I902500d589940e2af27be1d1961729dab409b4ae

Convert ril cpp to HIDL service.

Bug: 32020264

Test: Basic telephony sanity tests (voice calls, data, sms, APM).
Merged-in: I1327e7b07ed001e7fbb8d69a2c871bdc500f1a9f
Change-Id: I1327e7b07ed001e7fbb8d69a2c871bdc500f1a9f

Merge "Converting SAP from protobuf to hidl."

Merge "Adding simple radio service and callbacks."

Converting SAP from protobuf to hidl.

Test: Basic SAP sanity.
Bug: 32020264
Change-Id: I87dd2bd320736d06e10733dd40743c83e548d3c9

Adding simple radio service and callbacks.

Test: Sanity tests by telephony test team.
Bug: 32020264
Change-Id: Ib2b3abb54084a1b48c5fd74042fe534e3d9fb69f

Merge "Add deprecation warning to IMS_REGISTRATION RIL Calls"

Merge "telephony: add input data information in ril.h"

Add deprecation warning to IMS_REGISTRATION RIL Calls

-Deprecation warning for RIL_REQUEST_IMS_REGISTRATION_STATE
-Deprecation warning for RIL_UNSOL_RESPONSE_IMS_NETWORK_STATE_CHANGED

Bug: 33750436
Test: complation
Change-Id: Ieb7c2fc5061998e4d04a9d044cf3d883d01e995d

Merge "ril: Regenerate RIL response strings"

ril: Regenerate RIL response strings

Cleanup the strings output to the radio log. This change is cosmetic,
adding and updating strings to the log, without affecting usage of
these RIL requests, responses, and unsols.

Auto-generated based on header files, which also reorders some cases
to match the presentation order in the headers.
* ril_unsol_commands.h:
  Find: ^\s+\{RIL_UNSOL_([^,\ ]+).+
  Replace: case RIL_UNSOL_$1: return "UNSOL_$1";
* ril_commands.h:
  Find: ^\s+\{RIL_REQUEST_([^,\ ]+).+
  Replace: case RIL_REQUEST_$1: return "$1";

RIL_RESPONSE_* has been reordered to sit (alphabetically) between
RIL_REQUEST_* and RIL_UNSOL_*. The string format has been updated to
remove "RIL_" and just report "RESPONSE_*".

Change file modes in libril by removing the execute bit from source
files.

Change-Id: Ie6f1b49ff0245212e64d4e563a26ea93191deb22

telephony: add input data information in ril.h

Add input data information for RIL_REQUEST_SET_INITIAL_ATTACH_APN.