ril: service: Fix breakage introduced by f7c75aa4e0

Before:
    if (response == NULL || numStrings != 3) {
After:
    if (response == NULL || numStrings != mqanelements - 2) {

This patch fixes the logic so we don't change the value check.

"mqanelements" is not related to getOperatorResponse, so get get rid of
the affected code.

Change-Id: I86c6ae7b7492a5d04fbc8dc415c4f615e7d05bab

audio: Guard mixer path dependency fix

Change-Id: Id93d7941ae322a83488da3cea62c6eb36a6e9160

power: Allow POWER_HINT_DISABLE_TOUCH in low power mode

Remove the unused len var while we're at it.

Change-Id: Iabaf3b8704072a8758b213a54530eaffbf0c7ae5

ril: Add support for imsType parcel member

This is needed for Samsung devices like the Galaxy A5 2017.

Change-Id: I3b89cf8ac232fef5629b439dbf5d2f21bdb264a2

ril: Add support for roamingProtocol parcel member

This is needed for Samsung devices like the Galaxy A5 2017.

Change-Id: I549130b838377d8a8668ac59b51c314fe9e7f5ab

AdvancedDisplay: Rename Negative to Accessibility

* Samsung now supports many more modes than just inverted colors
* Add strings for these extra modes while we're at it

Change-Id: If2a469c994658d521d98b91adec1a19badb8092f

AdvancedDisplay: Add support for new mDNIe scenarios

* BROWSER_APP
* eBOOK_APP
* EMAIL_APP
* TDMB_APP

Change-Id: I44be4d46f7a077295f0631a71ec652e61c3f9411

AdvancedDisplay: Fix strings

* It just so happens that not all devices will have the
  same options available, so allow them to specify what
  they have without breaking tranlations by moving the
  scenario names into individual strings from the arrays
* Update a few strings to be actually translatable
* Fix an off-by-one error for every display mode after
  "LineageOS (default)": UI is the same thing as the
  default mode, so remove it to make the mode selection
  actually choose the user-selected mode

Change-Id: Id5d659d79c10ea097c35b7416ef7c1cc93937756

power: Silence excessive logging

Change-Id: I519122aaa77e73cf0290610a3d9d4aa79700e843

audio: Support new and old mixer paths locations

Change-Id: Ifa2a55cbfe45ab9d53cd23098012443f9b32ca31

audio: Print error in case of nonexistant mixer paths

Change-Id: I131ed6518ce8f26638a3490ee1be7a5d93b4e5b8
(cherry picked from commit 0524e26d2a1d155ce1dc9609b8e8741fd5f1500f)

audio: Avoid mixer path dependency conflicts

On some devices, certain input and output mixer controls share the same
endpoint, which means that if an input stream is disabled (and the mixer
is updated), that shared endpoint may be disabled as well.

This would result in no audio, because no one is aware that the output
stream in fact does still need the shared endpoint to be enabled.

We prevent this by enabling the output mixers after disabling input mixers,
if any.

Change-Id: I4d80fcd0648a11099f27177da3cd29095b6a8c34
(cherry picked from commit 507e8ce5bffb36fae8b1cd4e5498fa19a6273ee6)

wifiloader: Make sure WiFi firmware path is writable

Change-Id: Ib0ae2a004d4207a4ccb3d551c9a6a092920595ca

wifiloader: Notify WiFi HAL on module load success

Change-Id: I0c7de00aec571a29f79137bb5b50178b0be91408

dtbhtool: Print better warnings for PLATFORM/SUBTYPE mismatch

Change-Id: I486239d47255d5a5347e7b8ac7348afb8bc5bddb
(cherry picked from commit 8f74f0a7e9839d4f6f499aecdcc6c042ca4671a0)

dtbhtool: implement unpackdtbhimg

Parses a DTBH image and outputs the information needed
for samsung_dtb.h and dumps all DTBs contained in the file.

Change-Id: Ia724f25c3b33423baeefec85c9b0d42967fce6f2
(cherry picked from commit 25b3f58c5bb9e9cfd2f0282047a2ea13141496cc)

Added new Murata MAC range

OUI a0:c9:a0 was observed on hero devices bought in December 2017

Change-Id: I0a2404780b1b5b017bb89468d235a2174a245989
(cherry picked from commit 939c81d0e0581a08b896636ce0f1d5bb96946dc9)

Added another new Murata MAC range

Another herolte user observed OUI 8c:45:00

Change-Id: I7eaf4fef29c6dbc6a962de0ff745c9703f65fcfc
(cherry picked from commit 70d3e3166f748e96f72c1c96aa2971f4981e9981)

libsec-ril: Include liblog

Change-Id: I87df49446a2729f19a5d5611a88a4a80faf0350b

libril: fix network operator search

The number of mQANElements should be defined in the system
prop ro.ril.telephony.mqanelements

Change-Id: Ie5bb8ba80c5ac93b7502da3b1bb3d2b4404ecd5e

libril: fix in call presentation

Change-Id: Ifd4a20c0bcd34f380716cc96cdd4c99b4b909266

libril: handle Samsung proprietary commands

Change-Id: Ief8402c8e75c677c7f2e32794f13867b924b209f

libril: port Samsung responseDataCallList to O libril

Change-Id: I1a1980e25b2ae1c496420fe3bfb0e8999e62f864

libril: port samsung changes to signalStrength

Change-Id: I39c1f5dcb44bcd0cf265243276d7649a9c2fdc8c

libril: add Samsung flags to makefile

Change-Id: I5deb1f7582b207acf908691a7ac69a9ebbc62505

libril: bring back signalStrength v5 and v6

Change-Id: Ie0513986e9cabe9b363198e0192b18a0a8fe97de

ril: import Oreo AOSP libril

Change-Id: I00750bad812dc263e9c74a504b48512a892376c8

audio: Fix BT sample rate handling properly

This should now support both old and new headsets,
as we rely on codec negotiation in the bluetooth
stack for narrow/wide band switching.

Change-Id: Ie9d308dfc55726fd1591a7d158f610bd267340e6
(cherry picked from commit d462f7b86e957f858959ef5d80fd1744689ee6ea)

audio: End voice call in set_mode()

Previously, we would only end the voice call once the next output
stream happened (and parameters would be set).

Change-Id: Id0d2711605e57ad69d9b26b36afb562444349358
(cherry picked from commit 5169e86c10b7429b01d6ab0500a0509141dcbfe1)

audio: Fix incall device switch handling

The whole code block was no-op, since we would override
(in|out)_snd_device at a later point anyways.

Change-Id: Ieb2b0918c9c72f18d195d6b34b5c9768fce130b3
(cherry picked from commit f33a728b205f7656ca6211a07e3dd69c623b75ca)

audio: Improve code readability

Change-Id: I5b1f24ab6098a037559603a92b4a1a38987488f5
(cherry picked from commit 4b29ec95d88d7cb99a7c55f0d3297a3b8c2149c4)

audio: Initial support for amplifier HALs

Change-Id: I04ef5b89ca595a0df02f6a4a68748bd0d0cdb245
(cherry picked from commit 7a82a0e691d884045c78d1cb9f7ab0dd51a5d871)

libril: Fix multisim flags

Change-Id: Idb2ea518c115c1dbfbbbbb79094938ce006f40d8
(cherry picked from commit c98cc605c67e950bf14aa9f0d14966382ec45a0c)

AdvancedDisplay: Restore back button to the action bar

Change-Id: I7ae2d1ccbb0a55fc28c31d4791aaae1169b8221f

AdvancedDisplay: Re-integrate into Settings

 * Apply the default Oreo theme and inherit the layouts from Google
   for the Settings app, in order to keep UI consistency.

 * Get rid of SettingsDrawerActivity as it no longer fits in.

 * Add required appcompat and preference support libraries.

 * Integrate into display settings category using IA.

Change-Id: I8438e0ebec27085df2be4668f8f7170d2010fb20

AdvancedDisplay: cyanogenmod -> lineageos

Change-Id: Ie6d64f0995f266d2dbe3e16c8032820cc5164c1f

dtbhtool: Use right local module class to avoid relocation packaging

* UTILITY_EXECUTABLES is not a thing anymore, but EXECUTABLES are
  not packed

Change-Id: I1b27c4e23c08579f478af0756e86263e9ab4d740

dtbhtool: Define both host and target libdtbimg

* The utilities need it and not defining the target library results
  in a missing dependency

Change-Id: I745d8586a4a2cc5e3b8bdbe4a7e9dc19e1161db6

dtbhtool: Use BoringSSL instead of mincrypt

* mincrypt is gone as of LineageOS/android_system_core@6b11c13ae

Change-Id: Ia3b1dee747510daff45a345ae603181be5ab6023

macloader: Stop allowing G and O write perms to the cidfile

* Only macloader should ever write to this, but others (e.g,
  libbt-vendor.so) read from it. 0644 is more than enough.

Change-Id: I3d2fb3dfec1ba81c9c382ed18c93114e641df4a0

macloader: Move device dependent modules to /vendor

Change-Id: I184ec79c8fc4d6047146b84bcc935aef129fc7f4

fingerprint: bauth: Bump version to 2.1

* Required for fingerprint HIDL HAL in O.

Change-Id: Iec8450521a51bbfddf2225274a7a6982baa42561

consumerir: Move device dependent modules to /vendor

Change-Id: Ia05938d479babea4e0d7d25d52c818a43cedf362

liblights: Move device dependent modules to /vendor

Change-Id: Ia7534e9188048b95e6c85497cfeb5e28a279629b

samsung: hardware: Rebrand from cmhw to lineagehw

Change-Id: I9576ca2d6bf5dfb9d0259e314f20aa0fe7f775b5

dtbhtool: Use libfdt from external/dtc

Change-Id: Ia3e38120965125de684e39213f94ad53238d55ce

dtbToolCM -> dtbToolLineage

Change-Id: I515ce4855cbaa2157a25dac56f18a6b1ca60ec84

wifiloader: Load kernel module if present

Samsung does this in the macloader service,
but this is the more appropriate place for
our platform.

Change-Id: I6335fea1d73d48f3c3f9938f33b536c5cda52acb

Automatic translation import

Change-Id: Iac6ca07164f94ff319af48ecf8cae29a76052c02

secril-client: close fds on I/O errors

if a kernel device returns an I/O error, it is better not to chew
the CPU resources retrying I/O operation on a not ready device.

Fixes periodical 100%-core load in audioserver after a voice call
completion.

Change-Id: I2b66ffd18c85823a11e50598a6b4e3c0245a0083

power: Keep boost fd opened as well

Change-Id: I016b28c29786fc948a007d4ec3106be8a662eafd

macloader: Drop semcove case and enum

The all_ranges structure has one entry less than MAX_TYPES because
semcove type is not defined there, but there's no check for NULL and if
the mac isn't found it segfaults.

Change-Id: Ie218ea0a82b03992085db0f2d1cfc7079e7415a0

secril-client: Fix RIL client token management in SendOemRequestHookRaw()

On some devices, like SM-T805 (Galaxy Tab S 10.5 LTE), the RIL client
code produces errors "SendOemRequestHookRaw: No token" due to large
amount of tokens allocated by OEM messages requiring no response from
RIL daemon (like set call audio path, set call volume, set mite etc).

The indicator of an OEM message with no response expected is:

     RegisterRequestCompleteHandler(client, REQ_XXX, NULL);

inside the RIL client message method. To prevent the token pool from
overflows, the proposed patch de-registers the token and frees its call
history if the handler for this message req_id is set to NULL.

Change-Id: Id414263bf471115797cae1a9ed628249734b9347

secril-client: return exact error code from blockingWrite()

Change-Id: Ibb6d3636648480c53a2fd19044eefe5b97f1eceb

Revert "power: Temporary logging fix"

This reverts commit ff86846e97f2d159cc98af80938246db0d53c39a.

Change-Id: I46aa4d6606c3c519322b2e2cd048bd312a15fa4d

power: Allow for a dynamic cluster configuration

Change-Id: I542b36d26cad23f29e70ab70d996c95ca3ab11a7

power: Temporary logging fix

This will be reverted once the more advanced mechanism
(https://review.lineageos.org/#/c/179092/) goes in.

The boost fd logging will be restored once
https://review.lineageos.org/#/c/180313/ makes it into
the code base.

Change-Id: Ifc1b4a2ba3e1c4283d74eafec45c5fbf07153f03

power: Print information on boot

Change-Id: I1a8ba54f1b27c2e96c6bae77046dc207cdf35025

power: sysfs_read: Do not store newlines

Change-Id: If20b509b489b056ac3b37c208c42bc5407f61802

AdvancedDisplay: Refactor

* Integrate with Settings
* Remove back button

Change-Id: Ie3bcd43ef7225bdc3b177446906335f59561b584

Automatic translation import

Change-Id: I352dfea4f441b2b1a7d14b4e665f1fe77eecc99c

exynos: multimedia: add support for OMX_IndexParamVideoIntraRefresh parameter

filiprrs: This fixes miracast, and possibly resolves issues with features
like chromecast, screen recording, wifi display etc.

Change-Id: I35f789a9ae29df0198b21c98dc866d8886799893

liblights: Fix fast blinking notification LEDs.

Seems that the fast blinking notification LEDs experienced on some
Samsung devices was due to the removal of these two lines that clean
the 'blink' array. Propose re-inserting to correct the fast blinking
LED issue.

Change-Id: I88ea828a50468aae494b1dd54290607308de543a

liblights: Add BLN support

Change-Id: I3745b7c4ca7adadd62e116c71cd7357a4e8ed015

liblights: Fix variable button brightness code

* Because we're using #ifdef in set_light_buttons(),
  defining VAR_BUTTON_BRIGHTNESS to anything, including 0,
  results in the button brightness code being enabled
* That results in broken touchkey backlight for devices
  that don't override the samsung_lights header, or defined
  this macro to 0 in their device's samsung_lights header
* Comment out the macro to correct this logic

Change-Id: Icf554b0deac975f729f1a70e517341dae74f3657

Automatic translation import

Change-Id: I376a4f179af4bd395ca9c17da93f5101aca4c68c

liblights: Add support for variable touchkey brightness

Change-Id: Icc5c589e88dc342ae50b432187bae65a9b3a23d7

power: Write IO busy parameter for big cluster

Change-Id: I9e09f640b956be87729df735225fcfe8f066ab48

modemloader: Add support for recent devices

 * Try to read the hardware revision from the
   bootloader before parsing /proc/cpuinfo
 * Set more properties to catch all recent device
   needs as well
 * Introduce a property to let other services (cbd)
   know they can start

Change-Id: I50d8ec37921ec2559bdd7fe852d830f10fe1c12e

Automatic translation import

Change-Id: I15bc33336962fd84fcb328d5fda461e5ce0bd495

power: Allow POWER_HINT_LOW_POWER hint to disable power save mode

Change-Id: I3b4c90f05267ca4182187361e5d373adb6de25d7

Automatic translation import

Change-Id: I7cdf0153bcb66bf25d80c694022018b5d89287e2

AdvancedDisplay - Rename default scenario

Change-Id: I6122234ef08b78958e28d3a127b3c73757a43445

mkbootimg: add exynos subdir to possible_dtb_dirs

Change-Id: I70e5e0817ff0b63da7df3ab63f42b815170096e4

power: Sanity check touchscreen path

* Let's not always assume this isn't NULL

Change-Id: I181cb014fc06d0f0ca795c3ecc374ccf9dbb57f1
Signed-off-by: Paul Keith <javelinanddart@bestas.gr>

Automatic translation import

Change-Id: I201d7c8ce8c9495085e68a76e14aa8d67caa31de

power: Align to AOSP API

AOSP power hints can have a NULL data parameter,
only our custom hints are guaranteed to provide data.

Change-Id: Ibea874e75a7092b3f9c1c56dc5182cdf73f35532

power: Remove unused variables

Change-Id: I805a12837bd013e71c098479279dcc5895ea1e1d

lights: Put back fd checks before closing

While close(NULL) is indeed a harmless noop, fd can
end up being < 0 for us.

Change-Id: I56dcd7fb61c72d3ce750b13329ff42e11ab63c84

lights_helper: I/O API cleanup

 * buf => buffer for consistency
 * explicitly cast strtol retval to int
 * close(NULL) is a no-op, remove superfluous checking
 * replace variying buffer sizes by INT_MAX_STRLEN
 * always return signed errno codes

Change-Id: I972e33e2435ec7d68a8ffdc5ab88a14d8005ff55

audience: Cleanup write_int()

Change-Id: Idfec2093c63a864d5f9d37353adbf6f8fc5d4fcd

audience: Use O_WRONLY flag for write_int()

Change-Id: I9a0bdc78644e7a4875d7e49051722fff3d47ee6b

audience: Always log write_int() fails

Change-Id: I747c38646599ae3f687279ca8a95db804f3ad015

audience: Warn when we cannot handle the output device

Change-Id: I0cddd94f22c5e912436682d713a1ae441723cfee

power: Add POWER_HINT_DISABLE_TOUCH support

Change-Id: I363df278598e3402692424539638f3547c648e26

Automatic translation import

Change-Id: I58d68d3bc81f39d0b452a9aa1f408a0e6830de63

PowerHAL: Update the hispeed_freq value before using it

The hispeed_freq value is read only in the inizialization process.
If we decide to change it later writing a new value to the node
this new value wont be taken into consideration by the power hal.
Lets update the hispeed_freq value in the power hal before using
it to limit the max cpu freq when we go into power save mode.

Change-Id: I4b8c7f67259750b2931b18c3871874022aa6054d

audio: Split BT SCO media/incall paths

Change-Id: I23a621855e8b96edcd92d8740dfec270ff7e7407

audio: add support for bluetooth wideband voice calls

Change-Id: I02bf23a83c52389cc02ba1f61572eb651418fe24

power: Rework governor path abstraction

Construct the paths for the driver based on the device
specific header data.

Change-Id: Ic7c72a2bbe6f199e1297f41817840059595f516d

Automatic translation import

Change-Id: Ib6bc0641aa25b3f4071b3ad2e57a5c084d418916

power: Cleanup hint handling

Change-Id: I828a68105272fef3fe454561d98825f8703bca0b

power: Discard hints in power save mode

Change-Id: I9c3701261e49023761599ebaa8cd3e09a88a40c6

power: Keep boostpulse fd opened

No one else should be writing to this, so keep the fd
opened.

Change-Id: I4258f7f7d69250f4832a67807031ab3faa9ff586

power: Keep boostpulse fd opened

No one else should be writing to this, so keep the fd
opened.

Change-Id: I4258f7f7d69250f4832a67807031ab3faa9ff586

liblights: Always apply linear scaling if max brightness isn't 255

* As of now the code account only max_brightness > 255
  -> Check if max_brightness != 255 and apply the scaling
     even if the max_brightness is less then 255

Change-Id: I809ece5844c8ccf20ae6507f23d308fe3ddfe94b

Automatic translation import

Change-Id: I27a0fd483b301f46556d5e37819ee2bdcf346046

power: Initialize device power paths

Change-Id: I56481946e2e3f4a80f2d3a7713ce61ed7923666d

power: Cleanup touchkey handling

 * Simplify interactive changes
 * Better support for non-touchkey devices
 * Only perform read I/O in samsung_power_set_interactive
   if needed (improves wake->panel_on delay)

Change-Id: If7bd6a1833ac7605db437ef3664230f9a452d58b

lights: Add a log tag

Change-Id: I7f4643324ed5c52b60ceede5f1ad8f24131cf3c0