[libcamera-devel,v2,0/4] libcamera: enforce stream configuration
mbox series

Message ID 20190224173701.8587-1-niklas.soderlund@ragnatech.se
Headers show
Series
  • libcamera: enforce stream configuration
Related show

Message

Niklas Söderlund Feb. 24, 2019, 5:36 p.m. UTC 
Hi,

This series extends the documentation and implements the stricter
enforcing of that the configuration requested exactly can be satisfied
by the hardware discussed face to face. For a more detailed description
see patch 1/4.

The last patch 4/4 improves the cam utility to make better use of the
stricter requirements and prevents it from error out when requesting a
format which the hardware can't match and instead informs the user of
the real problem.

* Changes since v1
- Improve grammar in 1/4 and in commit messages, thanks Laurent!

Niklas Söderlund (4):
  libcamera: pipeline_handler: extend documentation for
    configureStreams()
  libcamera: pipeline: uvcvideo: enforce stream configuration
  libcamera: pipeline: vimc: enforce stream configuration
  cam: fix return type of configureStreams()

 src/cam/main.cpp                    |  7 ++-----
 src/libcamera/pipeline/uvcvideo.cpp | 12 +++++++++++-
 src/libcamera/pipeline/vimc.cpp     | 12 +++++++++++-
 src/libcamera/pipeline_handler.cpp  |  8 ++++++++
 4 files changed, 32 insertions(+), 7 deletions(-)

Comments

Niklas Söderlund Feb. 25, 2019, 10:28 p.m. UTC | #1 
On 2019-02-24 18:36:57 +0100, Niklas Söderlund wrote:
> Hi,
> 
> This series extends the documentation and implements the stricter
> enforcing of that the configuration requested exactly can be satisfied
> by the hardware discussed face to face. For a more detailed description
> see patch 1/4.
> 
> The last patch 4/4 improves the cam utility to make better use of the
> stricter requirements and prevents it from error out when requesting a
> format which the hardware can't match and instead informs the user of
> the real problem.

Series pushed to master.

> 
> * Changes since v1
> - Improve grammar in 1/4 and in commit messages, thanks Laurent!
> 
> Niklas Söderlund (4):
>   libcamera: pipeline_handler: extend documentation for
>     configureStreams()
>   libcamera: pipeline: uvcvideo: enforce stream configuration
>   libcamera: pipeline: vimc: enforce stream configuration
>   cam: fix return type of configureStreams()
> 
>  src/cam/main.cpp                    |  7 ++-----
>  src/libcamera/pipeline/uvcvideo.cpp | 12 +++++++++++-
>  src/libcamera/pipeline/vimc.cpp     | 12 +++++++++++-
>  src/libcamera/pipeline_handler.cpp  |  8 ++++++++
>  4 files changed, 32 insertions(+), 7 deletions(-)
> 
> -- 
> 2.20.1
>