[media] media: Make parameter of media_entity_remote_pad() const

Message ID 1499083691-30112-1-git-send-email-todor.tomov@linaro.org
State Accepted
Commit 6538b02d210f52ef2a2e67d59fcb58be98451fbd
Headers show

Commit Message

Todor Tomov July 3, 2017, 12:08 p.m.
The local pad parameter in media_entity_remote_pad() is not modified.
Make that explicit by adding a const modifier.

Signed-off-by: Todor Tomov <todor.tomov@linaro.org>

---
 drivers/media/media-entity.c | 2 +-
 include/media/media-entity.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
1.9.1

Comments

Sakari Ailus July 3, 2017, 12:28 p.m. | #1
On Mon, Jul 03, 2017 at 03:08:11PM +0300, Todor Tomov wrote:
> The local pad parameter in media_entity_remote_pad() is not modified.

> Make that explicit by adding a const modifier.

> 

> Signed-off-by: Todor Tomov <todor.tomov@linaro.org>


Thanks!

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>


-- 
Sakari Ailus
e-mail: sakari.ailus@iki.fi	XMPP: sailus@retiisi.org.uk

Patch

diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
index c68239e..60f9de0 100644
--- a/drivers/media/media-entity.c
+++ b/drivers/media/media-entity.c
@@ -854,7 +854,7 @@  struct media_link *
 }
 EXPORT_SYMBOL_GPL(media_entity_find_link);
 
-struct media_pad *media_entity_remote_pad(struct media_pad *pad)
+struct media_pad *media_entity_remote_pad(const struct media_pad *pad)
 {
 	struct media_link *link;
 
diff --git a/include/media/media-entity.h b/include/media/media-entity.h
index b2203ee..bb3a57c 100644
--- a/include/media/media-entity.h
+++ b/include/media/media-entity.h
@@ -804,7 +804,7 @@  struct media_link *media_entity_find_link(struct media_pad *source,
  * Return: returns 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_pad(struct media_pad *pad);
+struct media_pad *media_entity_remote_pad(const struct media_pad *pad);
 
 /**
  * media_entity_get - Get a reference to the parent module