diff mbox

[1/3] drm/radeon: constify more struct drm_*_helper funcs pointers

Message ID 1428913302-16195-1-git-send-email-jani.nikula@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jani Nikula April 13, 2015, 8:21 a.m. UTC
Some non-const pointers were added since the last constification, fix
them.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/radeon/radeon_connectors.c | 4 ++--
 drivers/gpu/drm/radeon/radeon_dp_mst.c     | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

Comments

Christian König April 13, 2015, 8:28 a.m. UTC | #1
On 13.04.2015 10:21, Jani Nikula wrote:
> Some non-const pointers were added since the last constification, fix
> them.
>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

For this one Reviewed-by: Christian König <christian.koenig@amd.com>.

Can't say anything for patch #2, and #3 already has my rb IIRC.

Regards,
Christian.

> ---
>   drivers/gpu/drm/radeon/radeon_connectors.c | 4 ++--
>   drivers/gpu/drm/radeon/radeon_dp_mst.c     | 2 +-
>   2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index f150c2495fa4..cebb65e07e1d 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -751,7 +751,7 @@ static int radeon_connector_set_property(struct drm_connector *connector, struct
>   		if (connector->encoder)
>   			radeon_encoder = to_radeon_encoder(connector->encoder);
>   		else {
> -			struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
> +			const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
>   			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector));
>   		}
>   
> @@ -762,7 +762,7 @@ static int radeon_connector_set_property(struct drm_connector *connector, struct
>   
>   		if (connector->encoder->crtc) {
>   			struct drm_crtc *crtc  = connector->encoder->crtc;
> -			struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
> +			const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
>   			struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc);
>   
>   			radeon_crtc->output_csc = radeon_encoder->output_csc;
> diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> index 5952ff2bb647..1017338a49d9 100644
> --- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
> +++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> @@ -604,7 +604,7 @@ radeon_dp_create_fake_mst_encoder(struct radeon_connector *connector)
>   	struct radeon_encoder *radeon_encoder;
>   	struct radeon_encoder_mst *mst_enc;
>   	struct drm_encoder *encoder;
> -	struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
> +	const struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
>   	struct drm_encoder *enc_master = connector_funcs->best_encoder(&connector->base);
>   
>   	DRM_DEBUG_KMS("enc master is %p\n", enc_master);
diff mbox

Patch

diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
index f150c2495fa4..cebb65e07e1d 100644
--- a/drivers/gpu/drm/radeon/radeon_connectors.c
+++ b/drivers/gpu/drm/radeon/radeon_connectors.c
@@ -751,7 +751,7 @@  static int radeon_connector_set_property(struct drm_connector *connector, struct
 		if (connector->encoder)
 			radeon_encoder = to_radeon_encoder(connector->encoder);
 		else {
-			struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
+			const struct drm_connector_helper_funcs *connector_funcs = connector->helper_private;
 			radeon_encoder = to_radeon_encoder(connector_funcs->best_encoder(connector));
 		}
 
@@ -762,7 +762,7 @@  static int radeon_connector_set_property(struct drm_connector *connector, struct
 
 		if (connector->encoder->crtc) {
 			struct drm_crtc *crtc  = connector->encoder->crtc;
-			struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
+			const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
 			struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc);
 
 			radeon_crtc->output_csc = radeon_encoder->output_csc;
diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
index 5952ff2bb647..1017338a49d9 100644
--- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
+++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
@@ -604,7 +604,7 @@  radeon_dp_create_fake_mst_encoder(struct radeon_connector *connector)
 	struct radeon_encoder *radeon_encoder;
 	struct radeon_encoder_mst *mst_enc;
 	struct drm_encoder *encoder;
-	struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
+	const struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
 	struct drm_encoder *enc_master = connector_funcs->best_encoder(&connector->base);
 
 	DRM_DEBUG_KMS("enc master is %p\n", enc_master);