* [PATCH 2/2] arm64: dts: ti: k3-am62l: Add alias node
2026-09-28 3:41 [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes Sai Sree Kartheek Adivi
@ 2026-09-28 3:41 ` Sai Sree Kartheek Adivi
2026-09-30 14:40 ` Bryan Brattlof
2026-09-28 3:55 ` [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes sashiko-bot
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Sai Sree Kartheek Adivi @ 2026-09-28 3:41 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, linux-arm-kernel,
devicetree, linux-kernel, s-adivi
Cc: hnagalla, jm
From: Judith Mendez <jm@ti.com>
Add alias node for am62l3. This is mainly to link mmc0
alias to eMMC and mmc1 alias to SD in u-boot.
Signed-off-by: Judith Mendez <jm@ti.com>
Signed-off-by: Sai Sree Kartheek Adivi <s-adivi@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
index 964387eb51788..f357dc56de1c7 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
@@ -21,6 +21,11 @@ / {
compatible = "ti,am62l3-evm", "ti,am62l3";
model = "Texas Instruments AM62L3 Evaluation Module";
+ aliases {
+ mmc0 = &sdhci0;
+ mmc1 = &sdhci1;
+ };
+
chosen {
stdout-path = &uart0;
};
--
2.54.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] arm64: dts: ti: k3-am62l: Add alias node
2026-09-28 3:41 ` [PATCH 2/2] arm64: dts: ti: k3-am62l: Add alias node Sai Sree Kartheek Adivi
@ 2026-09-30 14:40 ` Bryan Brattlof
0 siblings, 0 replies; 6+ messages in thread
From: Bryan Brattlof @ 2026-09-30 14:40 UTC (permalink / raw)
To: Sai Sree Kartheek Adivi
Cc: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, linux-arm-kernel,
devicetree, linux-kernel, hnagalla, jm
On September 28, 2026 thus sayeth Sai Sree Kartheek Adivi:
> From: Judith Mendez <jm@ti.com>
>
> Add alias node for am62l3. This is mainly to link mmc0
> alias to eMMC and mmc1 alias to SD in u-boot.
>
> Signed-off-by: Judith Mendez <jm@ti.com>
> Signed-off-by: Sai Sree Kartheek Adivi <s-adivi@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> index 964387eb51788..f357dc56de1c7 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
> @@ -21,6 +21,11 @@ / {
> compatible = "ti,am62l3-evm", "ti,am62l3";
> model = "Texas Instruments AM62L3 Evaluation Module";
>
> + aliases {
> + mmc0 = &sdhci0;
> + mmc1 = &sdhci1;
Why would we need to alias MMC nodes like this?
~Bryan
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes
2026-09-28 3:41 [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes Sai Sree Kartheek Adivi
2026-09-28 3:41 ` [PATCH 2/2] arm64: dts: ti: k3-am62l: Add alias node Sai Sree Kartheek Adivi
@ 2026-09-28 3:55 ` sashiko-bot
2026-09-28 16:49 ` Vignesh Raghavendra
2026-09-28 17:11 ` Judith Mendez
3 siblings, 0 replies; 6+ messages in thread
From: sashiko-bot @ 2026-09-28 3:55 UTC (permalink / raw)
To: Sai Sree Kartheek Adivi; +Cc: robh, devicetree, conor+dt
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dts: k3-am62l: tscadc0 reg exceeds parent cbass_main ranges
--
commit 06bc07c12d27dc31a3a3ae1cf4ef12f047b8b0e7
Author: Judith Mendez <jm@ti.com>
arm64: dts: ti: k3-am62l-main: Add ADC nodes
AM62L has an ADC in main domain, add ADC node in the respective
voltage domain file.
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> index 80615ca1e01a2..70157981aba5d 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
[ ... ]
> + tscadc0: tscadc@28001000 {
> + compatible = "ti,am654-tscadc", "ti,am3359-tscadc";
> + reg = <0x00 0x28001000 0x00 0x1000>;
[Severity: Low]
Does this reg size exceed the parent cbass_main ranges?
In arch/arm64/boot/dts/ti/k3-am62l.dtsi, the parent cbass_main bus defines
the Second Peripheral Window ending at 0x28001400 (0x0e000000 + 0x1a001400):
<0x00 0x0e000000 0x00 0x0e000000 0x00 0x1a001400>, /* Second Peripheral Window */
The tscadc0 node's reg size of 0x1000 bytes starting at 0x28001000 extends
up to 0x28002000, which exceeds the parent window by 0xC00 bytes. This causes
a device tree schema and dtc validation warning when building the dtbs.
While devm_ioremap_resource() still maps the full 0x1000 bytes, and the driver
only accesses registers up to 0x200 (well within the valid 0x400-byte window),
should the reg size or the parent range be adjusted to avoid the validation
warnings?
> + interrupts = <GIC_SPI 253 IRQ_TYPE_LEVEL_HIGH>;
> + power-domains = <&scmi_pds 0>;
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260928034115.1084840-1-s-adivi@ti.com?part=1
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes
2026-09-28 3:41 [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes Sai Sree Kartheek Adivi
2026-09-28 3:41 ` [PATCH 2/2] arm64: dts: ti: k3-am62l: Add alias node Sai Sree Kartheek Adivi
2026-09-28 3:55 ` [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes sashiko-bot
@ 2026-09-28 16:49 ` Vignesh Raghavendra
2026-09-28 17:11 ` Judith Mendez
3 siblings, 0 replies; 6+ messages in thread
From: Vignesh Raghavendra @ 2026-09-28 16:49 UTC (permalink / raw)
To: Sai Sree Kartheek Adivi, nm, kristo, robh, krzk+dt, conor+dt,
linux-arm-kernel, devicetree, linux-kernel
Cc: hnagalla, jm
On 28/09/26 09:11, Sai Sree Kartheek Adivi wrote:
> From: Judith Mendez <jm@ti.com>
>
> AM62L has an ADC in main domain, add ADC node in the respective
> voltage domain file.
>
> Signed-off-by: Judith Mendez <jm@ti.com>
> Signed-off-by: Sai Sree Kartheek Adivi <s-adivi@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62l-main.dtsi | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> index 80615ca1e01a2..70157981aba5d 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62l-main.dtsi
> @@ -544,6 +544,24 @@ elm0: ecc@25010000 {
> status = "disabled";
> };
>
> + tscadc0: tscadc@28001000 {
> + compatible = "ti,am654-tscadc", "ti,am3359-tscadc";
> + reg = <0x00 0x28001000 0x00 0x1000>;
> + interrupts = <GIC_SPI 253 IRQ_TYPE_LEVEL_HIGH>;
> + power-domains = <&scmi_pds 0>;
> + clocks = <&scmi_clk 4>;
> + assigned-clocks = <&scmi_clk 4>;
> + assigned-clock-parents = <&scmi_clk 2>;
> + assigned-clock-rates = <60000000>;
> + clock-names = "fck";
> + status = "disabled";
> +
> + adc {
> + #io-channel-cells = <1>;
> + compatible = "ti,am654-adc", "ti,am3359-adc";
compatible goes first
> + };
> + };
> +
> gpmc0: memory-controller@3b000000 {
> compatible = "ti,am64-gpmc";
> power-domains = <&scmi_pds 37>;
--
Regards
Vignesh
https://ti.com/opensource
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes
2026-09-28 3:41 [PATCH 1/2] arm64: dts: ti: k3-am62l-main: Add ADC nodes Sai Sree Kartheek Adivi
` (2 preceding siblings ...)
2026-09-28 16:49 ` Vignesh Raghavendra
@ 2026-09-28 17:11 ` Judith Mendez
3 siblings, 0 replies; 6+ messages in thread
From: Judith Mendez @ 2026-09-28 17:11 UTC (permalink / raw)
To: Sai Sree Kartheek Adivi, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt, linux-arm-kernel, devicetree, linux-kernel
Cc: hnagalla
Hi Sai,
On 9/27/26 10:41 PM, Sai Sree Kartheek Adivi wrote:
> From: Judith Mendez <jm@ti.com>
>
> AM62L has an ADC in main domain, add ADC node in the respective
> voltage domain file.
>
This was previously blocked because of TISCI parent clocking
issue in TF-A, is that now resolved?
~ Judith
^ permalink raw reply [flat|nested] 6+ messages in thread