Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Address Broadcast Radio HAL review notes." into oc-mr1-dev

Browse Source
This commit is contained in:
Tomasz Wasilczyk
2017-08-03 16:52:31 +00:00
committed by Android (Google) Code Review
parent 22b418448b 8316291cda
commit 36ea87b075
5 changed files with 14 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/java/android/hardware/radio/ITunerCallback.aidl
View File

@@ -23,7 +23,7 @@ import android.hardware.radio.RadioMetadata;
oneway interface ITunerCallback {
void onError(int status);
void onConfigurationChanged(in RadioManager.BandConfig config);
void onProgramInfoChanged();
void onCurrentProgramInfoChanged();
void onTrafficAnnouncement(boolean active);
void onEmergencyAnnouncement(boolean active);
void onAntennaState(boolean connected);

4
core/java/android/hardware/radio/TunerCallbackAdapter.java
View File

@@ -49,7 +49,7 @@ class TunerCallbackAdapter extends ITunerCallback.Stub {
synchronized (mLock) {
if (mTuner != null) throw new IllegalStateException();
mTuner = tuner;
if (mPendingProgramInfoChanged) onProgramInfoChanged();
if (mPendingProgramInfoChanged) onCurrentProgramInfoChanged();
}
}
@@ -64,7 +64,7 @@ class TunerCallbackAdapter extends ITunerCallback.Stub {
}
@Override
public void onProgramInfoChanged() {
public void onCurrentProgramInfoChanged() {
synchronized (mLock) {
if (mTuner == null) {
mPendingProgramInfoChanged = true;

4
services/core/java/com/android/server/broadcastradio/TunerCallback.java
View File

@@ -86,8 +86,8 @@ class TunerCallback implements ITunerCallback {
}
@Override
public void onProgramInfoChanged() {
dispatch(() -> mClientCallback.onProgramInfoChanged());
public void onCurrentProgramInfoChanged() {
dispatch(() -> mClientCallback.onCurrentProgramInfoChanged());
}
@Override

2
services/core/jni/BroadcastRadio/Tuner.cpp
View File

@@ -296,7 +296,7 @@ static void nativeTune(JNIEnv *env, jobject obj, jlong nativeContext, jobject jS
auto selector = convert::ProgramSelectorToHal(env, jSelector);
if (halTuner11 != nullptr) {
convert::ThrowIfFailed(env, halTuner11->tune_1_1(selector));
convert::ThrowIfFailed(env, halTuner11->tuneByProgramSelector(selector));
} else {
uint32_t channel, subChannel;
if (!V1_1::utils::getLegacyChannel(selector, &channel, &subChannel)) {

16
services/core/jni/BroadcastRadio/TunerCallback.cpp
View File

@@ -58,7 +58,7 @@ static struct {
jmethodID handleHwFailure;
jmethodID onError;
jmethodID onConfigurationChanged;
jmethodID onProgramInfoChanged;
jmethodID onCurrentProgramInfoChanged;
jmethodID onTrafficAnnouncement;
jmethodID onEmergencyAnnouncement;
jmethodID onAntennaState;
@@ -111,7 +111,7 @@ public:
virtual Return<void> backgroundScanAvailable(bool isAvailable);
virtual Return<void> backgroundScanComplete(ProgramListResult result);
virtual Return<void> programListChanged();
virtual Return<void> programInfoChanged();
virtual Return<void> currentProgramInfoChanged();
};
struct TunerCallbackContext {
@@ -192,7 +192,7 @@ Return<void> NativeCallback::tuneComplete_1_1(Result result, const ProgramSelect
mCallbackThread.enqueue([result, this](JNIEnv *env) {
if (result == Result::OK) {
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onProgramInfoChanged);
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onCurrentProgramInfoChanged);
} else {
TunerError cause = TunerError::CANCELLED;
if (result == Result::TIMEOUT) cause = TunerError::SCAN_TIMEOUT;
@@ -254,7 +254,7 @@ Return<void> NativeCallback::newMetadata(uint32_t channel, uint32_t subChannel,
}
mCallbackThread.enqueue([this, metadata](JNIEnv *env) {
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onProgramInfoChanged);
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onCurrentProgramInfoChanged);
});
return Return<void>();
@@ -297,11 +297,11 @@ Return<void> NativeCallback::programListChanged() {
return Return<void>();
}
Return<void> NativeCallback::programInfoChanged() {
Return<void> NativeCallback::currentProgramInfoChanged() {
ALOGV("%s", __func__);
mCallbackThread.enqueue([this](JNIEnv *env) {
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onProgramInfoChanged);
env->CallVoidMethod(mJCallback, gjni.TunerCallback.onCurrentProgramInfoChanged);
});
return Return<void>();
@@ -379,8 +379,8 @@ void register_android_server_broadcastradio_TunerCallback(JavaVM *vm, JNIEnv *en
gjni.TunerCallback.onError = GetMethodIDOrDie(env, tunerCbClass, "onError", "(I)V");
gjni.TunerCallback.onConfigurationChanged = GetMethodIDOrDie(env, tunerCbClass,
"onConfigurationChanged", "(Landroid/hardware/radio/RadioManager$BandConfig;)V");
gjni.TunerCallback.onProgramInfoChanged = GetMethodIDOrDie(env, tunerCbClass,
"onProgramInfoChanged", "()V");
gjni.TunerCallback.onCurrentProgramInfoChanged = GetMethodIDOrDie(env, tunerCbClass,
"onCurrentProgramInfoChanged", "()V");
gjni.TunerCallback.onTrafficAnnouncement = GetMethodIDOrDie(env, tunerCbClass,
"onTrafficAnnouncement", "(Z)V");
gjni.TunerCallback.onEmergencyAnnouncement = GetMethodIDOrDie(env, tunerCbClass,