[PATCH v2] arm64: dts: ti: k3-j784s4: Add GPU node

Antonios Christidis posted 1 patch 3 weeks, 2 days ago
arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 14 ++++++++++++++
1 file changed, 14 insertions(+)
[PATCH v2] arm64: dts: ti: k3-j784s4: Add GPU node
Posted by Antonios Christidis 3 weeks, 2 days ago
Add the Series BXS GPU node for j784s4 device tree.

Signed-off-by: Antonios Christidis <a-christidis@ti.com>
---
v2:
- Separated from the original series ([1]) per reviewer feedback ([2])

[1]: https://lore.kernel.org/all/20260224-gpu_dts-v1-0-cc5ddffe140c@ti.com/
[2]: https://lore.kernel.org/all/20260225010507.flvt775fs5kfe7ez@unknotted/
---
 arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
index 78fcd0c40abc..ddb9385cd942 100644
--- a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
@@ -137,6 +137,20 @@ serdes2: serdes@5020000 {
 		};
 	};
 
+	gpu: gpu@4e20000000 {
+		compatible = "ti,j721s2-gpu", "img,img-bxs-4-64", "img,img-rogue";
+		reg = <0x4e 0x20000000 0x00 0x80000>;
+		clocks = <&k3_clks 181 1>;
+		clock-names = "core";
+		assigned-clocks = <&k3_clks 181 1>;
+		assigned-clock-rates = <800000000>;
+		interrupts = <GIC_SPI 24 IRQ_TYPE_LEVEL_HIGH>;
+		power-domains = <&k3_pds 181 TI_SCI_PD_EXCLUSIVE>,
+			<&k3_pds 182 TI_SCI_PD_EXCLUSIVE>;
+		power-domain-names = "a", "b";
+		dma-coherent;
+	};
+
 	c71_3: dsp@67800000 {
 		compatible = "ti,j721s2-c71-dsp";
 		reg = <0x00 0x67800000 0x00 0x00080000>,

---
base-commit: 735d2f48cadaa9a87e7c7601667878de70c771c5
change-id: 20260506-j784s4_gpu_node_upstream-be81ebf0ca2b

Best regards,
-- 
Antonios Christidis <a-christidis@ti.com>
Re: [PATCH v2] arm64: dts: ti: k3-j784s4: Add GPU node
Posted by Nishanth Menon 2 weeks ago
On 15:18-20260506, Antonios Christidis wrote:
> Add the Series BXS GPU node for j784s4 device tree.
> 

Adding documentation why j721s2 compatible is correct for j784s4 is
needed.

[...]
> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> index 78fcd0c40abc..ddb9385cd942 100644
> --- a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> @@ -137,6 +137,20 @@ serdes2: serdes@5020000 {
>  		};
>  	};
>  
> +	gpu: gpu@4e20000000 {


https://docs.kernel.org/devicetree/bindings/dts-coding-style.html

"Nodes on any bus, thus using unit addresses for children, shall be
ordered by unit address in ascending order. Alternatively for some
subarchitectures, nodes of the same type can be grouped together,
e.g. all I2C controllers one after another even if this breaks unit
address ordering."

Any specific reason to keep this patch ordering?

> +		compatible = "ti,j721s2-gpu", "img,img-bxs-4-64", "img,img-rogue";

^^ j721s2-gpu: please clarify in commit message.

[...]

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource