[PATCH] amdgpu/dc: drop dml_util_is_420

[PATCH] amdgpu/dc: drop dml_util_is_420

[Dave Airlie]

From: Dave Airlie <airlied@redhat.com>

This is unused code.

Signed-off-by: Dave Airlie <airlied@redhat.com>
---
 .../gpu/drm/amd/display/dc/dml/dml_common_defs.c   | 33 ----------------------
 .../gpu/drm/amd/display/dc/dml/dml_common_defs.h   |  1 -
 2 files changed, 34 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
index c242b8d..7c0eb52 100644
--- a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
+++ b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
@@ -59,36 +59,3 @@ unsigned int dml_round_to_multiple(
 	else
 		return (num - remainder);
 }
-
-bool dml_util_is_420(enum source_format_class sorce_format)
-{
-	bool val = false;
-
-	switch (sorce_format) {
-	case dm_444_16:
-		val = false;
-		break;
-	case dm_444_32:
-		val = false;
-		break;
-	case dm_444_64:
-		val = false;
-		break;
-	case dm_420_8:
-		val = true;
-		break;
-	case dm_420_10:
-		val = true;
-		break;
-	case dm_422_8:
-		val = false;
-		break;
-	case dm_422_10:
-		val = false;
-		break;
-	default:
-		BREAK_TO_DEBUGGER();
-	}
-
-	return val;
-}
diff --git a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
index c621f83..a2da3da 100644
--- a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
+++ b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
@@ -32,7 +32,6 @@
 
 #define DTRACE(str, ...) dm_logger_write(mode_lib->logger, LOG_DML, str, ##__VA_ARGS__);
 
-bool dml_util_is_420(enum source_format_class sorce_format);
 double dml_round(double a);
 unsigned int dml_round_to_multiple(
 			unsigned int num, unsigned int multiple, bool up);
-- 
2.9.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: [PATCH] amdgpu/dc: drop dml_util_is_420

[Harry Wentland]

On 2017-09-27 08:55 PM, Dave Airlie wrote:
> From: Dave Airlie <airlied@redhat.com>
> 
> This is unused code.
> 
> Signed-off-by: Dave Airlie <airlied@redhat.com>

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Harry

> ---
>  .../gpu/drm/amd/display/dc/dml/dml_common_defs.c   | 33 ----------------------
>  .../gpu/drm/amd/display/dc/dml/dml_common_defs.h   |  1 -
>  2 files changed, 34 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
> index c242b8d..7c0eb52 100644
> --- a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
> +++ b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.c
> @@ -59,36 +59,3 @@ unsigned int dml_round_to_multiple(
>  	else
>  		return (num - remainder);
>  }
> -
> -bool dml_util_is_420(enum source_format_class sorce_format)
> -{
> -	bool val = false;
> -
> -	switch (sorce_format) {
> -	case dm_444_16:
> -		val = false;
> -		break;
> -	case dm_444_32:
> -		val = false;
> -		break;
> -	case dm_444_64:
> -		val = false;
> -		break;
> -	case dm_420_8:
> -		val = true;
> -		break;
> -	case dm_420_10:
> -		val = true;
> -		break;
> -	case dm_422_8:
> -		val = false;
> -		break;
> -	case dm_422_10:
> -		val = false;
> -		break;
> -	default:
> -		BREAK_TO_DEBUGGER();
> -	}
> -
> -	return val;
> -}
> diff --git a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
> index c621f83..a2da3da 100644
> --- a/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
> +++ b/drivers/gpu/drm/amd/display/dc/dml/dml_common_defs.h
> @@ -32,7 +32,6 @@
>  
>  #define DTRACE(str, ...) dm_logger_write(mode_lib->logger, LOG_DML, str, ##__VA_ARGS__);
>  
> -bool dml_util_is_420(enum source_format_class sorce_format);
>  double dml_round(double a);
>  unsigned int dml_round_to_multiple(
>  			unsigned int num, unsigned int multiple, bool up);
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx