On Tue, 29 Aug 2023, Ville Syrjala <ville.syrj...@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
>
> We aren't intending to mutate the SDVO device mapping structs,
> so make them const.
>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

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

> ---
>  drivers/gpu/drm/i915/display/intel_sdvo.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_sdvo.c 
> b/drivers/gpu/drm/i915/display/intel_sdvo.c
> index b988e05274c1..0bd815bf2e69 100644
> --- a/drivers/gpu/drm/i915/display/intel_sdvo.c
> +++ b/drivers/gpu/drm/i915/display/intel_sdvo.c
> @@ -2579,7 +2579,7 @@ intel_sdvo_select_ddc_bus(struct intel_sdvo *sdvo,
>                         struct intel_sdvo_connector *connector)
>  {
>       struct drm_i915_private *dev_priv = to_i915(sdvo->base.base.dev);
> -     struct sdvo_device_mapping *mapping;
> +     const struct sdvo_device_mapping *mapping;
>       int ddc_bus;
>  
>       if (sdvo->base.port == PORT_B)
> @@ -2602,7 +2602,7 @@ static void
>  intel_sdvo_select_i2c_bus(struct intel_sdvo *sdvo)
>  {
>       struct drm_i915_private *dev_priv = to_i915(sdvo->base.base.dev);
> -     struct sdvo_device_mapping *mapping;
> +     const struct sdvo_device_mapping *mapping;
>       u8 pin;
>  
>       if (sdvo->base.port == PORT_B)
> @@ -2647,7 +2647,7 @@ static u8
>  intel_sdvo_get_slave_addr(struct intel_sdvo *sdvo)
>  {
>       struct drm_i915_private *dev_priv = to_i915(sdvo->base.base.dev);
> -     struct sdvo_device_mapping *my_mapping, *other_mapping;
> +     const struct sdvo_device_mapping *my_mapping, *other_mapping;
>  
>       if (sdvo->base.port == PORT_B) {
>               my_mapping = &dev_priv->display.vbt.sdvo_mappings[0];

-- 
Jani Nikula, Intel Open Source Graphics Center

Reply via email to