All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Rob Clark <rob.clark@linaro.org>, dri-devel@lists.freedesktop.org
Cc: patches@linaro.org, daniel.vetter@ffwll.ch, bskeggs@redhat.com,
	laurent.pinchart@ideasonboard.com, alexander.deucher@amd.com,
	Rob Clark <rob@ti.com>,
	alan@linux.intel.com
Subject: Re: [PATCH 9/9] drm: remove legacy drm_connector_property fxns
Date: Mon, 15 Oct 2012 10:04:58 +0300	[thread overview]
Message-ID: <87hapwmd0l.fsf@intel.com> (raw)
In-Reply-To: <1350007562-22323-10-git-send-email-rob.clark@linaro.org>

On Fri, 12 Oct 2012, Rob Clark <rob.clark@linaro.org> wrote:
> From: Rob Clark <rob@ti.com>
>
> Replace references to and remove the connector property fxns, which
> have been superseded with the more general object property fxns:

"fxns"? meh.

For this patch,

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

>
>   + drm_connector_attach_property -> drm_object_attach_property
>   + drm_connector_property_set_value -> drm_object_property_set_value
>   + drm_connector_property_get_value -> drm_object_property_get_value
>
> Signed-off-by: Rob Clark <rob@ti.com>
> ---
>  drivers/gpu/drm/drm_crtc.c      |   31 +++++--------------------------
>  drivers/gpu/drm/drm_fb_helper.c |    2 +-
>  drivers/gpu/drm/drm_sysfs.c     |    6 +++---
>  include/drm/drm_crtc.h          |    8 --------
>  4 files changed, 9 insertions(+), 38 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index 9c346a5..cee96f4 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -561,11 +561,11 @@ int drm_connector_init(struct drm_device *dev,
>  	dev->mode_config.num_connector++;
>  
>  	if (connector_type != DRM_MODE_CONNECTOR_VIRTUAL)
> -		drm_connector_attach_property(connector,
> +		drm_object_attach_property(&connector->base,
>  					      dev->mode_config.edid_property,
>  					      0);
>  
> -	drm_connector_attach_property(connector,
> +	drm_object_attach_property(&connector->base,
>  				      dev->mode_config.dpms_property, 0);
>  
>   out:
> @@ -2917,27 +2917,6 @@ void drm_property_destroy(struct drm_device *dev, struct drm_property *property)
>  }
>  EXPORT_SYMBOL(drm_property_destroy);
>  
> -void drm_connector_attach_property(struct drm_connector *connector,
> -			       struct drm_property *property, uint64_t init_val)
> -{
> -	drm_object_attach_property(&connector->base, property, init_val);
> -}
> -EXPORT_SYMBOL(drm_connector_attach_property);
> -
> -int drm_connector_property_set_value(struct drm_connector *connector,
> -				  struct drm_property *property, uint64_t value)
> -{
> -	return drm_object_property_set_value(&connector->base, property, value);
> -}
> -EXPORT_SYMBOL(drm_connector_property_set_value);
> -
> -int drm_connector_property_get_value(struct drm_connector *connector,
> -				  struct drm_property *property, uint64_t *val)
> -{
> -	return drm_object_property_get_value(&connector->base, property, val);
> -}
> -EXPORT_SYMBOL(drm_connector_property_get_value);
> -
>  void drm_object_attach_property(struct drm_mode_object *obj,
>  				struct drm_property *property,
>  				uint64_t init_val)
> @@ -3174,7 +3153,7 @@ int drm_mode_connector_update_edid_property(struct drm_connector *connector,
>  	/* Delete edid, when there is none. */
>  	if (!edid) {
>  		connector->edid_blob_ptr = NULL;
> -		ret = drm_connector_property_set_value(connector, dev->mode_config.edid_property, 0);
> +		ret = drm_object_property_set_value(&connector->base, dev->mode_config.edid_property, 0);
>  		return ret;
>  	}
>  
> @@ -3182,7 +3161,7 @@ int drm_mode_connector_update_edid_property(struct drm_connector *connector,
>  	connector->edid_blob_ptr = drm_property_create_blob(connector->dev,
>  							    size, edid);
>  
> -	ret = drm_connector_property_set_value(connector,
> +	ret = drm_object_property_set_value(&connector->base,
>  					       dev->mode_config.edid_property,
>  					       connector->edid_blob_ptr->base.id);
>  
> @@ -3246,7 +3225,7 @@ static int drm_mode_connector_set_obj_prop(struct drm_mode_object *obj,
>  
>  	/* store the property value if successful */
>  	if (!ret)
> -		drm_connector_property_set_value(connector, property, value);
> +		drm_object_property_set_value(&connector->base, property, value);
>  	return ret;
>  }
>  
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index b5d05f5..712b3e88 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -331,7 +331,7 @@ static void drm_fb_helper_dpms(struct fb_info *info, int dpms_mode)
>  		for (j = 0; j < fb_helper->connector_count; j++) {
>  			connector = fb_helper->connector_info[j]->connector;
>  			connector->funcs->dpms(connector, dpms_mode);
> -			drm_connector_property_set_value(connector,
> +			drm_object_property_set_value(&connector->base,
>  				dev->mode_config.dpms_property, dpms_mode);
>  		}
>  	}
> diff --git a/drivers/gpu/drm/drm_sysfs.c b/drivers/gpu/drm/drm_sysfs.c
> index 45ac8d6..1990b5d 100644
> --- a/drivers/gpu/drm/drm_sysfs.c
> +++ b/drivers/gpu/drm/drm_sysfs.c
> @@ -182,7 +182,7 @@ static ssize_t dpms_show(struct device *device,
>  	uint64_t dpms_status;
>  	int ret;
>  
> -	ret = drm_connector_property_get_value(connector,
> +	ret = drm_object_property_get_value(&connector->base,
>  					    dev->mode_config.dpms_property,
>  					    &dpms_status);
>  	if (ret)
> @@ -277,7 +277,7 @@ static ssize_t subconnector_show(struct device *device,
>  		return 0;
>  	}
>  
> -	ret = drm_connector_property_get_value(connector, prop, &subconnector);
> +	ret = drm_object_property_get_value(&connector->base, prop, &subconnector);
>  	if (ret)
>  		return 0;
>  
> @@ -318,7 +318,7 @@ static ssize_t select_subconnector_show(struct device *device,
>  		return 0;
>  	}
>  
> -	ret = drm_connector_property_get_value(connector, prop, &subconnector);
> +	ret = drm_object_property_get_value(&connector->base, prop, &subconnector);
>  	if (ret)
>  		return 0;
>  
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> index 0f532d4..a45cdc2 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -917,12 +917,6 @@ extern void drm_mode_set_crtcinfo(struct drm_display_mode *p,
>  extern void drm_mode_connector_list_update(struct drm_connector *connector);
>  extern int drm_mode_connector_update_edid_property(struct drm_connector *connector,
>  						struct edid *edid);
> -extern int drm_connector_property_set_value(struct drm_connector *connector,
> -					 struct drm_property *property,
> -					 uint64_t value);
> -extern int drm_connector_property_get_value(struct drm_connector *connector,
> -					 struct drm_property *property,
> -					 uint64_t *value);
>  extern int drm_object_property_set_value(struct drm_mode_object *obj,
>  					 struct drm_property *property,
>  					 uint64_t val);
> @@ -937,8 +931,6 @@ extern void drm_framebuffer_reference(struct drm_framebuffer *fb);
>  extern void drm_framebuffer_remove(struct drm_framebuffer *fb);
>  extern void drm_framebuffer_cleanup(struct drm_framebuffer *fb);
>  
> -extern void drm_connector_attach_property(struct drm_connector *connector,
> -					  struct drm_property *property, uint64_t init_val);
>  extern void drm_object_attach_property(struct drm_mode_object *obj,
>  				       struct drm_property *property,
>  				       uint64_t init_val);
> -- 
> 1.7.9.5
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2012-10-15  7:00 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-12  2:05 [PATCH 0/9] cleanup: remove legacy connector property functions Rob Clark
2012-10-12  2:05 ` [PATCH 1/9] drm/i915: drm_connector_property -> drm_object_property Rob Clark
2012-10-15  6:59   ` Jani Nikula
2012-11-21 16:42     ` Daniel Vetter
2012-11-21 21:13   ` Daniel Vetter
2012-10-12  2:05 ` [PATCH 2/9] drm/gma500: " Rob Clark
2012-10-12  2:05 ` [PATCH 3/9] drm/nouveau: " Rob Clark
2012-10-12  2:05 ` [PATCH 4/9] drm/radeon: " Rob Clark
2012-10-12  2:05 ` [PATCH 5/9] drm/shmob: " Rob Clark
2012-10-12  2:05 ` [PATCH 6/9] drm/udl: " Rob Clark
2012-10-12  2:06 ` [PATCH 7/9] drm/vmwgfx: " Rob Clark
2012-10-12  2:06 ` [PATCH 8/9] drm/i2c: " Rob Clark
2012-10-12  2:06 ` [PATCH 9/9] drm: remove legacy drm_connector_property fxns Rob Clark
2012-10-15  7:04   ` Jani Nikula [this message]
2012-10-17 10:52 ` [PATCH 0/9] cleanup: remove legacy connector property functions Laurent Pinchart

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87hapwmd0l.fsf@intel.com \
    --to=jani.nikula@linux.intel.com \
    --cc=alan@linux.intel.com \
    --cc=alexander.deucher@amd.com \
    --cc=bskeggs@redhat.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=patches@linaro.org \
    --cc=rob.clark@linaro.org \
    --cc=rob@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.