Re: [PATCH v3 3/3] arm64: dts: ti: Add support for AM62D2-EVM

[Nishanth Menon <nm@ti.com>]

Paresh,


On 19:22-20250508, Bryan Brattlof wrote:
> On May  8, 2025 thus sayeth Paresh Bhagat:
> > AM62D-EVM evaluation module (EVM) is a low-cost expandable platform board
> > designed for TI’s AM62D2 SoC. It supports the following interfaces:
> > 
> > * 4 GB LPDDR4 RAM
> > * x2 Gigabit Ethernet expansion connectors
> > * x4 3.5mm TRS Audio Jack Line In
> > * x4 3.5mm TRS Audio Jack Line Out
> > * x2 Audio expansion connectors
> > * x1 Type-A USB 2.0, x1 Type-C dual-role device (DRD) USB 2.0
> > * x1 UHS-1 capable µSD card slot
> > * 32 GB eMMC Flash
> > * 512 Mb OSPI NOR flash
> > * x4 UARTs via USB 2.0-B
> > * XDS110 for onboard JTAG debug using USB
> > * Temperature sensors, user push buttons and LEDs
> > 
> > AM62A7 and AM62D2 SoCs share several peripherals in wakeup, mcu, thermal,
> > and portions of the main domain. To improve reuse and reduce duplication,
> > common *-wakeup.dtsi, *-mcu.dtsi, *-thermal.dtsi, and *-main.dtsi files
> > have been introduced. Each board will have a dedicated DTS file that
> > includes both the shared and SoC-specific .dtsi files.
> > 

I suggest doing a cleanup and internal review before public next
level patch. We have enough number of patches in upstream to use as
reference as to how to do this correctly.


Let us look at this in the next window, I do encourage using the time to
review internally to use the best practises and have less discussion on
minor organizational topics.

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