diff options
| author | Hans de Goede <hdegoede@redhat.com> | 2022-09-30 22:05:44 +0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2022-11-25 11:12:20 +0300 |
| commit | c7194b21809ec53db2f8ae5a5e2389ed774d2af6 (patch) | |
| tree | cfb4c238146fbe469c842e775c843e17846b9c23 /drivers/staging/media/atomisp/pci/atomisp_subdev.h | |
| parent | af6b9dfddcec827fe9c0a7975217f9e126d37454 (diff) | |
| download | linux-c7194b21809ec53db2f8ae5a5e2389ed774d2af6.tar.xz | |
media: atomisp: On streamoff wait for buffers owned by the CSS to be given back
There is no guarantee that when we stop the pipeline all buffers owned
by the CSS are cleanly returned to the videobuf queue.
This is a problem with videobuf2 which will complain loudly when not
all buffers have been returned after the streamoff() queue op has
returned.
And this also allows removing a WARN() in the continuous mode path.
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/staging/media/atomisp/pci/atomisp_subdev.h')
| -rw-r--r-- | drivers/staging/media/atomisp/pci/atomisp_subdev.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/media/atomisp/pci/atomisp_subdev.h b/drivers/staging/media/atomisp/pci/atomisp_subdev.h index a1f4da35235d..65c2f8664f9d 100644 --- a/drivers/staging/media/atomisp/pci/atomisp_subdev.h +++ b/drivers/staging/media/atomisp/pci/atomisp_subdev.h @@ -81,7 +81,8 @@ struct atomisp_video_pipe { /* Store here the initial run mode */ unsigned int default_run_mode; - + /* Set from streamoff to disallow queuing further buffers in CSS */ + bool stopping; unsigned int buffers_in_css; /* |