summaryrefslogtreecommitdiff
path: root/test/hotplug-cameras.cpp
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2020-12-26 23:45:04 +0200
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2021-03-03 00:58:59 +0200
commit29bd5583c4ebbd62583c12d76d5d0aceb300b649 (patch)
treef7ea877006e96b34cef9fc2dee1bd6a09f1951f4 /test/hotplug-cameras.cpp
parent96bcf66ac7c079f5a5a6edfe075aed7529a683c9 (diff)
libcamera: pipeline: simple: Don't override stride at configure time
The stride (and frame size) calculation has been moved from configure time to configuration validate time by commit 89fb1efac240 ("libcamera: simple: Fill stride and frameSize at config validation"). This change has however left one stray setting of the stride when configuring the converter. Fix it. While at it, turn the SimpleConverter::configure() output configuration argument to a const reference to emphasize it can't be null and isn't modified by the function, and rename it from cfg to outputCfg to make its purpose clearer. Fixes: 89fb1efac240 ("libcamera: simple: Fill stride and frameSize at config validation") Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Tested-by: Phi-Bang Nguyen <pnguyen@baylibre.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'test/hotplug-cameras.cpp')
0 files changed, 0 insertions, 0 deletions