Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f6090107 authored by Chris Phoenix's avatar Chris Phoenix Committed by Yifan Hong
Browse files

BroadcastRadio HAL uses "default" service name 

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: I49622132d25c15597f46c19330a9d0b8dbffa704
parent d94a7d2d

audio/2.0/default/service.cpp

+1 −1
Original line number Diff line number Diff line
@@ -45,7 +45,7 @@ int main(int /* argc */, char* /* argv */ []) { 
    // Soundtrigger and FM radio might be not present.

    status = registerPassthroughServiceImplementation<ISoundTriggerHw>("sound_trigger.primary");

    ALOGE_IF(status != OK, "Error while registering soundtrigger service: %d", status);

    status = registerPassthroughServiceImplementation<IBroadcastRadioFactory>("broadcastradio");

    status = registerPassthroughServiceImplementation<IBroadcastRadioFactory>();

    ALOGE_IF(status != OK, "Error while registering fm radio service: %d", status);

    joinRpcThreadpool();

    return status;

broadcastradio/1.0/vts/functional/VtsHalBroadcastradioV1_0TargetTest.cpp

+1 −1
Original line number Diff line number Diff line
@@ -63,7 +63,7 @@ class BroadcastRadioHidlTest : public ::testing::Test { 
            }

        }

        sp<IBroadcastRadioFactory> factory =

              IBroadcastRadioFactory::getService("broadcastradio", getStub);

              IBroadcastRadioFactory::getService(getStub);

        if (factory != 0) {

            factory->connectModule(Class::AM_FM,

                             [&](Result retval, const ::android::sp<IBroadcastRadio>& result) {