[Buildroot] [PATCH] package: new: add k3conf tool package

[Buildroot] [PATCH] package: new: add k3conf tool package

[Sadik SM via buildroot]

From: sadik <s-sadik@ti.com>

add the k3conf tool in the package section

K3CONF is a Linux user-space standalone application
designed to provide a quick'n easy way to dynamically
diagnose Texas Instruments' K3 architecture based
processors. K3CONF is intended to provide similar
experience to that of OMAPCONF that runs on legacy TI platforms.

K3CONF currently supports Texas Instruments AM654, J721E, J7200, 
AM64x,AM62x,J721S2,J784S4,J722S, AM62Ax, AM62Px, and AM62Lx devices.
Along with the BeagleBoard variants of the above mentioned TI SOC's.

Signed-off-by: sadik <s-sadik@ti.com>
---
 DEVELOPERS                 |  3 +++
 package/Config.in          |  1 +
 package/k3conf/Config.in   |  4 ++++
 package/k3conf/k3conf.hash |  2 ++
 package/k3conf/k3conf.mk   | 16 ++++++++++++++++
 5 files changed, 26 insertions(+)
 create mode 100644 package/k3conf/Config.in
 create mode 100644 package/k3conf/k3conf.hash
 create mode 100644 package/k3conf/k3conf.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index 1b27df9beb..396452f261 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -2962,6 +2962,9 @@ F:	package/agent-proxy/
 N:	Ryan Wilkins <ryan@deadfrog.net>
 F:	package/biosdevname/
 
+N:  Sadik SM <s-sadik@ti.com>
+F:  package/k3conf
+
 N:	Saeed Kazemi <kazemi.ms@gmail.com>
 F:	package/eza/
 F:	package/fd/
diff --git a/package/Config.in b/package/Config.in
index 22970b4d45..a0aa112070 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -110,6 +110,7 @@ menu "Debugging, profiling and benchmark"
 	source "package/google-breakpad/Config.in"
 	source "package/hyperfine/Config.in"
 	source "package/iozone/Config.in"
+	source "package/k3conf/Config.in"
 	source "package/kexec/Config.in"
 	source "package/kexec-lite/Config.in"
 	source "package/kmemd/Config.in"
diff --git a/package/k3conf/Config.in b/package/k3conf/Config.in
new file mode 100644
index 0000000000..f282ad672b
--- /dev/null
+++ b/package/k3conf/Config.in
@@ -0,0 +1,4 @@
+config BR2_PACKAGE_K3CONF
+    bool "k3conf"
+    help
+      A Powerful Diagnostic Tool for Texas Instruments K3 based Processors.
diff --git a/package/k3conf/k3conf.hash b/package/k3conf/k3conf.hash
new file mode 100644
index 0000000000..47d9eca6a0
--- /dev/null
+++ b/package/k3conf/k3conf.hash
@@ -0,0 +1,2 @@
+# Locally calculated sha256sum
+sha256  a3459f80e7af5672da6b41bc0814114690dd549aac4cf81ba4e5dc84d42ab9c6  k3conf-v0.4-git4.tar.gz
diff --git a/package/k3conf/k3conf.mk b/package/k3conf/k3conf.mk
new file mode 100644
index 0000000000..b248fb63d5
--- /dev/null
+++ b/package/k3conf/k3conf.mk
@@ -0,0 +1,16 @@
+################################################################################
+#
+# k3conf
+#
+################################################################################
+
+K3CONF_VERSION = v0.4
+K3CONF_SITE = https://git.ti.com/git/k3conf/k3conf.git
+K3CONF_SITE_METHOD = git
+K3CONF_LICENSE = BSD-3-Clause
+K3CONF_LICENSE_FILES = LICENSE
+
+
+K3CONF_CONF_OPTS = -DCMAKE_BUILD_TYPE=Release
+
+$(eval $(cmake-package))
-- 
2.34.1

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

Re: [Buildroot] [PATCH] package: new: add k3conf tool package

[Bryan Brattlof via buildroot]

On December 15, 2025 thus sayeth Sadik SM via buildroot:
> From: sadik <s-sadik@ti.com>
> 
> add the k3conf tool in the package section
> 
> K3CONF is a Linux user-space standalone application
> designed to provide a quick'n easy way to dynamically
> diagnose Texas Instruments' K3 architecture based
> processors. K3CONF is intended to provide similar
> experience to that of OMAPCONF that runs on legacy TI platforms.
> 
> K3CONF currently supports Texas Instruments AM654, J721E, J7200, 
> AM64x,AM62x,J721S2,J784S4,J722S, AM62Ax, AM62Px, and AM62Lx devices.
> Along with the BeagleBoard variants of the above mentioned TI SOC's.
>

Works for me! I wonder if we should call out the kernel needs to provide 
/dev/mem for this tool[0] 

[0] https://git.ti.com/cgit/k3conf/k3conf/tree/common/mmio.c#n39

> Signed-off-by: sadik <s-sadik@ti.com>
> ---
>  DEVELOPERS                 |  3 +++
>  package/Config.in          |  1 +
>  package/k3conf/Config.in   |  4 ++++
>  package/k3conf/k3conf.hash |  2 ++
>  package/k3conf/k3conf.mk   | 16 ++++++++++++++++
>  5 files changed, 26 insertions(+)
>  create mode 100644 package/k3conf/Config.in
>  create mode 100644 package/k3conf/k3conf.hash
>  create mode 100644 package/k3conf/k3conf.mk
> 
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 1b27df9beb..396452f261 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -2962,6 +2962,9 @@ F:	package/agent-proxy/
>  N:	Ryan Wilkins <ryan@deadfrog.net>
>  F:	package/biosdevname/
>  
> +N:  Sadik SM <s-sadik@ti.com>
> +F:  package/k3conf

I think we may have some white-space issues here

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

Re: [Buildroot] [PATCH] package: new: add k3conf tool package

[Thomas Petazzoni via buildroot]

On Mon, 15 Dec 2025 08:00:51 -0600
Bryan Brattlof <bb@ti.com> wrote:

> Works for me! I wonder if we should call out the kernel needs to provide 
> /dev/mem for this tool[0] 
> 
> [0] https://git.ti.com/cgit/k3conf/k3conf/tree/common/mmio.c#n39

You can always do:

define K3CONF_LINUX_CONFIG_FIXUPS
        $(call KCONFIG_ENABLE_OPT,CONFIG_DEVMEM)
endef

if you want to be sure it is enabled.

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] package: new: add k3conf tool package

[Thomas Petazzoni via buildroot]

Hello,

Thanks a lot for your patch! Please find below a review of your patch.

First the commit title should be:

	package/k3conf: new package

On Mon, 15 Dec 2025 18:54:25 +0530
Sadik SM <s-sadik@ti.com> wrote:

> From: sadik <s-sadik@ti.com>

Please use your full first name / last name, with proper capitalization.

> add the k3conf tool in the package section

This sentence is not needed.

> 
> K3CONF is a Linux user-space standalone application
> designed to provide a quick'n easy way to dynamically
> diagnose Texas Instruments' K3 architecture based
> processors. K3CONF is intended to provide similar
> experience to that of OMAPCONF that runs on legacy TI platforms.
> 
> K3CONF currently supports Texas Instruments AM654, J721E, J7200, 
> AM64x,AM62x,J721S2,J784S4,J722S, AM62Ax, AM62Px, and AM62Lx devices.
> Along with the BeagleBoard variants of the above mentioned TI SOC's.
> 
> Signed-off-by: sadik <s-sadik@ti.com>

Please use your full first name / last name, with proper capitalization.

> diff --git a/DEVELOPERS b/DEVELOPERS
> index 1b27df9beb..396452f261 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -2962,6 +2962,9 @@ F:	package/agent-proxy/
>  N:	Ryan Wilkins <ryan@deadfrog.net>
>  F:	package/biosdevname/
>  
> +N:  Sadik SM <s-sadik@ti.com>
> +F:  package/k3conf

Please use the same indentation as in the rest of the document. Put a
final / in package/k3conf/, like for other entries.

Use your full first name / last name.


> diff --git a/package/k3conf/Config.in b/package/k3conf/Config.in
> new file mode 100644
> index 0000000000..f282ad672b
> --- /dev/null
> +++ b/package/k3conf/Config.in
> @@ -0,0 +1,4 @@
> +config BR2_PACKAGE_K3CONF
> +    bool "k3conf"
> +    help
> +      A Powerful Diagnostic Tool for Texas Instruments K3 based Processors.

Please find the indentation to match other Config.in files: one tab for
bool/help, one tab + two spaces for the help text.

Add the upstream URL of the project at the end of the help text.

You can run "make check-package" to check for coding style issues.

I think you can add a "depends on BR2_aarch64" dependency to this
package since anyway it doesn't make sense for other CPU architectures.


> diff --git a/package/k3conf/k3conf.hash b/package/k3conf/k3conf.hash
> new file mode 100644
> index 0000000000..47d9eca6a0
> --- /dev/null
> +++ b/package/k3conf/k3conf.hash
> @@ -0,0 +1,2 @@
> +# Locally calculated sha256sum
> +sha256  a3459f80e7af5672da6b41bc0814114690dd549aac4cf81ba4e5dc84d42ab9c6  k3conf-v0.4-git4.tar.gz

Please add a hash for the license file.

> diff --git a/package/k3conf/k3conf.mk b/package/k3conf/k3conf.mk
> new file mode 100644
> index 0000000000..b248fb63d5
> --- /dev/null
> +++ b/package/k3conf/k3conf.mk
> @@ -0,0 +1,16 @@
> +################################################################################
> +#
> +# k3conf
> +#
> +################################################################################
> +
> +K3CONF_VERSION = v0.4
> +K3CONF_SITE = https://git.ti.com/git/k3conf/k3conf.git
> +K3CONF_SITE_METHOD = git
> +K3CONF_LICENSE = BSD-3-Clause
> +K3CONF_LICENSE_FILES = LICENSE
> +
> +

Only one blank line.

> +K3CONF_CONF_OPTS = -DCMAKE_BUILD_TYPE=Release

This is not needed, it's already passed by the cmake-package
infrastructure.

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