* if (usecase == NULL) should actually be if (vc_usecase == NULL)
* Avoid unnecessary if-block indentation by updating the control
flow
Change-Id: If00ac405a1eac429862b953580e8614227015134
This was intentionally left enabled and does not spam anyways,
but allows users to extract helpful information without
recompiling.
This reverts commit e06577dc81.
Change-Id: I604ee674a1db466b3d3e364cb95f5688edd4947c
* abs() accepts ints as parameters,
but this breaks compile since
the times are longs, not ints
* Use labs() instead, which accepts
longs as parameters, to fix compile
Change-Id: I8f980a78380cdae18abd5b9602e281beae3ee4d3
Signed-off-by: Paul Keith <javelinanddart@aidenswann.com>
We have a value for SND_DEVICE_NONE so we should use that. There is no
field for SND_DEVICE_MIN so we need to check it is bigger.
Change-Id: I041930c8c4b552d45804a178aeb1aa981ae7c13f
* Instead of messing around with indices, look up
the requestNumber in the array.
* This has a cost of O(N) instead of O(1) with the
previous implementation, but we don't receive unsol
response codes frequently enough to be worried about
this.
* This was needed because a few vendor reponses, aka
RIL_UNSOL_SNDMGR_WB_AMR_REPORT at index 33 and
RIL_UNSOL_SNDMGR_CLOCK_CTRL at index 34
could not be addressed by their array indices anymore
because we cannot calculate their index by the unsol
response code we receive from the modem.
Change-Id: I27319e621c777fe19ae8908d7e0c4a46d6dd6d3b
When a voice call is stopped we switch back to the default primary
output (speaker). Then this function gets executed and as ther was
no active PCM because voice_session is handling that it segfaults
because the PCM is NULL.
Change-Id: I927504b7962b096c0d4c3642b48aee55c85ec013
The unused functions have just been commented out.
This also sets the CFLAGS to -Wall and -Werror.
Change-Id: Iea097426313a68a01a074627115853fccc680579
We want that to avoid an error message in the log, so revert the
revomal.
This reverts commit cdad20da45.
Change-Id: Ic90cd6b02e5b3e8aae73d824153c3084643a8acc
This just allocates the structre and creates the RIL connection. This is
per audio_device structure to make sure we to not run into issues with
audioserver threading.
Change-Id: I42e1b7ae57579f39c5f76566ef5b67d4e2c13e3f
* Don't return in the middle of a function, set the
return code and jump to the exit point instead.
* Close file pointers properly.
* Unify error code returns (macloader will always exit
with -1 on error now).
Change-Id: I32ac8ea85fcbabed45550a4d176ac999eff9f9e9
* This also causes NVMRAM calibration to be on by default
* Most devices should be using this, so don't make them override
Change-Id: Ied6e8f4ebd131ae992814bf83c9e7bb379ca9d79
Signed-off-by: Paul Keith <javelinanddart@aidenswann.com>
* Samsung uses fingerprint enrollment % completed
while AOSP uses fingerprint sample remaining.
* This patch converts this data to AOSP-friendly one via (x = 100 - x).
Change-Id: I27034cb148e0f33f7d3efae9c75123715452f178
Signed-off-by: Jesse Chan <jc@linux.com>
* Adjusted whitespace to minimize diffs
* Plugged memory leaks
* Fix use-after-free
* Use calloc instead of alloca
* Reworked the Samsung request/response code to be
more readable
Change-Id: Ifd07b525fc84e8a3fca584d4d412785974ca3482
* These should be lowercase, otherwise detection will fail
Change-Id: Ibdc446a8f9c58bc9b17b8e9c6cfd5f4894f7ecff
Signed-off-by: Paul Keith <javelinanddart@aidenswann.com>
Christopher N. Hesse