[v2,7/9] pipeline: rkisp1: Add color space information to debug log
diff mbox series

Message ID 20250411123641.2144530-8-stefan.klug@ideasonboard.com
State New
Headers show
Series
  • Wdr preparations
Related show

Commit Message

Stefan Klug April 11, 2025, 12:36 p.m. UTC
The 6.13 kernel driver has a bug the the JPEG/sYCC color space is not
applied correctly. Add the color space to the debug log output to be
able to detect that now and later.

Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

---

Changes in v2:
- Collected tag
---
 src/libcamera/pipeline/rkisp1/rkisp1.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Kieran Bingham April 12, 2025, 12:50 p.m. UTC | #1
Quoting Stefan Klug (2025-04-11 13:36:35)
> The 6.13 kernel driver has a bug the the JPEG/sYCC color space is not

Was the bug introduced in 6.13? Or exists only in versions before 6.13 ?
or only since?

Would probably be helpful/useful to clarify when applying.

--
Kieran


> applied correctly. Add the color space to the debug log output to be
> able to detect that now and later.
> 
> Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>
> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> 
> ---
> 
> Changes in v2:
> - Collected tag
> ---
>  src/libcamera/pipeline/rkisp1/rkisp1.cpp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> index 9d7a3bd57ada..2708cecca0b0 100644
> --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> @@ -891,7 +891,8 @@ int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
>  
>         LOG(RkISP1, Debug)
>                 << "ISP output pad configured with " << format
> -               << " crop " << outputCrop;
> +               << " crop " << outputCrop
> +               << " color space " << ColorSpace::toString(format.colorSpace);
>  
>         IPACameraSensorInfo sensorInfo;
>         ret = data->sensor_->sensorInfo(&sensorInfo);
> -- 
> 2.43.0
>

Patch
diff mbox series

diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
index 9d7a3bd57ada..2708cecca0b0 100644
--- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
+++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
@@ -891,7 +891,8 @@  int PipelineHandlerRkISP1::configure(Camera *camera, CameraConfiguration *c)
 
 	LOG(RkISP1, Debug)
 		<< "ISP output pad configured with " << format
-		<< " crop " << outputCrop;
+		<< " crop " << outputCrop
+		<< " color space " << ColorSpace::toString(format.colorSpace);
 
 	IPACameraSensorInfo sensorInfo;
 	ret = data->sensor_->sensorInfo(&sensorInfo);