> -----Original Message----- > From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] > Sent: Sunday, November 23, 2014 6:23 PM > To: Devshatwar, Nikhil; linux-media@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v2 1/4] media: ti-vpe: Use data offset for getting > dma_addr for a plane > > On 11/14/2014 12:20 PM, Nikhil Devshatwar wrote: > > The data_offset in v4l2_planes structure will help us point to the > > start of data content for that particular plane. This may be useful > > when a single buffer contains the data for different planes e.g. Y > > planes of two fields in the same buffer. With this, user space can > > pass queue top field and bottom field with same dmafd and different > data_offsets. > > > > Signed-off-by: Nikhil Devshatwar <nikhil.nd@xxxxxx> > > --- > > drivers/media/platform/ti-vpe/vpe.c | 12 ++++++++++-- > > 1 file changed, 10 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/media/platform/ti-vpe/vpe.c > > b/drivers/media/platform/ti-vpe/vpe.c > > index 0ae19ee..e59eb81 100644 > > --- a/drivers/media/platform/ti-vpe/vpe.c > > +++ b/drivers/media/platform/ti-vpe/vpe.c > > @@ -495,6 +495,14 @@ struct vpe_mmr_adb { > > > > #define VPE_SET_MMR_ADB_HDR(ctx, hdr, regs, offset_a) \ > > VPDMA_SET_MMR_ADB_HDR(ctx->mmr_adb, vpe_mmr_adb, hdr, regs, > > offset_a) > > + > > +static inline dma_addr_t vb2_dma_addr_plus_data_offset(struct > vb2_buffer *vb, > > + unsigned int plane_no) > > +{ > > + return vb2_dma_contig_plane_dma_addr(vb, plane_no) + > > + vb->v4l2_planes[plane_no].data_offset; > > +} > > + > > /* > > * Set the headers for all of the address/data block structures. > > */ > > @@ -1002,7 +1010,7 @@ static void add_out_dtd(struct vpe_ctx *ctx, > int port) > > int plane = fmt->coplanar ? p_data->vb_part : 0; > > > > vpdma_fmt = fmt->vpdma_fmt[plane]; > > - dma_addr = vb2_dma_contig_plane_dma_addr(vb, plane); > > + dma_addr = vb2_dma_addr_plus_data_offset(vb, plane); > > if (!dma_addr) { > > vpe_err(ctx->dev, > > "acquiring output buffer(%d) dma_addr > failed\n", @@ -1042,7 > > +1050,7 @@ static void add_in_dtd(struct vpe_ctx *ctx, int port) > > > > vpdma_fmt = fmt->vpdma_fmt[plane]; > > > > - dma_addr = vb2_dma_contig_plane_dma_addr(vb, plane); > > + dma_addr = vb2_dma_addr_plus_data_offset(vb, plane); > > if (!dma_addr) { > > vpe_err(ctx->dev, > > "acquiring input buffer(%d) dma_addr failed\n", > > > > I suspect this does not do what you want. data_offset is set by the > application for an output stream (i.e. from memory to the hardware) and > it is set by the driver for a capture stream (i.e. from hardware to > memory). It looks like you expect that it is set by the application for > capture streams as well, but that's not true. Yes, I agree I should use the new function (with data_offset) only in the add_in_dtd add_in_dtd is called for the OUTPUT streams only (And some internal buffers) > > Regards, > > Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
