From: Myrrh Periwinkle <noreply@github.com>
To: linux-bluetooth@vger.kernel.org
Subject: [bluez/bluez] 18c07d: audio: Don't initialize media transport volume fro...
Date: Tue, 05 Aug 2025 18:17:10 -0700 [thread overview]
Message-ID: <bluez/bluez/push/refs/heads/988588/000000-94ccc2@github.com> (raw)
Branch: refs/heads/988588
Home: https://github.com/bluez/bluez
Commit: 18c07de44a6b8c2bda66a0dcc455220df792e268
https://github.com/bluez/bluez/commit/18c07de44a6b8c2bda66a0dcc455220df792e268
Author: Myrrh Periwinkle <myrrhperiwinkle@qtmlabs.xyz>
Date: 2025-08-06 (Wed, 06 Aug 2025)
Changed paths:
M profiles/audio/avrcp.c
M profiles/audio/media.c
M profiles/audio/media.h
Log Message:
-----------
audio: Don't initialize media transport volume from media player
Media player objects may be shared between devices. As a result,
a device without support for hardware volume that is connected after one
that does may end up being erroneously considered hardware
volume-capable.
fa7828bdd ("transport: Fix not being able to initialize volume properly")
introduced btd_device_{get,set}_volume that is used as an alternative in
case no media player objects are present. Therefore, we can remove
media_player_get_device_volume and instead use btd_device_get_volume to
determine the initial volume.
Commit: 94ccc2ab32e46a7ab190dfbc4274e2ca8d0beef3
https://github.com/bluez/bluez/commit/94ccc2ab32e46a7ab190dfbc4274e2ca8d0beef3
Author: Myrrh Periwinkle <myrrhperiwinkle@qtmlabs.xyz>
Date: 2025-08-06 (Wed, 06 Aug 2025)
Changed paths:
M profiles/audio/avrcp.c
M profiles/audio/avrcp.h
M profiles/audio/media.c
Log Message:
-----------
audio: Remove media_player.volume
With the previous commit, the media_player.volume field is now
completely unused. As this was never the right place to store the
initial volume to begin with, remove this field entirely as well as its
associated supporting methods to prevent confusion.
Compare: https://github.com/bluez/bluez/compare/18c07de44a6b%5E...94ccc2ab32e4
To unsubscribe from these emails, change your notification settings at https://github.com/bluez/bluez/settings/notifications
reply other threads:[~2025-08-06 1:17 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bluez/bluez/push/refs/heads/988588/000000-94ccc2@github.com \
--to=noreply@github.com \
--cc=linux-bluetooth@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.