Re: [PATCH v5 3/5] [media] davinci: vpif_capture: fix start/stop streaming locking

2016-12-07 Thread Kevin Hilman
Laurent Pinchart writes: > Hi Kevin, > > Thank you for the patch. > > On Tuesday 06 Dec 2016 21:08:24 Kevin Hilman wrote: >> Video capture subdevs may be over I2C and may sleep during xfer, so we >> cannot do IRQ-disabled locking when calling the subdev. >> >>

Re: [PATCH v5 3/5] [media] davinci: vpif_capture: fix start/stop streaming locking

2016-12-07 Thread Laurent Pinchart
Hi Kevin, Thank you for the patch. On Tuesday 06 Dec 2016 21:08:24 Kevin Hilman wrote: > Video capture subdevs may be over I2C and may sleep during xfer, so we > cannot do IRQ-disabled locking when calling the subdev. > > The IRQ-disabled locking is meant to protect the DMA queue list >

[PATCH v5 3/5] [media] davinci: vpif_capture: fix start/stop streaming locking

2016-12-06 Thread Kevin Hilman
Video capture subdevs may be over I2C and may sleep during xfer, so we cannot do IRQ-disabled locking when calling the subdev. The IRQ-disabled locking is meant to protect the DMA queue list throughout the rest of the driver, so update the locking in [start|stop]_streaming to protect just this