[Intel-gfx] [PATCH] drm/i915: Make psr_safest_params and enable_psr2_sel_fetch parameters read only

Mun, Gwan-gyeong gwan-gyeong.mun at intel.com
Fri Feb 5 05:24:43 UTC 2021


Reviewed-by: Gwan-gyeong Mun <gwan-gyeong.mun at intel.com>
On Thu, 2021-02-04 at 07:33 -0800, José Roberto de Souza wrote:
> By mistake those 2 parameters were defined as read and write in the
> .h file while in the .c file it is read only.
> The intention here was to be read only to avoid the need of
> additional
> handling.
> 
> Cc: Gwan-gyeong Mun <gwan-gyeong.mun at intel.com>
> Cc: Petri Latvala <petri.latvala at intel.com>
> Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
> ---
>  drivers/gpu/drm/i915/i915_params.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_params.h
> b/drivers/gpu/drm/i915/i915_params.h
> index f031966af5b7..48f47e44e848 100644
> --- a/drivers/gpu/drm/i915/i915_params.h
> +++ b/drivers/gpu/drm/i915/i915_params.h
> @@ -54,8 +54,8 @@ struct drm_printer;
>         param(int, enable_dc, -1, 0400) \
>         param(int, enable_fbc, -1, 0600) \
>         param(int, enable_psr, -1, 0600) \
> -       param(bool, psr_safest_params, false, 0600) \
> -       param(bool, enable_psr2_sel_fetch, false, 0600) \
> +       param(bool, psr_safest_params, false, 0400) \
> +       param(bool, enable_psr2_sel_fetch, false, 0400) \
>         param(int, disable_power_well, -1, 0400) \
>         param(int, enable_ips, 1, 0600) \
>         param(int, invert_brightness, 0, 0600) \



More information about the Intel-gfx mailing list