Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1632322

[PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous subdevs

Show all headers | View raw

From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>

Devices such as the the ADV748x support multiple parallel stream routes
through a single chip. This leads towards needing to provide multiple
distinct entities and subdevs from a single device-tree node.

To distinguish these separate outputs, the device-tree binding must
specify each endpoint link with a unique (to the device) non-zero port
number.

This number allows async subdev registrations to identify the correct
subdevice to bind and link.

Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
---
 drivers/media/v4l2-core/v4l2-async.c  | 7 +++++++
 drivers/media/v4l2-core/v4l2-subdev.c | 1 +
 include/media/v4l2-async.h            | 1 +
 include/media/v4l2-subdev.h           | 2 ++
 4 files changed, 11 insertions(+)

diff --git a/drivers/media/v4l2-core/v4l2-async.c b/drivers/media/v4l2-core/v4l2-async.c
index 1815e54e8a38..875e6ce646ec 100644
--- a/drivers/media/v4l2-core/v4l2-async.c
+++ b/drivers/media/v4l2-core/v4l2-async.c
@@ -42,6 +42,13 @@ static bool match_devname(struct v4l2_subdev *sd,
 
 static bool match_of(struct v4l2_subdev *sd, struct v4l2_async_subdev *asd)
 {
+	/*
+	 * If set, we must match the device tree port, with the subdev port.
+	 * This is a fast match, so do this first
+	 */
+	if (sd->port && sd->port != asd->match.of.port)
+		return -1;
+
 	return !of_node_cmp(of_node_full_name(sd->of_node),
 			    of_node_full_name(asd->match.of.node));
 }
diff --git a/drivers/media/v4l2-core/v4l2-subdev.c b/drivers/media/v4l2-core/v4l2-subdev.c
index da78497ae5ed..67f816f90ac3 100644
--- a/drivers/media/v4l2-core/v4l2-subdev.c
+++ b/drivers/media/v4l2-core/v4l2-subdev.c
@@ -607,6 +607,7 @@ void v4l2_subdev_init(struct v4l2_subdev *sd, const struct v4l2_subdev_ops *ops)
 	sd->flags = 0;
 	sd->name[0] = '\0';
 	sd->grp_id = 0;
+	sd->port = 0;
 	sd->dev_priv = NULL;
 	sd->host_priv = NULL;
 #if defined(CONFIG_MEDIA_CONTROLLER)
diff --git a/include/media/v4l2-async.h b/include/media/v4l2-async.h
index 5b501309b6a7..2988960613ec 100644
--- a/include/media/v4l2-async.h
+++ b/include/media/v4l2-async.h
@@ -56,6 +56,7 @@ struct v4l2_async_subdev {
 	union {
 		struct {
 			const struct device_node *node;
+			u32 port;
 		} of;
 		struct {
 			const char *name;
diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
index 0ab1c5df6fac..1c1731b491e5 100644
--- a/include/media/v4l2-subdev.h
+++ b/include/media/v4l2-subdev.h
@@ -782,6 +782,7 @@ struct v4l2_subdev_platform_data {
  * @ctrl_handler: The control handler of this subdev. May be NULL.
  * @name: Name of the sub-device. Please notice that the name must be unique.
  * @grp_id: can be used to group similar subdevs. Value is driver-specific
+ * @port: driver-specific value to bind multiple subdevs with a single DT node.
  * @dev_priv: pointer to private data
  * @host_priv: pointer to private data used by the device where the subdev
  *	is attached.
@@ -814,6 +815,7 @@ struct v4l2_subdev {
 	struct v4l2_ctrl_handler *ctrl_handler;
 	char name[V4L2_SUBDEV_NAME_SIZE];
 	u32 grp_id;
+	u32 port;
 	void *dev_priv;
 	void *host_priv;
 	struct video_device *devnode;
-- 
2.7.4

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/5] RFC: ADV748x HDMI/Analog video receiver Kieran Bingham <kbingham@kernel.org> - 2017-04-27 20:30 +0200
  [PATCH 4/5] arm64: dts: r8a7795: salvator-x: enable VIN, CSI and ADV7482 Kieran Bingham <kbingham@kernel.org> - 2017-04-27 20:30 +0200
    Re: [PATCH 4/5] arm64: dts: r8a7795: salvator-x: enable VIN, CSI and  ADV7482 Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> - 2017-04-28 11:00 +0200
      Re: [PATCH 4/5] arm64: dts: r8a7795: salvator-x: enable VIN, CSI and ADV7482 Geert Uytterhoeven <geert@linux-m68k.org> - 2017-04-28 12:10 +0200
        Re: [PATCH 4/5] arm64: dts: r8a7795: salvator-x: enable VIN, CSI and  ADV7482 Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> - 2017-04-28 12:10 +0200
  [PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous subdevs Kieran Bingham <kbingham@kernel.org> - 2017-04-27 20:30 +0200
    Re: [PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous  subdevs Sakari Ailus <sakari.ailus@iki.fi> - 2017-04-27 23:50 +0200
      Re: [PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous  subdevs Kieran Bingham <kieran.bingham@ideasonboard.com> - 2017-04-28 00:20 +0200
        Re: [PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous  subdevs Sakari Ailus <sakari.ailus@iki.fi> - 2017-04-28 01:00 +0200
          Re: [PATCH 1/5] v4l2-subdev: Provide a port mapping for asynchronous  subdevs Kieran Bingham <kieran.bingham@ideasonboard.com> - 2017-04-28 02:10 +0200
  Re: [PATCH 0/5] RFC: ADV748x HDMI/Analog video receiver Simon Horman <horms@verge.net.au> - 2017-04-28 09:20 +0200
    Re: [PATCH 0/5] RFC: ADV748x HDMI/Analog video receiver Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> - 2017-04-28 10:50 +0200
      Re: [PATCH 0/5] RFC: ADV748x HDMI/Analog video receiver Simon Horman <horms@verge.net.au> - 2017-05-01 10:40 +0200

csiph-web