diff mbox

[RFC] media: Rename media_entity_remote_source to media_entity_remote_pad

Message ID 1358843095-4839-1-git-send-email-a.hajda@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andrzej Hajda Jan. 22, 2013, 8:24 a.m. UTC
Function media_entity_remote_source actually returns the remote pad to
the given one, regardless if this is the source or the sink pad.
Name media_entity_remote_pad is more adequate for this function.

Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 Documentation/media-framework.txt                |    2 +-
 drivers/media/media-entity.c                     |   13 ++++++-------
 drivers/media/platform/omap3isp/isp.c            |    6 +++---
 drivers/media/platform/omap3isp/ispccdc.c        |    2 +-
 drivers/media/platform/omap3isp/ispccp2.c        |    2 +-
 drivers/media/platform/omap3isp/ispcsi2.c        |    2 +-
 drivers/media/platform/omap3isp/ispvideo.c       |    6 +++---
 drivers/media/platform/s3c-camif/camif-capture.c |    2 +-
 drivers/media/platform/s5p-fimc/fimc-capture.c   |    8 ++++----
 drivers/media/platform/s5p-fimc/fimc-lite.c      |    4 ++--
 drivers/media/platform/s5p-fimc/fimc-mdevice.c   |    2 +-
 drivers/staging/media/davinci_vpfe/vpfe_video.c  |   12 ++++++------
 include/media/media-entity.h                     |    2 +-
 13 files changed, 31 insertions(+), 32 deletions(-)

Comments

Sakari Ailus Jan. 22, 2013, 4:37 p.m. UTC | #1
Thanks, Andrzej!

On Tue, Jan 22, 2013 at 09:24:55AM +0100, Andrzej Hajda wrote:
> Function media_entity_remote_source actually returns the remote pad to
> the given one, regardless if this is the source or the sink pad.
> Name media_entity_remote_pad is more adequate for this function.
> 
> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
>  Documentation/media-framework.txt                |    2 +-
>  drivers/media/media-entity.c                     |   13 ++++++-------
>  drivers/media/platform/omap3isp/isp.c            |    6 +++---
>  drivers/media/platform/omap3isp/ispccdc.c        |    2 +-
>  drivers/media/platform/omap3isp/ispccp2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispcsi2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispvideo.c       |    6 +++---
>  drivers/media/platform/s3c-camif/camif-capture.c |    2 +-
>  drivers/media/platform/s5p-fimc/fimc-capture.c   |    8 ++++----
>  drivers/media/platform/s5p-fimc/fimc-lite.c      |    4 ++--
>  drivers/media/platform/s5p-fimc/fimc-mdevice.c   |    2 +-
>  drivers/staging/media/davinci_vpfe/vpfe_video.c  |   12 ++++++------
>  include/media/media-entity.h                     |    2 +-
>  13 files changed, 31 insertions(+), 32 deletions(-)

Acked-by: Sakari Ailus <sakari.ailus@iki.fi>
Hi Andrzej,

Thanks for the patch.

On 01/22/2013 09:24 AM, Andrzej Hajda wrote:
> Function media_entity_remote_source actually returns the remote pad to
> the given one, regardless if this is the source or the sink pad.
> Name media_entity_remote_pad is more adequate for this function.
> 
> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>

Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

> ---
>  Documentation/media-framework.txt                |    2 +-
>  drivers/media/media-entity.c                     |   13 ++++++-------
>  drivers/media/platform/omap3isp/isp.c            |    6 +++---
>  drivers/media/platform/omap3isp/ispccdc.c        |    2 +-
>  drivers/media/platform/omap3isp/ispccp2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispcsi2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispvideo.c       |    6 +++---
>  drivers/media/platform/s3c-camif/camif-capture.c |    2 +-
>  drivers/media/platform/s5p-fimc/fimc-capture.c   |    8 ++++----
>  drivers/media/platform/s5p-fimc/fimc-lite.c      |    4 ++--
>  drivers/media/platform/s5p-fimc/fimc-mdevice.c   |    2 +-
>  drivers/staging/media/davinci_vpfe/vpfe_video.c  |   12 ++++++------
>  include/media/media-entity.h                     |    2 +-
>  13 files changed, 31 insertions(+), 32 deletions(-)

Regards,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sylwester Nawrocki May 26, 2013, 8:59 p.m. UTC | #3
On 01/22/2013 09:24 AM, Andrzej Hajda wrote:
> Function media_entity_remote_source actually returns the remote pad to
> the given one, regardless if this is the source or the sink pad.
> Name media_entity_remote_pad is more adequate for this function.

It seems this patch had slipped through the cracks.

Laurent, would you have any comments on this patch ?

I might have rebased it on top of this series [1], once it is reviewed
and merged.

[1] http://www.spinics.net/lists/linux-media/msg63423.html

Regards,
Sylwester

> Signed-off-by: Andrzej Hajda<a.hajda@samsung.com>
> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
> ---
>   Documentation/media-framework.txt                |    2 +-
>   drivers/media/media-entity.c                     |   13 ++++++-------
>   drivers/media/platform/omap3isp/isp.c            |    6 +++---
>   drivers/media/platform/omap3isp/ispccdc.c        |    2 +-
>   drivers/media/platform/omap3isp/ispccp2.c        |    2 +-
>   drivers/media/platform/omap3isp/ispcsi2.c        |    2 +-
>   drivers/media/platform/omap3isp/ispvideo.c       |    6 +++---
>   drivers/media/platform/s3c-camif/camif-capture.c |    2 +-
>   drivers/media/platform/s5p-fimc/fimc-capture.c   |    8 ++++----
>   drivers/media/platform/s5p-fimc/fimc-lite.c      |    4 ++--
>   drivers/media/platform/s5p-fimc/fimc-mdevice.c   |    2 +-
>   drivers/staging/media/davinci_vpfe/vpfe_video.c  |   12 ++++++------
>   include/media/media-entity.h                     |    2 +-
>   13 files changed, 31 insertions(+), 32 deletions(-)
>
> diff --git a/Documentation/media-framework.txt b/Documentation/media-framework.txt
> index 8028754..e68744a 100644
> --- a/Documentation/media-framework.txt
> +++ b/Documentation/media-framework.txt
> @@ -265,7 +265,7 @@ connected to another pad through an enabled link
>   	media_entity_find_link(struct media_pad *source,
>   			       struct media_pad *sink);
>
> -	media_entity_remote_source(struct media_pad *pad);
> +	media_entity_remote_pad(struct media_pad *pad);
>
>   Refer to the kerneldoc documentation for more information.
>
> diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
> index e1cd132..0438209 100644
> --- a/drivers/media/media-entity.c
> +++ b/drivers/media/media-entity.c
> @@ -560,17 +560,16 @@ media_entity_find_link(struct media_pad *source, struct media_pad *sink)
>   EXPORT_SYMBOL_GPL(media_entity_find_link);
>
>   /**
> - * media_entity_remote_source - Find the source pad at the remote end of a link
> - * @pad: Sink pad at the local end of the link
> + * media_entity_remote_pad - Find the pad at the remote end of a link
> + * @pad: Pad at the local end of the link
>    *
> - * Search for a remote source pad connected to the given sink pad by iterating
> - * over all links originating or terminating at that pad until an enabled link
> - * is found.
> + * Search for a remote pad connected to the given pad by iterating over all
> + * links originating or terminating at that pad until an enabled link is found.
>    *
>    * Return a pointer to the pad at the remote end of the first found enabled
>    * link, or NULL if no enabled link has been found.
>    */
> -struct media_pad *media_entity_remote_source(struct media_pad *pad)
> +struct media_pad *media_entity_remote_pad(struct media_pad *pad)
>   {
>   	unsigned int i;
>
> @@ -590,4 +589,4 @@ struct media_pad *media_entity_remote_source(struct media_pad *pad)
>   	return NULL;
>
>   }
> -EXPORT_SYMBOL_GPL(media_entity_remote_source);
> +EXPORT_SYMBOL_GPL(media_entity_remote_pad);
> diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c
> index a9f6de5..5bb1698 100644
> --- a/drivers/media/platform/omap3isp/isp.c
> +++ b/drivers/media/platform/omap3isp/isp.c
> @@ -757,7 +757,7 @@ static int isp_pipeline_enable(struct isp_pipeline *pipe,
>   		if (!(pad->flags&  MEDIA_PAD_FL_SINK))
>   			break;
>
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> @@ -847,7 +847,7 @@ static int isp_pipeline_disable(struct isp_pipeline *pipe)
>   		if (!(pad->flags&  MEDIA_PAD_FL_SINK))
>   			break;
>
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> @@ -963,7 +963,7 @@ static int isp_pipeline_is_last(struct media_entity *me)
>   	pipe = to_isp_pipeline(me);
>   	if (pipe->stream_state == ISP_PIPELINE_STREAM_STOPPED)
>   		return 0;
> -	pad = media_entity_remote_source(&pipe->output->pad);
> +	pad = media_entity_remote_pad(&pipe->output->pad);
>   	return pad->entity == me;
>   }
>
> diff --git a/drivers/media/platform/omap3isp/ispccdc.c b/drivers/media/platform/omap3isp/ispccdc.c
> index 60e60aa..907a205 100644
> --- a/drivers/media/platform/omap3isp/ispccdc.c
> +++ b/drivers/media/platform/omap3isp/ispccdc.c
> @@ -1120,7 +1120,7 @@ static void ccdc_configure(struct isp_ccdc_device *ccdc)
>   	u32 syn_mode;
>   	u32 ccdc_pattern;
>
> -	pad = media_entity_remote_source(&ccdc->pads[CCDC_PAD_SINK]);
> +	pad = media_entity_remote_pad(&ccdc->pads[CCDC_PAD_SINK]);
>   	sensor = media_entity_to_v4l2_subdev(pad->entity);
>   	if (ccdc->input == CCDC_INPUT_PARALLEL)
>   		pdata =&((struct isp_v4l2_subdevs_group *)sensor->host_priv)
> diff --git a/drivers/media/platform/omap3isp/ispccp2.c b/drivers/media/platform/omap3isp/ispccp2.c
> index 85f0de8..d134e60 100644
> --- a/drivers/media/platform/omap3isp/ispccp2.c
> +++ b/drivers/media/platform/omap3isp/ispccp2.c
> @@ -360,7 +360,7 @@ static int ccp2_if_configure(struct isp_ccp2_device *ccp2)
>
>   	ccp2_pwr_cfg(ccp2);
>
> -	pad = media_entity_remote_source(&ccp2->pads[CCP2_PAD_SINK]);
> +	pad = media_entity_remote_pad(&ccp2->pads[CCP2_PAD_SINK]);
>   	sensor = media_entity_to_v4l2_subdev(pad->entity);
>   	pdata = sensor->host_priv;
>
> diff --git a/drivers/media/platform/omap3isp/ispcsi2.c b/drivers/media/platform/omap3isp/ispcsi2.c
> index 783f4b0..6db245d 100644
> --- a/drivers/media/platform/omap3isp/ispcsi2.c
> +++ b/drivers/media/platform/omap3isp/ispcsi2.c
> @@ -573,7 +573,7 @@ static int csi2_configure(struct isp_csi2_device *csi2)
>   	if (csi2->contexts[0].enabled || csi2->ctrl.if_enable)
>   		return -EBUSY;
>
> -	pad = media_entity_remote_source(&csi2->pads[CSI2_PAD_SINK]);
> +	pad = media_entity_remote_pad(&csi2->pads[CSI2_PAD_SINK]);
>   	sensor = media_entity_to_v4l2_subdev(pad->entity);
>   	pdata = sensor->host_priv;
>
> diff --git a/drivers/media/platform/omap3isp/ispvideo.c b/drivers/media/platform/omap3isp/ispvideo.c
> index e0d73a6..ed3e136 100644
> --- a/drivers/media/platform/omap3isp/ispvideo.c
> +++ b/drivers/media/platform/omap3isp/ispvideo.c
> @@ -222,7 +222,7 @@ isp_video_remote_subdev(struct isp_video *video, u32 *pad)
>   {
>   	struct media_pad *remote;
>
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
>
>   	if (remote == NULL ||
>   	    media_entity_type(remote->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
> @@ -317,7 +317,7 @@ static int isp_video_validate_pipeline(struct isp_pipeline *pipe)
>   		 * entity can be found, and stop checking the pipeline if the
>   		 * source entity isn't a subdev.
>   		 */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL)
>   			return -EPIPE;
>
> @@ -904,7 +904,7 @@ static int isp_video_check_external_subdevs(struct isp_video *video,
>   			continue;
>
>   		/* ISP entities have always sink pad == 0. Find source. */
> -		source_pad = media_entity_remote_source(&ents[i]->pads[0]);
> +		source_pad = media_entity_remote_pad(&ents[i]->pads[0]);
>   		if (source_pad == NULL)
>   			continue;
>
> diff --git a/drivers/media/platform/s3c-camif/camif-capture.c b/drivers/media/platform/s3c-camif/camif-capture.c
> index a55793c..4397722 100644
> --- a/drivers/media/platform/s3c-camif/camif-capture.c
> +++ b/drivers/media/platform/s3c-camif/camif-capture.c
> @@ -845,7 +845,7 @@ static int camif_pipeline_validate(struct camif_dev *camif)
>   	int ret;
>
>   	/* Retrieve format at the sensor subdev source pad */
> -	pad = media_entity_remote_source(&camif->pads[0]);
> +	pad = media_entity_remote_pad(&camif->pads[0]);
>   	if (!pad || media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   		return -EPIPE;
>
> diff --git a/drivers/media/platform/s5p-fimc/fimc-capture.c b/drivers/media/platform/s5p-fimc/fimc-capture.c
> index 18a70e4..b5a0a2f 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-capture.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-capture.c
> @@ -799,7 +799,7 @@ static struct media_entity *fimc_pipeline_get_head(struct media_entity *me)
>   	struct media_pad *pad =&me->pads[0];
>
>   	while (!(pad->flags&  MEDIA_PAD_FL_SOURCE)) {
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (!pad)
>   			break;
>   		me = pad->entity;
> @@ -871,7 +871,7 @@ static int fimc_pipeline_try_format(struct fimc_ctx *ctx,
>   					return ret;
>   			}
>
> -			pad = media_entity_remote_source(&me->pads[sfmt.pad]);
> +			pad = media_entity_remote_pad(&me->pads[sfmt.pad]);
>   			if (!pad)
>   				return -EINVAL;
>   			me = pad->entity;
> @@ -1158,7 +1158,7 @@ static int fimc_pipeline_validate(struct fimc_dev *fimc)
>   	int ret;
>
>   	/* Start with the video capture node pad */
> -	pad = media_entity_remote_source(&vid_cap->vd_pad);
> +	pad = media_entity_remote_pad(&vid_cap->vd_pad);
>   	if (pad == NULL)
>   		return -EPIPE;
>   	/* FIMC.{N} subdevice */
> @@ -1183,7 +1183,7 @@ static int fimc_pipeline_validate(struct fimc_dev *fimc)
>   				return -EPIPE;
>   		}
>   		/* Retrieve format at the source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.c b/drivers/media/platform/s5p-fimc/fimc-lite.c
> index ef31c39..a48abb2 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-lite.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-lite.c
> @@ -774,7 +774,7 @@ static int fimc_pipeline_validate(struct fimc_lite *fimc)
>   				return -EPIPE;
>   		}
>   		/* Retrieve format at the source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> @@ -981,7 +981,7 @@ static struct v4l2_subdev *__find_remote_sensor(struct media_entity *me)
>
>   	while (pad->flags&  MEDIA_PAD_FL_SINK) {
>   		/* source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> diff --git a/drivers/media/platform/s5p-fimc/fimc-mdevice.c b/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> index 62f3a71..8399d31 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> @@ -54,7 +54,7 @@ static void fimc_pipeline_prepare(struct fimc_pipeline *p,
>   			break;
>
>   		/* source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> index 99ccbeb..de52978 100644
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> @@ -39,7 +39,7 @@ static struct media_entity *vpfe_get_input_entity
>   	struct vpfe_device *vpfe_dev = video->vpfe_dev;
>   	struct media_pad *remote;
>
> -	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
> +	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
>   	if (remote == NULL) {
>   		pr_err("Invalid media connection to isif/ccdc\n");
>   		return NULL;
> @@ -56,7 +56,7 @@ static int vpfe_update_current_ext_subdev(struct vpfe_video_device *video)
>   	struct media_pad *remote;
>   	int i;
>
> -	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
> +	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
>   	if (remote == NULL) {
>   		pr_err("Invalid media connection to isif/ccdc\n");
>   		return -EINVAL;
> @@ -89,7 +89,7 @@ static int vpfe_update_current_ext_subdev(struct vpfe_video_device *video)
>   static struct v4l2_subdev *
>   vpfe_video_remote_subdev(struct vpfe_video_device *video, u32 *pad)
>   {
> -	struct media_pad *remote = media_entity_remote_source(&video->pad);
> +	struct media_pad *remote = media_entity_remote_pad(&video->pad);
>
>   	if (remote == NULL || remote->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
>   		return NULL;
> @@ -114,7 +114,7 @@ __vpfe_video_get_format(struct vpfe_video_device *video,
>   		return -EINVAL;
>
>   	fmt.which = V4L2_SUBDEV_FORMAT_ACTIVE;
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
>   	fmt.pad = remote->index;
>
>   	ret = v4l2_subdev_call(subdev, pad, get_fmt, NULL,&fmt);
> @@ -245,7 +245,7 @@ static int vpfe_video_validate_pipeline(struct vpfe_pipeline *pipe)
>   			return -EPIPE;
>
>   		/* Retrieve the source format */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>   		if (pad == NULL ||
>   			pad->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
>   			break;
> @@ -667,7 +667,7 @@ static int vpfe_enum_fmt(struct file *file, void  *priv,
>   		return -EINVAL;
>   	}
>   	/* get the remote pad */
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
>   	if (remote == NULL) {
>   		v4l2_err(&vpfe_dev->v4l2_dev,
>   			 "invalid remote pad for video node\n");
> diff --git a/include/media/media-entity.h b/include/media/media-entity.h
> index 0c16f51..4eefedc 100644
> --- a/include/media/media-entity.h
> +++ b/include/media/media-entity.h
> @@ -132,7 +132,7 @@ int __media_entity_setup_link(struct media_link *link, u32 flags);
>   int media_entity_setup_link(struct media_link *link, u32 flags);
>   struct media_link *media_entity_find_link(struct media_pad *source,
>   		struct media_pad *sink);
> -struct media_pad *media_entity_remote_source(struct media_pad *pad);
> +struct media_pad *media_entity_remote_pad(struct media_pad *pad);
>
>   struct media_entity *media_entity_get(struct media_entity *entity);
>   void media_entity_put(struct media_entity *entity);
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Laurent Pinchart May 30, 2013, 3:08 a.m. UTC | #4
Hi Andrzej,

Thank you for the patch, and sorry for not handling this earlier.

On Tuesday 22 January 2013 09:24:55 Andrzej Hajda wrote:
> Function media_entity_remote_source actually returns the remote pad to
> the given one, regardless if this is the source or the sink pad.
> Name media_entity_remote_pad is more adequate for this function.
> 
> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

However, the patch doesn't apply on top of the latest linuxtv master branch. 
Could you please respin it ?

Mauro, as this patch touches several drivers in addition to the media core, 
would you prefer to apply it yourself, or should I take it in my tree and send 
a pull request ?

> ---
>  Documentation/media-framework.txt                |    2 +-
>  drivers/media/media-entity.c                     |   13 ++++++-------
>  drivers/media/platform/omap3isp/isp.c            |    6 +++---
>  drivers/media/platform/omap3isp/ispccdc.c        |    2 +-
>  drivers/media/platform/omap3isp/ispccp2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispcsi2.c        |    2 +-
>  drivers/media/platform/omap3isp/ispvideo.c       |    6 +++---
>  drivers/media/platform/s3c-camif/camif-capture.c |    2 +-
>  drivers/media/platform/s5p-fimc/fimc-capture.c   |    8 ++++----
>  drivers/media/platform/s5p-fimc/fimc-lite.c      |    4 ++--
>  drivers/media/platform/s5p-fimc/fimc-mdevice.c   |    2 +-
>  drivers/staging/media/davinci_vpfe/vpfe_video.c  |   12 ++++++------
>  include/media/media-entity.h                     |    2 +-
>  13 files changed, 31 insertions(+), 32 deletions(-)
> 
> diff --git a/Documentation/media-framework.txt
> b/Documentation/media-framework.txt index 8028754..e68744a 100644
> --- a/Documentation/media-framework.txt
> +++ b/Documentation/media-framework.txt
> @@ -265,7 +265,7 @@ connected to another pad through an enabled link
>  	media_entity_find_link(struct media_pad *source,
>  			       struct media_pad *sink);
> 
> -	media_entity_remote_source(struct media_pad *pad);
> +	media_entity_remote_pad(struct media_pad *pad);
> 
>  Refer to the kerneldoc documentation for more information.
> 
> diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
> index e1cd132..0438209 100644
> --- a/drivers/media/media-entity.c
> +++ b/drivers/media/media-entity.c
> @@ -560,17 +560,16 @@ media_entity_find_link(struct media_pad *source,
> struct media_pad *sink) EXPORT_SYMBOL_GPL(media_entity_find_link);
> 
>  /**
> - * media_entity_remote_source - Find the source pad at the remote end of a
> link
> - * @pad: Sink pad at the local end of the link
> + * media_entity_remote_pad - Find the pad at the remote end of a link
> + * @pad: Pad at the local end of the link
>   *
> - * Search for a remote source pad connected to the given sink pad by
> - * iterating over all links originating or terminating at that pad until an
> - * enabled link is found.
> + * Search for a remote pad connected to the given pad by iterating over all
> + * links originating or terminating at that pad until an enabled link is
> found.
>   *
>   * Return a pointer to the pad at the remote end of the first found enabled
>   * link, or NULL if no enabled link has been found.
>   */
> -struct media_pad *media_entity_remote_source(struct media_pad *pad)
> +struct media_pad *media_entity_remote_pad(struct media_pad *pad)
>  {
>  	unsigned int i;
> 
> @@ -590,4 +589,4 @@ struct media_pad *media_entity_remote_source(struct
> media_pad *pad) return NULL;
> 
>  }
> -EXPORT_SYMBOL_GPL(media_entity_remote_source);
> +EXPORT_SYMBOL_GPL(media_entity_remote_pad);
> diff --git a/drivers/media/platform/omap3isp/isp.c
> b/drivers/media/platform/omap3isp/isp.c index a9f6de5..5bb1698 100644
> --- a/drivers/media/platform/omap3isp/isp.c
> +++ b/drivers/media/platform/omap3isp/isp.c
> @@ -757,7 +757,7 @@ static int isp_pipeline_enable(struct isp_pipeline
> *pipe, if (!(pad->flags & MEDIA_PAD_FL_SINK))
>  			break;
> 
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> @@ -847,7 +847,7 @@ static int isp_pipeline_disable(struct isp_pipeline
> *pipe) if (!(pad->flags & MEDIA_PAD_FL_SINK))
>  			break;
> 
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> @@ -963,7 +963,7 @@ static int isp_pipeline_is_last(struct media_entity *me)
> pipe = to_isp_pipeline(me);
>  	if (pipe->stream_state == ISP_PIPELINE_STREAM_STOPPED)
>  		return 0;
> -	pad = media_entity_remote_source(&pipe->output->pad);
> +	pad = media_entity_remote_pad(&pipe->output->pad);
>  	return pad->entity == me;
>  }
> 
> diff --git a/drivers/media/platform/omap3isp/ispccdc.c
> b/drivers/media/platform/omap3isp/ispccdc.c index 60e60aa..907a205 100644
> --- a/drivers/media/platform/omap3isp/ispccdc.c
> +++ b/drivers/media/platform/omap3isp/ispccdc.c
> @@ -1120,7 +1120,7 @@ static void ccdc_configure(struct isp_ccdc_device
> *ccdc) u32 syn_mode;
>  	u32 ccdc_pattern;
> 
> -	pad = media_entity_remote_source(&ccdc->pads[CCDC_PAD_SINK]);
> +	pad = media_entity_remote_pad(&ccdc->pads[CCDC_PAD_SINK]);
>  	sensor = media_entity_to_v4l2_subdev(pad->entity);
>  	if (ccdc->input == CCDC_INPUT_PARALLEL)
>  		pdata = &((struct isp_v4l2_subdevs_group *)sensor->host_priv)
> diff --git a/drivers/media/platform/omap3isp/ispccp2.c
> b/drivers/media/platform/omap3isp/ispccp2.c index 85f0de8..d134e60 100644
> --- a/drivers/media/platform/omap3isp/ispccp2.c
> +++ b/drivers/media/platform/omap3isp/ispccp2.c
> @@ -360,7 +360,7 @@ static int ccp2_if_configure(struct isp_ccp2_device
> *ccp2)
> 
>  	ccp2_pwr_cfg(ccp2);
> 
> -	pad = media_entity_remote_source(&ccp2->pads[CCP2_PAD_SINK]);
> +	pad = media_entity_remote_pad(&ccp2->pads[CCP2_PAD_SINK]);
>  	sensor = media_entity_to_v4l2_subdev(pad->entity);
>  	pdata = sensor->host_priv;
> 
> diff --git a/drivers/media/platform/omap3isp/ispcsi2.c
> b/drivers/media/platform/omap3isp/ispcsi2.c index 783f4b0..6db245d 100644
> --- a/drivers/media/platform/omap3isp/ispcsi2.c
> +++ b/drivers/media/platform/omap3isp/ispcsi2.c
> @@ -573,7 +573,7 @@ static int csi2_configure(struct isp_csi2_device *csi2)
>  	if (csi2->contexts[0].enabled || csi2->ctrl.if_enable)
>  		return -EBUSY;
> 
> -	pad = media_entity_remote_source(&csi2->pads[CSI2_PAD_SINK]);
> +	pad = media_entity_remote_pad(&csi2->pads[CSI2_PAD_SINK]);
>  	sensor = media_entity_to_v4l2_subdev(pad->entity);
>  	pdata = sensor->host_priv;
> 
> diff --git a/drivers/media/platform/omap3isp/ispvideo.c
> b/drivers/media/platform/omap3isp/ispvideo.c index e0d73a6..ed3e136 100644
> --- a/drivers/media/platform/omap3isp/ispvideo.c
> +++ b/drivers/media/platform/omap3isp/ispvideo.c
> @@ -222,7 +222,7 @@ isp_video_remote_subdev(struct isp_video *video, u32
> *pad) {
>  	struct media_pad *remote;
> 
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
> 
>  	if (remote == NULL ||
>  	    media_entity_type(remote->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
> @@ -317,7 +317,7 @@ static int isp_video_validate_pipeline(struct
> isp_pipeline *pipe) * entity can be found, and stop checking the pipeline
> if the
>  		 * source entity isn't a subdev.
>  		 */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL)
>  			return -EPIPE;
> 
> @@ -904,7 +904,7 @@ static int isp_video_check_external_subdevs(struct
> isp_video *video, continue;
> 
>  		/* ISP entities have always sink pad == 0. Find source. */
> -		source_pad = media_entity_remote_source(&ents[i]->pads[0]);
> +		source_pad = media_entity_remote_pad(&ents[i]->pads[0]);
>  		if (source_pad == NULL)
>  			continue;
> 
> diff --git a/drivers/media/platform/s3c-camif/camif-capture.c
> b/drivers/media/platform/s3c-camif/camif-capture.c index a55793c..4397722
> 100644
> --- a/drivers/media/platform/s3c-camif/camif-capture.c
> +++ b/drivers/media/platform/s3c-camif/camif-capture.c
> @@ -845,7 +845,7 @@ static int camif_pipeline_validate(struct camif_dev
> *camif) int ret;
> 
>  	/* Retrieve format at the sensor subdev source pad */
> -	pad = media_entity_remote_source(&camif->pads[0]);
> +	pad = media_entity_remote_pad(&camif->pads[0]);
>  	if (!pad || media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  		return -EPIPE;
> 
> diff --git a/drivers/media/platform/s5p-fimc/fimc-capture.c
> b/drivers/media/platform/s5p-fimc/fimc-capture.c index 18a70e4..b5a0a2f
> 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-capture.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-capture.c
> @@ -799,7 +799,7 @@ static struct media_entity
> *fimc_pipeline_get_head(struct media_entity *me) struct media_pad *pad =
> &me->pads[0];
> 
>  	while (!(pad->flags & MEDIA_PAD_FL_SOURCE)) {
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (!pad)
>  			break;
>  		me = pad->entity;
> @@ -871,7 +871,7 @@ static int fimc_pipeline_try_format(struct fimc_ctx
> *ctx, return ret;
>  			}
> 
> -			pad = media_entity_remote_source(&me->pads[sfmt.pad]);
> +			pad = media_entity_remote_pad(&me->pads[sfmt.pad]);
>  			if (!pad)
>  				return -EINVAL;
>  			me = pad->entity;
> @@ -1158,7 +1158,7 @@ static int fimc_pipeline_validate(struct fimc_dev
> *fimc) int ret;
> 
>  	/* Start with the video capture node pad */
> -	pad = media_entity_remote_source(&vid_cap->vd_pad);
> +	pad = media_entity_remote_pad(&vid_cap->vd_pad);
>  	if (pad == NULL)
>  		return -EPIPE;
>  	/* FIMC.{N} subdevice */
> @@ -1183,7 +1183,7 @@ static int fimc_pipeline_validate(struct fimc_dev
> *fimc) return -EPIPE;
>  		}
>  		/* Retrieve format at the source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.c
> b/drivers/media/platform/s5p-fimc/fimc-lite.c index ef31c39..a48abb2 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-lite.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-lite.c
> @@ -774,7 +774,7 @@ static int fimc_pipeline_validate(struct fimc_lite
> *fimc) return -EPIPE;
>  		}
>  		/* Retrieve format at the source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> @@ -981,7 +981,7 @@ static struct v4l2_subdev *__find_remote_sensor(struct
> media_entity *me)
> 
>  	while (pad->flags & MEDIA_PAD_FL_SINK) {
>  		/* source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> diff --git a/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> b/drivers/media/platform/s5p-fimc/fimc-mdevice.c index 62f3a71..8399d31
> 100644
> --- a/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> +++ b/drivers/media/platform/s5p-fimc/fimc-mdevice.c
> @@ -54,7 +54,7 @@ static void fimc_pipeline_prepare(struct fimc_pipeline *p,
> break;
> 
>  		/* source pad */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> b/drivers/staging/media/davinci_vpfe/vpfe_video.c index 99ccbeb..de52978
> 100644
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> @@ -39,7 +39,7 @@ static struct media_entity *vpfe_get_input_entity
>  	struct vpfe_device *vpfe_dev = video->vpfe_dev;
>  	struct media_pad *remote;
> 
> -	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
> +	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
>  	if (remote == NULL) {
>  		pr_err("Invalid media connection to isif/ccdc\n");
>  		return NULL;
> @@ -56,7 +56,7 @@ static int vpfe_update_current_ext_subdev(struct
> vpfe_video_device *video) struct media_pad *remote;
>  	int i;
> 
> -	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
> +	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
>  	if (remote == NULL) {
>  		pr_err("Invalid media connection to isif/ccdc\n");
>  		return -EINVAL;
> @@ -89,7 +89,7 @@ static int vpfe_update_current_ext_subdev(struct
> vpfe_video_device *video) static struct v4l2_subdev *
>  vpfe_video_remote_subdev(struct vpfe_video_device *video, u32 *pad)
>  {
> -	struct media_pad *remote = media_entity_remote_source(&video->pad);
> +	struct media_pad *remote = media_entity_remote_pad(&video->pad);
> 
>  	if (remote == NULL || remote->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
>  		return NULL;
> @@ -114,7 +114,7 @@ __vpfe_video_get_format(struct vpfe_video_device *video,
> return -EINVAL;
> 
>  	fmt.which = V4L2_SUBDEV_FORMAT_ACTIVE;
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
>  	fmt.pad = remote->index;
> 
>  	ret = v4l2_subdev_call(subdev, pad, get_fmt, NULL, &fmt);
> @@ -245,7 +245,7 @@ static int vpfe_video_validate_pipeline(struct
> vpfe_pipeline *pipe) return -EPIPE;
> 
>  		/* Retrieve the source format */
> -		pad = media_entity_remote_source(pad);
> +		pad = media_entity_remote_pad(pad);
>  		if (pad == NULL ||
>  			pad->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
>  			break;
> @@ -667,7 +667,7 @@ static int vpfe_enum_fmt(struct file *file, void  *priv,
> return -EINVAL;
>  	}
>  	/* get the remote pad */
> -	remote = media_entity_remote_source(&video->pad);
> +	remote = media_entity_remote_pad(&video->pad);
>  	if (remote == NULL) {
>  		v4l2_err(&vpfe_dev->v4l2_dev,
>  			 "invalid remote pad for video node\n");
> diff --git a/include/media/media-entity.h b/include/media/media-entity.h
> index 0c16f51..4eefedc 100644
> --- a/include/media/media-entity.h
> +++ b/include/media/media-entity.h
> @@ -132,7 +132,7 @@ int __media_entity_setup_link(struct media_link *link,
> u32 flags); int media_entity_setup_link(struct media_link *link, u32
> flags);
>  struct media_link *media_entity_find_link(struct media_pad *source,
>  		struct media_pad *sink);
> -struct media_pad *media_entity_remote_source(struct media_pad *pad);
> +struct media_pad *media_entity_remote_pad(struct media_pad *pad);
> 
>  struct media_entity *media_entity_get(struct media_entity *entity);
>  void media_entity_put(struct media_entity *entity);
diff mbox

Patch

diff --git a/Documentation/media-framework.txt b/Documentation/media-framework.txt
index 8028754..e68744a 100644
--- a/Documentation/media-framework.txt
+++ b/Documentation/media-framework.txt
@@ -265,7 +265,7 @@  connected to another pad through an enabled link
 	media_entity_find_link(struct media_pad *source,
 			       struct media_pad *sink);
 
-	media_entity_remote_source(struct media_pad *pad);
+	media_entity_remote_pad(struct media_pad *pad);
 
 Refer to the kerneldoc documentation for more information.
 
diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
index e1cd132..0438209 100644
--- a/drivers/media/media-entity.c
+++ b/drivers/media/media-entity.c
@@ -560,17 +560,16 @@  media_entity_find_link(struct media_pad *source, struct media_pad *sink)
 EXPORT_SYMBOL_GPL(media_entity_find_link);
 
 /**
- * media_entity_remote_source - Find the source pad at the remote end of a link
- * @pad: Sink pad at the local end of the link
+ * media_entity_remote_pad - Find the pad at the remote end of a link
+ * @pad: Pad at the local end of the link
  *
- * Search for a remote source pad connected to the given sink pad by iterating
- * over all links originating or terminating at that pad until an enabled link
- * is found.
+ * Search for a remote pad connected to the given pad by iterating over all
+ * links originating or terminating at that pad until an enabled link is found.
  *
  * Return a pointer to the pad at the remote end of the first found enabled
  * link, or NULL if no enabled link has been found.
  */
-struct media_pad *media_entity_remote_source(struct media_pad *pad)
+struct media_pad *media_entity_remote_pad(struct media_pad *pad)
 {
 	unsigned int i;
 
@@ -590,4 +589,4 @@  struct media_pad *media_entity_remote_source(struct media_pad *pad)
 	return NULL;
 
 }
-EXPORT_SYMBOL_GPL(media_entity_remote_source);
+EXPORT_SYMBOL_GPL(media_entity_remote_pad);
diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c
index a9f6de5..5bb1698 100644
--- a/drivers/media/platform/omap3isp/isp.c
+++ b/drivers/media/platform/omap3isp/isp.c
@@ -757,7 +757,7 @@  static int isp_pipeline_enable(struct isp_pipeline *pipe,
 		if (!(pad->flags & MEDIA_PAD_FL_SINK))
 			break;
 
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
@@ -847,7 +847,7 @@  static int isp_pipeline_disable(struct isp_pipeline *pipe)
 		if (!(pad->flags & MEDIA_PAD_FL_SINK))
 			break;
 
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
@@ -963,7 +963,7 @@  static int isp_pipeline_is_last(struct media_entity *me)
 	pipe = to_isp_pipeline(me);
 	if (pipe->stream_state == ISP_PIPELINE_STREAM_STOPPED)
 		return 0;
-	pad = media_entity_remote_source(&pipe->output->pad);
+	pad = media_entity_remote_pad(&pipe->output->pad);
 	return pad->entity == me;
 }
 
diff --git a/drivers/media/platform/omap3isp/ispccdc.c b/drivers/media/platform/omap3isp/ispccdc.c
index 60e60aa..907a205 100644
--- a/drivers/media/platform/omap3isp/ispccdc.c
+++ b/drivers/media/platform/omap3isp/ispccdc.c
@@ -1120,7 +1120,7 @@  static void ccdc_configure(struct isp_ccdc_device *ccdc)
 	u32 syn_mode;
 	u32 ccdc_pattern;
 
-	pad = media_entity_remote_source(&ccdc->pads[CCDC_PAD_SINK]);
+	pad = media_entity_remote_pad(&ccdc->pads[CCDC_PAD_SINK]);
 	sensor = media_entity_to_v4l2_subdev(pad->entity);
 	if (ccdc->input == CCDC_INPUT_PARALLEL)
 		pdata = &((struct isp_v4l2_subdevs_group *)sensor->host_priv)
diff --git a/drivers/media/platform/omap3isp/ispccp2.c b/drivers/media/platform/omap3isp/ispccp2.c
index 85f0de8..d134e60 100644
--- a/drivers/media/platform/omap3isp/ispccp2.c
+++ b/drivers/media/platform/omap3isp/ispccp2.c
@@ -360,7 +360,7 @@  static int ccp2_if_configure(struct isp_ccp2_device *ccp2)
 
 	ccp2_pwr_cfg(ccp2);
 
-	pad = media_entity_remote_source(&ccp2->pads[CCP2_PAD_SINK]);
+	pad = media_entity_remote_pad(&ccp2->pads[CCP2_PAD_SINK]);
 	sensor = media_entity_to_v4l2_subdev(pad->entity);
 	pdata = sensor->host_priv;
 
diff --git a/drivers/media/platform/omap3isp/ispcsi2.c b/drivers/media/platform/omap3isp/ispcsi2.c
index 783f4b0..6db245d 100644
--- a/drivers/media/platform/omap3isp/ispcsi2.c
+++ b/drivers/media/platform/omap3isp/ispcsi2.c
@@ -573,7 +573,7 @@  static int csi2_configure(struct isp_csi2_device *csi2)
 	if (csi2->contexts[0].enabled || csi2->ctrl.if_enable)
 		return -EBUSY;
 
-	pad = media_entity_remote_source(&csi2->pads[CSI2_PAD_SINK]);
+	pad = media_entity_remote_pad(&csi2->pads[CSI2_PAD_SINK]);
 	sensor = media_entity_to_v4l2_subdev(pad->entity);
 	pdata = sensor->host_priv;
 
diff --git a/drivers/media/platform/omap3isp/ispvideo.c b/drivers/media/platform/omap3isp/ispvideo.c
index e0d73a6..ed3e136 100644
--- a/drivers/media/platform/omap3isp/ispvideo.c
+++ b/drivers/media/platform/omap3isp/ispvideo.c
@@ -222,7 +222,7 @@  isp_video_remote_subdev(struct isp_video *video, u32 *pad)
 {
 	struct media_pad *remote;
 
-	remote = media_entity_remote_source(&video->pad);
+	remote = media_entity_remote_pad(&video->pad);
 
 	if (remote == NULL ||
 	    media_entity_type(remote->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
@@ -317,7 +317,7 @@  static int isp_video_validate_pipeline(struct isp_pipeline *pipe)
 		 * entity can be found, and stop checking the pipeline if the
 		 * source entity isn't a subdev.
 		 */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL)
 			return -EPIPE;
 
@@ -904,7 +904,7 @@  static int isp_video_check_external_subdevs(struct isp_video *video,
 			continue;
 
 		/* ISP entities have always sink pad == 0. Find source. */
-		source_pad = media_entity_remote_source(&ents[i]->pads[0]);
+		source_pad = media_entity_remote_pad(&ents[i]->pads[0]);
 		if (source_pad == NULL)
 			continue;
 
diff --git a/drivers/media/platform/s3c-camif/camif-capture.c b/drivers/media/platform/s3c-camif/camif-capture.c
index a55793c..4397722 100644
--- a/drivers/media/platform/s3c-camif/camif-capture.c
+++ b/drivers/media/platform/s3c-camif/camif-capture.c
@@ -845,7 +845,7 @@  static int camif_pipeline_validate(struct camif_dev *camif)
 	int ret;
 
 	/* Retrieve format at the sensor subdev source pad */
-	pad = media_entity_remote_source(&camif->pads[0]);
+	pad = media_entity_remote_pad(&camif->pads[0]);
 	if (!pad || media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 		return -EPIPE;
 
diff --git a/drivers/media/platform/s5p-fimc/fimc-capture.c b/drivers/media/platform/s5p-fimc/fimc-capture.c
index 18a70e4..b5a0a2f 100644
--- a/drivers/media/platform/s5p-fimc/fimc-capture.c
+++ b/drivers/media/platform/s5p-fimc/fimc-capture.c
@@ -799,7 +799,7 @@  static struct media_entity *fimc_pipeline_get_head(struct media_entity *me)
 	struct media_pad *pad = &me->pads[0];
 
 	while (!(pad->flags & MEDIA_PAD_FL_SOURCE)) {
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (!pad)
 			break;
 		me = pad->entity;
@@ -871,7 +871,7 @@  static int fimc_pipeline_try_format(struct fimc_ctx *ctx,
 					return ret;
 			}
 
-			pad = media_entity_remote_source(&me->pads[sfmt.pad]);
+			pad = media_entity_remote_pad(&me->pads[sfmt.pad]);
 			if (!pad)
 				return -EINVAL;
 			me = pad->entity;
@@ -1158,7 +1158,7 @@  static int fimc_pipeline_validate(struct fimc_dev *fimc)
 	int ret;
 
 	/* Start with the video capture node pad */
-	pad = media_entity_remote_source(&vid_cap->vd_pad);
+	pad = media_entity_remote_pad(&vid_cap->vd_pad);
 	if (pad == NULL)
 		return -EPIPE;
 	/* FIMC.{N} subdevice */
@@ -1183,7 +1183,7 @@  static int fimc_pipeline_validate(struct fimc_dev *fimc)
 				return -EPIPE;
 		}
 		/* Retrieve format at the source pad */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.c b/drivers/media/platform/s5p-fimc/fimc-lite.c
index ef31c39..a48abb2 100644
--- a/drivers/media/platform/s5p-fimc/fimc-lite.c
+++ b/drivers/media/platform/s5p-fimc/fimc-lite.c
@@ -774,7 +774,7 @@  static int fimc_pipeline_validate(struct fimc_lite *fimc)
 				return -EPIPE;
 		}
 		/* Retrieve format at the source pad */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
@@ -981,7 +981,7 @@  static struct v4l2_subdev *__find_remote_sensor(struct media_entity *me)
 
 	while (pad->flags & MEDIA_PAD_FL_SINK) {
 		/* source pad */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
diff --git a/drivers/media/platform/s5p-fimc/fimc-mdevice.c b/drivers/media/platform/s5p-fimc/fimc-mdevice.c
index 62f3a71..8399d31 100644
--- a/drivers/media/platform/s5p-fimc/fimc-mdevice.c
+++ b/drivers/media/platform/s5p-fimc/fimc-mdevice.c
@@ -54,7 +54,7 @@  static void fimc_pipeline_prepare(struct fimc_pipeline *p,
 			break;
 
 		/* source pad */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 		    media_entity_type(pad->entity) != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c b/drivers/staging/media/davinci_vpfe/vpfe_video.c
index 99ccbeb..de52978 100644
--- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
+++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
@@ -39,7 +39,7 @@  static struct media_entity *vpfe_get_input_entity
 	struct vpfe_device *vpfe_dev = video->vpfe_dev;
 	struct media_pad *remote;
 
-	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
+	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
 	if (remote == NULL) {
 		pr_err("Invalid media connection to isif/ccdc\n");
 		return NULL;
@@ -56,7 +56,7 @@  static int vpfe_update_current_ext_subdev(struct vpfe_video_device *video)
 	struct media_pad *remote;
 	int i;
 
-	remote = media_entity_remote_source(&vpfe_dev->vpfe_isif.pads[0]);
+	remote = media_entity_remote_pad(&vpfe_dev->vpfe_isif.pads[0]);
 	if (remote == NULL) {
 		pr_err("Invalid media connection to isif/ccdc\n");
 		return -EINVAL;
@@ -89,7 +89,7 @@  static int vpfe_update_current_ext_subdev(struct vpfe_video_device *video)
 static struct v4l2_subdev *
 vpfe_video_remote_subdev(struct vpfe_video_device *video, u32 *pad)
 {
-	struct media_pad *remote = media_entity_remote_source(&video->pad);
+	struct media_pad *remote = media_entity_remote_pad(&video->pad);
 
 	if (remote == NULL || remote->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
 		return NULL;
@@ -114,7 +114,7 @@  __vpfe_video_get_format(struct vpfe_video_device *video,
 		return -EINVAL;
 
 	fmt.which = V4L2_SUBDEV_FORMAT_ACTIVE;
-	remote = media_entity_remote_source(&video->pad);
+	remote = media_entity_remote_pad(&video->pad);
 	fmt.pad = remote->index;
 
 	ret = v4l2_subdev_call(subdev, pad, get_fmt, NULL, &fmt);
@@ -245,7 +245,7 @@  static int vpfe_video_validate_pipeline(struct vpfe_pipeline *pipe)
 			return -EPIPE;
 
 		/* Retrieve the source format */
-		pad = media_entity_remote_source(pad);
+		pad = media_entity_remote_pad(pad);
 		if (pad == NULL ||
 			pad->entity->type != MEDIA_ENT_T_V4L2_SUBDEV)
 			break;
@@ -667,7 +667,7 @@  static int vpfe_enum_fmt(struct file *file, void  *priv,
 		return -EINVAL;
 	}
 	/* get the remote pad */
-	remote = media_entity_remote_source(&video->pad);
+	remote = media_entity_remote_pad(&video->pad);
 	if (remote == NULL) {
 		v4l2_err(&vpfe_dev->v4l2_dev,
 			 "invalid remote pad for video node\n");
diff --git a/include/media/media-entity.h b/include/media/media-entity.h
index 0c16f51..4eefedc 100644
--- a/include/media/media-entity.h
+++ b/include/media/media-entity.h
@@ -132,7 +132,7 @@  int __media_entity_setup_link(struct media_link *link, u32 flags);
 int media_entity_setup_link(struct media_link *link, u32 flags);
 struct media_link *media_entity_find_link(struct media_pad *source,
 		struct media_pad *sink);
-struct media_pad *media_entity_remote_source(struct media_pad *pad);
+struct media_pad *media_entity_remote_pad(struct media_pad *pad);
 
 struct media_entity *media_entity_get(struct media_entity *entity);
 void media_entity_put(struct media_entity *entity);