<feed xmlns='http://www.w3.org/2005/Atom'>
<title>renesas/kmsxx.git/kmscube, branch writeback</title>
<subtitle>kmsxx, cloned from https://github.com/tomba/kmsxx</subtitle>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/'/>
<entry>
<title>Drop libfmt</title>
<updated>2026-04-30T08:09:49+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2026-04-30T06:35:03+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=f9da4640c4098107aefc6c02b12b543384cccc50'/>
<id>f9da4640c4098107aefc6c02b12b543384cccc50</id>
<content type='text'>
We can use std::format() and a custom print() wrapper with C++20. When
moving to C++23, we can drop the wrapper.

Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We can use std::format() and a custom print() wrapper with C++20. When
moving to C++23, we can drop the wrapper.

Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fix: Initialize uninitialized member variables</title>
<updated>2025-12-18T10:23:57+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-09-13T15:37:51+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=65b0414775f0674cfb9aff0863a5be8c0d72dfaa'/>
<id>65b0414775f0674cfb9aff0863a5be8c0d72dfaa</id>
<content type='text'>
This commit addresses cppcheck warnings about member variables that are
not properly initialized in constructors (uninitMemberVar). Uninitialized
member variables can lead to undefined behavior and unpredictable program
execution, making this a critical bug fix.

Changes made:
- kmscube/cube-gles2.cpp: Initialize m_width and m_height to 0 in GlScene
  constructor to prevent undefined behavior when these values are used
- utils/kmstest.cpp: Initialize m_frame_num and m_flip_count to 0 in
  FlipState constructor to ensure proper frame counting behavior

These fixes prevent potential crashes and ensure deterministic behavior
by providing proper initial values for all member variables.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit addresses cppcheck warnings about member variables that are
not properly initialized in constructors (uninitMemberVar). Uninitialized
member variables can lead to undefined behavior and unpredictable program
execution, making this a critical bug fix.

Changes made:
- kmscube/cube-gles2.cpp: Initialize m_width and m_height to 0 in GlScene
  constructor to prevent undefined behavior when these values are used
- utils/kmstest.cpp: Initialize m_frame_num and m_flip_count to 0 in
  FlipState constructor to ensure proper frame counting behavior

These fixes prevent potential crashes and ensure deterministic behavior
by providing proper initial values for all member variables.
</pre>
</div>
</content>
</entry>
<entry>
<title>fix: Add explicit keyword to single-argument constructors</title>
<updated>2025-12-18T10:23:57+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-09-13T15:28:25+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=de2346e9ba06d15d5839fe0a3b3f61f474c5bedb'/>
<id>de2346e9ba06d15d5839fe0a3b3f61f474c5bedb</id>
<content type='text'>
This commit addresses cppcheck warnings about constructors with single
arguments that are not marked as explicit (noExplicitConstructor).
Single-argument constructors can perform implicit type conversions which
may lead to unexpected behavior and bugs. Marking them as explicit prevents
these implicit conversions and makes the code more predictable.

Changes made:
- kmscube/cube-egl.h: Added explicit keyword to EglState constructor
- kmscube/cube-gbm.cpp: Added explicit keyword to GbmDevice constructor

These changes improve type safety by requiring explicit construction calls
and preventing unintended implicit conversions.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit addresses cppcheck warnings about constructors with single
arguments that are not marked as explicit (noExplicitConstructor).
Single-argument constructors can perform implicit type conversions which
may lead to unexpected behavior and bugs. Marking them as explicit prevents
these implicit conversions and makes the code more predictable.

Changes made:
- kmscube/cube-egl.h: Added explicit keyword to EglState constructor
- kmscube/cube-gbm.cpp: Added explicit keyword to GbmDevice constructor

These changes improve type safety by requiring explicit construction calls
and preventing unintended implicit conversions.
</pre>
</div>
</content>
</entry>
<entry>
<title>fix: Resolve variable shadowing warnings</title>
<updated>2025-12-18T10:23:57+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-09-13T15:27:14+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=00a14befa954162723142571caf301f9679c6059'/>
<id>00a14befa954162723142571caf301f9679c6059</id>
<content type='text'>
This commit addresses cppcheck warnings about local variables that shadow
outer function names (shadowFunction). Variable shadowing can make code
confusing and error-prone as it's unclear which variable is being referenced.

Changes made:
- kmscube/cube-egl.cpp: Renamed 'config' to 'cfg' in loop to avoid shadowing
  the config() member function
- kmscube/cube-gbm.cpp: Renamed 'width' and 'height' local variables to
  'bo_width' and 'bo_height' to avoid shadowing width() and height() member
  functions

These changes improve code clarity and eliminate potential confusion about
variable scope and naming.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit addresses cppcheck warnings about local variables that shadow
outer function names (shadowFunction). Variable shadowing can make code
confusing and error-prone as it's unclear which variable is being referenced.

Changes made:
- kmscube/cube-egl.cpp: Renamed 'config' to 'cfg' in loop to avoid shadowing
  the config() member function
- kmscube/cube-gbm.cpp: Renamed 'width' and 'height' local variables to
  'bo_width' and 'bo_height' to avoid shadowing width() and height() member
  functions

These changes improve code clarity and eliminate potential confusion about
variable scope and naming.
</pre>
</div>
</content>
</entry>
<entry>
<title>fix: Replace C-style casts with C++ casts</title>
<updated>2025-12-18T10:23:57+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-09-13T15:23:09+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=0271a62e676d50940ec5f86d94cf66629894c25f'/>
<id>0271a62e676d50940ec5f86d94cf66629894c25f</id>
<content type='text'>
This commit addresses cppcheck warnings about C-style casts (cstyleCast)
by replacing them with appropriate C++ casts. C-style casts are considered
dangerous because they can perform unsafe conversions without compile-time
type checking, while C++ casts are more explicit and type-safe.

Changes made:
- static_cast for safe type conversions (e.g., void* to struct*)
- reinterpret_cast for pointer type conversions (e.g., uint8_t* to char*)
- Combined static_cast and reinterpret_cast for integer-to-pointer conversions

Fixed files:
- kmscube/cube-gles2.cpp: GLvoid* casts for OpenGL vertex attribute pointers
- kmscube/cube-wl.cpp: Wayland interface pointer casts
- kmscube/cube-x11.cpp: X11 window handle conversion
- utils/fbtest.cpp: mmap return value cast
- utils/kmstest.cpp: Framebuffer pointer arithmetic
- utils/kmsview.cpp: Framebuffer memory mapping cast
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit addresses cppcheck warnings about C-style casts (cstyleCast)
by replacing them with appropriate C++ casts. C-style casts are considered
dangerous because they can perform unsafe conversions without compile-time
type checking, while C++ casts are more explicit and type-safe.

Changes made:
- static_cast for safe type conversions (e.g., void* to struct*)
- reinterpret_cast for pointer type conversions (e.g., uint8_t* to char*)
- Combined static_cast and reinterpret_cast for integer-to-pointer conversions

Fixed files:
- kmscube/cube-gles2.cpp: GLvoid* casts for OpenGL vertex attribute pointers
- kmscube/cube-wl.cpp: Wayland interface pointer casts
- kmscube/cube-x11.cpp: X11 window handle conversion
- utils/fbtest.cpp: mmap return value cast
- utils/kmstest.cpp: Framebuffer pointer arithmetic
- utils/kmsview.cpp: Framebuffer memory mapping cast
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix a few clang warnings</title>
<updated>2025-04-22T07:27:27+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-04-21T19:01:07+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=7a4e4cbf5e70cd304b751f9862792e0d6af85eb1'/>
<id>7a4e4cbf5e70cd304b751f9862792e0d6af85eb1</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Use the fourcc helpers</title>
<updated>2025-03-26T13:44:00+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2025-02-05T10:03:34+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=a35d57fa9088fbccd468ad095c89aba096b2494e'/>
<id>a35d57fa9088fbccd468ad095c89aba096b2494e</id>
<content type='text'>
Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Implement native visual matching</title>
<updated>2024-12-09T10:35:38+00:00</updated>
<author>
<name>Kaido Kert</name>
<email>kaidokert@gmail.com</email>
</author>
<published>2024-11-29T04:47:25+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=aaab406251540429522c5ef7808ee049c65a06d2'/>
<id>aaab406251540429522c5ef7808ee049c65a06d2</id>
<content type='text'>
Implement matching GBM buffer format to EGL NATIVE_VISUAL_ID.
eglChooseConfig cannot match on NATIVE_VISUAL_ID, but GBM/EGL
requires matching formats. Similar logic is implemented in
kmscube code in match_config_to_visual.

X11, Wayland and Null cube demos remain unchanged.

Tested on Raspi-4 and VirtualBox/Ubuntu
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Implement matching GBM buffer format to EGL NATIVE_VISUAL_ID.
eglChooseConfig cannot match on NATIVE_VISUAL_ID, but GBM/EGL
requires matching formats. Similar logic is implemented in
kmscube code in match_config_to_visual.

X11, Wayland and Null cube demos remain unchanged.

Tested on Raspi-4 and VirtualBox/Ubuntu
</pre>
</div>
</content>
</entry>
<entry>
<title>Make gbm binary respect numframes</title>
<updated>2024-11-29T07:36:16+00:00</updated>
<author>
<name>Kaido Kert</name>
<email>kaidokert@gmail.com</email>
</author>
<published>2024-11-29T02:04:12+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=6cf6e88715ac034f568603bce9a1b8f4a30c12ce'/>
<id>6cf6e88715ac034f568603bce9a1b8f4a30c12ce</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>meson: Add summary</title>
<updated>2023-02-24T16:10:42+00:00</updated>
<author>
<name>Tomi Valkeinen</name>
<email>tomi.valkeinen@ideasonboard.com</email>
</author>
<published>2023-02-24T14:02:47+00:00</published>
<link rel='alternate' type='text/html' href='https://gitolite.ideasonboard.com/renesas/kmsxx.git/commit/?id=3c9e134287362f3273ff04a86f32838b65b2651e'/>
<id>3c9e134287362f3273ff04a86f32838b65b2651e</id>
<content type='text'>
Add summary print for meson configuration.

Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add summary print for meson configuration.

Signed-off-by: Tomi Valkeinen &lt;tomi.valkeinen@ideasonboard.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
