mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: ti: k3-am62: copy bootph flags to Linux
@ 2025-06-24 14:50 Bryan Brattlof
  2025-06-26  4:02 ` Vignesh Raghavendra
  0 siblings, 1 reply; 3+ messages in thread
From: Bryan Brattlof @ 2025-06-24 14:50 UTC (permalink / raw)
  To: Nishanth Menon, Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-kernel, devicetree, linux-kernel, Bryan Brattlof

To keep things as organized as possible, copy the bootph-all properties
from U-Boot for the packet DMA controller to indicate it should be
available during all phases of bootup.

Signed-off-by: Bryan Brattlof <bb@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
index 9e0b6eee9ac77d66869915b2d7bec3e2275c03ea..2a727caf36ec44e4a023c5985d1bad9b80737633 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
@@ -150,6 +150,7 @@ main_pktdma: dma-controller@485c0000 {
 				    "ring", "tchan", "rchan", "rflow";
 			msi-parent = <&inta_main_dmss>;
 			#dma-cells = <2>;
+			bootph-all;
 
 			ti,sci = <&dmsc>;
 			ti,sci-dev-id = <30>;

---
base-commit: 19272b37aa4f83ca52bdf9c16d5d81bdd1354494
change-id: 20250623-62-uboot-cleanup-dbec66f9e257

Best regards,
-- 
Bryan Brattlof <bb@ti.com>


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

* Re: [PATCH] arm64: dts: ti: k3-am62: copy bootph flags to Linux
  2025-06-24 14:50 [PATCH] arm64: dts: ti: k3-am62: copy bootph flags to Linux Bryan Brattlof
@ 2025-06-26  4:02 ` Vignesh Raghavendra
  2025-06-26 11:33   ` Bryan Brattlof
  0 siblings, 1 reply; 3+ messages in thread
From: Vignesh Raghavendra @ 2025-06-26  4:02 UTC (permalink / raw)
  To: Bryan Brattlof, Nishanth Menon, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-kernel, devicetree, linux-kernel



On 24/06/25 20:20, Bryan Brattlof wrote:
> To keep things as organized as possible, copy the bootph-all properties
> from U-Boot for the packet DMA controller to indicate it should be
> available during all phases of bootup.
> 

Above isn't a valid reason to add bootph-all properties to a node. You
would need to say why this DMA controller is needed during all phases of
boot, does this enable a certain boot-mode? Fix $subject to reflect the
same.

> Signed-off-by: Bryan Brattlof <bb@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> index 9e0b6eee9ac77d66869915b2d7bec3e2275c03ea..2a727caf36ec44e4a023c5985d1bad9b80737633 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> @@ -150,6 +150,7 @@ main_pktdma: dma-controller@485c0000 {
>  				    "ring", "tchan", "rchan", "rflow";
>  			msi-parent = <&inta_main_dmss>;
>  			#dma-cells = <2>;
> +			bootph-all;
>  
>  			ti,sci = <&dmsc>;
>  			ti,sci-dev-id = <30>;
> 
> ---
> base-commit: 19272b37aa4f83ca52bdf9c16d5d81bdd1354494
> change-id: 20250623-62-uboot-cleanup-dbec66f9e257
> 
> Best regards,

-- 
Regards
Vignesh
https://ti.com/opensource


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

* Re: [PATCH] arm64: dts: ti: k3-am62: copy bootph flags to Linux
  2025-06-26  4:02 ` Vignesh Raghavendra
@ 2025-06-26 11:33   ` Bryan Brattlof
  0 siblings, 0 replies; 3+ messages in thread
From: Bryan Brattlof @ 2025-06-26 11:33 UTC (permalink / raw)
  To: Vignesh Raghavendra
  Cc: Nishanth Menon, Tero Kristo, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, linux-arm-kernel, devicetree, linux-kernel

On June 26, 2025 thus sayeth Vignesh Raghavendra:
> 
> 
> On 24/06/25 20:20, Bryan Brattlof wrote:
> > To keep things as organized as possible, copy the bootph-all properties
> > from U-Boot for the packet DMA controller to indicate it should be
> > available during all phases of bootup.
> > 
> 
> Above isn't a valid reason to add bootph-all properties to a node. You
> would need to say why this DMA controller is needed during all phases of
> boot, does this enable a certain boot-mode? Fix $subject to reflect the
> same.

Ok will do

~Bryan

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

end of thread, other threads:[~2025-06-26 11:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-06-24 14:50 [PATCH] arm64: dts: ti: k3-am62: copy bootph flags to Linux Bryan Brattlof
2025-06-26  4:02 ` Vignesh Raghavendra
2025-06-26 11:33   ` Bryan Brattlof

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

all inboxes | Powered by JetHome®