[v16,8/9] libcamera: simple: Move colour space logging after adjustment
diff mbox series

Message ID 20251127211932.122463-9-mzamazal@redhat.com
State New
Headers show
Series
  • Enable raw streams with software ISP
Related show

Commit Message

Milan Zamazal Nov. 27, 2025, 9:19 p.m. UTC 
The log message about adjusting an unspecified colour space should log
the value after, not before, the adjustment.

Signed-off-by: Milan Zamazal <mzamazal@redhat.com>
---
 src/libcamera/pipeline/simple/simple.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Patch
diff mbox series 

diff --git a/src/libcamera/pipeline/simple/simple.cpp b/src/libcamera/pipeline/simple/simple.cpp
index cbfc06fb6..2820d1254 100644
--- a/src/libcamera/pipeline/simple/simple.cpp
+++ b/src/libcamera/pipeline/simple/simple.cpp
@@ -1305,9 +1305,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
@@ -1315,6 +1312,9 @@  CameraConfiguration::Status SimpleCameraConfiguration::validate()
 			 * to Adjusted.
 			 */
 			cfg.colorSpace->adjust(cfg.pixelFormat);
+			LOG(SimplePipeline, Debug)
+				<< "Unspecified color space set to "
+				<< cfg.colorSpace.value().toString();
 		} else {
 			if (cfg.colorSpace->adjust(cfg.pixelFormat)) {
 				LOG(SimplePipeline, Debug)