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

Commit df1a8ec0 authored by Jack Yu's avatar Jack Yu
Browse files

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: Ieb2322113557be700aeae78346c969465e7a9dd3
parent 77c3822d

radio/1.0/IRadio.hal

+0 −14
Original line number Diff line number Diff line
@@ -730,20 +730,6 @@ interface IRadio { 
     */

    oneway getDataCallList(int32_t serial);



    /*

     * Indicates the current state of the screen. When the screen is off, the

     * Radio must notify the baseband to suppress certain notifications (eg,

     * signal strength and changes in LAC/CID or BID/SID/NID/latitude/longitude)

     * in an effort to conserve power. These notifications must resume when the

     * screen is on.

     *

     * @param serial Serial number of request.

     * @param enable true = screen on, false = screen off.

     *

     * Response function is IRadioResponse.sendScreenStateResponse()

     */

    oneway sendScreenState(int32_t serial, bool enable);



    /*

     * Enables/disables supplementary service related notifications from the network.

     * Notifications are reported via unsolSuppSvcNotification().

radio/1.0/IRadioResponse.hal

+0 −11
Original line number Diff line number Diff line
@@ -921,17 +921,6 @@ interface IRadioResponse { 
     */

    oneway getDataCallListResponse(RadioResponseInfo info, vec<SetupDataCallResult> dcResponse);



    /*

     * @param info Response info struct containing response type, serial no. and error

     *

     * Valid errors returned:

     *   RadioError:NONE

     *   RadioError:RADIO_NOT_AVAILABLE

     *   RadioError:INVALID_ARGUMENTS

     *   RadioError:GENERIC_FAILURE

     */

    oneway sendScreenStateResponse(RadioResponseInfo info);



    /*

     * @param info Response info struct containing response type, serial no. and error

     *

radio/1.0/vts/functional/radio_hidl_hal_utils.h

+0 −2
Original line number Diff line number Diff line
@@ -234,8 +234,6 @@ public: 
    Return<void> sendOemRilRequestStringsResponse(const RadioResponseInfo& info,

            const ::android::hardware::hidl_vec<::android::hardware::hidl_string>& data);



    Return<void> sendScreenStateResponse(const RadioResponseInfo& info);



    Return<void> setSuppServiceNotificationsResponse(

            const RadioResponseInfo& info);

radio/1.0/vts/functional/radio_response.cpp

+0 −5
Original line number Diff line number Diff line
@@ -314,11 +314,6 @@ Return<void> RadioResponse::sendOemRilRequestStringsResponse( 
    return Void();

}



Return<void> RadioResponse::sendScreenStateResponse(

        const RadioResponseInfo& /*info*/) {

    return Void();

}



Return<void> RadioResponse::setSuppServiceNotificationsResponse(

        const RadioResponseInfo& /*info*/) {

    return Void();