From e6ea01046ee17d542fd26cb0b75d5d2f401ab75d Mon Sep 17 00:00:00 2001
From: Wim Taymans <wtaymans@redhat.com>
Date: Tue, 9 Feb 2021 10:57:16 +0100
Subject: [PATCH] media-session: add some logging to bluez volume handling

See #699
---
 src/examples/media-session/bluez-monitor.c | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/examples/media-session/bluez-monitor.c b/src/examples/media-session/bluez-monitor.c
index 20ca6ad82..28ed94b3d 100644
--- a/src/examples/media-session/bluez-monitor.c
+++ b/src/examples/media-session/bluez-monitor.c
@@ -278,8 +278,10 @@ static void bluez_device_event(void *data, const struct spa_event *event)
 			SPA_EVENT_DEVICE_Props, SPA_POD_OPT_Pod(&props)) < 0)
 		return;
 
-	if ((node = bluez5_find_node(device, id)) == NULL)
+	if ((node = bluez5_find_node(device, id)) == NULL) {
+		pw_log_warn("device %p: unknown node %d", device, id);
 		return;
+	}
 
 	switch (type) {
 	case SPA_DEVICE_EVENT_ObjectConfig:
@@ -288,9 +290,15 @@ static void bluez_device_event(void *data, const struct spa_event *event)
 		 * pw_client_node_get_node() and perform the set_param on
 		 * that node proxy instead of waiting for the session manager
 		 * proxy. */
-		if (props != NULL && node->snode->obj.proxy != NULL)
-			pw_node_set_param((struct pw_node*)node->snode->obj.proxy,
-				SPA_PARAM_Props, 0, props);
+		if (props != NULL) {
+			if (node->snode->obj.proxy != NULL) {
+				pw_node_set_param((struct pw_node*)node->snode->obj.proxy,
+					SPA_PARAM_Props, 0, props);
+			} else {
+				pw_log_warn("device %p: node %d not ready for volume yet",
+						device, id);
+			}
+		}
 		break;
 	default:
 		break;