[PATCH 1/4] media: stm32-dcmi: do not fall into error on buffer starvation

[Hugues Fruchet]

Return silently instead of falling into error when running
out of available buffers when restarting capture.
Capture will be restarted when new buffers will be
provided by V4L2 client.

Signed-off-by: Hugues Fruchet 
---
 drivers/media/platform/stm32/stm32-dcmi.c | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/media/platform/stm32/stm32-dcmi.c 
b/drivers/media/platform/stm32/stm32-dcmi.c
index b796334..a3fbfac 100644
--- a/drivers/media/platform/stm32/stm32-dcmi.c
+++ b/drivers/media/platform/stm32/stm32-dcmi.c
@@ -228,13 +228,10 @@ static int dcmi_restart_capture(struct stm32_dcmi *dcmi)
 
/* Restart a new DMA transfer with next buffer */
if (list_empty(>buffers)) {
-   dev_err(dcmi->dev, "%s: No more buffer queued, cannot capture 
buffer\n",
-   __func__);
-   dcmi->errors_count++;
+   dev_dbg(dcmi->dev, "Capture restart is deferred to next buffer 
queueing\n");
dcmi->active = NULL;
-
spin_unlock_irq(>irqlock);
-   return -EINVAL;
+   return 0;
}
 
dcmi->active = list_entry(dcmi->buffers.next,
-- 
1.9.1

[PATCH 1/4] media: stm32-dcmi: do not fall into error on buffer starvation

[Hugues Fruchet]

Return silently instead of falling into error when running
out of available buffers when restarting capture.
Capture will be restarted when new buffers will be
provided by V4L2 client.

Signed-off-by: Hugues Fruchet 
---
 drivers/media/platform/stm32/stm32-dcmi.c | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/media/platform/stm32/stm32-dcmi.c 
b/drivers/media/platform/stm32/stm32-dcmi.c
index b796334..a3fbfac 100644
--- a/drivers/media/platform/stm32/stm32-dcmi.c
+++ b/drivers/media/platform/stm32/stm32-dcmi.c
@@ -228,13 +228,10 @@ static int dcmi_restart_capture(struct stm32_dcmi *dcmi)
 
/* Restart a new DMA transfer with next buffer */
if (list_empty(>buffers)) {
-   dev_err(dcmi->dev, "%s: No more buffer queued, cannot capture 
buffer\n",
-   __func__);
-   dcmi->errors_count++;
+   dev_dbg(dcmi->dev, "Capture restart is deferred to next buffer 
queueing\n");
dcmi->active = NULL;
-
spin_unlock_irq(>irqlock);
-   return -EINVAL;
+   return 0;
}
 
dcmi->active = list_entry(dcmi->buffers.next,
-- 
1.9.1