[PATCH] libcamera: formats: Change bytesPerGroup of RGB16 from 3 to 2
Hou Qi
qi.hou at nxp.com
Fri Oct 25 05:31:25 CEST 2024
Change the bytesPerGroup in plane[0] of RGB16 format from 3 to 2,
otherwise calculated stride using below formula will be incorrect.
/* ceil(width / pixelsPerGroup) * bytesPerGroup */
unsigned int stride = (width + pixelsPerGroup - 1) / pixelsPerGroup
* planes[plane].bytesPerGroup;
---
src/libcamera/formats.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp
index dbefb094..bfcdfc08 100644
--- a/src/libcamera/formats.cpp
+++ b/src/libcamera/formats.cpp
@@ -157,7 +157,7 @@ const std::map<PixelFormat, PixelFormatInfo> pixelFormatInfo{
.colourEncoding = PixelFormatInfo::ColourEncodingRGB,
.packed = false,
.pixelsPerGroup = 1,
- .planes = {{ { 3, 1 }, { 0, 0 }, { 0, 0 } }},
+ .planes = {{ { 2, 1 }, { 0, 0 }, { 0, 0 } }},
} },
{ formats::RGB565_BE, {
.name = "RGB565_BE",
@@ -167,7 +167,7 @@ const std::map<PixelFormat, PixelFormatInfo> pixelFormatInfo{
.colourEncoding = PixelFormatInfo::ColourEncodingRGB,
.packed = false,
.pixelsPerGroup = 1,
- .planes = {{ { 3, 1 }, { 0, 0 }, { 0, 0 } }},
+ .planes = {{ { 2, 1 }, { 0, 0 }, { 0, 0 } }},
} },
{ formats::BGR888, {
.name = "BGR888",
--
2.34.1
More information about the libcamera-devel
mailing list