audio/transport: Propagate errors from avrcp_set_volume to DBus
authorMarijn Suijten <marijn.suijten@somainline.org>
Sat, 11 Mar 2023 01:12:02 +0000 (02:12 +0100)
committerAyush Garg <ayush.garg@samsung.com>
Fri, 5 Jan 2024 10:41:34 +0000 (16:11 +0530)
commit09ec719abcc3697160fc2fcc2e5b560a5f8dd5b4
tree7dda3844a6a67c41a1e2fca4257a2e030bd0205d
parent18c89c768c38301055944627eca2fb9be80eb18e
audio/transport: Propagate errors from avrcp_set_volume to DBus

Any error while setting absolute volume on the peer, or notifying the
peer of changes was previously going completely unnoticed.  Propagate it
to the logs and back to the DBus "Volume" property setter so that they
aren't misled into thinking that the AVRCP command succeeded.

Note that an error is mostly harmless when the setter of the property is
an audio sink and the peer the audio source: in this case we're only
_notifying_ the peer of the change when it has already been applied on
the sink.

Also improve the other two error messages to more closely describe what
was "invalid" about the argument.
profiles/audio/transport.c