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

Tom Rini trini at konsulko.com
Wed May 29 18:13:16 CEST 2024


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)? 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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240529/312245f2/attachment.sig>


More information about the U-Boot mailing list