[Buildroot] [PATCH 0/2] package/kmsxx: Add support for pykms

[Buildroot] [PATCH 0/2] package/kmsxx: Add support for pykms

[Paul Elder]

This series adds support for pykms in kmsxx. This enables the python
bindings for kmsxx.

The first patch updates kmsxx to the current HEAD, and the second patch
adds support for pykms as a separate build config option.

Paul Elder (2):
  package/kmsxx: Bump version to current head as of 2026.10
  package/kmsxx: Add option for pykms

 package/kmsxx/Config.in  |  7 +++++++
 package/kmsxx/kmsxx.hash |  2 +-
 package/kmsxx/kmsxx.mk   | 15 ++++++++++-----
 3 files changed, 18 insertions(+), 6 deletions(-)

-- 
2.47.3

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 1/2] package/kmsxx: Bump version to current head as of 2026.10

[Paul Elder]

Bump version of kmsxx to the current head as of 2026/10/02. Remove
build options as appropriate.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>

---
The git commit ID didn't seem like a meaningful version to put in the
commit message but neither did it feel useful to leave it blank so I
just filled it in with the date... I hope that's acceptable.
---
 package/kmsxx/kmsxx.hash | 2 +-
 package/kmsxx/kmsxx.mk   | 7 +++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/package/kmsxx/kmsxx.hash b/package/kmsxx/kmsxx.hash
index c141ef7e0725..1d74a0602598 100644
--- a/package/kmsxx/kmsxx.hash
+++ b/package/kmsxx/kmsxx.hash
@@ -1,3 +1,3 @@
 # Locally calculated
-sha256  99759fe9334369ccd80ab0bbfe1110ee47e74f1e20abc2d7b2f028067ea0a0c3  kmsxx-824bbb1f4cd062d66b457faca50f904b34dfd96c.tar.gz
+sha256  8671e99766590055cafd58793eaf56782044e2cf1203a8a390a5726457d39f7d  kmsxx-4d2008f5893a5d6ea2b2bf5e70bc539bfcaf6db0.tar.gz
 sha256  fab3dd6bdab226f1c08630b1dd917e11fcb4ec5e1e020e2c16f83a0a13863e85  LICENSE
diff --git a/package/kmsxx/kmsxx.mk b/package/kmsxx/kmsxx.mk
index f65cc668d484..5aeb48910c7e 100644
--- a/package/kmsxx/kmsxx.mk
+++ b/package/kmsxx/kmsxx.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-KMSXX_VERSION = 824bbb1f4cd062d66b457faca50f904b34dfd96c
+KMSXX_VERSION = 4d2008f5893a5d6ea2b2bf5e70bc539bfcaf6db0
 KMSXX_SITE = $(call github,tomba,kmsxx,$(KMSXX_VERSION))
 KMSXX_LICENSE = MPL-2.0
 KMSXX_LICENSE_FILES = LICENSE
@@ -13,8 +13,7 @@ KMSXX_DEPENDENCIES = fmt libdrm host-pkgconf
 KMSXX_CONF_OPTS = \
 	-Dkmscube=false \
 	-Dpykms=disabled \
-	-Domap=disabled \
-	-Dsystem-pybind11=enabled
+	-Domap=disabled
 
 ifeq ($(BR2_TOOLCHAIN_HAS_GCC_BUG_85180),y)
 KMSXX_CXXFLAGS += $(TARGET_CXXFLAGS) -O0
@@ -23,7 +22,7 @@ endif
 ifeq ($(BR2_PACKAGE_KMSXX_INSTALL_TESTS),y)
 KMSXX_CONF_OPTS += -Dutils=true
 # extra handling for some utils not installed by default
-KMSXX_EXTRA_UTILS = kmsview kmscapture omap-wbcap omap-wbm2m
+KMSXX_EXTRA_UTILS = kmsview kmscapture
 ifeq ($(BR2_PACKAGE_LIBEVDEV),y)
 KMSXX_DEPENDENCIES += libevdev
 KMSXX_EXTRA_UTILS += kmstouch
-- 
2.47.3

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 2/2] package/kmsxx: Add option for pykms

[Paul Elder]

Add option to kmsxx to enable building pykms. While at it, add kmsprint
and kmstest to the extra utils that are built as they are useful for
testing and development.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
---
 package/kmsxx/Config.in |  7 +++++++
 package/kmsxx/kmsxx.mk  | 10 ++++++++--
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/package/kmsxx/Config.in b/package/kmsxx/Config.in
index fa327ba18de1..dbf2c4ec2a42 100644
--- a/package/kmsxx/Config.in
+++ b/package/kmsxx/Config.in
@@ -16,6 +16,13 @@ config BR2_PACKAGE_KMSXX
 
 if BR2_PACKAGE_KMSXX
 
+config BR2_PACKAGE_KMSXX_PYKMS
+	bool "Enable Python bindings"
+	depends on BR2_PACKAGE_PYTHON3
+	depends on BR2_PACKAGE_PYTHON_PYBIND
+	help
+	  This option enables the pykms python bindings.
+
 config BR2_PACKAGE_KMSXX_INSTALL_TESTS
 	bool "Install test programs"
 	help
diff --git a/package/kmsxx/kmsxx.mk b/package/kmsxx/kmsxx.mk
index 5aeb48910c7e..f52a1047011a 100644
--- a/package/kmsxx/kmsxx.mk
+++ b/package/kmsxx/kmsxx.mk
@@ -12,17 +12,23 @@ KMSXX_INSTALL_STAGING = YES
 KMSXX_DEPENDENCIES = fmt libdrm host-pkgconf
 KMSXX_CONF_OPTS = \
 	-Dkmscube=false \
-	-Dpykms=disabled \
 	-Domap=disabled
 
 ifeq ($(BR2_TOOLCHAIN_HAS_GCC_BUG_85180),y)
 KMSXX_CXXFLAGS += $(TARGET_CXXFLAGS) -O0
 endif
 
+ifeq ($(BR2_PACKAGE_KMSXX_PYKMS),y)
+KMSXX_CONF_OPTS += -Dpykms=enabled
+KMSXX_DEPENDENCIES += python3 python-pybind
+else
+KMSXX_CONF_OPTS += -Dpykms=disabled
+endif
+
 ifeq ($(BR2_PACKAGE_KMSXX_INSTALL_TESTS),y)
 KMSXX_CONF_OPTS += -Dutils=true
 # extra handling for some utils not installed by default
-KMSXX_EXTRA_UTILS = kmsview kmscapture
+KMSXX_EXTRA_UTILS = kmsview kmscapture kmsprint kmstest
 ifeq ($(BR2_PACKAGE_LIBEVDEV),y)
 KMSXX_DEPENDENCIES += libevdev
 KMSXX_EXTRA_UTILS += kmstouch
-- 
2.47.3

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/2] package/kmsxx: Bump version to current head as of 2026.10

[Laurent Pinchart]

On Fri, Oct 02, 2026 at 07:23:32PM +0900, Paul Elder wrote:
> Bump version of kmsxx to the current head as of 2026/10/02. Remove
> build options as appropriate.
> 
> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> 
> ---
> The git commit ID didn't seem like a meaningful version to put in the
> commit message but neither did it feel useful to leave it blank so I
> just filled it in with the date... I hope that's acceptable.

That seems fine, but https://xkcd.com/1179/ :-)

> ---
>  package/kmsxx/kmsxx.hash | 2 +-
>  package/kmsxx/kmsxx.mk   | 7 +++----
>  2 files changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/package/kmsxx/kmsxx.hash b/package/kmsxx/kmsxx.hash
> index c141ef7e0725..1d74a0602598 100644
> --- a/package/kmsxx/kmsxx.hash
> +++ b/package/kmsxx/kmsxx.hash
> @@ -1,3 +1,3 @@
>  # Locally calculated
> -sha256  99759fe9334369ccd80ab0bbfe1110ee47e74f1e20abc2d7b2f028067ea0a0c3  kmsxx-824bbb1f4cd062d66b457faca50f904b34dfd96c.tar.gz
> +sha256  8671e99766590055cafd58793eaf56782044e2cf1203a8a390a5726457d39f7d  kmsxx-4d2008f5893a5d6ea2b2bf5e70bc539bfcaf6db0.tar.gz
>  sha256  fab3dd6bdab226f1c08630b1dd917e11fcb4ec5e1e020e2c16f83a0a13863e85  LICENSE
> diff --git a/package/kmsxx/kmsxx.mk b/package/kmsxx/kmsxx.mk
> index f65cc668d484..5aeb48910c7e 100644
> --- a/package/kmsxx/kmsxx.mk
> +++ b/package/kmsxx/kmsxx.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>  
> -KMSXX_VERSION = 824bbb1f4cd062d66b457faca50f904b34dfd96c
> +KMSXX_VERSION = 4d2008f5893a5d6ea2b2bf5e70bc539bfcaf6db0
>  KMSXX_SITE = $(call github,tomba,kmsxx,$(KMSXX_VERSION))
>  KMSXX_LICENSE = MPL-2.0
>  KMSXX_LICENSE_FILES = LICENSE
> @@ -13,8 +13,7 @@ KMSXX_DEPENDENCIES = fmt libdrm host-pkgconf
>  KMSXX_CONF_OPTS = \
>  	-Dkmscube=false \
>  	-Dpykms=disabled \
> -	-Domap=disabled \
> -	-Dsystem-pybind11=enabled
> +	-Domap=disabled
>  
>  ifeq ($(BR2_TOOLCHAIN_HAS_GCC_BUG_85180),y)
>  KMSXX_CXXFLAGS += $(TARGET_CXXFLAGS) -O0
> @@ -23,7 +22,7 @@ endif
>  ifeq ($(BR2_PACKAGE_KMSXX_INSTALL_TESTS),y)
>  KMSXX_CONF_OPTS += -Dutils=true
>  # extra handling for some utils not installed by default
> -KMSXX_EXTRA_UTILS = kmsview kmscapture omap-wbcap omap-wbm2m
> +KMSXX_EXTRA_UTILS = kmsview kmscapture
>  ifeq ($(BR2_PACKAGE_LIBEVDEV),y)
>  KMSXX_DEPENDENCIES += libevdev
>  KMSXX_EXTRA_UTILS += kmstouch

-- 
Regards,

Laurent Pinchart
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 2/2] package/kmsxx: Add option for pykms

[Laurent Pinchart]

Hi Paul,

Thank you for the patch.

On Fri, Oct 02, 2026 at 07:23:33PM +0900, Paul Elder wrote:
> Add option to kmsxx to enable building pykms. While at it, add kmsprint
> and kmstest to the extra utils that are built as they are useful for
> testing and development.

Those are two unrelated changes, I would have split them in two patches.
I don't know if buildroot has strict requirements in that area.

> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> ---
>  package/kmsxx/Config.in |  7 +++++++
>  package/kmsxx/kmsxx.mk  | 10 ++++++++--
>  2 files changed, 15 insertions(+), 2 deletions(-)
> 
> diff --git a/package/kmsxx/Config.in b/package/kmsxx/Config.in
> index fa327ba18de1..dbf2c4ec2a42 100644
> --- a/package/kmsxx/Config.in
> +++ b/package/kmsxx/Config.in
> @@ -16,6 +16,13 @@ config BR2_PACKAGE_KMSXX
>  
>  if BR2_PACKAGE_KMSXX
>  
> +config BR2_PACKAGE_KMSXX_PYKMS
> +	bool "Enable Python bindings"
> +	depends on BR2_PACKAGE_PYTHON3
> +	depends on BR2_PACKAGE_PYTHON_PYBIND
> +	help
> +	  This option enables the pykms python bindings.
> +
>  config BR2_PACKAGE_KMSXX_INSTALL_TESTS
>  	bool "Install test programs"
>  	help
> diff --git a/package/kmsxx/kmsxx.mk b/package/kmsxx/kmsxx.mk
> index 5aeb48910c7e..f52a1047011a 100644
> --- a/package/kmsxx/kmsxx.mk
> +++ b/package/kmsxx/kmsxx.mk
> @@ -12,17 +12,23 @@ KMSXX_INSTALL_STAGING = YES
>  KMSXX_DEPENDENCIES = fmt libdrm host-pkgconf
>  KMSXX_CONF_OPTS = \
>  	-Dkmscube=false \
> -	-Dpykms=disabled \
>  	-Domap=disabled
>  
>  ifeq ($(BR2_TOOLCHAIN_HAS_GCC_BUG_85180),y)
>  KMSXX_CXXFLAGS += $(TARGET_CXXFLAGS) -O0
>  endif
>  
> +ifeq ($(BR2_PACKAGE_KMSXX_PYKMS),y)
> +KMSXX_CONF_OPTS += -Dpykms=enabled
> +KMSXX_DEPENDENCIES += python3 python-pybind
> +else
> +KMSXX_CONF_OPTS += -Dpykms=disabled
> +endif
> +
>  ifeq ($(BR2_PACKAGE_KMSXX_INSTALL_TESTS),y)
>  KMSXX_CONF_OPTS += -Dutils=true
>  # extra handling for some utils not installed by default
> -KMSXX_EXTRA_UTILS = kmsview kmscapture
> +KMSXX_EXTRA_UTILS = kmsview kmscapture kmsprint kmstest
>  ifeq ($(BR2_PACKAGE_LIBEVDEV),y)
>  KMSXX_DEPENDENCIES += libevdev
>  KMSXX_EXTRA_UTILS += kmstouch

-- 
Regards,

Laurent Pinchart
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/2] package/kmsxx: Bump version to current head as of 2026.10

[Thomas Petazzoni via buildroot]

On Fri, Oct 02, 2026 at 07:23:32PM +0900, Paul Elder wrote:
> Bump version of kmsxx to the current head as of 2026/10/02. Remove
> build options as appropriate.
> 
> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> 

Applied to master, thanks!

Best regards,

Thomas Petazzoni
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 2/2] package/kmsxx: Add option for pykms

[Thomas Petazzoni via buildroot]

On Fri, Oct 02, 2026 at 07:23:33PM +0900, Paul Elder wrote:
> Add option to kmsxx to enable building pykms. While at it, add kmsprint
> and kmstest to the extra utils that are built as they are useful for
> testing and development.
> 
> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>

As Laurent suggested, it's best to split into separate commits changes
that are separate.

> +config BR2_PACKAGE_KMSXX_PYKMS
> +	bool "Enable Python bindings"
> +	depends on BR2_PACKAGE_PYTHON3
> +	depends on BR2_PACKAGE_PYTHON_PYBIND

This one should really be a "select".

> +ifeq ($(BR2_PACKAGE_KMSXX_PYKMS),y)
> +KMSXX_CONF_OPTS += -Dpykms=enabled
> +KMSXX_DEPENDENCIES += python3 python-pybind

Are you sure python-pybind is a build-time dependency?

> +else
> +KMSXX_CONF_OPTS += -Dpykms=disabled
> +endif

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 0/2] package/kmsxx: Add support for pykms

[Laurent Pinchart]

On Fri, Oct 02, 2026 at 04:05:29PM +0300, Tomi Valkeinen wrote:
> On 02/10/2026 13:23, Paul Elder wrote:
> > This series adds support for pykms in kmsxx. This enables the python
> > bindings for kmsxx.
> > 
> > The first patch updates kmsxx to the current HEAD, and the second patch
> > adds support for pykms as a separate build config option.
> 
> I don't recall the exact problems, but pybind11 was difficult... Or was 
> it about installing the module to the right place. But if it works now, 
> good =).

The issue with the current buildroot package is that the version of
kmsxx compiled and linked fine, but pykms.so contained unresolved symbol
when the meson utils option was disabled. You have fixed that, so we
decided to just bump the version as an easy fix.

> However, I have not used kms++'s python bindings for a long time, and 
> consider them deprecated. Have you tried https://github.com/tomba/pykms?

Paul's paul to enable the kmsxx Python bindings come from their usage in
kms-tests (https://git.ideasonboard.com/renesas/kms-tests). If pykms
gets packaged in buildroot we could consider it. For the time being, is
there any drawback in enabling the kmsxx Python bindings ?

-- 
Regards,

Laurent Pinchart
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot