On Wed Jul 28, 2021 at 2:18 PM WEST, Laurent Pinchart wrote:
> Hello,
>
> On Wed, Jul 28, 2021 at 10:19:36AM +0100, Rui Miguel Silva wrote:
> > On Wed Jul 28, 2021 at 9:50 AM WEST, Martin Kepplinger wrote:
> > > Am Mittwoch, dem 19.05.2021 um 03:58 +0300 schrieb Laurent Pinchart:
> > > > When the stream fails to start, the first two buffers in the queue
> > > > have
> > > > been moved to the active_vb2_buf array and are returned to vb2 by
> > > > imx7_csi_dma_unsetup_vb2_buf(). The function is called with the
> > > > buffer
> > > > state set to VB2_BUF_STATE_ERROR unconditionally, which is correct
> > > > when
> > > > stopping the stream, but not when the start operation fails. In that
> > > > case, the state should be set to VB2_BUF_STATE_QUEUED. Fix it.
> > > >
> > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> > > > ---
> > > > drivers/staging/media/imx/imx7-media-csi.c | 15 +++++++++------
> >
> > <snip>
> >
> > > > + imx7_csi_deinit(csi, VB2_BUF_STATE_ERROR);
> > > > }
> > > >
> > > > csi->is_streaming = !!enable;
> > >
> > >
> > > This patch has not yet been accepted. Any specific reason? I need it.
> >
> > Good question, I gave my reviewed in May [0], maybe got lost in the
> > merge process somewhere. Mauro?
>
> I've just sent a pull request with all the pending i.MX8 patches,
> including Martin's i.MX8MQ CSI-2 receiver driver.
Thanks Laurent.
------
Cheers,
Rui
>
