v4l: add support for CSI-1 bus
Commit Message
From: Sakari Ailus <sakari.ailus@iki.fi>
The function to parse CSI2 bus parameters was called
v4l2_of_parse_csi_bus(), rename it as v4l2_of_parse_csi2_bus() in
anticipation of CSI1/CCP2 support.
Obtain data bus type from bus-type property. Only try parsing bus
specific properties in this case.
Add CSI1 and CCP2 bus type to enum v4l2_mbus_type. CCP2, or CSI-1, is
an older single data lane serial bus.
Separate lane parsing from CSI-2 bus parameter parsing. The CSI-1 will
need these as well, separate them into a different
function. have_clk_lane and num_data_lanes arguments may be NULL; the
CSI-1 bus will have no use for them.
Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
Comments
Hi Sakari,
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.10-rc1 next-20161224]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Pavel-Machek/v4l-add-support-for-CSI-1-bus/20161229-043410
base: git://linuxtv.org/media_tree.git master
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All warnings (new ones prefixed by >>):
drivers/media/platform/pxa_camera.c: In function 'pxa_mbus_config_compatible':
>> drivers/media/platform/pxa_camera.c:595:2: warning: enumeration value 'V4L2_MBUS_CSI1' not handled in switch [-Wswitch]
switch (cfg->type) {
^~~~~~
>> drivers/media/platform/pxa_camera.c:595:2: warning: enumeration value 'V4L2_MBUS_CCP2' not handled in switch [-Wswitch]
vim +/V4L2_MBUS_CSI1 +595 drivers/media/platform/pxa_camera.c
34b27b13 Hans Verkuil 2016-09-11 579
34b27b13 Hans Verkuil 2016-09-11 580 static const struct pxa_mbus_pixelfmt *pxa_mbus_get_fmtdesc(
34b27b13 Hans Verkuil 2016-09-11 581 u32 code)
34b27b13 Hans Verkuil 2016-09-11 582 {
34b27b13 Hans Verkuil 2016-09-11 583 return pxa_mbus_find_fmtdesc(code, mbus_fmt, ARRAY_SIZE(mbus_fmt));
34b27b13 Hans Verkuil 2016-09-11 584 }
34b27b13 Hans Verkuil 2016-09-11 585
34b27b13 Hans Verkuil 2016-09-11 586 static unsigned int pxa_mbus_config_compatible(const struct v4l2_mbus_config *cfg,
34b27b13 Hans Verkuil 2016-09-11 587 unsigned int flags)
34b27b13 Hans Verkuil 2016-09-11 588 {
34b27b13 Hans Verkuil 2016-09-11 589 unsigned long common_flags;
34b27b13 Hans Verkuil 2016-09-11 590 bool hsync = true, vsync = true, pclk, data, mode;
34b27b13 Hans Verkuil 2016-09-11 591 bool mipi_lanes, mipi_clock;
34b27b13 Hans Verkuil 2016-09-11 592
34b27b13 Hans Verkuil 2016-09-11 593 common_flags = cfg->flags & flags;
34b27b13 Hans Verkuil 2016-09-11 594
34b27b13 Hans Verkuil 2016-09-11 @595 switch (cfg->type) {
34b27b13 Hans Verkuil 2016-09-11 596 case V4L2_MBUS_PARALLEL:
34b27b13 Hans Verkuil 2016-09-11 597 hsync = common_flags & (V4L2_MBUS_HSYNC_ACTIVE_HIGH |
34b27b13 Hans Verkuil 2016-09-11 598 V4L2_MBUS_HSYNC_ACTIVE_LOW);
34b27b13 Hans Verkuil 2016-09-11 599 vsync = common_flags & (V4L2_MBUS_VSYNC_ACTIVE_HIGH |
34b27b13 Hans Verkuil 2016-09-11 600 V4L2_MBUS_VSYNC_ACTIVE_LOW);
34b27b13 Hans Verkuil 2016-09-11 601 /* fall through */
34b27b13 Hans Verkuil 2016-09-11 602 case V4L2_MBUS_BT656:
34b27b13 Hans Verkuil 2016-09-11 603 pclk = common_flags & (V4L2_MBUS_PCLK_SAMPLE_RISING |
:::::: The code at line 595 was first introduced by commit
:::::: 34b27b13a2cd43e0d27e3ce13b96119efbbbedbc [media] pxa_camera: merge soc_mediabus.c into pxa_camera.c
:::::: TO: Hans Verkuil <hans.verkuil@cisco.com>
:::::: CC: Mauro Carvalho Chehab <mchehab@s-opensource.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Hi Pavel,
On Wed, Dec 28, 2016 at 07:31:16PM +0100, Pavel Machek wrote:
> From: Sakari Ailus <sakari.ailus@iki.fi>
>
> The function to parse CSI2 bus parameters was called
> v4l2_of_parse_csi_bus(), rename it as v4l2_of_parse_csi2_bus() in
> anticipation of CSI1/CCP2 support.
>
> Obtain data bus type from bus-type property. Only try parsing bus
> specific properties in this case.
>
> Add CSI1 and CCP2 bus type to enum v4l2_mbus_type. CCP2, or CSI-1, is
> an older single data lane serial bus.
>
> Separate lane parsing from CSI-2 bus parameter parsing. The CSI-1 will
> need these as well, separate them into a different
> function. have_clk_lane and num_data_lanes arguments may be NULL; the
> CSI-1 bus will have no use for them.
The patch adds the CCP2 and CSI1 bus types to the enum, but it does not yet
add support for parsing either endpoints. How about separating the enum
changes to a different patch and keeping this one just for splitting lane
parsing from CSI-2 specific code?
>
> Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
> Signed-off-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>
> diff --git a/drivers/media/v4l2-core/v4l2-of.c b/drivers/media/v4l2-core/v4l2-of.c
> index 93b3368..60bbc5f 100644
> --- a/drivers/media/v4l2-core/v4l2-of.c
> +++ b/drivers/media/v4l2-core/v4l2-of.c
> @@ -20,53 +20,88 @@
>
> #include <media/v4l2-of.h>
>
> -static int v4l2_of_parse_csi_bus(const struct device_node *node,
> - struct v4l2_of_endpoint *endpoint)
> +enum v4l2_of_bus_type {
> + V4L2_OF_BUS_TYPE_CSI2 = 0,
> + V4L2_OF_BUS_TYPE_PARALLEL,
> +};
> +
> +static int v4l2_of_parse_lanes(const struct device_node *node,
> + unsigned char *clock_lane,
> + bool *have_clk_lane,
> + unsigned char *data_lanes,
> + bool *lane_polarities,
> + unsigned short *__num_data_lanes,
> + unsigned int max_data_lanes)
> {
> - struct v4l2_of_bus_mipi_csi2 *bus = &endpoint->bus.mipi_csi2;
> struct property *prop;
> - bool have_clk_lane = false;
> - unsigned int flags = 0;
> + unsigned short num_data_lanes = 0;
> u32 v;
>
> prop = of_find_property(node, "data-lanes", NULL);
> if (prop) {
> const __be32 *lane = NULL;
> - unsigned int i;
>
> - for (i = 0; i < ARRAY_SIZE(bus->data_lanes); i++) {
> + for (num_data_lanes = 0; num_data_lanes < max_data_lanes;
> + num_data_lanes++) {
> lane = of_prop_next_u32(prop, lane, &v);
> if (!lane)
> break;
> - bus->data_lanes[i] = v;
> + data_lanes[num_data_lanes] = v;
> }
> - bus->num_data_lanes = i;
> }
> + if (__num_data_lanes)
> + *__num_data_lanes = num_data_lanes;
>
> prop = of_find_property(node, "lane-polarities", NULL);
> if (prop) {
> const __be32 *polarity = NULL;
> unsigned int i;
>
> - for (i = 0; i < ARRAY_SIZE(bus->lane_polarities); i++) {
> + for (i = 0; i < 1 + max_data_lanes; i++) {
> polarity = of_prop_next_u32(prop, polarity, &v);
> if (!polarity)
> break;
> - bus->lane_polarities[i] = v;
> + lane_polarities[i] = v;
> }
>
> - if (i < 1 + bus->num_data_lanes /* clock + data */) {
> + if (i < 1 + num_data_lanes /* clock + data */) {
> pr_warn("%s: too few lane-polarities entries (need %u, got %u)\n",
> - node->full_name, 1 + bus->num_data_lanes, i);
> + node->full_name, 1 + num_data_lanes, i);
> return -EINVAL;
> }
> }
>
> + if (have_clk_lane)
> + *have_clk_lane = false;
> +
> if (!of_property_read_u32(node, "clock-lanes", &v)) {
> - bus->clock_lane = v;
> - have_clk_lane = true;
> + *clock_lane = v;
> + if (have_clk_lane)
> + *have_clk_lane = true;
> }
>
> + return 0;
> +}
> +
> +static int v4l2_of_parse_csi2_bus(const struct device_node *node,
> + struct v4l2_of_endpoint *endpoint)
> +{
> + struct v4l2_of_bus_mipi_csi2 *bus = &endpoint->bus.mipi_csi2;
> + bool have_clk_lane = false;
> + unsigned int flags = 0;
> + int rval;
> + u32 v;
> +
> + rval = v4l2_of_parse_lanes(node, &bus->clock_lane, &have_clk_lane,
> + bus->data_lanes, bus->lane_polarities,
> + &bus->num_data_lanes,
> + ARRAY_SIZE(bus->data_lanes));
> + if (rval)
> + return rval;
> +
> + BUILD_BUG_ON(1 + ARRAY_SIZE(bus->data_lanes)
> + != ARRAY_SIZE(bus->lane_polarities));
> +
> if (of_get_property(node, "clock-noncontinuous", &v))
> flags |= V4L2_MBUS_CSI2_NONCONTINUOUS_CLOCK;
> else if (have_clk_lane || bus->num_data_lanes > 0)
> @@ -151,6 +186,7 @@ static void v4l2_of_parse_parallel_bus(const struct device_node *node,
> int v4l2_of_parse_endpoint(const struct device_node *node,
> struct v4l2_of_endpoint *endpoint)
> {
> + u32 bus_type;
> int rval;
>
> of_graph_parse_endpoint(node, &endpoint->base);
> @@ -158,17 +194,33 @@ int v4l2_of_parse_endpoint(const struct device_node *node,
> memset(&endpoint->bus_type, 0, sizeof(*endpoint) -
> offsetof(typeof(*endpoint), bus_type));
>
> - rval = v4l2_of_parse_csi_bus(node, endpoint);
> - if (rval)
> - return rval;
> - /*
> - * Parse the parallel video bus properties only if none
> - * of the MIPI CSI-2 specific properties were found.
> - */
> - if (endpoint->bus.mipi_csi2.flags == 0)
> - v4l2_of_parse_parallel_bus(node, endpoint);
> + rval = of_property_read_u32(node, "bus-type", &bus_type);
> + if (rval < 0) {
> + endpoint->bus_type = 0;
> + rval = v4l2_of_parse_csi2_bus(node, endpoint);
> + if (rval)
> + return rval;
> + /*
> + * Parse the parallel video bus properties only if none
> + * of the MIPI CSI-2 specific properties were found.
> + */
> + if (endpoint->bus.mipi_csi2.flags == 0)
> + v4l2_of_parse_parallel_bus(node, endpoint);
> +
> + return 0;
> + }
>
> - return 0;
> + switch (bus_type) {
> + case V4L2_OF_BUS_TYPE_CSI2:
> + return v4l2_of_parse_csi2_bus(node, endpoint);
> + case V4L2_OF_BUS_TYPE_PARALLEL:
> + v4l2_of_parse_parallel_bus(node, endpoint);
> + return 0;
> + default:
> + pr_warn("bad bus-type %u, device_node \"%s\"\n",
> + bus_type, node->full_name);
> + return -EINVAL;
> + }
> }
> EXPORT_SYMBOL(v4l2_of_parse_endpoint);
>
> diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h
> index 34cc99e..315c167 100644
> --- a/include/media/v4l2-mediabus.h
> +++ b/include/media/v4l2-mediabus.h
> @@ -69,11 +69,15 @@
> * @V4L2_MBUS_PARALLEL: parallel interface with hsync and vsync
> * @V4L2_MBUS_BT656: parallel interface with embedded synchronisation, can
> * also be used for BT.1120
> + * @V4L2_MBUS_CSI1: MIPI CSI-1 serial interface
> + * @V4L2_MBUS_CCP2: CCP2 (Compact Camera Port 2)
> * @V4L2_MBUS_CSI2: MIPI CSI-2 serial interface
> */
> enum v4l2_mbus_type {
> V4L2_MBUS_PARALLEL,
> V4L2_MBUS_BT656,
> + V4L2_MBUS_CSI1,
> + V4L2_MBUS_CCP2,
> V4L2_MBUS_CSI2,
> };
>
>
@@ -20,53 +20,88 @@
#include <media/v4l2-of.h>
-static int v4l2_of_parse_csi_bus(const struct device_node *node,
- struct v4l2_of_endpoint *endpoint)
+enum v4l2_of_bus_type {
+ V4L2_OF_BUS_TYPE_CSI2 = 0,
+ V4L2_OF_BUS_TYPE_PARALLEL,
+};
+
+static int v4l2_of_parse_lanes(const struct device_node *node,
+ unsigned char *clock_lane,
+ bool *have_clk_lane,
+ unsigned char *data_lanes,
+ bool *lane_polarities,
+ unsigned short *__num_data_lanes,
+ unsigned int max_data_lanes)
{
- struct v4l2_of_bus_mipi_csi2 *bus = &endpoint->bus.mipi_csi2;
struct property *prop;
- bool have_clk_lane = false;
- unsigned int flags = 0;
+ unsigned short num_data_lanes = 0;
u32 v;
prop = of_find_property(node, "data-lanes", NULL);
if (prop) {
const __be32 *lane = NULL;
- unsigned int i;
- for (i = 0; i < ARRAY_SIZE(bus->data_lanes); i++) {
+ for (num_data_lanes = 0; num_data_lanes < max_data_lanes;
+ num_data_lanes++) {
lane = of_prop_next_u32(prop, lane, &v);
if (!lane)
break;
- bus->data_lanes[i] = v;
+ data_lanes[num_data_lanes] = v;
}
- bus->num_data_lanes = i;
}
+ if (__num_data_lanes)
+ *__num_data_lanes = num_data_lanes;
prop = of_find_property(node, "lane-polarities", NULL);
if (prop) {
const __be32 *polarity = NULL;
unsigned int i;
- for (i = 0; i < ARRAY_SIZE(bus->lane_polarities); i++) {
+ for (i = 0; i < 1 + max_data_lanes; i++) {
polarity = of_prop_next_u32(prop, polarity, &v);
if (!polarity)
break;
- bus->lane_polarities[i] = v;
+ lane_polarities[i] = v;
}
- if (i < 1 + bus->num_data_lanes /* clock + data */) {
+ if (i < 1 + num_data_lanes /* clock + data */) {
pr_warn("%s: too few lane-polarities entries (need %u, got %u)\n",
- node->full_name, 1 + bus->num_data_lanes, i);
+ node->full_name, 1 + num_data_lanes, i);
return -EINVAL;
}
}
+ if (have_clk_lane)
+ *have_clk_lane = false;
+
if (!of_property_read_u32(node, "clock-lanes", &v)) {
- bus->clock_lane = v;
- have_clk_lane = true;
+ *clock_lane = v;
+ if (have_clk_lane)
+ *have_clk_lane = true;
}
+ return 0;
+}
+
+static int v4l2_of_parse_csi2_bus(const struct device_node *node,
+ struct v4l2_of_endpoint *endpoint)
+{
+ struct v4l2_of_bus_mipi_csi2 *bus = &endpoint->bus.mipi_csi2;
+ bool have_clk_lane = false;
+ unsigned int flags = 0;
+ int rval;
+ u32 v;
+
+ rval = v4l2_of_parse_lanes(node, &bus->clock_lane, &have_clk_lane,
+ bus->data_lanes, bus->lane_polarities,
+ &bus->num_data_lanes,
+ ARRAY_SIZE(bus->data_lanes));
+ if (rval)
+ return rval;
+
+ BUILD_BUG_ON(1 + ARRAY_SIZE(bus->data_lanes)
+ != ARRAY_SIZE(bus->lane_polarities));
+
if (of_get_property(node, "clock-noncontinuous", &v))
flags |= V4L2_MBUS_CSI2_NONCONTINUOUS_CLOCK;
else if (have_clk_lane || bus->num_data_lanes > 0)
@@ -151,6 +186,7 @@ static void v4l2_of_parse_parallel_bus(const struct device_node *node,
int v4l2_of_parse_endpoint(const struct device_node *node,
struct v4l2_of_endpoint *endpoint)
{
+ u32 bus_type;
int rval;
of_graph_parse_endpoint(node, &endpoint->base);
@@ -158,17 +194,33 @@ int v4l2_of_parse_endpoint(const struct device_node *node,
memset(&endpoint->bus_type, 0, sizeof(*endpoint) -
offsetof(typeof(*endpoint), bus_type));
- rval = v4l2_of_parse_csi_bus(node, endpoint);
- if (rval)
- return rval;
- /*
- * Parse the parallel video bus properties only if none
- * of the MIPI CSI-2 specific properties were found.
- */
- if (endpoint->bus.mipi_csi2.flags == 0)
- v4l2_of_parse_parallel_bus(node, endpoint);
+ rval = of_property_read_u32(node, "bus-type", &bus_type);
+ if (rval < 0) {
+ endpoint->bus_type = 0;
+ rval = v4l2_of_parse_csi2_bus(node, endpoint);
+ if (rval)
+ return rval;
+ /*
+ * Parse the parallel video bus properties only if none
+ * of the MIPI CSI-2 specific properties were found.
+ */
+ if (endpoint->bus.mipi_csi2.flags == 0)
+ v4l2_of_parse_parallel_bus(node, endpoint);
+
+ return 0;
+ }
- return 0;
+ switch (bus_type) {
+ case V4L2_OF_BUS_TYPE_CSI2:
+ return v4l2_of_parse_csi2_bus(node, endpoint);
+ case V4L2_OF_BUS_TYPE_PARALLEL:
+ v4l2_of_parse_parallel_bus(node, endpoint);
+ return 0;
+ default:
+ pr_warn("bad bus-type %u, device_node \"%s\"\n",
+ bus_type, node->full_name);
+ return -EINVAL;
+ }
}
EXPORT_SYMBOL(v4l2_of_parse_endpoint);
@@ -69,11 +69,15 @@
* @V4L2_MBUS_PARALLEL: parallel interface with hsync and vsync
* @V4L2_MBUS_BT656: parallel interface with embedded synchronisation, can
* also be used for BT.1120
+ * @V4L2_MBUS_CSI1: MIPI CSI-1 serial interface
+ * @V4L2_MBUS_CCP2: CCP2 (Compact Camera Port 2)
* @V4L2_MBUS_CSI2: MIPI CSI-2 serial interface
*/
enum v4l2_mbus_type {
V4L2_MBUS_PARALLEL,
V4L2_MBUS_BT656,
+ V4L2_MBUS_CSI1,
+ V4L2_MBUS_CCP2,
V4L2_MBUS_CSI2,
};