libcamera: simple: Move colour space logging after adjustment
The log message about adjusting an unspecified colour space should log
the value after, not before, the adjustment.
Fixes: 5a33bc10e9 ("libcamera: software_isp: Assign colour spaces in configurations")
Reviewed-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Umang Jain <uajain@igalia.com>
Signed-off-by: Milan Zamazal <mzamazal@redhat.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
committed by
Kieran Bingham
parent
f4c3dee217
commit
0a9cf7e0eb
@@ -1249,9 +1249,6 @@ CameraConfiguration::Status SimpleCameraConfiguration::validate()
|
||||
default:
|
||||
cfg.colorSpace = ColorSpace::Raw;
|
||||
}
|
||||
LOG(SimplePipeline, Debug)
|
||||
<< "Unspecified color space set to "
|
||||
<< cfg.colorSpace.value().toString();
|
||||
/*
|
||||
* Adjust the assigned color space to make sure everything is OK.
|
||||
* Since this is assigning an unspecified color space rather than
|
||||
@@ -1259,6 +1256,9 @@ CameraConfiguration::Status SimpleCameraConfiguration::validate()
|
||||
* to Adjusted.
|
||||
*/
|
||||
cfg.colorSpace->adjust(pixelFormat);
|
||||
LOG(SimplePipeline, Debug)
|
||||
<< "Unspecified color space set to "
|
||||
<< cfg.colorSpace.value().toString();
|
||||
} else {
|
||||
if (cfg.colorSpace->adjust(pixelFormat)) {
|
||||
LOG(SimplePipeline, Debug)
|
||||
|
||||
Reference in New Issue
Block a user