video: gc2145: add CSI mode of the GC2145 sensor

Add possibility to use the gc2145 sensor in CSI mode by
adding the bus-type property in the device-tree.

Signed-off-by: Alain Volmat <[email protected]>

Author: Alain Volmat

drivers/video/gc2145.c

@@ -12,7 +12,7 @@
 #include <zephyr/drivers/video-controls.h>
 #include <zephyr/drivers/i2c.h>
 #include <zephyr/drivers/gpio.h>
-
+#include <zephyr/dt-bindings/video/video-interfaces.h>
 #include <zephyr/logging/log.h>
 
 #include "video_ctrls.h"
@@ -42,6 +42,54 @@ LOG_MODULE_REGISTER(video_gc2145, CONFIG_VIDEO_LOG_LEVEL);
 #define GC2145_REG_SUBSAMPLE_MODE       0x9A
 #define GC2145_SUBSAMPLE_MODE_SMOOTH    0x0E
 
+/* MIPI-CSI registers - on page 3 */
+#define GC2145_REG_DPHY_MODE1		0x01
+#define GC2145_DPHY_MODE1_CLK_EN	BIT(0)
+#define GC2145_DPHY_MODE1_LANE0_EN	BIT(1)
+#define GC2145_DPHY_MODE1_LANE1_EN	BIT(2)
+#define GC2145_DPHY_MODE1_CLK_LANE_P2S_SEL	BIT(7)
+
+#define GC2145_REG_DPHY_MODE2	0x02
+#define GC2145_DPHY_MODE2_CLK_DIFF(a)		((a) & 0x07)
+#define GC2145_DPHY_MODE2_LANE0_DIFF(a)	(((a) & 0x07) << 4)
+
+#define GC2145_REG_DPHY_MODE3	0x03
+#define GC2145_DPHY_MODE3_LANE1_DIFF(a)	((a) & 0x07)
+#define GC2145_DPHY_MODE3_CLK_DELAY		BIT(4)
+#define GC2145_DPHY_MODE3_LANE0_DELAY		BIT(5)
+#define GC2145_DPHY_MODE3_LANE1_DELAY		BIT(6)
+
+#define GC2145_REG_FIFO_FULL_LVL_LOW	0x04
+#define GC2145_REG_FIFO_FULL_LVL_HIGH	0x05
+#define GC2145_REG_FIFO_MODE		0x06
+#define GC2145_FIFO_MODE_READ_GATE	BIT(3)
+#define GC2145_FIFO_MODE_MIPI_CLK_MODULE	BIT(7)
+
+#define GC2145_REG_BUF_CSI2_MODE	0x10
+#define GC2145_CSI2_MODE_DOUBLE		BIT(0)
+#define GC2145_CSI2_MODE_RAW8		BIT(2)
+#define GC2145_CSI2_MODE_MIPI_EN	BIT(4)
+#define GC2145_CSI2_MODE_EN		BIT(7)
+
+#define GC2145_REG_MIPI_DT		0x11
+#define GC2145_REG_LWC_LOW		0x12
+#define GC2145_REG_LWC_HIGH		0x13
+#define GC2145_REG_DPHY_MODE		0x15
+#define GC2145_DPHY_MODE_TRIGGER_PROG	BIT(4)
+
+#define GC2145_REG_FIFO_GATE_MODE	0x17
+#define GC2145_REG_T_LPX		0x21
+#define GC2145_REG_T_CLK_HS_PREPARE	0x22
+#define GC2145_REG_T_CLK_ZERO		0x23
+#define GC2145_REG_T_CLK_PRE		0x24
+#define GC2145_REG_T_CLK_POST		0x25
+#define GC2145_REG_T_CLK_TRAIL		0x26
+#define GC2145_REG_T_HS_EXIT		0x27
+#define GC2145_REG_T_WAKEUP		0x28
+#define GC2145_REG_T_HS_PREPARE		0x29
+#define GC2145_REG_T_HS_ZERO		0x2a
+#define GC2145_REG_T_HS_TRAIL		0x2b
+
 #define UXGA_HSIZE 1600
 #define UXGA_VSIZE 1200
 
@@ -681,6 +729,32 @@ static const struct gc2145_reg default_regs[] = {
 	{0x00, 0x00},
 };
 
+static const struct gc2145_reg default_mipi_csi_regs[] = {
+	/* Switch to page 3 */
+	{0xfe, 0x03},
+	{GC2145_REG_DPHY_MODE1, GC2145_DPHY_MODE1_CLK_EN |
+				GC2145_DPHY_MODE1_LANE0_EN | GC2145_DPHY_MODE1_LANE1_EN |
+				GC2145_DPHY_MODE1_CLK_LANE_P2S_SEL},
+	{GC2145_REG_DPHY_MODE2, GC2145_DPHY_MODE2_CLK_DIFF(2) |
+				GC2145_DPHY_MODE2_LANE0_DIFF(2)},
+	{GC2145_REG_DPHY_MODE3, GC2145_DPHY_MODE3_LANE1_DIFF(0) |
+				GC2145_DPHY_MODE3_CLK_DELAY},
+	{GC2145_REG_FIFO_MODE, GC2145_FIFO_MODE_READ_GATE |
+			       GC2145_FIFO_MODE_MIPI_CLK_MODULE},
+	{GC2145_REG_DPHY_MODE, GC2145_DPHY_MODE_TRIGGER_PROG},
+
+	/* Clock & Data lanes timing */
+	{GC2145_REG_T_LPX, 0x10},
+	{GC2145_REG_T_CLK_HS_PREPARE, 0x04},
+	{GC2145_REG_T_CLK_ZERO, 0x10},
+	{GC2145_REG_T_CLK_PRE, 0x10},
+	{GC2145_REG_T_CLK_POST, 0x10},
+	{GC2145_REG_T_CLK_TRAIL, 0x05},
+	{GC2145_REG_T_HS_PREPARE, 0x03},
+	{GC2145_REG_T_HS_ZERO, 0x0a},
+	{GC2145_REG_T_HS_TRAIL, 0x06},
+};
+
 struct gc2145_config {
 	struct i2c_dt_spec i2c;
 #if DT_INST_NODE_HAS_PROP(0, pwdn_gpios)
@@ -689,11 +763,13 @@ struct gc2145_config {
 #if DT_INST_NODE_HAS_PROP(0, reset_gpios)
 	struct gpio_dt_spec reset_gpio;
 #endif
+	int bus_type;
 };
 
 struct gc2145_ctrls {
 	struct video_ctrl hflip;
 	struct video_ctrl vflip;
+	struct video_ctrl pixelrate;
 };
 
 struct gc2145_data {
@@ -1038,10 +1114,77 @@ static uint8_t gc2145_check_connection(const struct device *dev)
 	return 0;
 }
 
+static int gc2145_config_csi(const struct device *dev, uint32_t pixelformat,
+			     uint32_t width, uint32_t height)
+{
+	const struct gc2145_config *cfg = dev->config;
+	uint16_t fifo_full_level = width == 1600 ? 0x0001 : 0x0190;
+	uint16_t lwc = width * 2;
+	uint8_t csi_dt;
+	int ret;
+
+	switch(pixelformat) {
+	case VIDEO_PIX_FMT_RGB565:
+		csi_dt = VIDEO_MIPI_CSI2_DT_RGB565;
+		break;
+	case VIDEO_PIX_FMT_YUYV:
+		csi_dt = VIDEO_MIPI_CSI2_DT_YUV422_8;
+		break;
+	default:
+		LOG_ERR("Unsupported pixelformat for CSI");
+		return -EINVAL;
+	}
+
+	/* 320x240 resolution doesn't work (yet ?) in CSI */
+	if (width == 320 && height == 240) {
+		LOG_ERR("Unsupported resolution 320x240 for CSI");
+		return -EINVAL;
+	}
+
+	/* Apply fixed settings for MIPI-CSI. After that active page is 3 */
+	ret = gc2145_write_all(dev, default_mipi_csi_regs, ARRAY_SIZE(default_mipi_csi_regs));
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_LWC_LOW, lwc & 0xff);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_LWC_HIGH, lwc >> 8);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_FIFO_FULL_LVL_LOW, fifo_full_level & 0xff);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_FIFO_FULL_LVL_HIGH, fifo_full_level >> 8);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_FIFO_GATE_MODE, 0xf0);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_MIPI_DT, csi_dt);
+	if (ret) {
+		return ret;
+	}
+
+	return gc2145_write_reg(&cfg->i2c, 0xfe, 0x0);
+}
+
 static int gc2145_set_fmt(const struct device *dev, enum video_endpoint_id ep,
 			  struct video_format *fmt)
 {
 	struct gc2145_data *drv_data = dev->data;
+	const struct gc2145_config *cfg = dev->config;
 	size_t res = ARRAY_SIZE(fmts);
 	int ret;
 
@@ -1080,6 +1223,14 @@ static int gc2145_set_fmt(const struct device *dev, enum video_endpoint_id ep,
 		return ret;
 	}
 
+	if (cfg->bus_type == VIDEO_BUS_TYPE_CSI2_DPHY) {
+		ret = gc2145_config_csi(dev, fmt->pixelformat, fmt->width, fmt->height);
+		if (ret < 0) {
+			LOG_ERR("Failed to configure MIPI-CSI");
+			return ret;
+		}
+	}
+
 	return 0;
 }
 
@@ -1093,14 +1244,44 @@ static int gc2145_get_fmt(const struct device *dev, enum video_endpoint_id ep,
 	return 0;
 }
 
-static int gc2145_set_stream(const struct device *dev, bool enable)
+static int gc2145_set_stream_dvp(const struct device *dev, bool enable)
 {
 	const struct gc2145_config *cfg = dev->config;
 
 	return enable ? gc2145_write_reg(&cfg->i2c, 0xf2, 0x0f)
 		      : gc2145_write_reg(&cfg->i2c, 0xf2, 0x00);
 }
 
+static int gc2145_set_stream_csi(const struct device *dev, bool enable)
+{
+	const struct gc2145_config *cfg = dev->config;
+	int ret;
+
+	ret = gc2145_write_reg(&cfg->i2c, 0xfe, 0x03);
+	if (ret) {
+		return ret;
+	}
+
+	ret = gc2145_write_reg(&cfg->i2c, GC2145_REG_BUF_CSI2_MODE,
+			       enable ? GC2145_CSI2_MODE_RAW8 | GC2145_CSI2_MODE_DOUBLE |
+					GC2145_CSI2_MODE_EN | GC2145_CSI2_MODE_MIPI_EN
+				      : 0);
+	if (ret) {
+		return ret;
+	}
+
+	return gc2145_write_reg(&cfg->i2c, 0xfe, 0x0);
+}
+
+static int gc2145_set_stream(const struct device *dev, bool enable)
+{
+	const struct gc2145_config *cfg = dev->config;
+
+	return cfg->bus_type == VIDEO_BUS_TYPE_PARALLEL ?
+	       gc2145_set_stream_dvp(dev, enable) :
+	       gc2145_set_stream_csi(dev, enable);
+}
+
 static int gc2145_get_caps(const struct device *dev, enum video_endpoint_id ep,
 			   struct video_caps *caps)
 {
@@ -1130,6 +1311,7 @@ static DEVICE_API(video, gc2145_driver_api) = {
 	.set_ctrl = gc2145_set_ctrl,
 };
 
+#define GC2145_PIXEL_RATE	120000000
 static int gc2145_init_controls(const struct device *dev)
 {
 	int ret;
@@ -1142,8 +1324,18 @@ static int gc2145_init_controls(const struct device *dev)
 		return ret;
 	}
 
-	return video_init_ctrl(&ctrls->vflip, dev, VIDEO_CID_VFLIP,
-			       (struct video_ctrl_range){.min = 0, .max = 1, .step = 1, .def = 0});
+	ret = video_init_ctrl(&ctrls->vflip, dev, VIDEO_CID_VFLIP,
+			      (struct video_ctrl_range){.min = 0, .max = 1, .step = 1, .def = 0});
+	if (ret) {
+		return ret;
+	}
+
+
+	return video_init_ctrl(&ctrls->pixelrate, dev, VIDEO_CID_PIXEL_RATE,
+			       (struct video_ctrl_range){.min64 = GC2145_PIXEL_RATE,
+							 .max64 = GC2145_PIXEL_RATE,
+							 .step64 = 1,
+							 .def64 = GC2145_PIXEL_RATE});
 }
 
 static int gc2145_init(const struct device *dev)
@@ -1205,6 +1397,8 @@ static const struct gc2145_config gc2145_cfg_0 = {
 #if DT_INST_NODE_HAS_PROP(0, reset_gpios)
 	.reset_gpio = GPIO_DT_SPEC_INST_GET(0, reset_gpios),
 #endif
+	.bus_type = DT_PROP_OR(DT_CHILD(DT_INST_CHILD(0, port), endpoint), bus_type,
+			       VIDEO_BUS_TYPE_PARALLEL),
 };
 static struct gc2145_data gc2145_data_0;
 

dts/bindings/video/galaxycore,gc2145.yaml

@@ -18,3 +18,7 @@ properties:
     description: |
       The PWDN pin is asserted to power down the sensor. The sensor
       receives this as an active high signal
+
+child-binding:
+  child-binding:
+    include: video-interfaces.yaml