[media] adv7604: Add endpoint properties to DT bindings

Add support for the hsync-active, vsync-active and pclk-sample
properties to the DT bindings and control BT.656 mode implicitly.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

diff --git a/Documentation/devicetree/bindings/media/i2c/adv7604.txt b/Documentation/devicetree/bindings/media/i2c/adv7604.txt
index 2efb48f20fb6247c021610316480f7dd00fc066f..c27cede3bd68adeb72fae1053e09dfffe920a2b2 100644 (file)
--- a/Documentation/devicetree/bindings/media/i2c/adv7604.txt
+++ b/Documentation/devicetree/bindings/media/i2c/adv7604.txt
@@ -33,6 +33,19 @@ Optional Properties:
 
   - reset-gpios: Reference to the GPIO connected to the device's reset pin.
 
+Optional Endpoint Properties:
+
+  The following three properties are defined in video-interfaces.txt and are
+  valid for source endpoints only.
+
+  - hsync-active: Horizontal synchronization polarity. Defaults to active low.
+  - vsync-active: Vertical synchronization polarity. Defaults to active low.
+  - pclk-sample: Pixel clock polarity. Defaults to output on the falling edge.
+
+  If none of hsync-active, vsync-active and pclk-sample is specified the
+  endpoint will use embedded BT.656 synchronization.
+
+
 Example:
 
        hdmi_receiver@4c {

diff --git a/drivers/media/i2c/adv7604.c b/drivers/media/i2c/adv7604.c
index 1323189a45fac9e779be4de93d3ea0ecccd46bda..1493de6c246f08dfb3d0e1621d48eec05d85ea60 100644 (file)
--- a/drivers/media/i2c/adv7604.c
+++ b/drivers/media/i2c/adv7604.c
@@ -41,6 +41,7 @@
 #include <media/v4l2-ctrls.h>
 #include <media/v4l2-device.h>
 #include <media/v4l2-dv-timings.h>
+#include <media/v4l2-of.h>
 
 static int debug;
 module_param(debug, int, 0644);
@@ -2678,6 +2679,37 @@ MODULE_DEVICE_TABLE(of, adv7604_of_id);
 
 static int adv7604_parse_dt(struct adv7604_state *state)
 {
+       struct v4l2_of_endpoint bus_cfg;
+       struct device_node *endpoint;
+       struct device_node *np;
+       unsigned int flags;
+
+       np = state->i2c_clients[ADV7604_PAGE_IO]->dev.of_node;
+
+       /* Parse the endpoint. */
+       endpoint = of_graph_get_next_endpoint(np, NULL);
+       if (!endpoint)
+               return -EINVAL;
+
+       v4l2_of_parse_endpoint(endpoint, &bus_cfg);
+       of_node_put(endpoint);
+
+       flags = bus_cfg.bus.parallel.flags;
+
+       if (flags & V4L2_MBUS_HSYNC_ACTIVE_HIGH)
+               state->pdata.inv_hs_pol = 1;
+
+       if (flags & V4L2_MBUS_VSYNC_ACTIVE_HIGH)
+               state->pdata.inv_vs_pol = 1;
+
+       if (flags & V4L2_MBUS_PCLK_SAMPLE_RISING)
+               state->pdata.inv_llc_pol = 1;
+
+       if (bus_cfg.bus_type == V4L2_MBUS_BT656) {
+               state->pdata.insert_av_codes = 1;
+               state->pdata.op_656_range = 1;
+       }
+
        /* Disable the interrupt for now as no DT-based board uses it. */
        state->pdata.int1_config = ADV7604_INT1_CONFIG_DISABLED;
 
@@ -2700,9 +2732,7 @@ static int adv7604_parse_dt(struct adv7604_state *state)
        state->pdata.disable_cable_det_rst = 0;
        state->pdata.default_input = -1;
        state->pdata.blank_data = 1;
-       state->pdata.op_656_range = 1;
        state->pdata.alt_data_sat = 1;
-       state->pdata.insert_av_codes = 1;
        state->pdata.op_format_mode_sel = ADV7604_OP_FORMAT_MODE0;
        state->pdata.bus_order = ADV7604_BUS_ORDER_RGB;