drm/amd/display: Amend descriptions within enum pipe_split_policy

Message ID 20221107143715.279841-1-carlos.bilbao@amd.com
State New
Headers
Series drm/amd/display: Amend descriptions within enum pipe_split_policy |

Commit Message

Carlos Bilbao Nov. 7, 2022, 2:37 p.m. UTC
  Correct descriptions of two last fields of enum pipe_split_policy, updating
comments with proper field names.

Signed-off-by: Carlos Bilbao <carlos.bilbao@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dc.h | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
  

Comments

Alex Deucher Nov. 8, 2022, 4:24 p.m. UTC | #1
On Mon, Nov 7, 2022 at 9:37 AM Carlos Bilbao <carlos.bilbao@amd.com> wrote:
>
> Correct descriptions of two last fields of enum pipe_split_policy, updating
> comments with proper field names.
>
> Signed-off-by: Carlos Bilbao <carlos.bilbao@amd.com>

Acked-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  drivers/gpu/drm/amd/display/dc/dc.h | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
> index bfc5474c0f4c..277631a899d8 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc.h
> @@ -457,15 +457,16 @@ enum pipe_split_policy {
>         MPC_SPLIT_DYNAMIC = 0,
>
>         /**
> -        * @MPC_SPLIT_DYNAMIC: Avoid pipe split, which means that DC will not
> +        * @MPC_SPLIT_AVOID: Avoid pipe split, which means that DC will not
>          * try any sort of split optimization.
>          */
>         MPC_SPLIT_AVOID = 1,
>
>         /**
> -        * @MPC_SPLIT_DYNAMIC: With this option, DC will only try to optimize
> -        * the pipe utilization when using a single display; if the user
> -        * connects to a second display, DC will avoid pipe split.
> +        * @MPC_SPLIT_AVOID_MULT_DISP: With this option, DC will only try
> +        * to optimize the pipe utilization when using a single display;
> +        * if the user connects to a second display, DC will avoid pipe
> +        * split.
>          */
>         MPC_SPLIT_AVOID_MULT_DISP = 2,
>  };
> --
> 2.34.1
>
  
Harry Wentland Nov. 8, 2022, 7:45 p.m. UTC | #2
On 11/7/22 09:37, Carlos Bilbao wrote:
> Correct descriptions of two last fields of enum pipe_split_policy, updating
> comments with proper field names.
> 
> Signed-off-by: Carlos Bilbao <carlos.bilbao@amd.com>

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

Harry

> ---
>  drivers/gpu/drm/amd/display/dc/dc.h | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
> index bfc5474c0f4c..277631a899d8 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc.h
> @@ -457,15 +457,16 @@ enum pipe_split_policy {
>  	MPC_SPLIT_DYNAMIC = 0,
>  
>  	/**
> -	 * @MPC_SPLIT_DYNAMIC: Avoid pipe split, which means that DC will not
> +	 * @MPC_SPLIT_AVOID: Avoid pipe split, which means that DC will not
>  	 * try any sort of split optimization.
>  	 */
>  	MPC_SPLIT_AVOID = 1,
>  
>  	/**
> -	 * @MPC_SPLIT_DYNAMIC: With this option, DC will only try to optimize
> -	 * the pipe utilization when using a single display; if the user
> -	 * connects to a second display, DC will avoid pipe split.
> +	 * @MPC_SPLIT_AVOID_MULT_DISP: With this option, DC will only try
> +	 * to optimize the pipe utilization when using a single display;
> +	 * if the user connects to a second display, DC will avoid pipe
> +	 * split.
>  	 */
>  	MPC_SPLIT_AVOID_MULT_DISP = 2,
>  };
  

Patch

diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
index bfc5474c0f4c..277631a899d8 100644
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -457,15 +457,16 @@  enum pipe_split_policy {
 	MPC_SPLIT_DYNAMIC = 0,
 
 	/**
-	 * @MPC_SPLIT_DYNAMIC: Avoid pipe split, which means that DC will not
+	 * @MPC_SPLIT_AVOID: Avoid pipe split, which means that DC will not
 	 * try any sort of split optimization.
 	 */
 	MPC_SPLIT_AVOID = 1,
 
 	/**
-	 * @MPC_SPLIT_DYNAMIC: With this option, DC will only try to optimize
-	 * the pipe utilization when using a single display; if the user
-	 * connects to a second display, DC will avoid pipe split.
+	 * @MPC_SPLIT_AVOID_MULT_DISP: With this option, DC will only try
+	 * to optimize the pipe utilization when using a single display;
+	 * if the user connects to a second display, DC will avoid pipe
+	 * split.
 	 */
 	MPC_SPLIT_AVOID_MULT_DISP = 2,
 };