Re: [PATCH v2] drm: Make the .mode_fixup() operations mode argument a const pointer

[Alex Deucher <alexdeucher@xxxxxxxxx>]

On Tue, Jul 17, 2012 at 11:56 AM, Laurent Pinchart
<laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
> The passed mode must not be modified by the operation, make it const.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>


Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> ---
>  drivers/gpu/drm/ast/ast_mode.c                  |    6 +++---
>  drivers/gpu/drm/cirrus/cirrus_mode.c            |    6 +++---
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c        |    2 +-
>  drivers/gpu/drm/exynos/exynos_drm_drv.h         |    2 +-
>  drivers/gpu/drm/exynos/exynos_drm_encoder.c     |    2 +-
>  drivers/gpu/drm/exynos/exynos_drm_hdmi.c        |    2 +-
>  drivers/gpu/drm/exynos/exynos_drm_hdmi.h        |    2 +-
>  drivers/gpu/drm/exynos/exynos_hdmi.c            |    2 +-
>  drivers/gpu/drm/gma500/cdv_intel_crt.c          |    2 +-
>  drivers/gpu/drm/gma500/cdv_intel_display.c      |    2 +-
>  drivers/gpu/drm/gma500/cdv_intel_hdmi.c         |    2 +-
>  drivers/gpu/drm/gma500/cdv_intel_lvds.c         |    2 +-
>  drivers/gpu/drm/gma500/mdfld_dsi_dpi.c          |    2 +-
>  drivers/gpu/drm/gma500/mdfld_dsi_dpi.h          |    2 +-
>  drivers/gpu/drm/gma500/mdfld_intel_display.c    |    2 +-
>  drivers/gpu/drm/gma500/oaktrail_crtc.c          |    2 +-
>  drivers/gpu/drm/gma500/oaktrail_hdmi.c          |    2 +-
>  drivers/gpu/drm/gma500/psb_intel_display.c      |    2 +-
>  drivers/gpu/drm/gma500/psb_intel_drv.h          |    2 +-
>  drivers/gpu/drm/gma500/psb_intel_lvds.c         |    2 +-
>  drivers/gpu/drm/gma500/psb_intel_sdvo.c         |    6 +++---
>  drivers/gpu/drm/i2c/ch7006_drv.c                |    2 +-
>  drivers/gpu/drm/i2c/ch7006_mode.c               |    2 +-
>  drivers/gpu/drm/i2c/ch7006_priv.h               |    2 +-
>  drivers/gpu/drm/i2c/sil164_drv.c                |    2 +-
>  drivers/gpu/drm/i915/dvo.h                      |    2 +-
>  drivers/gpu/drm/i915/intel_crt.c                |    2 +-
>  drivers/gpu/drm/i915/intel_dp.c                 |    3 ++-
>  drivers/gpu/drm/i915/intel_dvo.c                |    2 +-
>  drivers/gpu/drm/i915/intel_hdmi.c               |    2 +-
>  drivers/gpu/drm/i915/intel_lvds.c               |    2 +-
>  drivers/gpu/drm/i915/intel_sdvo.c               |    6 +++---
>  drivers/gpu/drm/i915/intel_tv.c                 |    3 ++-
>  drivers/gpu/drm/mgag200/mgag200_mode.c          |    8 ++++----
>  drivers/gpu/drm/nouveau/nv04_crtc.c             |    2 +-
>  drivers/gpu/drm/nouveau/nv04_dac.c              |    2 +-
>  drivers/gpu/drm/nouveau/nv04_dfp.c              |    2 +-
>  drivers/gpu/drm/nouveau/nv17_tv.c               |    2 +-
>  drivers/gpu/drm/nouveau/nv50_crtc.c             |    2 +-
>  drivers/gpu/drm/nouveau/nv50_dac.c              |    3 ++-
>  drivers/gpu/drm/nouveau/nv50_sor.c              |    3 ++-
>  drivers/gpu/drm/nouveau/nvd0_display.c          |    8 +++++---
>  drivers/gpu/drm/radeon/atombios_crtc.c          |    2 +-
>  drivers/gpu/drm/radeon/atombios_dp.c            |    2 +-
>  drivers/gpu/drm/radeon/atombios_encoders.c      |    4 ++--
>  drivers/gpu/drm/radeon/radeon_display.c         |    4 ++--
>  drivers/gpu/drm/radeon/radeon_legacy_crtc.c     |    2 +-
>  drivers/gpu/drm/radeon/radeon_legacy_encoders.c |    2 +-
>  drivers/gpu/drm/radeon/radeon_mode.h            |    4 ++--
>  drivers/gpu/drm/udl/udl_encoder.c               |    2 +-
>  drivers/gpu/drm/udl/udl_modeset.c               |    2 +-
>  drivers/staging/omapdrm/omap_crtc.c             |    2 +-
>  drivers/staging/omapdrm/omap_encoder.c          |    2 +-
>  include/drm/drm_crtc_helper.h                   |    4 ++--
>  include/drm/drm_encoder_slave.h                 |    2 +-
>  55 files changed, 78 insertions(+), 72 deletions(-)
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel