On 07/20/2016 02:03 AM, Steve Longerbeam wrote:
> This patch adds a "newavmode" boolean property as part of the v4l2 endpoint
> properties. This indicates an Analog Devices decoder is generating EAV/SAV
> codes to suit Analog Devices encoders.
>
> Signed-off-by: Steve Longerbeam <steve_longerb...@mentor.com>
> Cc: Mauro Carvalho Chehab <mche...@osg.samsung.com>
> Cc: Javier Martinez Canillas <jav...@osg.samsung.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Sakari Ailus <sakari.ai...@linux.intel.com>
> ---
> Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++
> drivers/media/v4l2-core/v4l2-of.c | 4 ++++
> include/media/v4l2-mediabus.h | 5 +++++
> 3 files changed, 11 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt
> b/Documentation/devicetree/bindings/media/video-interfaces.txt
> index 9cd2a36..6f2df51 100644
> --- a/Documentation/devicetree/bindings/media/video-interfaces.txt
> +++ b/Documentation/devicetree/bindings/media/video-interfaces.txt
> @@ -88,6 +88,8 @@ Optional endpoint properties
> - field-even-active: field signal level during the even field data
> transmission.
> - pclk-sample: sample data on rising (1) or falling (0) edge of the pixel
> clock
> signal.
> +- newavmode: a boolean property to indicate an Analog Devices decoder is
> + operating in NEWAVMODE. Valid for BT.656 busses only.
This property is adv7180 specific and does not belong here.
Add this to Documentation/devicetree/bindings/media/i2c/adv7180.txt instead.
Nacked-by: Hans Verkuil <hans.verk...@cisco.com>
Regards,
Hans
> - sync-on-green-active: active state of Sync-on-green (SoG) signal, 0/1 for
> LOW/HIGH respectively.
> - data-lanes: an array of physical data lane indexes. Position of an entry
> diff --git a/drivers/media/v4l2-core/v4l2-of.c
> b/drivers/media/v4l2-core/v4l2-of.c
> index 93b3368..719a7d1 100644
> --- a/drivers/media/v4l2-core/v4l2-of.c
> +++ b/drivers/media/v4l2-core/v4l2-of.c
> @@ -109,6 +109,10 @@ static void v4l2_of_parse_parallel_bus(const struct
> device_node *node,
> flags |= v ? V4L2_MBUS_DATA_ACTIVE_HIGH :
> V4L2_MBUS_DATA_ACTIVE_LOW;
>
> + if (endpoint->bus_type == V4L2_MBUS_BT656 &&
> + of_get_property(node, "newavmode", &v))
> + flags |= V4L2_MBUS_NEWAVMODE;
> +
> if (of_get_property(node, "slave-mode", &v))
> flags |= V4L2_MBUS_SLAVE;
> else
> diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h
> index 34cc99e..0bd5f0e 100644
> --- a/include/media/v4l2-mediabus.h
> +++ b/include/media/v4l2-mediabus.h
> @@ -43,6 +43,11 @@
> /* Active state of Sync-on-green (SoG) signal, 0/1 for LOW/HIGH
> respectively. */
> #define V4L2_MBUS_VIDEO_SOG_ACTIVE_HIGH (1 << 12)
> #define V4L2_MBUS_VIDEO_SOG_ACTIVE_LOW (1 << 13)
> +/*
> + * BT.656 specific flags
> + */
> +/* Analog Device's NEWAVMODE */
> +#define V4L2_MBUS_NEWAVMODE (1 << 14)
>
> /* Serial flags */
> /* How many lanes the client can use */
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
