All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anshul Dalal <anshuld@ti.com>
To: Anshul Dalal <anshuld@ti.com>, <u-boot@lists.denx.de>,
	<upstream@lists.phytec.de>
Cc: Tom Rini <trini@konsulko.com>, Nishanth Menon <nm@ti.com>,
	Robert Nelson <robertcnelson@gmail.com>,
	Wadim Egorov <w.egorov@phytec.de>,
	"Francesco Dolcini" <francesco.dolcini@toradex.com>,
	Garrett Giordano <ggiordano@phytec.com>,
	Manorit Chawdhry <m-chawdhry@ti.com>,
	"Bryan Brattlof" <bb@ti.com>, Paresh Bhagat <p-bhagat@ti.com>,
	Vitor Soares <vitor.soares@toradex.com>,
	Parth Pancholi <parth.pancholi@toradex.com>,
	Andrew Davis <afd@ti.com>, Gowtham Tammana <g-tammana@ti.com>,
	Vignesh R <vigneshr@ti.com>
Subject: Re: [PATCH master] arch: arm: dts: k3: refactor common nodes to k3-*-r5.dtsi
Date: Mon, 2 Feb 2026 13:55:19 +0530	[thread overview]
Message-ID: <DG4BZ97UWPF0.FCYZ7RUNHS94@ti.com> (raw)
In-Reply-To: <20260122-r5_dt_refactor-v1-1-5442a4321756@ti.com>

On Thu Jan 22, 2026 at 7:38 PM IST, Anshul Dalal wrote:
> This patch refactors the nodes in each board's R5 device-tree to common
> SoC level dtsi. No functional change is intended from this patch.
>
> Signed-off-by: Anshul Dalal <anshuld@ti.com>

Any updates on this patch?

> ---
> I have verified the DT delta before and after the patch and there is
> no difference except for the addition of "tick-timer" under /aliases and
> "clock-names" under the a53/a72 cores on some platforms which should
> cause no functional change. An exhaustive diff for all DTs is provided
> below[1].
>
> The following affected platforms have not been boot tested:
>  - k3-am625-r5-beagleplay
>  - k3-am67a-r5-beagley-ai
>  - k3-am642-r5-phycore-som-2gb
>  - k3-am625-r5-phycore-som-2gb
>  - k3-am62a7-r5-phycore-som-2gb
>  - k3-am625-verdin-r5
>  - k3-am62p5-verdin-r5
>
> Though I'm not sure if the patch should be split per SoC for ease of
> review?
>
> [1]:
>
>   k3-am625-r5-beagleplay:
>    --- build/before/spl/u-boot-spl.dtb
>    +++ build/after/spl/u-boot-spl.dtb
>    @@ -31,6 +31,7 @@
>     		remoteproc0 = "/bus@f0000/sysctrler";
>     		remoteproc1 = "/a53@0";
>     		serial2 = "/bus@f0000/serial@2800000";
>    +		tick-timer = "/bus@f0000/timer@2400000";
>     		usb0 = "/bus@f0000/dwc3-usb@f900000/usb@31000000";
>     	};
>
>   k3-am625-r5-phycore-som-2gb:
>    --- build/before/spl/u-boot-spl.dtb
>    +++ build/after/spl/u-boot-spl.dtb
>    @@ -10,6 +10,7 @@
>     		assigned-clock-parents = <0x02 0x3d 0x02>;
>     		assigned-clock-rates = <0xbebc200 0x47868c00>;
>     		assigned-clocks = <0x02 0x3d 0x00 0x02 0x87 0x00>;
>    +		clock-names = "gtc";
>     		clocks = <0x02 0x3d 0x00>;
>     		compatible = "ti,am654-rproc";
>     		power-domains = <0x03 0x3d 0x01 0x03 0x87 0x01 0x03 0xa6 0x01>;
>    @@ -31,6 +32,7 @@
>     		serial2 = "/bus@f0000/serial@2800000";
>     		serial3 = "/bus@f0000/serial@2810000";
>     		spi0 = "/bus@f0000/bus@fc00000/spi@fc40000";
>    +		tick-timer = "/bus@f0000/timer@2400000";
>     		usb0 = "/bus@f0000/dwc3-usb@f900000/usb@31000000";
>     	};
>
>   k3-am625-verdin-r5:
>    --- build/before/spl/u-boot-spl.dtb
>    +++ build/after/spl/u-boot-spl.dtb
>    @@ -10,6 +10,7 @@
>     		assigned-clock-parents = <0x02 0x3d 0x02 0x00 0x02 0x9d 0x16>;
>     		assigned-clock-rates = <0xbebc200 0x2faf0800 0x17d7840>;
>     		assigned-clocks = <0x02 0x3d 0x00 0x02 0x87 0x00 0x02 0x9d 0x14>;
>    +		clock-names = "gtc", "core";
>     		clocks = <0x02 0x3d 0x00>;
>     		compatible = "ti,am654-rproc";
>     		power-domains = <0x03 0x3d 0x01 0x03 0x87 0x01 0x03 0xa6 0x01>;
>    @@ -29,6 +30,7 @@
>     		serial0 = "/bus@f0000/serial@2810000";
>     		serial1 = "/bus@f0000/bus@b00000/target-module@2b300050/serial@0";
>     		serial2 = "/bus@f0000/serial@2800000";
>    +		tick-timer = "/bus@f0000/timer@2400000";
>     		usb0 = "/bus@f0000/dwc3-usb@f900000/usb@31000000";
>     	};
>
>   k3-am62a7-r5-phycore-som-2gb:
>    --- build/before/spl/u-boot-spl.dtb
>    +++ build/after/spl/u-boot-spl.dtb
>    @@ -10,6 +10,7 @@
>     		assigned-clock-parents = <0x02 0x3d 0x02>;
>     		assigned-clock-rates = <0xbebc200 0x47868c00>;
>     		assigned-clocks = <0x02 0x3d 0x00 0x02 0x87 0x00>;
>    +		clock-names = "gtc";
>     		clocks = <0x02 0x3d 0x00>;
>     		compatible = "ti,am654-rproc";
>
> power-domains = <0x03 0x3d 0x01 0x03 0x87 0x01 0x03 0k3-am642-r5-phycore-som-2gbxa6 0x01>;:
>    --- build/before/spl/u-boot-spl.dtb
>    +++ build/after/spl/u-boot-spl.dtb
>    @@ -10,6 +10,7 @@
>     		assigned-clock-parents = <0x02 0x3d 0x02>;
>     		assigned-clock-rates = <0xbebc200 0x3b9aca00>;
>     		assigned-clocks = <0x02 0x3d 0x00 0x02 0x87 0x00>;
>    +		clock-names = "gtc";
>     		clocks = <0x02 0x3d 0x00>;
>     		compatible = "ti,am654-rproc";
>     		power-domains = <0x03 0x3d 0x01 0x03 0x87 0x01 0x03 0x89 0x01>;
> ---
>  arch/arm/dts/k3-am62-r5-lp-sk.dts              | 82 +-----------------------
>  arch/arm/dts/k3-am625-r5-beagleplay.dts        | 72 +++------------------
>  arch/arm/dts/k3-am625-r5-phycore-som-2gb.dts   | 76 ++--------------------
>  arch/arm/dts/k3-am625-r5-sk.dts                | 86 +------------------------
>  arch/arm/dts/k3-am625-r5.dtsi                  | 88 ++++++++++++++++++++++++++
>  arch/arm/dts/k3-am625-verdin-r5.dts            | 81 ++++++------------------
>  arch/arm/dts/k3-am6254atl-r5-sk.dts            | 86 +------------------------
>  arch/arm/dts/k3-am62a7-r5-phycore-som-2gb.dts  | 58 ++---------------
>  arch/arm/dts/k3-am62a7-r5-sk.dts               | 60 +-----------------
>  arch/arm/dts/k3-am62a7-r5.dtsi                 | 69 ++++++++++++++++++++
>  arch/arm/dts/k3-am62d2-r5-evm.dts              | 72 +--------------------
>  arch/arm/dts/k3-am62d2-r5.dtsi                 | 75 ++++++++++++++++++++++
>  arch/arm/dts/k3-am62p5-r5-sk.dts               | 69 +-------------------
>  arch/arm/dts/k3-am62p5-r5.dtsi                 | 77 ++++++++++++++++++++++
>  arch/arm/dts/k3-am62p5-verdin-r5.dts           | 76 ++--------------------
>  arch/arm/dts/k3-am642-r5-evm.dts               | 81 +-----------------------
>  arch/arm/dts/k3-am642-r5-phycore-som-2gb.dts   | 76 ++--------------------
>  arch/arm/dts/k3-am642-r5-sk.dts                | 81 +-----------------------
>  arch/arm/dts/k3-am642-r5.dtsi                  | 84 ++++++++++++++++++++++++
>  arch/arm/dts/k3-am654-r5-base-board.dts        | 74 +---------------------
>  arch/arm/dts/k3-am654-r5.dtsi                  | 77 ++++++++++++++++++++++
>  arch/arm/dts/k3-am67a-r5-beagley-ai.dts        | 67 +-------------------
>  arch/arm/dts/k3-am67a-r5.dtsi                  | 70 ++++++++++++++++++++
>  arch/arm/dts/k3-j7200-r5-common-proc-board.dts | 82 +-----------------------
>  arch/arm/dts/k3-j7200-r5.dtsi                  | 86 +++++++++++++++++++++++++
>  arch/arm/dts/k3-j722s-r5-evm.dts               | 81 +-----------------------
>  arch/arm/dts/k3-j722s-r5.dtsi                  | 84 ++++++++++++++++++++++++
>  27 files changed, 768 insertions(+), 1302 deletions(-)
>

[snip]

> ---
> base-commit: fd871fc6bb497c6ddd7b39622480b89add78f57b
> change-id: 20260122-r5_dt_refactor-4535f5177593
>
> Best regards,


  parent reply	other threads:[~2026-02-02  8:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-22 14:08 [PATCH master] arch: arm: dts: k3: refactor common nodes to k3-*-r5.dtsi Anshul Dalal
2026-01-22 14:44 ` Francesco Dolcini
2026-01-22 14:48   ` Francesco Dolcini
2026-02-02  8:25 ` Anshul Dalal [this message]
2026-02-02 14:38   ` Tom Rini
2026-02-03  6:51 ` Wadim Egorov
2026-02-03 13:39 ` Bryan Brattlof
2026-02-06  8:23   ` Anshul Dalal
2026-02-06 13:47     ` Bryan Brattlof
2026-02-06 15:18 ` Bryan Brattlof
2026-02-06 18:15 ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DG4BZ97UWPF0.FCYZ7RUNHS94@ti.com \
    --to=anshuld@ti.com \
    --cc=afd@ti.com \
    --cc=bb@ti.com \
    --cc=francesco.dolcini@toradex.com \
    --cc=g-tammana@ti.com \
    --cc=ggiordano@phytec.com \
    --cc=m-chawdhry@ti.com \
    --cc=nm@ti.com \
    --cc=p-bhagat@ti.com \
    --cc=parth.pancholi@toradex.com \
    --cc=robertcnelson@gmail.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=upstream@lists.phytec.de \
    --cc=vigneshr@ti.com \
    --cc=vitor.soares@toradex.com \
    --cc=w.egorov@phytec.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.