From: Arindam Nath <arindam.nath@amd.com>
Date: Mon, 5 Dec 2016 19:21:26 +0530
Subject: drm/amd/display: remove DM_NOT_IMPL messages
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Git-commit: 49c07a99b2bdd6e6ed621f728d58d8535cc87a61
Patch-mainline: v4.15-rc1
References: FATE#326289 FATE#326079 FATE#326049 FATE#322398 FATE#326166
Some functions like dm_bandwidth_update are placeholders
for future implementations. Right now, they simply
print messages like "DM_NOT_IMPL: dm_bandwidth_update".
This message even though informational, sometimes can
create some confusion for users.
Since these functions are only skeletons for future, we
simply remove the rather un-unecessary messages from these
functions.
Signed-off-by: Arindam Nath <arindam.nath@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Harry Wentland <Harry.Wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Petr Tesarik <ptesarik@suse.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -48,10 +48,6 @@
#include "modules/inc/mod_freesync.h"
-/* Debug facilities */
-#define AMDGPU_DM_NOT_IMPL(fmt, ...) \
- DRM_INFO("DM_NOT_IMPL: " fmt, ##__VA_ARGS__)
-
/*
* dm_vblank_get_counter
*
@@ -1269,20 +1265,18 @@ void amdgpu_dm_destroy_drm_device(struct
*/
static void dm_bandwidth_update(struct amdgpu_device *adev)
{
- AMDGPU_DM_NOT_IMPL("%s\n", __func__);
+ /* TODO: implement later */
}
static void dm_set_backlight_level(struct amdgpu_encoder *amdgpu_encoder,
u8 level)
{
/* TODO: translate amdgpu_encoder to display_index and call DAL */
- AMDGPU_DM_NOT_IMPL("%s\n", __func__);
}
static u8 dm_get_backlight_level(struct amdgpu_encoder *amdgpu_encoder)
{
/* TODO: translate amdgpu_encoder to display_index and call DAL */
- AMDGPU_DM_NOT_IMPL("%s\n", __func__);
return 0;
}