build/patch/kernel/archive/sunxi-5.15/patches.megous/media-sun6i-csi-Add-support-for-multiple-endpoints.patch

446 lines
14 KiB
Diff

From b48a91bfa1ef8c92951f74b90c1c4bf520e4a9f8 Mon Sep 17 00:00:00 2001
From: Ondrej Jirman <megous@megous.com>
Date: Mon, 17 Aug 2020 22:49:38 +0200
Subject: [PATCH 147/478] media: sun6i-csi: Add support for multiple endpoints
Devices like tablets and phones can have multiple sensors attached
to the parallel camera interface bus. Add support for switching
between multiple cameras.
Signed-off-by: Ondrej Jirman <megous@megous.com>
---
.../platform/sunxi/sun6i-csi/sun6i_csi.c | 118 ++++++++++--------
.../platform/sunxi/sun6i-csi/sun6i_csi.h | 24 ++--
.../platform/sunxi/sun6i-csi/sun6i_video.c | 51 +++++---
3 files changed, 112 insertions(+), 81 deletions(-)
diff --git a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
index 3e9c31f8ec04..ab2f813e6c48 100644
--- a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
+++ b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
@@ -48,7 +48,8 @@ static inline struct sun6i_csi_dev *sun6i_csi_to_dev(struct sun6i_csi *csi)
/* TODO add 10&12 bit YUV, RGB support */
bool sun6i_csi_is_format_supported(struct sun6i_csi *csi,
- u32 pixformat, u32 mbus_code)
+ u32 pixformat, u32 mbus_code,
+ struct v4l2_fwnode_endpoint* vep)
{
struct sun6i_csi_dev *sdev = sun6i_csi_to_dev(csi);
@@ -57,9 +58,9 @@ bool sun6i_csi_is_format_supported(struct sun6i_csi *csi,
* 8bit and 16bit bus width.
* Identify the media bus format from device tree.
*/
- if ((sdev->csi.v4l2_ep.bus_type == V4L2_MBUS_PARALLEL
- || sdev->csi.v4l2_ep.bus_type == V4L2_MBUS_BT656)
- && sdev->csi.v4l2_ep.bus.parallel.bus_width == 16) {
+ if ((vep->bus_type == V4L2_MBUS_PARALLEL
+ || vep->bus_type == V4L2_MBUS_BT656)
+ && vep->bus.parallel.bus_width == 16) {
switch (pixformat) {
case V4L2_PIX_FMT_HM12:
case V4L2_PIX_FMT_NV12:
@@ -376,9 +377,9 @@ static enum csi_input_seq get_csi_input_seq(struct sun6i_csi_dev *sdev,
return CSI_INPUT_SEQ_YUYV;
}
-static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev)
+static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev,
+ struct v4l2_fwnode_endpoint* vep)
{
- struct v4l2_fwnode_endpoint *endpoint = &sdev->csi.v4l2_ep;
struct sun6i_csi *csi = &sdev->csi;
unsigned char bus_width;
u32 flags;
@@ -390,7 +391,7 @@ static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev)
|| csi->config.field == V4L2_FIELD_INTERLACED_BT)
input_interlaced = true;
- bus_width = endpoint->bus.parallel.bus_width;
+ bus_width = vep->bus.parallel.bus_width;
regmap_read(sdev->regmap, CSI_IF_CFG_REG, &cfg);
@@ -405,11 +406,11 @@ static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev)
else
cfg |= CSI_IF_CFG_SRC_TYPE_PROGRESSED;
- switch (endpoint->bus_type) {
+ switch (vep->bus_type) {
case V4L2_MBUS_PARALLEL:
cfg |= CSI_IF_CFG_MIPI_IF_CSI;
- flags = endpoint->bus.parallel.flags;
+ flags = vep->bus.parallel.flags;
cfg |= (bus_width == 16) ? CSI_IF_CFG_CSI_IF_YUV422_16BIT :
CSI_IF_CFG_CSI_IF_YUV422_INTLV;
@@ -428,7 +429,7 @@ static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev)
case V4L2_MBUS_BT656:
cfg |= CSI_IF_CFG_MIPI_IF_CSI;
- flags = endpoint->bus.parallel.flags;
+ flags = vep->bus.parallel.flags;
cfg |= (bus_width == 16) ? CSI_IF_CFG_CSI_IF_BT1120 :
CSI_IF_CFG_CSI_IF_BT656;
@@ -441,7 +442,7 @@ static void sun6i_csi_setup_bus(struct sun6i_csi_dev *sdev)
break;
default:
dev_warn(sdev->dev, "Unsupported bus type: %d\n",
- endpoint->bus_type);
+ vep->bus_type);
break;
}
@@ -577,7 +578,8 @@ static void sun6i_csi_set_window(struct sun6i_csi_dev *sdev)
}
int sun6i_csi_update_config(struct sun6i_csi *csi,
- struct sun6i_csi_config *config)
+ struct sun6i_csi_config *config,
+ struct v4l2_fwnode_endpoint* vep)
{
struct sun6i_csi_dev *sdev = sun6i_csi_to_dev(csi);
@@ -586,7 +588,7 @@ int sun6i_csi_update_config(struct sun6i_csi *csi,
memcpy(&csi->config, config, sizeof(csi->config));
- sun6i_csi_setup_bus(sdev);
+ sun6i_csi_setup_bus(sdev, vep);
sun6i_csi_set_format(sdev);
sun6i_csi_set_window(sdev);
@@ -634,37 +636,29 @@ void sun6i_csi_set_stream(struct sun6i_csi *csi, bool enable)
/* -----------------------------------------------------------------------------
* Media Controller and V4L2
*/
-static int sun6i_csi_link_entity(struct sun6i_csi *csi,
- struct media_entity *entity,
- struct fwnode_handle *fwnode)
+static int sun6i_csi_link_subdev(struct sun6i_csi *csi, struct v4l2_subdev *sd,
+ int link_flags)
{
- struct media_entity *sink;
- struct media_pad *sink_pad;
- int src_pad_index;
- int ret;
+ struct media_entity *source = &sd->entity;
+ struct media_entity *sink = &csi->video.vdev.entity;
+ int src_pad, sink_pad = csi->video.pad.index, ret;
- ret = media_entity_get_fwnode_pad(entity, fwnode, MEDIA_PAD_FL_SOURCE);
- if (ret < 0) {
+ src_pad = media_entity_get_fwnode_pad(source, sd->fwnode,
+ MEDIA_PAD_FL_SOURCE);
+ if (src_pad < 0) {
dev_err(csi->dev, "%s: no source pad in external entity %s\n",
- __func__, entity->name);
+ __func__, source->name);
return -EINVAL;
}
- src_pad_index = ret;
+ dev_info(csi->dev, "creating %s:%u -> %s:%u link\n",
+ source->name, src_pad, sink->name, sink_pad);
- sink = &csi->video.vdev.entity;
- sink_pad = &csi->video.pad;
-
- dev_dbg(csi->dev, "creating %s:%u -> %s:%u link\n",
- entity->name, src_pad_index, sink->name, sink_pad->index);
- ret = media_create_pad_link(entity, src_pad_index, sink,
- sink_pad->index,
- MEDIA_LNK_FL_ENABLED |
- MEDIA_LNK_FL_IMMUTABLE);
+ ret = media_create_pad_link(source, src_pad, sink, sink_pad,
+ link_flags);
if (ret < 0) {
dev_err(csi->dev, "failed to create %s:%u -> %s:%u link\n",
- entity->name, src_pad_index,
- sink->name, sink_pad->index);
+ source->name, src_pad, sink->name, sink_pad);
return ret;
}
@@ -677,22 +671,24 @@ static int sun6i_subdev_notify_complete(struct v4l2_async_notifier *notifier)
notifier);
struct v4l2_device *v4l2_dev = &csi->v4l2_dev;
struct v4l2_subdev *sd;
- int ret;
+ int ret, link_flags = MEDIA_LNK_FL_ENABLED;
dev_dbg(csi->dev, "notify complete, all subdevs registered\n");
- sd = list_first_entry(&v4l2_dev->subdevs, struct v4l2_subdev, list);
- if (!sd)
- return -EINVAL;
-
- ret = sun6i_csi_link_entity(csi, &sd->entity, sd->fwnode);
- if (ret < 0)
- return ret;
-
ret = v4l2_device_register_subdev_nodes(&csi->v4l2_dev);
if (ret < 0)
return ret;
+ // link subdevs source pads to the controller sink pad, enable
+ // the first link
+ list_for_each_entry(sd, &v4l2_dev->subdevs, list) {
+ ret = sun6i_csi_link_subdev(csi, sd, link_flags);
+ if (ret < 0)
+ return ret;
+
+ link_flags = 0;
+ }
+
return media_device_register(&csi->media_dev);
}
@@ -700,24 +696,41 @@ static const struct v4l2_async_notifier_operations sun6i_csi_async_ops = {
.complete = sun6i_subdev_notify_complete,
};
+static const struct media_device_ops sun6i_csi_media_ops = {
+ .link_notify = v4l2_pipeline_link_notify,
+};
+
+/* CSI module has one port that can support multiple endpoints.
+ * Typically front and back camera on a tablet.
+ *
+ * Multiple endpoints are supported by selectively enabling only
+ * one of the endpoint devices at a time and shutting down the
+ * rest.
+ */
static int sun6i_csi_fwnode_parse(struct device *dev,
struct v4l2_fwnode_endpoint *vep,
struct v4l2_async_subdev *asd)
{
- struct sun6i_csi *csi = dev_get_drvdata(dev);
+ struct sun6i_csi_async_subdev* casd =
+ container_of(asd, struct sun6i_csi_async_subdev, asd);
+
+ if (vep->base.port) {
+ dev_warn(dev, "Too many ports\n");
+ return -ENOTCONN;
+ }
- if (vep->base.port || vep->base.id) {
- dev_warn(dev, "Only support a single port with one endpoint\n");
+ if (vep->base.id >= MAX_ENDPOINTS) {
+ dev_warn(dev, "Too many endpoints\n");
return -ENOTCONN;
}
switch (vep->bus_type) {
case V4L2_MBUS_PARALLEL:
case V4L2_MBUS_BT656:
- csi->v4l2_ep = *vep;
+ casd->vep = *vep;
return 0;
default:
- dev_err(dev, "Unsupported media bus type\n");
+ dev_warn(dev, "Unsupported media bus type\n");
return -ENOTCONN;
}
}
@@ -741,6 +754,7 @@ static int sun6i_csi_v4l2_init(struct sun6i_csi *csi)
strscpy(csi->media_dev.model, "Allwinner Video Capture Device",
sizeof(csi->media_dev.model));
csi->media_dev.hw_revision = 0;
+ csi->media_dev.ops = &sun6i_csi_media_ops;
snprintf(csi->media_dev.bus_info, sizeof(csi->media_dev.bus_info),
"platform:%s", dev_name(csi->dev));
@@ -768,9 +782,9 @@ static int sun6i_csi_v4l2_init(struct sun6i_csi *csi)
goto unreg_v4l2;
ret = v4l2_async_notifier_parse_fwnode_endpoints(csi->dev,
- &csi->notifier,
- sizeof(struct v4l2_async_subdev),
- sun6i_csi_fwnode_parse);
+ &csi->notifier,
+ sizeof(struct sun6i_csi_async_subdev),
+ sun6i_csi_fwnode_parse);
if (ret)
goto clean_video;
diff --git a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h
index b00eaab85891..3be680938255 100644
--- a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h
+++ b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h
@@ -11,10 +11,11 @@
#include <media/v4l2-ctrls.h>
#include <media/v4l2-device.h>
#include <media/v4l2-fwnode.h>
+#include <media/v4l2-mc.h>
#include "sun6i_video.h"
-struct sun6i_csi;
+#define MAX_ENDPOINTS 4
/**
* struct sun6i_csi_config - configs for sun6i csi
@@ -37,25 +38,26 @@ struct sun6i_csi {
struct v4l2_ctrl_handler ctrl_handler;
struct v4l2_device v4l2_dev;
struct media_device media_dev;
-
struct v4l2_async_notifier notifier;
-
- /* video port settings */
- struct v4l2_fwnode_endpoint v4l2_ep;
-
struct sun6i_csi_config config;
-
struct sun6i_video video;
};
+struct sun6i_csi_async_subdev {
+ struct v4l2_async_subdev asd; /* must be first */
+ struct v4l2_fwnode_endpoint vep;
+};
+
/**
* sun6i_csi_is_format_supported() - check if the format supported by csi
* @csi: pointer to the csi
* @pixformat: v4l2 pixel format (V4L2_PIX_FMT_*)
* @mbus_code: media bus format code (MEDIA_BUS_FMT_*)
+ * @vep: parsed CSI side bus endpoint configuration
*/
-bool sun6i_csi_is_format_supported(struct sun6i_csi *csi, u32 pixformat,
- u32 mbus_code);
+bool sun6i_csi_is_format_supported(struct sun6i_csi *csi,
+ u32 pixformat, u32 mbus_code,
+ struct v4l2_fwnode_endpoint* vep);
/**
* sun6i_csi_set_power() - power on/off the csi
@@ -68,9 +70,11 @@ int sun6i_csi_set_power(struct sun6i_csi *csi, bool enable);
* sun6i_csi_update_config() - update the csi register settings
* @csi: pointer to the csi
* @config: see struct sun6i_csi_config
+ * @vep: parsed CSI side bus endpoint configuration
*/
int sun6i_csi_update_config(struct sun6i_csi *csi,
- struct sun6i_csi_config *config);
+ struct sun6i_csi_config *config,
+ struct v4l2_fwnode_endpoint* vep);
/**
* sun6i_csi_update_buf_addr() - update the csi frame buffer address
diff --git a/drivers/media/platform/sunxi/sun6i-csi/sun6i_video.c b/drivers/media/platform/sunxi/sun6i-csi/sun6i_video.c
index ee336f090aa9..1633107f4668 100644
--- a/drivers/media/platform/sunxi/sun6i-csi/sun6i_video.c
+++ b/drivers/media/platform/sunxi/sun6i-csi/sun6i_video.c
@@ -138,6 +138,7 @@ static int sun6i_video_start_streaming(struct vb2_queue *vq, unsigned int count)
struct sun6i_csi_buffer *next_buf;
struct sun6i_csi_config config;
struct v4l2_subdev *subdev;
+ struct sun6i_csi_async_subdev* casd;
unsigned long flags;
int ret;
@@ -158,13 +159,15 @@ static int sun6i_video_start_streaming(struct vb2_queue *vq, unsigned int count)
goto stop_media_pipeline;
}
+ casd = container_of(subdev->asd, struct sun6i_csi_async_subdev, asd);
+
config.pixelformat = video->fmt.fmt.pix.pixelformat;
config.code = video->mbus_code;
config.field = video->fmt.fmt.pix.field;
config.width = video->fmt.fmt.pix.width;
config.height = video->fmt.fmt.pix.height;
- ret = sun6i_csi_update_config(video->csi, &config);
+ ret = sun6i_csi_update_config(video->csi, &config, &casd->vep);
if (ret < 0)
goto stop_media_pipeline;
@@ -560,27 +563,17 @@ static const struct v4l2_file_operations sun6i_video_fops = {
/* -----------------------------------------------------------------------------
* Media Operations
*/
-static int sun6i_video_link_validate_get_format(struct media_pad *pad,
- struct v4l2_subdev_format *fmt)
-{
- if (is_media_entity_v4l2_subdev(pad->entity)) {
- struct v4l2_subdev *sd =
- media_entity_to_v4l2_subdev(pad->entity);
-
- fmt->which = V4L2_SUBDEV_FORMAT_ACTIVE;
- fmt->pad = pad->index;
- return v4l2_subdev_call(sd, pad, get_fmt, NULL, fmt);
- }
-
- return -EINVAL;
-}
static int sun6i_video_link_validate(struct media_link *link)
{
struct video_device *vdev = container_of(link->sink->entity,
struct video_device, entity);
+ struct v4l2_subdev *sd =
+ media_entity_to_v4l2_subdev(link->source->entity);
+ struct sun6i_csi_async_subdev* casd =
+ container_of(sd->asd, struct sun6i_csi_async_subdev, asd);
struct sun6i_video *video = video_get_drvdata(vdev);
- struct v4l2_subdev_format source_fmt;
+ struct v4l2_subdev_format source_fmt = {};
int ret;
video->mbus_code = 0;
@@ -591,13 +584,20 @@ static int sun6i_video_link_validate(struct media_link *link)
return -ENOLINK;
}
- ret = sun6i_video_link_validate_get_format(link->source, &source_fmt);
+ if (!is_media_entity_v4l2_subdev(link->source->entity))
+ return -EINVAL;
+
+ source_fmt.which = V4L2_SUBDEV_FORMAT_ACTIVE;
+ source_fmt.pad = link->source->index;
+
+ ret = v4l2_subdev_call(sd, pad, get_fmt, NULL, &source_fmt);
if (ret < 0)
return ret;
if (!sun6i_csi_is_format_supported(video->csi,
video->fmt.fmt.pix.pixelformat,
- source_fmt.format.code)) {
+ source_fmt.format.code,
+ &casd->vep)) {
dev_err(video->csi->dev,
"Unsupported pixformat: 0x%x with mbus code: 0x%x!\n",
video->fmt.fmt.pix.pixelformat,
@@ -619,8 +619,21 @@ static int sun6i_video_link_validate(struct media_link *link)
return 0;
}
+static int sun6i_video_link_setup(struct media_entity *entity,
+ const struct media_pad *local,
+ const struct media_pad *remote, u32 flags)
+{
+ if (flags & MEDIA_LNK_FL_ENABLED) {
+ if (media_entity_remote_pad(local))
+ return -EBUSY;
+ }
+
+ return 0;
+}
+
static const struct media_entity_operations sun6i_video_media_ops = {
- .link_validate = sun6i_video_link_validate
+ .link_validate = sun6i_video_link_validate,
+ .link_setup = sun6i_video_link_setup,
};
int sun6i_video_init(struct sun6i_video *video, struct sun6i_csi *csi,
--
2.35.3