Re: [PATCH] drm/amd/pm: Clean up errors in vega10_powertune.c

[Alex Deucher]

Applied.  Thanks!

On Mon, Jul 31, 2023 at 11:38 PM Ran Sun  wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: that open brace { should be on the previous line
> ERROR: space prohibited after that open parenthesis '('
>
> Signed-off-by: Ran Sun 
> ---
>  .../amd/pm/powerplay/hwmgr/vega10_powertune.c | 89 +++
>  1 file changed, 31 insertions(+), 58 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c 
> b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
> index 309a9d3bc1b7..3007b054c873 100644
> --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
> +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
> @@ -30,8 +30,7 @@
>  #include "pp_debug.h"
>  #include "soc15_common.h"
>
> -static const struct vega10_didt_config_reg SEDiDtTuningCtrlConfig_Vega10[] =
> -{
> +static const struct vega10_didt_config_reg SEDiDtTuningCtrlConfig_Vega10[] = 
> {
>  /* 
> -
>   *  Offset Mask  
>Shift  
> Value
>   * 
> -
> @@ -55,8 +54,7 @@ static const struct vega10_didt_config_reg 
> SEDiDtTuningCtrlConfig_Vega10[] =
> {   0x  }  /* End of list */
>  };
>
> -static const struct vega10_didt_config_reg SEDiDtCtrl3Config_vega10[] =
> -{
> +static const struct vega10_didt_config_reg SEDiDtCtrl3Config_vega10[] = {
>  /* 
> -
>   *  Offset   Mask
>  Shift
> Value
>   * 
> -
> @@ -120,8 +118,7 @@ static const struct vega10_didt_config_reg 
> SEDiDtCtrl3Config_vega10[] =
> {   0x  }  /* End of list */
>  };
>
> -static const struct vega10_didt_config_reg SEDiDtCtrl2Config_Vega10[] =
> -{
> +static const struct vega10_didt_config_reg SEDiDtCtrl2Config_Vega10[] = {
>  /* 
> -
>   *  OffsetMask   
>   Shift  Value
>   * 
> -
> @@ -149,8 +146,7 @@ static const struct vega10_didt_config_reg 
> SEDiDtCtrl2Config_Vega10[] =
> {   0x  }  /* End of list */
>  };
>
> -static const struct vega10_didt_config_reg SEDiDtCtrl1Config_Vega10[] =
> -{
> +static const struct vega10_didt_config_reg SEDiDtCtrl1Config_Vega10[] = {
>  /* 
> -
>   *  Offset Mask  
>Shift  
> Value
>   * 
> -
> @@ -172,8 +168,7 @@ static const struct vega10_didt_config_reg 
> SEDiDtCtrl1Config_Vega10[] =
>  };
>
>
> -static const struct vega10_didt_config_reg SEDiDtWeightConfig_Vega10[] =
> -{
> +static const struct vega10_didt_config_reg SEDiDtWeightConfig_Vega10[] = {
>  /* 
> -
>   *  Offset Mask  
> Shift 
> Value
>   * 
> -
> @@ -201,8 +196,7 @@ static const struct vega10_didt_config_reg 
> SEDiDtWeightConfig_Vega10[] =
> {   0x  }  /* End of list */
>  }

[PATCH] drm/amd/pm: Clean up errors in vega10_powertune.c

[Ran Sun]

Fix the following errors reported by checkpatch:

ERROR: that open brace { should be on the previous line
ERROR: space prohibited after that open parenthesis '('

Signed-off-by: Ran Sun 
---
 .../amd/pm/powerplay/hwmgr/vega10_powertune.c | 89 +++
 1 file changed, 31 insertions(+), 58 deletions(-)

diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c 
b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
index 309a9d3bc1b7..3007b054c873 100644
--- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
+++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_powertune.c
@@ -30,8 +30,7 @@
 #include "pp_debug.h"
 #include "soc15_common.h"
 
-static const struct vega10_didt_config_reg SEDiDtTuningCtrlConfig_Vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtTuningCtrlConfig_Vega10[] = {
 /* 
-
  *  Offset Mask
 Shift  Value
  * 
-
@@ -55,8 +54,7 @@ static const struct vega10_didt_config_reg 
SEDiDtTuningCtrlConfig_Vega10[] =
{   0x  }  /* End of list */
 };
 
-static const struct vega10_didt_config_reg SEDiDtCtrl3Config_vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtCtrl3Config_vega10[] = {
 /* 
-
  *  Offset   Mask  
   ShiftValue
  * 
-
@@ -120,8 +118,7 @@ static const struct vega10_didt_config_reg 
SEDiDtCtrl3Config_vega10[] =
{   0x  }  /* End of list */
 };
 
-static const struct vega10_didt_config_reg SEDiDtCtrl2Config_Vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtCtrl2Config_Vega10[] = {
 /* 
-
  *  OffsetMask 
Shift  Value
  * 
-
@@ -149,8 +146,7 @@ static const struct vega10_didt_config_reg 
SEDiDtCtrl2Config_Vega10[] =
{   0x  }  /* End of list */
 };
 
-static const struct vega10_didt_config_reg SEDiDtCtrl1Config_Vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtCtrl1Config_Vega10[] = {
 /* 
-
  *  Offset Mask
 Shift  Value
  * 
-
@@ -172,8 +168,7 @@ static const struct vega10_didt_config_reg 
SEDiDtCtrl1Config_Vega10[] =
 };
 
 
-static const struct vega10_didt_config_reg SEDiDtWeightConfig_Vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtWeightConfig_Vega10[] = {
 /* 
-
  *  Offset Mask
  Shift Value
  * 
-
@@ -201,8 +196,7 @@ static const struct vega10_didt_config_reg 
SEDiDtWeightConfig_Vega10[] =
{   0x  }  /* End of list */
 };
 
-static const struct vega10_didt_config_reg SEDiDtCtrl0Config_Vega10[] =
-{
+static const struct vega10_didt_config_reg SEDiDtCtrl0Config_Vega10[] = {
 /* 
--