NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

libcamera: camera: Fix validateColorSpaces to choose "main" colour space

Browse source 
The intention is that the "main" colour space is the colour space of
the largest non-raw stream. Unfortunately the use of "config_[i].size"
is clearly incorrect, and has been copied from prior versions of the
code. This patch corrects the error.

Signed-off-by: David Plowman <david.plowman@raspberrypi.com>
Reviewed-by: Naushir Patuck <naush@raspberrypi.com>
Reviewed-by: Umang Jain <umang.jain@ideasonboard.com>
Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
This commit is contained in:
David Plowman 2023-01-12 12:10:43 +00:00 • committed by Umang Jain
parent d81505b834
commit 13986d6ce3
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/libcamera/camera.cpp
View file

@ -361,6 +361,7 @@ CameraConfiguration::Status CameraConfiguration::validateColorSpaces(ColorSpaceF
* largest non-raw stream with a defined color space (if there is one).
*/
std::optional<ColorSpace> colorSpace;
Size size;
for (auto [i, cfg] : utils::enumerate(config_)) {
if (!cfg.colorSpace)
@ -369,9 +370,10 @@ CameraConfiguration::Status CameraConfiguration::validateColorSpaces(ColorSpaceF
if (cfg.colorSpace->adjust(cfg.pixelFormat))
status = Adjusted;
if (cfg.colorSpace != ColorSpace::Raw &&
(!colorSpace || cfg.size > config_[i].size))
if (cfg.colorSpace != ColorSpace::Raw && cfg.size > size) {
colorSpace = cfg.colorSpace;
size = cfg.size;
}
}
if (!colorSpace || !(flags & ColorSpaceFlag::StreamsShareColorSpace))