[PATCH 0/4] arm64: versal2: Add support for new AMD Versal Gen 2 SoC

[Michal Simek]

Hi Tom,

On 5/29/24 18:13, Tom Rini wrote:
> On Wed, May 29, 2024 at 04:47:57PM +0200, Michal Simek wrote:
> 
>> Hi,
>>
>> I am sending patches for adding initial support for new AMD Versal Gen 2
>> SoC. If you want to find out more information please take a look at this
>> page:
>> https://www.amd.com/en/products/adaptive-socs-and-fpgas/versal.html
>>
>> Thanks,
>> Michal
>>
>>
>> Michal Simek (4):
>>    arm64: versal2: Add support for AMD Versal Gen 2
>>    soc: versal2: Add SoC driver for AMD Versal Gen 2
>>    mmc: versal2: Update zynq_sdhci driver to support AMD Versal Gen 2
>>    spi: versal2: Enable spi drivers for Versal Gen 2
> 
> My only concerns are name-related and not-blocking. First, should we
> perhaps instead have arch/arm/mach-versal/{gen2,net}/ (and presumably
> gen2-net down the line)?

There are completely 3 different SOCs only names are similar
Versal is 2 cores a72
Versal NET 16 cores a78
Versal Gen 2 8 cores a78
with different features.

Creating subfolders under current mach-versal would be very confusing.
We can definitely try to make it as small as possible but still they are 
separated families.

We tried to reuse existing platforms but it is hard to keep it in sync that's 
why new platform is better way to go.
But as you know we are trying to reuse for example board/xilinx/common/ folder 
as much as possible that we don't have code duplication.


> Second, and I see I can't say we should follow
> the Linux kernel since there's no versal dts files upstream, should we
> still be "xilinx" and not "amd" ? I do see the kernel shoves all the
> imx8 (as an example of a modern part) stuff under
> arch/arm64/boot/dts/freescale/ and if nothing else we should be
> consistent between projects whenever possible. Thanks.

I am not going to send DTSes to U-Boot and just push them to Linux kernel at 
right time. And target is arch/arm64/boot/dts/amd which already exists for old 
amd socs and elba (pensando). Versal Gen 2 should go there.

It means Versal and Versal NET will be branded under Xilinx and Versal Gen 2 
will be branded under AMD. Also we are going to stop to use xlnx, dt prefixes 
for new IPs and will start to use amd, prefix.

Thanks,
Michal