On 9/10/20 1:20 PM, Nishanth Menon wrote:

On 12:54-20200910, Suman Anna wrote:
[...]

quoted

quoted

quoted

+		};

I think we could introduce base infrastructure stuff like intr and
inta nodes here? Also, the gpio_intr?

FYI, they are currently being added in Patch 1 from Grygorii's "[v2,0/4] arm64:
dts: ti: k3-j7200: add dma and mcu cpsw" series,
https://patchwork.kernel.org/cover/11763711/

The overall series seems to have some dependencies, so better to separate out
those nodes and include as an additional add-on patch to this series, atleast it
can unblock all others who use the TI-SCI Interrupt node.

There is dependency on udma which in turn needs chipid stuff merged,
but I dont see anything preventing inta intr (which are infrastructure
components) from being merged in. It is not just udma driver that uses these
infrastructure.

Yep, I also meant the same.

[...]

quoted

quoted

quoted

+
+/ {
+	model = "Texas Instruments K3 J7200 SoC";
+	compatible = "ti,j7200";
+	interrupt-parent = <&gic500>;
+	#address-cells = <2>;
+	#size-cells = <2>;
+
+	aliases {
+		serial0 = &wkup_uart0;
+		serial1 = &mcu_uart0;
+		serial2 = &main_uart0;
+		serial3 = &main_uart1;
+		serial4 = &main_uart2;
+		serial5 = &main_uart3;
+		serial6 = &main_uart4;
+		serial7 = &main_uart5;
+		serial8 = &main_uart6;
+		serial9 = &main_uart7;
+		serial10 = &main_uart8;
+		serial11 = &main_uart9;
+	};
+

might be nice to leave a chosen { }; here to indicate board
files fill it up.. just to maintain consistency with rest of SoC dtsis?

Doesn't serve any purpose IMO. I remember commenting about that blank node to
remove it during some earlier reviews.

I know. just consistency for board files to add things in -> you can
see similar usage else where as well. if we are going to drop it, we might
drop from am65 and j721e as well. for bootloaders that look for the node, it
tends to be consistently present OR not.. I'd rather they be consistently
be available than to deal with mix for downstream boards. No strong feelings
either way, but.. lets stay consistent throughout.

$ git grep 'chosen { };' arch/arm64/boot/dts/
arch/arm64/boot/dts/apm/apm-merlin.dts: chosen { };
arch/arm64/boot/dts/apm/apm-mustang.dts:        chosen { };
arch/arm64/boot/dts/arm/foundation-v8.dtsi:     chosen { };
arch/arm64/boot/dts/arm/fvp-base-revc.dts:      chosen { };
arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts:     chosen { };
arch/arm64/boot/dts/hisilicon/hip06-d03.dts:    chosen { };
arch/arm64/boot/dts/mediatek/mt8173-evb.dts:    chosen { };
arch/arm64/boot/dts/qcom/msm8916.dtsi:  chosen { };
arch/arm64/boot/dts/qcom/msm8992.dtsi:  chosen { };
arch/arm64/boot/dts/qcom/msm8994.dtsi:  chosen { };
arch/arm64/boot/dts/qcom/msm8996.dtsi:  chosen { };
arch/arm64/boot/dts/qcom/msm8998.dtsi:  chosen { };
arch/arm64/boot/dts/qcom/qcs404.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sc7180.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sdm630.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sdm660.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sdm845.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sm8150.dtsi:   chosen { };
arch/arm64/boot/dts/qcom/sm8250.dtsi:   chosen { };
arch/arm64/boot/dts/ti/k3-am65.dtsi:    chosen { };
arch/arm64/boot/dts/ti/k3-j721e.dtsi:   chosen { };

Yeah ok to add back then.

regards
Suman