* [PATCH v2 0/3] media: CSI-2 configuration to frame desc
@ 2022-04-26 7:02 Tomi Valkeinen
2022-04-26 7:02 ` [PATCH v2 1/3] media: Add bus type to frame descriptors Tomi Valkeinen
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Tomi Valkeinen @ 2022-04-26 7:02 UTC (permalink / raw)
To: linux-media, sakari.ailus, Jacopo Mondi, Laurent Pinchart,
niklas.soderlund+renesas, Mauro Carvalho Chehab, Hans Verkuil,
Pratyush Yadav, satish.nagireddy
Cc: Tomi Valkeinen
Hi,
v2 of the csi-2 frame desc change. v1 can be found from:
https://lore.kernel.org/all/20220421142906.527918-1-tomi.valkeinen@ideasonboard.com/
These improve the frame descriptors so that the drivers can fill in the
CSI-2 datatype and virtual channel, and make the CAL driver use that
data if available.
The only change to v1 is in the CAL patch, so that it allows frame descs
with multiple streams (but prints a dev_dbg in that case) and always
picks the first one.
Tomi
Sakari Ailus (2):
media: Add bus type to frame descriptors
media: Add CSI-2 bus configuration to frame descriptors
Tomi Valkeinen (1):
media: ti: cal: use frame desc to get vc and dt
drivers/media/platform/ti/cal/cal-camerarx.c | 27 +++++++++++
drivers/media/platform/ti/cal/cal.c | 49 +++++++++++++++++++-
drivers/media/platform/ti/cal/cal.h | 2 +
include/media/v4l2-subdev.h | 36 ++++++++++++++
4 files changed, 112 insertions(+), 2 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread* [PATCH v2 1/3] media: Add bus type to frame descriptors 2022-04-26 7:02 [PATCH v2 0/3] media: CSI-2 configuration to frame desc Tomi Valkeinen @ 2022-04-26 7:02 ` Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 2/3] media: Add CSI-2 bus configuration " Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt Tomi Valkeinen 2 siblings, 0 replies; 5+ messages in thread From: Tomi Valkeinen @ 2022-04-26 7:02 UTC (permalink / raw) To: linux-media, sakari.ailus, Jacopo Mondi, Laurent Pinchart, niklas.soderlund+renesas, Mauro Carvalho Chehab, Hans Verkuil, Pratyush Yadav, satish.nagireddy Cc: Tomi Valkeinen From: Sakari Ailus <sakari.ailus@linux.intel.com> Add the media bus type to the frame descriptor. CSI-2 specific information will be added in next patch to the frame descriptor. Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> - Make the bus type a named enum Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- include/media/v4l2-subdev.h | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h index 6c153b33bb04..16d6559f81ed 100644 --- a/include/media/v4l2-subdev.h +++ b/include/media/v4l2-subdev.h @@ -344,12 +344,32 @@ struct v4l2_mbus_frame_desc_entry { #define V4L2_FRAME_DESC_ENTRY_MAX 4 +/** + * enum v4l2_mbus_frame_desc_type - media bus frame description type + * + * @V4L2_MBUS_FRAME_DESC_TYPE_UNDEFINED: + * Undefined frame desc type. Drivers should not use this, it is + * for backwards compatibility. + * @V4L2_MBUS_FRAME_DESC_TYPE_PARALLEL: + * Parallel media bus. + * @V4L2_MBUS_FRAME_DESC_TYPE_CSI2: + * CSI-2 media bus. Frame desc parameters must be set in + * &struct v4l2_mbus_frame_desc_entry->csi2. + */ +enum v4l2_mbus_frame_desc_type { + V4L2_MBUS_FRAME_DESC_TYPE_UNDEFINED = 0, + V4L2_MBUS_FRAME_DESC_TYPE_PARALLEL, + V4L2_MBUS_FRAME_DESC_TYPE_CSI2, +}; + /** * struct v4l2_mbus_frame_desc - media bus data frame description + * @type: type of the bus (enum v4l2_mbus_frame_desc_type) * @entry: frame descriptors array * @num_entries: number of entries in @entry array */ struct v4l2_mbus_frame_desc { + enum v4l2_mbus_frame_desc_type type; struct v4l2_mbus_frame_desc_entry entry[V4L2_FRAME_DESC_ENTRY_MAX]; unsigned short num_entries; }; -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2 2/3] media: Add CSI-2 bus configuration to frame descriptors 2022-04-26 7:02 [PATCH v2 0/3] media: CSI-2 configuration to frame desc Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 1/3] media: Add bus type to frame descriptors Tomi Valkeinen @ 2022-04-26 7:02 ` Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt Tomi Valkeinen 2 siblings, 0 replies; 5+ messages in thread From: Tomi Valkeinen @ 2022-04-26 7:02 UTC (permalink / raw) To: linux-media, sakari.ailus, Jacopo Mondi, Laurent Pinchart, niklas.soderlund+renesas, Mauro Carvalho Chehab, Hans Verkuil, Pratyush Yadav, satish.nagireddy Cc: Tomi Valkeinen From: Sakari Ailus <sakari.ailus@linux.intel.com> Add CSI-2 bus specific configuration to the frame descriptors. This allows obtaining the virtual channel and data type information for each stream the transmitter is sending. Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Reviewed-by: Jacopo Mondi <jacopo+renesas@jmondi.org> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- include/media/v4l2-subdev.h | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h index 16d6559f81ed..e6b24bbdd6b6 100644 --- a/include/media/v4l2-subdev.h +++ b/include/media/v4l2-subdev.h @@ -312,6 +312,17 @@ struct v4l2_subdev_audio_ops { int (*s_stream)(struct v4l2_subdev *sd, int enable); }; +/** + * struct v4l2_mbus_frame_desc_entry_csi2 + * + * @vc: CSI-2 virtual channel + * @dt: CSI-2 data type ID + */ +struct v4l2_mbus_frame_desc_entry_csi2 { + u8 vc; + u8 dt; +}; + /** * enum v4l2_mbus_frame_desc_flags - media bus frame description flags * @@ -335,11 +346,16 @@ enum v4l2_mbus_frame_desc_flags { * %FRAME_DESC_FL_BLOB is not set. * @length: number of octets per frame, valid if @flags * %V4L2_MBUS_FRAME_DESC_FL_LEN_MAX is set. + * @bus: Bus-specific frame descriptor parameters + * @bus.csi2: CSI-2-specific bus configuration */ struct v4l2_mbus_frame_desc_entry { enum v4l2_mbus_frame_desc_flags flags; u32 pixelcode; u32 length; + union { + struct v4l2_mbus_frame_desc_entry_csi2 csi2; + } bus; }; #define V4L2_FRAME_DESC_ENTRY_MAX 4 -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt 2022-04-26 7:02 [PATCH v2 0/3] media: CSI-2 configuration to frame desc Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 1/3] media: Add bus type to frame descriptors Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 2/3] media: Add CSI-2 bus configuration " Tomi Valkeinen @ 2022-04-26 7:02 ` Tomi Valkeinen 2022-04-26 9:28 ` Jacopo Mondi 2 siblings, 1 reply; 5+ messages in thread From: Tomi Valkeinen @ 2022-04-26 7:02 UTC (permalink / raw) To: linux-media, sakari.ailus, Jacopo Mondi, Laurent Pinchart, niklas.soderlund+renesas, Mauro Carvalho Chehab, Hans Verkuil, Pratyush Yadav, satish.nagireddy Cc: Tomi Valkeinen Use get_frame_desc() to get the frame desc from the connected source, and use the provided virtual channel and datatype instead of hardcoded ones. get_frame_desc() can contain multiple streams, but as we don't support multiple streams yet, we will just always use the first stream. If the source doesn't support get_frame_desc(), fall back to the previous method of always capturing virtual channel 0 and any datatype. Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> --- drivers/media/platform/ti/cal/cal-camerarx.c | 27 +++++++++++ drivers/media/platform/ti/cal/cal.c | 49 +++++++++++++++++++- drivers/media/platform/ti/cal/cal.h | 2 + 3 files changed, 76 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/ti/cal/cal-camerarx.c b/drivers/media/platform/ti/cal/cal-camerarx.c index 6b43a1525b45..e69fed117fea 100644 --- a/drivers/media/platform/ti/cal/cal-camerarx.c +++ b/drivers/media/platform/ti/cal/cal-camerarx.c @@ -583,6 +583,33 @@ static int cal_camerarx_parse_dt(struct cal_camerarx *phy) return ret; } +int cal_camerarx_get_remote_frame_desc(struct cal_camerarx *phy, + struct v4l2_mbus_frame_desc *desc) +{ + struct media_pad *pad; + int ret; + + if (!phy->source) + return -EPIPE; + + pad = media_entity_remote_pad(&phy->pads[CAL_CAMERARX_PAD_SINK]); + if (!pad) + return -EPIPE; + + ret = v4l2_subdev_call(phy->source, pad, get_frame_desc, pad->index, + desc); + if (ret) + return ret; + + if (desc->type != V4L2_MBUS_FRAME_DESC_TYPE_CSI2) { + dev_err(phy->cal->dev, + "Frame descriptor does not describe CSI-2 link"); + return -EINVAL; + } + + return 0; +} + /* ------------------------------------------------------------------ * V4L2 Subdev Operations * ------------------------------------------------------------------ diff --git a/drivers/media/platform/ti/cal/cal.c b/drivers/media/platform/ti/cal/cal.c index 4a4a6c5983f7..df66b8192349 100644 --- a/drivers/media/platform/ti/cal/cal.c +++ b/drivers/media/platform/ti/cal/cal.c @@ -469,10 +469,57 @@ static bool cal_ctx_wr_dma_stopped(struct cal_ctx *ctx) return stopped; } +static int +cal_get_remote_frame_desc_entry(struct cal_camerarx *phy, + struct v4l2_mbus_frame_desc_entry *entry) +{ + struct v4l2_mbus_frame_desc fd; + int ret; + + ret = cal_camerarx_get_remote_frame_desc(phy, &fd); + if (ret) { + if (ret != -ENOIOCTLCMD) + dev_err(phy->cal->dev, + "Failed to get remote frame desc: %d\n", ret); + return ret; + } + + if (fd.num_entries == 0) { + dev_err(phy->cal->dev, + "No streams found in the remote frame descriptor\n"); + + return -ENODEV; + } + + if (fd.num_entries > 1) + dev_dbg(phy->cal->dev, + "Multiple streams not supported in remote frame descriptor, using the first one\n"); + + *entry = fd.entry[0]; + + return 0; +} + int cal_ctx_prepare(struct cal_ctx *ctx) { + struct v4l2_mbus_frame_desc_entry entry; int ret; + ret = cal_get_remote_frame_desc_entry(ctx->phy, &entry); + + if (ret == -ENOIOCTLCMD) { + ctx->vc = 0; + ctx->datatype = CAL_CSI2_CTX_DT_ANY; + } else if (!ret) { + ctx_dbg(2, ctx, "Framedesc: len %u, vc %u, dt %#x\n", + entry.length, entry.bus.csi2.vc, entry.bus.csi2.dt); + + ctx->vc = entry.bus.csi2.vc; + ctx->datatype = entry.bus.csi2.dt; + } else { + return ret; + } + ctx->use_pix_proc = !ctx->fmtinfo->meta; if (ctx->use_pix_proc) { @@ -936,8 +983,6 @@ static struct cal_ctx *cal_ctx_create(struct cal_dev *cal, int inst) ctx->dma_ctx = inst; ctx->csi2_ctx = inst; ctx->cport = inst; - ctx->vc = 0; - ctx->datatype = CAL_CSI2_CTX_DT_ANY; ret = cal_ctx_v4l2_init(ctx); if (ret) diff --git a/drivers/media/platform/ti/cal/cal.h b/drivers/media/platform/ti/cal/cal.h index 527e22d022f3..61409ddced98 100644 --- a/drivers/media/platform/ti/cal/cal.h +++ b/drivers/media/platform/ti/cal/cal.h @@ -323,6 +323,8 @@ const struct cal_format_info *cal_format_by_code(u32 code); void cal_quickdump_regs(struct cal_dev *cal); +int cal_camerarx_get_remote_frame_desc(struct cal_camerarx *phy, + struct v4l2_mbus_frame_desc *desc); void cal_camerarx_disable(struct cal_camerarx *phy); void cal_camerarx_i913_errata(struct cal_camerarx *phy); struct cal_camerarx *cal_camerarx_create(struct cal_dev *cal, -- 2.34.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt 2022-04-26 7:02 ` [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt Tomi Valkeinen @ 2022-04-26 9:28 ` Jacopo Mondi 0 siblings, 0 replies; 5+ messages in thread From: Jacopo Mondi @ 2022-04-26 9:28 UTC (permalink / raw) To: Tomi Valkeinen Cc: linux-media, sakari.ailus, Jacopo Mondi, Laurent Pinchart, niklas.soderlund+renesas, Mauro Carvalho Chehab, Hans Verkuil, Pratyush Yadav, satish.nagireddy Hi Tomi, On Tue, Apr 26, 2022 at 10:02:40AM +0300, Tomi Valkeinen wrote: > Use get_frame_desc() to get the frame desc from the connected source, > and use the provided virtual channel and datatype instead of hardcoded > ones. > > get_frame_desc() can contain multiple streams, but as we don't support > multiple streams yet, we will just always use the first stream. > > If the source doesn't support get_frame_desc(), fall back to the > previous method of always capturing virtual channel 0 and any datatype. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Reviewed-by: Jacopo Mondi <jacopo@jmondi.org> Thanks j > --- > drivers/media/platform/ti/cal/cal-camerarx.c | 27 +++++++++++ > drivers/media/platform/ti/cal/cal.c | 49 +++++++++++++++++++- > drivers/media/platform/ti/cal/cal.h | 2 + > 3 files changed, 76 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/ti/cal/cal-camerarx.c b/drivers/media/platform/ti/cal/cal-camerarx.c > index 6b43a1525b45..e69fed117fea 100644 > --- a/drivers/media/platform/ti/cal/cal-camerarx.c > +++ b/drivers/media/platform/ti/cal/cal-camerarx.c > @@ -583,6 +583,33 @@ static int cal_camerarx_parse_dt(struct cal_camerarx *phy) > return ret; > } > > +int cal_camerarx_get_remote_frame_desc(struct cal_camerarx *phy, > + struct v4l2_mbus_frame_desc *desc) > +{ > + struct media_pad *pad; > + int ret; > + > + if (!phy->source) > + return -EPIPE; > + > + pad = media_entity_remote_pad(&phy->pads[CAL_CAMERARX_PAD_SINK]); > + if (!pad) > + return -EPIPE; > + > + ret = v4l2_subdev_call(phy->source, pad, get_frame_desc, pad->index, > + desc); > + if (ret) > + return ret; > + > + if (desc->type != V4L2_MBUS_FRAME_DESC_TYPE_CSI2) { > + dev_err(phy->cal->dev, > + "Frame descriptor does not describe CSI-2 link"); > + return -EINVAL; > + } > + > + return 0; > +} > + > /* ------------------------------------------------------------------ > * V4L2 Subdev Operations > * ------------------------------------------------------------------ > diff --git a/drivers/media/platform/ti/cal/cal.c b/drivers/media/platform/ti/cal/cal.c > index 4a4a6c5983f7..df66b8192349 100644 > --- a/drivers/media/platform/ti/cal/cal.c > +++ b/drivers/media/platform/ti/cal/cal.c > @@ -469,10 +469,57 @@ static bool cal_ctx_wr_dma_stopped(struct cal_ctx *ctx) > return stopped; > } > > +static int > +cal_get_remote_frame_desc_entry(struct cal_camerarx *phy, > + struct v4l2_mbus_frame_desc_entry *entry) > +{ > + struct v4l2_mbus_frame_desc fd; > + int ret; > + > + ret = cal_camerarx_get_remote_frame_desc(phy, &fd); > + if (ret) { > + if (ret != -ENOIOCTLCMD) > + dev_err(phy->cal->dev, > + "Failed to get remote frame desc: %d\n", ret); > + return ret; > + } > + > + if (fd.num_entries == 0) { > + dev_err(phy->cal->dev, > + "No streams found in the remote frame descriptor\n"); > + > + return -ENODEV; > + } > + > + if (fd.num_entries > 1) > + dev_dbg(phy->cal->dev, > + "Multiple streams not supported in remote frame descriptor, using the first one\n"); > + > + *entry = fd.entry[0]; > + > + return 0; > +} > + > int cal_ctx_prepare(struct cal_ctx *ctx) > { > + struct v4l2_mbus_frame_desc_entry entry; > int ret; > > + ret = cal_get_remote_frame_desc_entry(ctx->phy, &entry); > + > + if (ret == -ENOIOCTLCMD) { > + ctx->vc = 0; > + ctx->datatype = CAL_CSI2_CTX_DT_ANY; > + } else if (!ret) { > + ctx_dbg(2, ctx, "Framedesc: len %u, vc %u, dt %#x\n", > + entry.length, entry.bus.csi2.vc, entry.bus.csi2.dt); > + > + ctx->vc = entry.bus.csi2.vc; > + ctx->datatype = entry.bus.csi2.dt; > + } else { > + return ret; > + } > + > ctx->use_pix_proc = !ctx->fmtinfo->meta; > > if (ctx->use_pix_proc) { > @@ -936,8 +983,6 @@ static struct cal_ctx *cal_ctx_create(struct cal_dev *cal, int inst) > ctx->dma_ctx = inst; > ctx->csi2_ctx = inst; > ctx->cport = inst; > - ctx->vc = 0; > - ctx->datatype = CAL_CSI2_CTX_DT_ANY; > > ret = cal_ctx_v4l2_init(ctx); > if (ret) > diff --git a/drivers/media/platform/ti/cal/cal.h b/drivers/media/platform/ti/cal/cal.h > index 527e22d022f3..61409ddced98 100644 > --- a/drivers/media/platform/ti/cal/cal.h > +++ b/drivers/media/platform/ti/cal/cal.h > @@ -323,6 +323,8 @@ const struct cal_format_info *cal_format_by_code(u32 code); > > void cal_quickdump_regs(struct cal_dev *cal); > > +int cal_camerarx_get_remote_frame_desc(struct cal_camerarx *phy, > + struct v4l2_mbus_frame_desc *desc); > void cal_camerarx_disable(struct cal_camerarx *phy); > void cal_camerarx_i913_errata(struct cal_camerarx *phy); > struct cal_camerarx *cal_camerarx_create(struct cal_dev *cal, > -- > 2.34.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-04-26 10:04 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-04-26 7:02 [PATCH v2 0/3] media: CSI-2 configuration to frame desc Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 1/3] media: Add bus type to frame descriptors Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 2/3] media: Add CSI-2 bus configuration " Tomi Valkeinen 2022-04-26 7:02 ` [PATCH v2 3/3] media: ti: cal: use frame desc to get vc and dt Tomi Valkeinen 2022-04-26 9:28 ` Jacopo Mondi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.