Re: [Intel-gfx] [PATCH 4/6] drm/i915/bios: Fix aggressiveness typos

[Jani Nikula <jani.nikula@linux.intel.com>]

On Fri, 27 May 2022, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Fix various typos around "aggressiveness". Note that
> the VBT spec also sometimes missspells it as
> "agressiveness" so I guess that's where some of the typos
> came from.

Could nitpick unrelated things like space before ":" in bitfields or the
struct naming, but this patch does what it says on the box.

Reviewed-by: Jani Nikula <jani.nikula@intel.com>


>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_vbt_defs.h | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_vbt_defs.h b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> index 4b98bab3b890..39109f204c6d 100644
> --- a/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> +++ b/drivers/gpu/drm/i915/display/intel_vbt_defs.h
> @@ -856,9 +856,9 @@ struct als_data_entry {
>  	u16 lux;
>  } __packed;
>  
> -struct agressiveness_profile_entry {
> -	u8 dpst_agressiveness : 4;
> -	u8 lace_agressiveness : 4;
> +struct aggressiveness_profile_entry {
> +	u8 dpst_aggressiveness : 4;
> +	u8 lace_aggressiveness : 4;
>  } __packed;
>  
>  struct bdb_lfp_power {
> @@ -873,7 +873,7 @@ struct bdb_lfp_power {
>  	u16 dmrrs;
>  	u16 adb;
>  	u16 lace_enabled_status;
> -	struct agressiveness_profile_entry aggressivenes[16];
> +	struct aggressiveness_profile_entry aggressiveness[16];
>  	u16 hobl; /* 232+ */
>  	u16 vrr_feature_enabled; /* 233+ */
>  } __packed;

-- 
Jani Nikula, Intel Open Source Graphics Center