[PATCH BlueZ] audio/media: Fix build with --disable-hfp

[PATCH BlueZ] audio/media: Fix build with --disable-hfp

[Jacob Siverskog]

The HFP MediaEndpoint support calls into hfp-hf.c, which is only built
when HFP is enabled, so linking bluetoothd fails with --disable-hfp:

  profiles/audio/media.c:242: undefined reference to `hfp_hf_sco_remove'
  profiles/audio/media.c:1467: undefined reference to `hfp_hf_get_device'

Guard the HFP code with HAVE_HFP, as is already done for the endpoint
table entry and in transport.c.

Fixes: 89b91a4806fb ("audio/hfp-hf: Add MediaEndpoint for HFP codecs")
Assisted-by: Claude:claude-opus-5-5
---
 profiles/audio/media.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/profiles/audio/media.c b/profiles/audio/media.c
index 34ff251c7..abd3cb992 100644
--- a/profiles/audio/media.c
+++ b/profiles/audio/media.c
@@ -238,10 +238,12 @@ static void media_endpoint_destroy(struct media_endpoint *endpoint)
 		endpoint->pac = NULL;
 	}
 
+#ifdef HAVE_HFP
 	if (endpoint->sco) {
 		hfp_hf_sco_remove(endpoint->adapter->btd_adapter, endpoint);
 		endpoint->sco = false;
 	}
+#endif
 
 	g_dbus_remove_watch(btd_get_dbus_connection(), endpoint->watch);
 	g_free(endpoint->capabilities);
@@ -1457,6 +1459,7 @@ static bool endpoint_init_asha(struct media_endpoint *endpoint,
 	return true;
 }
 
+#ifdef HAVE_HFP
 gboolean hfp_hf_set_configuration(struct media_endpoint *endpoint,
 					const char *ep_path,
 					media_endpoint_cb_t cb,
@@ -1514,6 +1517,7 @@ static bool endpoint_init_sco(struct media_endpoint *endpoint,
 
 	return hfp_hf_sco_listen(endpoint->adapter->btd_adapter, endpoint);
 }
+#endif /* HAVE_HFP */
 
 static bool endpoint_properties_exists(const char *uuid,
 						struct btd_device *dev,
-- 
2.53.0

RE: [BlueZ] audio/media: Fix build with --disable-hfp

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 1021 bytes --]

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/series/1196992/

---Test result---

Test Summary:
CheckPatch                    PASS      0.41 seconds
GitLint                       PASS      0.28 seconds
BuildEll                      PASS      21.04 seconds
BluezMake                     PASS      307.87 seconds
MakeCheck                     PASS      2.94 seconds
MakeDistcheck                 PASS      146.23 seconds
CheckValgrind                 PASS      157.22 seconds
CheckSmatch                   PASS      297.61 seconds
bluezmakeextell               PASS      98.87 seconds
TestFunctional                PASS      1161.34 seconds
IncrementalBuild              PASS      310.90 seconds
ScanBuild                     PASS      1172.54 seconds



https://github.com/bluez/bluez/pull/2736

---
Regards,
Linux Bluetooth