lkml.org 
[lkml]   [2023]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] drm/amd/pm: Clean up errors in smu_v11_0_pptable.h
Applied.  Thanks!

On Mon, Jul 31, 2023 at 5:22 AM Ran Sun <sunran001@208suo.com> wrote:
>
> Fix the following errors reported by checkpatch:
>
> ERROR: trailing whitespace
> ERROR: open brace '{' following struct go on the same line
> ERROR: code indent should use tabs where possible
>
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
> .../gpu/drm/amd/pm/swsmu/inc/smu_v11_0_pptable.h | 15 ++++++---------
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/smu_v11_0_pptable.h b/drivers/gpu/drm/amd/pm/swsmu/inc/smu_v11_0_pptable.h
> index 0116e3d04fad..df7430876e0c 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/inc/smu_v11_0_pptable.h
> +++ b/drivers/gpu/drm/amd/pm/swsmu/inc/smu_v11_0_pptable.h
> @@ -101,8 +101,7 @@ enum SMU_11_0_ODSETTING_ID {
> };
> #define SMU_11_0_MAX_ODSETTING 32 //Maximum Number of ODSettings
>
> -struct smu_11_0_overdrive_table
> -{
> +struct smu_11_0_overdrive_table {
> uint8_t revision; //Revision = SMU_11_0_PP_OVERDRIVE_VERSION
> uint8_t reserve[3]; //Zero filled field reserved for future use
> uint32_t feature_count; //Total number of supported features
> @@ -127,8 +126,7 @@ enum SMU_11_0_PPCLOCK_ID {
> };
> #define SMU_11_0_MAX_PPCLOCK 16 //Maximum Number of PP Clocks
>
> -struct smu_11_0_power_saving_clock_table
> -{
> +struct smu_11_0_power_saving_clock_table {
> uint8_t revision; //Revision = SMU_11_0_PP_POWERSAVINGCLOCK_VERSION
> uint8_t reserve[3]; //Zero filled field reserved for future use
> uint32_t count; //power_saving_clock_count = SMU_11_0_PPCLOCK_COUNT
> @@ -136,8 +134,7 @@ struct smu_11_0_power_saving_clock_table
> uint32_t min[SMU_11_0_MAX_PPCLOCK]; //PowerSavingClock Mode Clock Minimum array In MHz
> };
>
> -struct smu_11_0_powerplay_table
> -{
> +struct smu_11_0_powerplay_table {
> struct atom_common_table_header header;
> uint8_t table_revision;
> uint16_t table_size; //Driver portion table size. The offset to smc_pptable including header size
> @@ -145,14 +142,14 @@ struct smu_11_0_powerplay_table
> uint32_t golden_revision;
> uint16_t format_id;
> uint32_t platform_caps; //POWERPLAYABLE::ulPlatformCaps
> -
> +
> uint8_t thermal_controller_type; //one of SMU_11_0_PP_THERMALCONTROLLER
>
> uint16_t small_power_limit1;
> uint16_t small_power_limit2;
> uint16_t boost_power_limit;
> - uint16_t od_turbo_power_limit; //Power limit setting for Turbo mode in Performance UI Tuning.
> - uint16_t od_power_save_power_limit; //Power limit setting for PowerSave/Optimal mode in Performance UI Tuning.
> + uint16_t od_turbo_power_limit; //Power limit setting for Turbo mode in Performance UI Tuning.
> + uint16_t od_power_save_power_limit; //Power limit setting for PowerSave/Optimal mode in Performance UI Tuning.
> uint16_t software_shutdown_temp;
>
> uint16_t reserve[6]; //Zero filled field reserved for future use
> --
> 2.17.1
>

\
 
 \ /
  Last update: 2023-08-14 19:53    [W:0.056 / U:0.168 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site