[PATCH 2/4] arm: dts: Add support for AM62x LP SK

[Nishanth Menon]

On 15:00-20230928, Nitin Yadav wrote:
> Hi,
> 
> On 27/09/23 17:22, Nishanth Menon wrote:
> > On 13:51-20230927, Nitin Yadav wrote:
> >> The AM62x LP SK board is similar to the AM62x SK board,
> >> but has some significant changes that requires different
> >> device tree.
> >>
> >> The differences are mainly:
> >> - AM62x SoC in the AMC package that meets AECQ100 automotive standard.
> >> - LPDDR4 versus DDR4 on the AM62x SK.
> >> - TPS65219 PMIC instead of discrete regulators.
> >> - IO expander pin names are wired differently.
> >> - Second ethernet port is currently disabled as the boards do not have
> >>   the part physically installed.
> >> - OSPI NAND vs OSPI NOR.
> >> - No WLAN chip instead a SDIO M.2 connector.
> >>
> >> Signed-off-by: Nitin Yadav <n-yadav at ti.com>
> >> ---
> >>  arch/arm/dts/Makefile                         |    2 +
> >>  arch/arm/dts/k3-am62-lp-sk-u-boot.dtsi        |    9 +
> >>  arch/arm/dts/k3-am62-lp-sk.dts                |  231 ++
> > 
> >>  arch/arm/dts/k3-am62-r5-lp-sk.dts             |   21 +
> >>  arch/arm/dts/k3-am62x-ddr-lp4-50-800-800.dtsi | 2190 +++++++++++++++++
> >>  5 files changed, 2453 insertions(+)
> >>  create mode 100644 arch/arm/dts/k3-am62-lp-sk-u-boot.dtsi
> >>  create mode 100644 arch/arm/dts/k3-am62-lp-sk.dts
> >>  create mode 100644 arch/arm/dts/k3-am62-r5-lp-sk.dts
> >>  create mode 100644 arch/arm/dts/k3-am62x-ddr-lp4-50-800-800.dtsi
> >>
> >> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> >> index bde2176ec7..72ea57885f 100644
> >> --- a/arch/arm/dts/Makefile
> >> +++ b/arch/arm/dts/Makefile
> >> @@ -1337,6 +1337,8 @@ dtb-$(CONFIG_SOC_K3_AM642) += k3-am642-evm.dtb \
> >>  
> >>  dtb-$(CONFIG_SOC_K3_AM625) += k3-am625-sk.dtb \
> >>  			      k3-am625-r5-sk.dtb \
> >> +			      k3-am62-lp-sk.dtb \
> >> +			      k3-am62-r5-lp-sk.dtb \
> >>  			      k3-am625-beagleplay.dtb \
> >>  			      k3-am625-r5-beagleplay.dtb \
> >>  			      k3-am625-verdin-wifi-dev.dtb \
> >> diff --git a/arch/arm/dts/k3-am62-lp-sk-u-boot.dtsi b/arch/arm/dts/k3-am62-lp-sk-u-boot.dtsi
> >> new file mode 100644
> >> index 0000000000..7da94fe4b6
> >> --- /dev/null
> >> +++ b/arch/arm/dts/k3-am62-lp-sk-u-boot.dtsi
> >> @@ -0,0 +1,9 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> +/*
> >> + * AM62x LP SK dts file for SPLs
> >> + * Copyright (C) 2021-2023 Texas Instruments Incorporated - https://www.ti.com/
> >> + */
> >> +
> >> +#include "k3-am62x-sk-common-u-boot.dtsi"
> >> +
> >> +#include "k3-am62-lp-sk-binman.dtsi"
> > 
> > Are you sure you don't need the dt phase properties for regulators for
> > mmc to work?
> > 
> >> diff --git a/arch/arm/dts/k3-am62-lp-sk.dts b/arch/arm/dts/k3-am62-lp-sk.dts
> > 
> > Is this coming in from v6.6-rc1 if so document that in commit message.
> > 
> > [...]
> > 
> >> diff --git a/arch/arm/dts/k3-am62-r5-lp-sk.dts b/arch/arm/dts/k3-am62-r5-lp-sk.dts
> >> new file mode 100644
> >> index 0000000000..ed2c028bad
> >> --- /dev/null
> >> +++ b/arch/arm/dts/k3-am62-r5-lp-sk.dts
> >> @@ -0,0 +1,21 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> +/*
> >> + * AM62x LP SK dts file for R5 SPL
> >> + * Copyright (C) 2021-2023 Texas Instruments Incorporated - https://www.ti.com/
> >> + */
> >> +
> >> +#include "k3-am62-lp-sk.dts"
> >> +#include "k3-am62x-ddr-lp4-50-800-800.dtsi"
> >> +#include "k3-am62-ddr.dtsi"
> >> +
> >> +#include "k3-am62-lp-sk-u-boot.dtsi"
> >> +#include "k3-am62x-r5-sk-common.dtsi"
> >> +
> >> +/ {
> >> +	memory at 80000000 {
> >> +		device_type = "memory";
> >> +		/* 2G RAM */
> >> +		reg = <0x00000000 0x80000000 0x00000000 0x80000000>;
> >> +		bootph-pre-ram;
> >> +	};
> >> +};
> > 
> > NAK. should come from board.dts.
> 
> am62sip support is also in the queue where we have only 512M of RAM.
> That's why I'm adding this node in r5-sk.dts

NAK.

Do this cleanup in the context of the new platform addition - when you
get it in the kernel first.

Side note: I don't see anything in ti.com for the search text.


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