media: v4l: fwnode: Detect bus type correctly

In case the device supports multiple video bus types on an endpoint, the
V4L2 fwnode framework attempts to detect the type based on the available
information. This wasn't working really well, and sometimes could lead to
the V4L2 fwnode endpoint struct as being mishandled between the bus types.

Default to Bt.656 if no properties suggesting a bus type are found.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Tested-by: Steve Longerbeam <steve_longerbeam@mentor.com>
Tested-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c b/drivers/media/v4l2-core/v4l2-fwnode.c
index d6ba3e5d4356594d2c36beedffd662920d88a194..aa3d28c4a50b8521a20d0af5f46b96d232539fc1 100644 (file)
--- a/drivers/media/v4l2-core/v4l2-fwnode.c
+++ b/drivers/media/v4l2-core/v4l2-fwnode.c
@@ -114,8 +114,11 @@ static int v4l2_fwnode_endpoint_parse_csi2_bus(struct fwnode_handle *fwnode,
                flags |= V4L2_MBUS_CSI2_CONTINUOUS_CLOCK;
        }
 
-       bus->flags = flags;
-       vep->bus_type = V4L2_MBUS_CSI2_DPHY;
+       if (lanes_used || have_clk_lane ||
+           (flags & ~V4L2_MBUS_CSI2_CONTINUOUS_CLOCK)) {
+               bus->flags = flags;
+               vep->bus_type = V4L2_MBUS_CSI2_DPHY;
+       }
 
        return 0;
 }
@@ -145,11 +148,6 @@ static void v4l2_fwnode_endpoint_parse_parallel_bus(
                pr_debug("field-even-active %s\n", v ? "high" : "low");
        }
 
-       if (flags)
-               vep->bus_type = V4L2_MBUS_PARALLEL;
-       else
-               vep->bus_type = V4L2_MBUS_BT656;
-
        if (!fwnode_property_read_u32(fwnode, "pclk-sample", &v)) {
                flags |= v ? V4L2_MBUS_PCLK_SAMPLE_RISING :
                        V4L2_MBUS_PCLK_SAMPLE_FALLING;
@@ -192,13 +190,21 @@ static void v4l2_fwnode_endpoint_parse_parallel_bus(
        }
 
        bus->flags = flags;
-
+       if (flags & (V4L2_MBUS_HSYNC_ACTIVE_HIGH |
+                    V4L2_MBUS_HSYNC_ACTIVE_LOW |
+                    V4L2_MBUS_VSYNC_ACTIVE_HIGH |
+                    V4L2_MBUS_VSYNC_ACTIVE_LOW |
+                    V4L2_MBUS_FIELD_EVEN_HIGH |
+                    V4L2_MBUS_FIELD_EVEN_LOW))
+               vep->bus_type = V4L2_MBUS_PARALLEL;
+       else
+               vep->bus_type = V4L2_MBUS_BT656;
 }
 
 static void
 v4l2_fwnode_endpoint_parse_csi1_bus(struct fwnode_handle *fwnode,
                                    struct v4l2_fwnode_endpoint *vep,
-                                   u32 bus_type)
+                                   enum v4l2_fwnode_bus_type bus_type)
 {
        struct v4l2_fwnode_bus_mipi_csi1 *bus = &vep->bus.mipi_csi1;
        u32 v;
@@ -250,11 +256,8 @@ static int __v4l2_fwnode_endpoint_parse(struct fwnode_handle *fwnode,
                rval = v4l2_fwnode_endpoint_parse_csi2_bus(fwnode, vep);
                if (rval)
                        return rval;
-               /*
-                * Parse the parallel video bus properties only if none
-                * of the MIPI CSI-2 specific properties were found.
-                */
-               if (vep->bus.mipi_csi2.flags == 0)
+
+               if (vep->bus_type == V4L2_MBUS_UNKNOWN)
                        v4l2_fwnode_endpoint_parse_parallel_bus(fwnode, vep);
 
                break;

diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h
index 26e1c644ded6629a079f1a316600e5f832620173..df1d552e9df6070f5f665d04f6c362773d8f9b98 100644 (file)
--- a/include/media/v4l2-mediabus.h
+++ b/include/media/v4l2-mediabus.h
@@ -70,6 +70,7 @@
 
 /**
  * enum v4l2_mbus_type - media bus type
+ * @V4L2_MBUS_UNKNOWN: unknown bus type, no V4L2 mediabus configuration
  * @V4L2_MBUS_PARALLEL:        parallel interface with hsync and vsync
  * @V4L2_MBUS_BT656:   parallel interface with embedded synchronisation, can
  *                     also be used for BT.1120
@@ -79,6 +80,7 @@
  * @V4L2_MBUS_CSI2_CPHY: MIPI CSI-2 serial interface, with C-PHY
  */
 enum v4l2_mbus_type {
+       V4L2_MBUS_UNKNOWN,
        V4L2_MBUS_PARALLEL,
        V4L2_MBUS_BT656,
        V4L2_MBUS_CSI1,