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

Commit 0d3637a0 authored by vivek mehta's avatar vivek mehta Committed by Eric Laurent
Browse files

hal: audio: on routing command check first voice usecase 

- when routing command is received since primary output selects device
  for non voice usecase first and then selects device for voice
  usecase this adds one unnecessary device switch and additional latency
  in case of voice call setup latency.
- modify device selection logic by starting with voice usecase first
  if audio mode is in call.

Bug: 30270281

Change-Id: I2b9bb383b525793158dd6f8d2402fa73ab4c53ef
parent 3190ba36

hal/audio_hw.c

+9 −8
Original line number Diff line number Diff line
@@ -1777,14 +1777,6 @@ static int out_set_parameters(struct audio_stream *stream, const char *kvpairs) 
            bool same_dev = out->devices == new_dev;

            out->devices = new_dev;



            if (!out->standby) {

                if (!same_dev) {

                    ALOGV("update routing change");

                    out->routing_change = true;

                }

                select_devices(adev, out->usecase);

            }



            if (output_drives_call(adev, out)) {

                if (!voice_is_in_call(adev)) {

                    if (adev->mode == AUDIO_MODE_IN_CALL) {
@@ -1796,6 +1788,15 @@ static int out_set_parameters(struct audio_stream *stream, const char *kvpairs) 
                    voice_update_devices_for_all_voice_usecases(adev);

                }

            }



            if (!out->standby) {

                if (!same_dev) {

                    ALOGV("update routing change");

                    out->routing_change = true;

                }

                select_devices(adev, out->usecase);

            }



        }



        pthread_mutex_unlock(&adev->lock);