Change log message

[Version] 0.2.32
[Profile] Mobile, Wearable
[Issue Type] Fix bugs

Change-Id: I3951d405e6e869344c8dbf28a7be6b4026c0ec2d

1 files changed, 54 insertions, 54 deletions

diff --git a/src/mm_radio_priv_hal.c b/src/mm_radio_priv_hal.c
index 6e9cffd..18493a9 100644..100755
--- a/src/mm_radio_priv_hal.c
+++ b/src/mm_radio_priv_hal.c
@@ -197,22 +197,22 @@ int _mmradio_create_radio(mm_radio_t *radio)
 	/* create command lock */
 	ret = pthread_mutex_init(&radio->cmd_lock, NULL);
 	if (ret) {
-		MMRADIO_LOG_ERROR("mutex creation failed");
+		MMRADIO_LOG_ERROR("failed to create mutex");
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 
 	ret = pthread_mutex_init(&radio->volume_lock, NULL);
 	if (ret) {
-		MMRADIO_LOG_ERROR("volume mutex creation failed");
+		MMRADIO_LOG_ERROR("failed to create volume mutex");
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 
 	MMRADIO_SET_STATE(radio, MM_RADIO_STATE_NULL);
 
 	/* initialize resource manager */
-	ret = mmradio_resource_manager_init(&radio->resource_manager, radio);
+	ret = mmradio_resource_manager_init(&(radio->resource_manager), radio);
 	if (ret) {
-		MMRADIO_LOG_ERROR("failed to initialize resource manager\n");
+		MMRADIO_LOG_ERROR("failed to initialize resource manager");
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 
@@ -224,14 +224,14 @@ int _mmradio_create_radio(mm_radio_t *radio)
 
 	if (ret) {
 		/* NOTE : we are dealing it as an error since we cannot expect it's behavior */
-		MMRADIO_LOG_ERROR("mmradio_audio_focus_register is failed");
+		MMRADIO_LOG_ERROR("failed to register sound focus");
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 #endif
 
 	ret = radio_hal_interface_init(&(radio->hal_inf));
 	if (ret) {
-		MMRADIO_LOG_ERROR("mmradio hal interface init failed");
+		MMRADIO_LOG_ERROR("failed to init mmradio hal interface");
 		return ret;
 	}
 
@@ -255,7 +255,7 @@ int _mmradio_realize(mm_radio_t *radio)
 
 	ret = pthread_mutex_init(&radio->seek_cancel_mutex, NULL);
 	if (ret < 0) {
-		MMRADIO_LOG_ERROR("Mutex creation failed %d", ret);
+		MMRADIO_LOG_ERROR("failed to create seek cancel mutex %d", ret);
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 
@@ -273,7 +273,7 @@ int _mmradio_realize(mm_radio_t *radio)
 #ifdef TIZEN_FEATURE_SOUND_VSTREAM
 	ret = sound_manager_create_stream_information_internal(SOUND_STREAM_TYPE_RADIO, NULL, radio, &radio->stream_info);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("sound_manager_create_stream_information_internal error");
+		MMRADIO_LOG_ERROR("failed to create stream information");
 		MMRADIO_LOG_FLEAVE();
 		return ret;
 	}
@@ -287,7 +287,7 @@ int _mmradio_realize(mm_radio_t *radio)
 
 	ret = sound_manager_get_max_volume(SOUND_TYPE_MEDIA, &max);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_WARNING("sound_manager_get_max_volume error");
+		MMRADIO_LOG_WARNING("failed to get max volume");
 		radio->max_media_volume = DEFAULT_MAX_MEDIA_VOLUME;
 	} else {
 		radio->max_media_volume = max;
@@ -296,7 +296,7 @@ int _mmradio_realize(mm_radio_t *radio)
 	ret = mm_sound_volume_get_value(VOLUME_TYPE_MEDIA, &volume);
 
 	if (ret != MM_ERROR_NONE)
-		MMRADIO_LOG_WARNING("failed to get MEDIA_VOLUME");
+		MMRADIO_LOG_WARNING("failed to get media volume");
 
 	MMRADIO_VOLUME_LOCK(radio);
 	radio->media_volume = volume;
@@ -304,7 +304,7 @@ int _mmradio_realize(mm_radio_t *radio)
 
 	ret = mm_sound_add_volume_changed_callback(__mmradio_volume_changed_cb, (void *)radio, &radio->volume_subs_id);
 	if (ret != MM_ERROR_NONE)
-		MMRADIO_LOG_WARNING("mm_sound_add_volume_changed_callback error");
+		MMRADIO_LOG_WARNING("failed to register volume changed callback");
 
 	MMRADIO_SET_STATE(radio, MM_RADIO_STATE_READY);
 
@@ -361,18 +361,18 @@ int _mmradio_destroy(mm_radio_t *radio)
 
 	ret = radio_hal_interface_deinit(radio->hal_inf);
 	if (ret) {
-		MMRADIO_LOG_ERROR("mmradio hal interface deinit failed");
+		MMRADIO_LOG_ERROR("failed to deinitialize radio hal interface");
 		return ret;
 	}
 
 	/* destroy command lock */
 	ret = pthread_mutex_destroy(&radio->cmd_lock);
 	if (ret)
-		MMRADIO_LOG_ERROR("mutex destroy failed\n");
+		MMRADIO_LOG_ERROR("failed to destory muxtex");
 
 	ret = pthread_mutex_destroy(&radio->volume_lock);
 	if (ret)
-		MMRADIO_LOG_ERROR("volume mutex destroy failed\n");
+		MMRADIO_LOG_ERROR("failed to destory volume mutex");
 
 #ifdef TIZEN_FEATURE_SOUND_FOCUS
 	ret = mmradio_sound_focus_deregister(&radio->sound_focus);
@@ -384,7 +384,7 @@ int _mmradio_destroy(mm_radio_t *radio)
 
 	ret = mmradio_resource_manager_deinit(&radio->resource_manager);
 	if (ret) {
-		MMRADIO_LOG_ERROR("failed to initialize resource manager\n");
+		MMRADIO_LOG_ERROR("failed to deinitialize resource manager");
 		return MM_ERROR_RADIO_INTERNAL;
 	}
 
@@ -409,7 +409,7 @@ int _mmradio_set_frequency(mm_radio_t *radio, int freq)
 
 	ret = radio_hal_set_frequency(radio->hal_inf, freq);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_set_frequency error");
+		MMRADIO_LOG_ERROR("failed to set radio hal frequency");
 		MMRADIO_LOG_FLEAVE();
 		return ret;
 	}
@@ -433,7 +433,7 @@ int _mmradio_get_frequency(mm_radio_t *radio, int *pFreq)
 
 	ret = radio_hal_get_frequency(radio->hal_inf, &freq);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_get_frequency error");
+		MMRADIO_LOG_ERROR("failed to get radio hal frequency");
 		*pFreq = 0;
 		return ret;
 	}
@@ -461,7 +461,7 @@ int _mmradio_mute(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_mute is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_mute error");
+		MMRADIO_LOG_ERROR("failed to set radio hal mute");
 		MMRADIO_LOG_FLEAVE();
 		return ret;
 	}
@@ -485,7 +485,7 @@ int _mmradio_unmute(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_unmute is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_unmute error");
+		MMRADIO_LOG_ERROR("failed to set radio hal unmute");
 		MMRADIO_LOG_FLEAVE();
 		return ret;
 	}
@@ -547,7 +547,7 @@ int _mmradio_start(mm_radio_t *radio)
 	if (radio->sound_focus.handle > 0) {
 		ret = mmradio_acquire_sound_focus(&radio->sound_focus);
 		if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("failed to set sound focus");
+			MMRADIO_LOG_ERROR("failed to acquire sound focus");
 			return ret;
 		}
 	}
@@ -579,7 +579,7 @@ int _mmradio_start(mm_radio_t *radio)
 	if (!radio->is_ready) {
 		ret = mmradio_resource_manager_prepare(&radio->resource_manager, MM_RADIO_RESOURCE_TYPE_RADIO);
 		if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("resource manager prepare fail");
+			MMRADIO_LOG_ERROR("failed to prepare resource manager");
 			return MM_ERROR_RADIO_INTERNAL;
 		}
 		ret = mmradio_resource_manager_acquire(&radio->resource_manager);
@@ -592,7 +592,7 @@ int _mmradio_start(mm_radio_t *radio)
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_prepare is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_prepare_device error");
+			MMRADIO_LOG_ERROR("failed to prepare radio hal");
 			goto error2;
 		}
 
@@ -600,7 +600,7 @@ int _mmradio_start(mm_radio_t *radio)
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_open is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_init error");
+			MMRADIO_LOG_ERROR("failed to open radio hal");
 			goto error1;
 		}
 		radio->is_ready = TRUE;
@@ -610,11 +610,11 @@ int _mmradio_start(mm_radio_t *radio)
 
 	ret = mm_sound_volume_get_value(VOLUME_TYPE_MEDIA, &volume);
 	if (ret != MM_ERROR_NONE)
-		MMRADIO_LOG_WARNING("failed to get MEDIA_VOLUME");
+		MMRADIO_LOG_WARNING("failed to get media volume");
 
 	ret = __mmradio_set_media_volume(radio, volume);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("failed to media volume");
+		MMRADIO_LOG_ERROR("failed to set media volume");
 		goto error1;
 	}
 
@@ -622,21 +622,21 @@ int _mmradio_start(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_start is not supported");
 	} else if (ret) {
-		MMRADIO_LOG_ERROR("failed to radio_hal_start");
+		MMRADIO_LOG_ERROR("failed to start radio hal");
 		goto error1;
 	}
 
 	/* set stored frequency */
 	ret = radio_hal_set_frequency(radio->hal_inf, radio->freq);
 	if (ret) {
-		MMRADIO_LOG_ERROR("failed to radio_hal_set_frequency");
+		MMRADIO_LOG_ERROR("failed to set radio hal frequency");
 		goto error1;
 	}
 
 #ifdef TIZEN_FEATURE_SOUND_VSTREAM
 	ret = sound_manager_start_virtual_stream(radio->vstream);
 	if (ret) {
-		MMRADIO_LOG_ERROR("failed to sound_manager_start_virtual_stream");
+		MMRADIO_LOG_ERROR("failed to start sound manager virtual stream");
 		goto error;
 	}
 #endif
@@ -684,7 +684,7 @@ int _mmradio_stop(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_unmute is not supported");
 	} else if (ret) {
-		MMRADIO_LOG_ERROR("failed to radio_hal_stop");
+		MMRADIO_LOG_ERROR("failed to stop radio hal");
 		return ret;
 	}
 
@@ -693,7 +693,7 @@ int _mmradio_stop(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_close is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_close_device error");
+		MMRADIO_LOG_ERROR("failed to close radio hal");
 		return ret;
 	}
 
@@ -701,7 +701,7 @@ int _mmradio_stop(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_unprepare is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_close_device error");
+		MMRADIO_LOG_ERROR("failed to unprepare radio hal");
 		return ret;
 	}
 
@@ -722,7 +722,7 @@ int _mmradio_stop(mm_radio_t *radio)
 		if (resource_state == MM_RADIO_RESOURCE_STATE_PREPARED) {
 			ret = mmradio_resource_manager_unprepare(&radio->resource_manager);
 			if (ret != MM_ERROR_NONE)
-				MMRADIO_LOG_ERROR("resource manager unprepare fail");
+				MMRADIO_LOG_ERROR("failed to unprepare resource manager");
 		}
 	}
 
@@ -730,7 +730,7 @@ int _mmradio_stop(mm_radio_t *radio)
 	if (radio->sound_focus.handle > 0) {
 		ret = mmradio_release_sound_focus(&radio->sound_focus);
 		if (ret) {
-			MMRADIO_LOG_ERROR("mmradio_release_audio_focus is failed");
+			MMRADIO_LOG_ERROR("failed to release sound focus");
 			return ret;
 		}
 	}
@@ -765,7 +765,7 @@ int _mmradio_seek(mm_radio_t *radio, MMRadioSeekDirectionType direction)
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_mute is not supported");
 		} else if (ret) {
-			MMRADIO_LOG_ERROR("failed to radio_hal_mute");
+			MMRADIO_LOG_ERROR("failed to set radio hal mute");
 			return ret;
 		}
 		radio->seek_unmute = TRUE;
@@ -777,7 +777,7 @@ int _mmradio_seek(mm_radio_t *radio, MMRadioSeekDirectionType direction)
 	ret = pthread_create(&radio->seek_thread, NULL, (void *)__mmradio_seek_thread, (void *)radio);
 
 	if (ret) {
-		MMRADIO_LOG_DEBUG("failed create thread");
+		MMRADIO_LOG_DEBUG("failed to create thread");
 		radio->is_seeking = FALSE;
 		radio->seek_cancel = TRUE;
 		if (radio->seek_unmute) {
@@ -785,7 +785,7 @@ int _mmradio_seek(mm_radio_t *radio, MMRadioSeekDirectionType direction)
 			if (ret == MM_ERROR_NOT_SUPPORT_API) {
 				MMRADIO_LOG_WARNING("radio_hal_mute is not supported");
 			} else if (ret) {
-				MMRADIO_LOG_ERROR("failed to radio_hal_mute");
+				MMRADIO_LOG_ERROR("failed to set radio hal mute");
 				radio->seek_unmute = FALSE;
 				return ret;
 			}
@@ -845,7 +845,7 @@ int _mmradio_start_scan(mm_radio_t *radio)
 	if (!radio->is_ready) {
 		ret = mmradio_resource_manager_prepare(&radio->resource_manager, MM_RADIO_RESOURCE_TYPE_RADIO);
 		if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("resource manager prepare fail");
+			MMRADIO_LOG_ERROR("failed to prepare resource manager");
 			return MM_ERROR_RADIO_INTERNAL;
 		}
 		ret = mmradio_resource_manager_acquire(&radio->resource_manager);
@@ -859,7 +859,7 @@ int _mmradio_start_scan(mm_radio_t *radio)
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_prepare is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_prepare_device error");
+			MMRADIO_LOG_ERROR("failed to prepare radio hal");
 			return ret;
 		}
 
@@ -867,7 +867,7 @@ int _mmradio_start_scan(mm_radio_t *radio)
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_open is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_init error");
+			MMRADIO_LOG_ERROR("failed to open radio hal");
 			MMRADIO_LOG_FLEAVE();
 			return ret;
 		}
@@ -937,7 +937,7 @@ int _mm_radio_get_signal_strength(mm_radio_t *radio, int *value)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_unmute is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_get_signal_strength error");
+		MMRADIO_LOG_ERROR("failed to get radio hal signal strength");
 		*value = 0;
 		MMRADIO_LOG_FLEAVE();
 		return ret;
@@ -961,7 +961,7 @@ void __mmradio_scan_thread(mm_radio_t *radio)
 	if (ret == MM_ERROR_NOT_SUPPORT_API) {
 		MMRADIO_LOG_WARNING("radio_hal_mute is not supported");
 	} else if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_mute error");
+		MMRADIO_LOG_ERROR("faied to set radio hal mute");
 		goto FINISHED;
 	}
 	ret = radio_hal_set_frequency(radio->hal_inf, radio->region_setting.band_min);
@@ -1034,14 +1034,14 @@ FINISHED:
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_unmute is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_unmute error");
+			MMRADIO_LOG_ERROR("failed to set radio hal unmute");
 			goto FINISHED_ERR;
 		}
 		ret = radio_hal_set_frequency(radio->hal_inf, prev_freq);
 		if (ret == MM_ERROR_NOT_SUPPORT_API) {
 			MMRADIO_LOG_WARNING("radio_hal_set_frequency is not supported");
 		} else if (ret != MM_ERROR_NONE) {
-			MMRADIO_LOG_ERROR("radio_hal_set_frequency error");
+			MMRADIO_LOG_ERROR("failed to set radio hal frequency");
 			goto FINISHED_ERR;
 		}
 	}
@@ -1056,13 +1056,13 @@ FINISHED_ERR:
 		if (ret == MM_ERROR_NOT_SUPPORT_API)
 			MMRADIO_LOG_WARNING("radio_hal_close is not supported");
 		else if (ret != MM_ERROR_NONE)
-			MMRADIO_LOG_ERROR("radio_hal_close_device error");
+			MMRADIO_LOG_ERROR("failed to close radio hal");
 
 		ret = radio_hal_unprepare(radio->hal_inf);
 		if (ret == MM_ERROR_NOT_SUPPORT_API)
 			MMRADIO_LOG_WARNING("radio_hal_unprepare is not supported");
 		else if (ret != MM_ERROR_NONE)
-			MMRADIO_LOG_ERROR("radio_hal_close_device error");
+			MMRADIO_LOG_ERROR("failed to unprepare radio hal");
 
 		radio->is_ready = FALSE;
 
@@ -1079,7 +1079,7 @@ FINISHED_ERR:
 			if (resource_state == MM_RADIO_RESOURCE_STATE_PREPARED) {
 				ret = mmradio_resource_manager_unprepare(&radio->resource_manager);
 				if (ret != MM_ERROR_NONE)
-					MMRADIO_LOG_ERROR("resource manager unprepare fail");
+					MMRADIO_LOG_ERROR("failed to unprepare resource manager");
 			}
 		}
 
@@ -1140,7 +1140,7 @@ void __mmradio_seek_thread(mm_radio_t *radio)
 		ret = radio_hal_seek(radio->hal_inf, radio->seek_direction);
 		pthread_mutex_unlock(&radio->seek_cancel_mutex);
 		if (ret) {
-			MMRADIO_LOG_ERROR("radio_hal_seek failed");
+			MMRADIO_LOG_ERROR("failed to seek radio hal");
 			goto SEEK_FAILED;
 		}
 
@@ -1178,7 +1178,7 @@ void __mmradio_seek_thread(mm_radio_t *radio)
 			 */
 			ret = radio_hal_unmute(radio->hal_inf);
 			if (ret) {
-				MMRADIO_LOG_ERROR("failed to tune to new frequency");
+				MMRADIO_LOG_ERROR("failed to set unmute radio hal");
 				goto SEEK_FAILED;
 			}
 			radio->seek_unmute = FALSE;
@@ -1204,7 +1204,7 @@ SEEK_FAILED:
 		*/
 		ret = radio_hal_unmute(radio->hal_inf);
 		if (ret)
-			MMRADIO_LOG_ERROR("failed to tune to new frequency");
+			MMRADIO_LOG_ERROR("failed to set unmute radio hal");
 		radio->seek_unmute = FALSE;
 	}
 	/* freq -1 means it's failed to seek */
@@ -1222,7 +1222,7 @@ static bool __mmradio_post_message(mm_radio_t *radio, enum MMMessageType msgtype
 	MMRADIO_LOG_FENTER();
 
 	if (!radio->msg_cb) {
-		MMRADIO_LOG_WARNING("failed to post a message");
+		MMRADIO_LOG_WARNING("failed to post a message because msg cb didn't register");
 		return false;
 	}
 
@@ -1556,7 +1556,7 @@ static void __mmradio_volume_changed_cb(volume_type_t type, unsigned int volume,
 		MMRADIO_LOG_INFO("Change FM Radio volume to %d", volume);
 		ret = __mmradio_set_media_volume(radio, volume);
 		if (ret != MM_ERROR_NONE)
-			MMRADIO_LOG_ERROR("__mmradio_set_media_volume error");
+			MMRADIO_LOG_ERROR("failed to set media volume");
 		return;
 	}
 }
@@ -1621,7 +1621,7 @@ int _mmradio_set_volume(mm_radio_t *radio, float volume)
 
 	ret = radio_hal_set_volume(radio->hal_inf, volume);
 	if (ret != MM_ERROR_NONE)
-		MMRADIO_LOG_ERROR("radio_hal_set_volume error");
+		MMRADIO_LOG_ERROR("failed to set radio hal volume");
 
 	MMRADIO_LOG_FLEAVE();
 
@@ -1641,7 +1641,7 @@ int _mmradio_get_volume(mm_radio_t *radio, float *pVolume)
 
 	ret = radio_hal_get_volume(radio->hal_inf, &volume);
 	if (ret != MM_ERROR_NONE) {
-		MMRADIO_LOG_ERROR("radio_hal_get_volume error");
+		MMRADIO_LOG_ERROR("failed to get radio hal volume");
 		*pVolume = 0;
 		return ret;
 	}
@@ -1673,7 +1673,7 @@ static int __mmradio_set_media_volume(mm_radio_t *radio, unsigned int level)
 
 	ret = radio_hal_set_media_volume(radio->hal_inf, level);
 	if (ret != MM_ERROR_NONE)
-		MMRADIO_LOG_ERROR("radio_hal_set_media_volume error");
+		MMRADIO_LOG_ERROR("failed to set radio hal media volume");
 
 	MMRADIO_LOG_FLEAVE();