From ad756056a4052e6eb128072d7852c0dbf0c0ed59 Mon Sep 17 00:00:00 2001 From: Tai Vuong Date: Fri, 22 Dec 2017 15:08:51 -0500 Subject: SPEC-1207 Several improvement on AHL error messages, this required a lot of changes in the code, please review carefully and decide if you want to include now or after CES Change-Id: If1aacecdd2aa9afd3ce4314c3b6334f9334faff4 Signed-off-by: Tai Vuong --- ahl-binding/ahl-deviceenum.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'ahl-binding/ahl-deviceenum.c') diff --git a/ahl-binding/ahl-deviceenum.c b/ahl-binding/ahl-deviceenum.c index 43e5af5..75b0715 100644 --- a/ahl-binding/ahl-deviceenum.c +++ b/ahl-binding/ahl-deviceenum.c @@ -237,6 +237,7 @@ void TermEndpointInfo( gpointer data ) int EnumerateDevices(json_object * in_jDeviceArray, char * in_pAudioRole, EndpointTypeT in_deviceType, GPtrArray * out_pEndpointArray) { g_assert_nonnull(in_jDeviceArray); + g_assert_nonnull(in_pAudioRole); int iNumberDevices = json_object_array_length(in_jDeviceArray); // Parse and validate list of available devices @@ -249,7 +250,7 @@ int EnumerateDevices(json_object * in_jDeviceArray, char * in_pAudioRole, Endpoi json_object * jDevice = json_object_array_get_idx(in_jDeviceArray,i); if (jDevice == NULL) { - AFB_WARNING("Invalid device array -> %s",json_object_to_json_string(in_jDeviceArray)); + AFB_WARNING("Invalid device array for audiorole:%s -> %s",in_pAudioRole,json_object_to_json_string(in_jDeviceArray)); continue; } // strip domain name from URI @@ -258,7 +259,7 @@ int EnumerateDevices(json_object * in_jDeviceArray, char * in_pAudioRole, Endpoi err = SeparateDomainFromDeviceURI(pFullDeviceURICopy,&pDeviceURIDomain,&pDeviceURIPCM); if (err) { - AFB_WARNING("Invalid device URI string -> %s",pFullDeviceURICopy); + AFB_WARNING("Invalid device URI string -> %s for audiorole:%s",pFullDeviceURICopy,in_pAudioRole); continue; } @@ -289,13 +290,13 @@ int EnumerateDevices(json_object * in_jDeviceArray, char * in_pAudioRole, Endpoi err = snd_pcm_open(&pPcmHandle, pEndpointInfo->gsDeviceURI, streamType, 0); if (err < 0) { - AFB_NOTICE("Alsa PCM device was not found -> %s", pEndpointInfo->gsDeviceURI); + AFB_NOTICE("Alsa PCM device was not found -> %s for audiorole:%s", pEndpointInfo->gsDeviceURI,in_pAudioRole); continue; } err = FillALSAPCMInfo(pPcmHandle,pEndpointInfo); if (err) { - AFB_WARNING("Unable to retrieve PCM information for PCM -> %s",pEndpointInfo->gsDeviceURI); + AFB_WARNING("Unable to retrieve PCM information for PCM -> %s for audiorole:%s",pEndpointInfo->gsDeviceURI,in_pAudioRole); snd_pcm_close(pPcmHandle); continue; } @@ -318,7 +319,7 @@ int EnumerateDevices(json_object * in_jDeviceArray, char * in_pAudioRole, Endpoi } else { // Unknown domain - AFB_WARNING("Unknown domain in device URI string -> %s",pFullDeviceURI); + AFB_WARNING("Unknown domain in device URI string -> %s for audiorole:%s",pFullDeviceURI,in_pAudioRole); continue; } -- cgit 1.2.3-korg