Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
@ 2023-07-05  7:16 Patrick Oppenlander
  2023-08-15 23:09 ` Patrick Oppenlander
  2023-09-02 17:15 ` Thomas Petazzoni via buildroot
  0 siblings, 2 replies; 12+ messages in thread
From: Patrick Oppenlander @ 2023-07-05  7:16 UTC (permalink / raw)
  To: buildroot

This is duplicated from the U-Boot makefile & Kconfig.

Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
devicetree. For example, the AM64x SDRAM controller supports inline ECC
which needs to be enabled in the R5 loader devicetree.

Please note that some versions of U-Boot have broken external devicetree
support and may require manual patching to build. See commit
3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.

Signed-off-by: Patrick Oppenlander <patrick.oppenlander@gmail.com>
---
 boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
 2 files changed, 15 insertions(+)

diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
index 2419babe70..580abf59ec 100644
--- a/boot/ti-k3-r5-loader/Config.in
+++ b/boot/ti-k3-r5-loader/Config.in
@@ -92,3 +92,13 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
  Path to the TI K3 R5 Loader configuration file.

 endif
+
+config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
+ string "Device Tree Source file paths"
+ help
+ Space-separated list of paths to device tree source files
+ that will be copied to arch/ARCH/dts/ before starting the
+ build.
+
+ To use this device tree source file, the TI K3 R5 Loader
+ configuration file must refer to it.
diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
index 8311e1b401..012f4ee52d 100644
--- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
+++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
@@ -77,7 +77,12 @@ TI_K3_R5_LOADER_MAKE_OPTS +=
TI_SECURE_DEV_PKG=$(TI_CORE_SECDEV_K3_INSTALL_DIR)
 endif
 endif

+TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call
qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
+
 define TI_K3_R5_LOADER_BUILD_CMDS
+ $(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
+ cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
+ )
  $(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D)
$(TI_K3_R5_LOADER_MAKE_OPTS)
 endef

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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-07-05  7:16 [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation Patrick Oppenlander
@ 2023-08-15 23:09 ` Patrick Oppenlander
  2023-09-02 17:15 ` Thomas Petazzoni via buildroot
  1 sibling, 0 replies; 12+ messages in thread
From: Patrick Oppenlander @ 2023-08-15 23:09 UTC (permalink / raw)
  To: buildroot; +Cc: Andreas Dannenberg

On Wed, Jul 5, 2023 at 5:16 PM Patrick Oppenlander
<patrick.oppenlander@gmail.com> wrote:
>
> This is duplicated from the U-Boot makefile & Kconfig.
>
> Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
> devicetree. For example, the AM64x SDRAM controller supports inline ECC
> which needs to be enabled in the R5 loader devicetree.
>
> Please note that some versions of U-Boot have broken external devicetree
> support and may require manual patching to build. See commit
> 3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.
>

Ping adding Andreas to cc.

Hi Andreas, could you take a look at this. For our project we need to
enable inline ECC support, which requires customisation of the R5
loader devicetree.

Thanks,

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-07-05  7:16 [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation Patrick Oppenlander
  2023-08-15 23:09 ` Patrick Oppenlander
@ 2023-09-02 17:15 ` Thomas Petazzoni via buildroot
  2023-09-03 23:06   ` patrick.oppenlander
  1 sibling, 1 reply; 12+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-09-02 17:15 UTC (permalink / raw)
  To: Patrick Oppenlander; +Cc: buildroot

Hello Patrick,

On Wed, 5 Jul 2023 17:16:34 +1000
Patrick Oppenlander <patrick.oppenlander@gmail.com> wrote:

> +config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
> + string "Device Tree Source file paths"
> + help
> + Space-separated list of paths to device tree source files
> + that will be copied to arch/ARCH/dts/ before starting the
> + build.
> +
> + To use this device tree source file, the TI K3 R5 Loader
> + configuration file must refer to it.
> diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> index 8311e1b401..012f4ee52d 100644
> --- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> +++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> @@ -77,7 +77,12 @@ TI_K3_R5_LOADER_MAKE_OPTS +=
> TI_SECURE_DEV_PKG=$(TI_CORE_SECDEV_K3_INSTALL_DIR)
>  endif
>  endif
> 
> +TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call
> qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
> +
>  define TI_K3_R5_LOADER_BUILD_CMDS
> + $(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
> + cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
> + )
>   $(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D)
> $(TI_K3_R5_LOADER_MAKE_OPTS)
>  endef
> 

This patch is badly line-wrapped. Could you resend with "git send-email"?

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-02 17:15 ` Thomas Petazzoni via buildroot
@ 2023-09-03 23:06   ` patrick.oppenlander
  2023-09-04 16:26     ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 12+ messages in thread
From: patrick.oppenlander @ 2023-09-03 23:06 UTC (permalink / raw)
  To: buildroot; +Cc: Thomas Petazzoni, Patrick Oppenlander

From: Patrick Oppenlander <patrick.oppenlander@gmail.com>

Hi Thomas,

sorry about that. Resend below.

Patrick

---
 boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
 2 files changed, 15 insertions(+)

diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
index 2419babe70..580abf59ec 100644
--- a/boot/ti-k3-r5-loader/Config.in
+++ b/boot/ti-k3-r5-loader/Config.in
@@ -92,3 +92,13 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
 	  Path to the TI K3 R5 Loader configuration file.
 
 endif
+
+config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
+	string "Device Tree Source file paths"
+	help
+	  Space-separated list of paths to device tree source files
+	  that will be copied to arch/ARCH/dts/ before starting the
+	  build.
+
+	  To use this device tree source file, the TI K3 R5 Loader
+	  configuration file must refer to it.
diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
index 8311e1b401..012f4ee52d 100644
--- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
+++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
@@ -77,7 +77,12 @@ TI_K3_R5_LOADER_MAKE_OPTS += TI_SECURE_DEV_PKG=$(TI_CORE_SECDEV_K3_INSTALL_DIR)
 endif
 endif
 
+TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
+
 define TI_K3_R5_LOADER_BUILD_CMDS
+	$(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
+		cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
+	)
 	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
 endef
 
-- 
2.42.0

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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-03 23:06   ` patrick.oppenlander
@ 2023-09-04 16:26     ` Thomas Petazzoni via buildroot
  2023-09-05  0:33       ` Patrick Oppenlander
  2023-09-05  0:33       ` patrick.oppenlander
  0 siblings, 2 replies; 12+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-09-04 16:26 UTC (permalink / raw)
  To: patrick.oppenlander; +Cc: buildroot

On Mon,  4 Sep 2023 09:06:09 +1000
patrick.oppenlander@gmail.com wrote:

> From: Patrick Oppenlander <patrick.oppenlander@gmail.com>
> 
> Hi Thomas,
> 
> sorry about that. Resend below.
> 
> Patrick

This also doesn't work, because now your commit log would be:

=============================================================
Hi Thomas,

sorry about that. Resend below.

Patrick
=============================================================

Your commit log should be a proper commit log, which explains the
change and its rationale.

Thanks a lot!

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-04 16:26     ` Thomas Petazzoni via buildroot
@ 2023-09-05  0:33       ` Patrick Oppenlander
  2023-09-05  0:33       ` patrick.oppenlander
  1 sibling, 0 replies; 12+ messages in thread
From: Patrick Oppenlander @ 2023-09-05  0:33 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: buildroot

Hi Thomas,

I messed that up, sorry. I'll send it again.

Patrick

On Tue, Sep 5, 2023 at 2:26 AM Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> On Mon,  4 Sep 2023 09:06:09 +1000
> patrick.oppenlander@gmail.com wrote:
>
> > From: Patrick Oppenlander <patrick.oppenlander@gmail.com>
> >
> > Hi Thomas,
> >
> > sorry about that. Resend below.
> >
> > Patrick
>
> This also doesn't work, because now your commit log would be:
>
> =============================================================
> Hi Thomas,
>
> sorry about that. Resend below.
>
> Patrick
> =============================================================
>
> Your commit log should be a proper commit log, which explains the
> change and its rationale.
>
> Thanks a lot!
>
> 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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-04 16:26     ` Thomas Petazzoni via buildroot
  2023-09-05  0:33       ` Patrick Oppenlander
@ 2023-09-05  0:33       ` patrick.oppenlander
  2023-09-05  7:07         ` Thomas Petazzoni via buildroot
  1 sibling, 1 reply; 12+ messages in thread
From: patrick.oppenlander @ 2023-09-05  0:33 UTC (permalink / raw)
  To: buildroot; +Cc: Thomas Petazzoni, Patrick Oppenlander

From: Patrick Oppenlander <patrick.oppenlander@gmail.com>

This is duplicated from the U-Boot makefile & Kconfig.

Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
devicetree. For example, the AM64x SDRAM controller supports inline ECC
which needs to be enabled in the R5 loader devicetree.

Please note that some versions of U-Boot have broken external devicetree
support and may require manual patching to build. See commit
3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.
---
 boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
 2 files changed, 15 insertions(+)

diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
index 2419babe70..580abf59ec 100644
--- a/boot/ti-k3-r5-loader/Config.in
+++ b/boot/ti-k3-r5-loader/Config.in
@@ -92,3 +92,13 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
 	  Path to the TI K3 R5 Loader configuration file.
 
 endif
+
+config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
+	string "Device Tree Source file paths"
+	help
+	  Space-separated list of paths to device tree source files
+	  that will be copied to arch/ARCH/dts/ before starting the
+	  build.
+
+	  To use this device tree source file, the TI K3 R5 Loader
+	  configuration file must refer to it.
diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
index 8311e1b401..012f4ee52d 100644
--- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
+++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
@@ -77,7 +77,12 @@ TI_K3_R5_LOADER_MAKE_OPTS += TI_SECURE_DEV_PKG=$(TI_CORE_SECDEV_K3_INSTALL_DIR)
 endif
 endif
 
+TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
+
 define TI_K3_R5_LOADER_BUILD_CMDS
+	$(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
+		cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
+	)
 	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
 endef
 
-- 
2.42.0

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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-05  0:33       ` patrick.oppenlander
@ 2023-09-05  7:07         ` Thomas Petazzoni via buildroot
  2023-09-07  0:30           ` patrick.oppenlander
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-09-05  7:07 UTC (permalink / raw)
  To: patrick.oppenlander; +Cc: buildroot

Hello Patrick,

On Tue,  5 Sep 2023 10:33:48 +1000
patrick.oppenlander@gmail.com wrote:

> From: Patrick Oppenlander <patrick.oppenlander@gmail.com>
> 
> This is duplicated from the U-Boot makefile & Kconfig.
> 
> Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
> devicetree. For example, the AM64x SDRAM controller supports inline ECC
> which needs to be enabled in the R5 loader devicetree.
> 
> Please note that some versions of U-Boot have broken external devicetree
> support and may require manual patching to build. See commit
> 3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.

We're almost there! We now need you to add your Signed-off-by line...

... here!

> ---
>  boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
>  2 files changed, 15 insertions(+)

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation
  2023-09-05  7:07         ` Thomas Petazzoni via buildroot
@ 2023-09-07  0:30           ` patrick.oppenlander
  2024-04-30 13:37             ` [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization Gero Schwäricke via buildroot
  0 siblings, 1 reply; 12+ messages in thread
From: patrick.oppenlander @ 2023-09-07  0:30 UTC (permalink / raw)
  To: buildroot; +Cc: Patrick Oppenlander, Thomas Petazzoni

From: Patrick Oppenlander <patrick.oppenlander@gmail.com>

This is duplicated from the U-Boot makefile & Kconfig.

Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
devicetree. For example, the AM64x SDRAM controller supports inline ECC
which needs to be enabled in the R5 loader devicetree.

Please note that some versions of U-Boot have broken external devicetree
support and may require manual patching to build. See commit
3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.

Signed-off-by: Patrick Oppenlander <patrick.oppenlander@gmail.com>
---
 boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
 2 files changed, 15 insertions(+)

diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
index 2419babe70..580abf59ec 100644
--- a/boot/ti-k3-r5-loader/Config.in
+++ b/boot/ti-k3-r5-loader/Config.in
@@ -92,3 +92,13 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
 	  Path to the TI K3 R5 Loader configuration file.
 
 endif
+
+config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
+	string "Device Tree Source file paths"
+	help
+	  Space-separated list of paths to device tree source files
+	  that will be copied to arch/ARCH/dts/ before starting the
+	  build.
+
+	  To use this device tree source file, the TI K3 R5 Loader
+	  configuration file must refer to it.
diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
index 8311e1b401..012f4ee52d 100644
--- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
+++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
@@ -77,7 +77,12 @@ TI_K3_R5_LOADER_MAKE_OPTS += TI_SECURE_DEV_PKG=$(TI_CORE_SECDEV_K3_INSTALL_DIR)
 endif
 endif
 
+TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
+
 define TI_K3_R5_LOADER_BUILD_CMDS
+	$(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
+		cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
+	)
 	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
 endef
 
-- 
2.41.0

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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization
  2023-09-07  0:30           ` patrick.oppenlander
@ 2024-04-30 13:37             ` Gero Schwäricke via buildroot
  2024-04-30 13:42               ` Gero Schwäricke via buildroot
  2024-05-01 21:53               ` Romain Naour
  0 siblings, 2 replies; 12+ messages in thread
From: Gero Schwäricke via buildroot @ 2024-04-30 13:37 UTC (permalink / raw)
  To: buildroot
  Cc: Xuanhao Shi, Gero Schwäricke, Patrick Oppenlander,
	Anand Gadiyar

This is duplicated from the U-Boot makefile & Kconfig.

Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
devicetree. For example, the AM64x SDRAM controller supports inline ECC
which needs to be enabled in the R5 loader devicetree.

Please note that some versions of U-Boot have broken external devicetree
support and may require manual patching to build. See commit
3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.

Signed-off-by: Patrick Oppenlander <patrick.oppenlander@gmail.com>
Signed-off-by: Gero Schwäricke <gero.schwaericke@grandcentrix.net>
---
v1 -> v2:
  - make it apply on current master
  - move the config option into  if BR2_TARGET_TI_K3_R5_LOADER
  - fix help string mentioning ARCH though it's fixed to arm in .mk
---
 boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
 boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
 2 files changed, 15 insertions(+)

diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
index 6d1adbeac3..df885cc2af 100644
--- a/boot/ti-k3-r5-loader/Config.in
+++ b/boot/ti-k3-r5-loader/Config.in
@@ -95,6 +95,16 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
 	help
 	  Path to the TI K3 R5 Loader configuration file.
 
+config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
+	string "Device Tree Source file paths"
+	help
+	  Space-separated list of paths to device tree source files
+	  that will be copied to arch/arm/dts/ before starting the
+	  build.
+
+	  To use this device tree source file, the U-Boot configuration
+	  file must refer to it.
+
 config BR2_TARGET_TI_K3_R5_LOADER_TIBOOT3_BIN
 	string "tiboot3.bin binary name"
 	default "tiboot3.bin"
diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
index 891783e40f..98a6d473f1 100644
--- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
+++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
@@ -88,7 +88,12 @@ define TI_K3_R5_LOADER_INSTALL_SWSFW_ITB
 	fi
 endef
 
+TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
+
 define TI_K3_R5_LOADER_BUILD_CMDS
+	$(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
+		cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
+	)
 	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
 endef
 
-- 
2.43.2


-- 



–––


grandcentrix GmbH | A Vodafone Company · Holzmarkt 1 · 50676 
*Cologne* | Phoenixseestrasse 18 · 44263 *Dortmund* · Germany
| in 
<https://www.linkedin.com/company/grandcentrix> | phone: +49-221-677-860-0 
| email: hello@grandcentrix.net <mailto:hello@grandcentrix.net>


grandcentrix cares about privacy 
<https://grandcentrix.net/de/pages/privacymail/>

Amtsgericht Köln | HRB  
70119 | Geschäftsführer: M. Krömer,  R. Hänel | USt.-IdNr.: DE266333969

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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization
  2024-04-30 13:37             ` [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization Gero Schwäricke via buildroot
@ 2024-04-30 13:42               ` Gero Schwäricke via buildroot
  2024-05-01 21:53               ` Romain Naour
  1 sibling, 0 replies; 12+ messages in thread
From: Gero Schwäricke via buildroot @ 2024-04-30 13:42 UTC (permalink / raw)
  To: buildroot

Note: This is v2 for

  [PATCH] ti-k3-r5-loader: support devicetree customisation

  https://patchwork.ozlabs.org/project/buildroot/patch/20230905003412.67305-2-patrick.oppenlander@gmail.com/

I think patchwork didn't catch that because I spelled the subject line
differently.

Best,
Gero

-- 



---


grandcentrix GmbH | A Vodafone Company · Holzmarkt 1 · 50676 
*Cologne* | Phoenixseestrasse 18 · 44263 *Dortmund* · Germany
| in 
<https://www.linkedin.com/company/grandcentrix> | phone: +49-221-677-860-0 
| email: hello@grandcentrix.net <mailto:hello@grandcentrix.net>


grandcentrix cares about privacy 
<https://grandcentrix.net/de/pages/privacymail/>

Amtsgericht Köln | HRB  
70119 | Geschäftsführer: M. Krömer,  R. Hänel | USt.-IdNr.: DE266333969

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization
  2024-04-30 13:37             ` [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization Gero Schwäricke via buildroot
  2024-04-30 13:42               ` Gero Schwäricke via buildroot
@ 2024-05-01 21:53               ` Romain Naour
  1 sibling, 0 replies; 12+ messages in thread
From: Romain Naour @ 2024-05-01 21:53 UTC (permalink / raw)
  To: Gero Schwäricke, buildroot
  Cc: Xuanhao Shi, Patrick Oppenlander, Anand Gadiyar

Hello Gero,

Le 30/04/2024 à 15:37, Gero Schwäricke via buildroot a écrit :
> This is duplicated from the U-Boot makefile & Kconfig.
> 
> Similarly to U-Boot custom boards sometimes need to adjust the U-Boot
> devicetree. For example, the AM64x SDRAM controller supports inline ECC
> which needs to be enabled in the R5 loader devicetree.
> 
> Please note that some versions of U-Boot have broken external devicetree
> support and may require manual patching to build. See commit
> 3609e1dc5f4d4c238dcd23e045ba6223310feffa in the U-Boot tree.

I added the URL to gitlab instead:
https://source.denx.de/u-boot/u-boot/-/commit/3609e1dc5f4d4c238dcd23e045ba6223310feffa

Applied to master, thanks.

Best regards,
Romain


> 
> Signed-off-by: Patrick Oppenlander <patrick.oppenlander@gmail.com>
> Signed-off-by: Gero Schwäricke <gero.schwaericke@grandcentrix.net>
> ---
> v1 -> v2:
>   - make it apply on current master
>   - move the config option into  if BR2_TARGET_TI_K3_R5_LOADER
>   - fix help string mentioning ARCH though it's fixed to arm in .mk
> ---
>  boot/ti-k3-r5-loader/Config.in          | 10 ++++++++++
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk |  5 +++++
>  2 files changed, 15 insertions(+)
> 
> diff --git a/boot/ti-k3-r5-loader/Config.in b/boot/ti-k3-r5-loader/Config.in
> index 6d1adbeac3..df885cc2af 100644
> --- a/boot/ti-k3-r5-loader/Config.in
> +++ b/boot/ti-k3-r5-loader/Config.in
> @@ -95,6 +95,16 @@ config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_CONFIG_FILE
>  	help
>  	  Path to the TI K3 R5 Loader configuration file.
>  
> +config BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH
> +	string "Device Tree Source file paths"
> +	help
> +	  Space-separated list of paths to device tree source files
> +	  that will be copied to arch/arm/dts/ before starting the
> +	  build.
> +
> +	  To use this device tree source file, the U-Boot configuration
> +	  file must refer to it.
> +
>  config BR2_TARGET_TI_K3_R5_LOADER_TIBOOT3_BIN
>  	string "tiboot3.bin binary name"
>  	default "tiboot3.bin"
> diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> index 891783e40f..98a6d473f1 100644
> --- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> +++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> @@ -88,7 +88,12 @@ define TI_K3_R5_LOADER_INSTALL_SWSFW_ITB
>  	fi
>  endef
>  
> +TI_K3_R5_LOADER_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_TI_K3_R5_LOADER_CUSTOM_DTS_PATH))
> +
>  define TI_K3_R5_LOADER_BUILD_CMDS
> +	$(if $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH),
> +		cp -f $(TI_K3_R5_LOADER_CUSTOM_DTS_PATH) $(@D)/arch/arm/dts/
> +	)
>  	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
>  endef
>  

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-05-01 21:54 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-05  7:16 [Buildroot] [PATCH] ti-k3-r5-loader: support devicetree customisation Patrick Oppenlander
2023-08-15 23:09 ` Patrick Oppenlander
2023-09-02 17:15 ` Thomas Petazzoni via buildroot
2023-09-03 23:06   ` patrick.oppenlander
2023-09-04 16:26     ` Thomas Petazzoni via buildroot
2023-09-05  0:33       ` Patrick Oppenlander
2023-09-05  0:33       ` patrick.oppenlander
2023-09-05  7:07         ` Thomas Petazzoni via buildroot
2023-09-07  0:30           ` patrick.oppenlander
2024-04-30 13:37             ` [Buildroot] [PATCH v2] ti-k3-r5-loader: support devicetree customization Gero Schwäricke via buildroot
2024-04-30 13:42               ` Gero Schwäricke via buildroot
2024-05-01 21:53               ` Romain Naour

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox