On 27 February 2012 09:49, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > Hi Javier > > On Wed, 22 Feb 2012, Javier Martin wrote: > >> Signed-off-by: Javier Martin <javier.martin@xxxxxxxxxxxxxxxxx> >> --- >> Changes since v1: >> - Adapt to [PATCH v4 4/4] media i.MX27 camera: handle overflows properly. >> >> --- >> drivers/media/video/mx2_camera.c | 26 ++++++++++++-------------- >> 1 files changed, 12 insertions(+), 14 deletions(-) >> >> diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c >> index 0ade14e..7793264 100644 >> --- a/drivers/media/video/mx2_camera.c >> +++ b/drivers/media/video/mx2_camera.c > > [snip] > >> @@ -1314,7 +1312,7 @@ static irqreturn_t mx27_camera_emma_irq(int irq_emma, void *data) >> pcdev->base_emma + PRP_CNTL); >> writel(cntl, pcdev->base_emma + PRP_CNTL); >> >> - buf = list_entry(pcdev->active_bufs.next, >> + buf = list_first_entry(pcdev->active_bufs.next, > > This is the only hunk, that you've changed. I'll fix this to be > > + buf = list_first_entry(&pcdev->active_bufs, > >> struct mx2_buffer, queue); >> mx27_camera_frame_done_emma(pcdev, >> buf->bufnum, true); > Looks OK. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander. Cantabria. Spain +34 942 25 32 60 www.vista-silicon.com -- 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
