-----Original Message----- From: Tom Rini trini@konsulko.com Sent: Thursday, August 29, 2024 4:18 PM To: Gaskell, Oliver Oliver.Gaskell@analog.com Cc: Nathan Barrett-Morrison nathan.morrison@timesys.com; Greg Malysa greg.malysa@timesys.com; Ian Roberts ian.roberts@timesys.com; Bimpikas, Vasileios Vasileios.Bimpikas@analog.com; Agarwal, Utsav Utsav.Agarwal@analog.com; Artamonovs, Arturs Arturs.Artamonovs@analog.com; Rayagonda Kokatanur rayagonda.kokatanur@broadcom.com; u-boot@lists.denx.de; Linux Factory adsp-linux@analog.com; Trevor Woerner twoerner@gmail.com Subject: Re: [PATCH 00/20] arm: Initial support for Analog Devices SC5xx boards
[External]
On Thu, Aug 29, 2024 at 03:06:31PM +0000, Gaskell, Oliver wrote:
-----Original Message----- From: Tom Rini trini@konsulko.com Sent: Thursday, August 29, 2024 3:52 PM To: Gaskell, Oliver Oliver.Gaskell@analog.com Cc: Nathan Barrett-Morrison nathan.morrison@timesys.com; Greg Malysa greg.malysa@timesys.com; Ian Roberts ian.roberts@timesys.com; Bimpikas, Vasileios Vasileios.Bimpikas@analog.com; Agarwal, Utsav Utsav.Agarwal@analog.com; Artamonovs, Arturs Arturs.Artamonovs@analog.com; Rayagonda Kokatanur rayagonda.kokatanur@broadcom.com; u-boot@lists.denx.de; Linux
Factory
adsp-linux@analog.com; Trevor Woerner twoerner@gmail.com Subject: Re: [PATCH 00/20] arm: Initial support for Analog Devices
SC5xx
boards
[External]
On Wed, Aug 28, 2024 at 11:02:08AM +0100, Oliver Gaskell via B4 Relay wrote:
ADSP-SC5xx is a series of ARM-based DSPs. This comprises the armv7 based SC57x, SC58x and SC594 series, and
the
armv8 based SC598.
This patch series includes configurations, init code, and minimal
DTs
to enable Analog Devices' evaluation boards for these SoCs to boot through SPL and into U-Boot Proper, as well as devicetree schemas
for
the added DTs.
This patch series depends on ("arm: Add Analog Devices SC5xx Machine Type") (https://lists.denx.de/pipermail/u-boot/2024-
April/552043.html)
Since that series has been merged since early May, what is this series based on top of? It should be based on top of current (or close enough to not matter) -next. Also, has everything been passed through checkpatch.pl and the results looked at? Thanks.
I made it on top of -rc1, just tested and I think it applies cleanly to -rc3 and next. Wasn't sure on the policy re. patch series
dependencies
so just wanted to be 100% sure that was clear.
It has been run through checkpatch, there's a few warnings that I think are OK. Only warning I missed is about the Co-developed-by tags not having matching Signed-off-by tags - if this is an issue I can make a v2 that fixes this.
OK, thanks for elaborating. I forget, what is the plan to upstream these dts files?
We're currently in the process of upstreaming support for a single platform (SC598) to Linux, and others will likely follow, so we'd like to have the minimal device trees in this patch series in U-Boot for now so it's possible to boot the boards from source. As and when the devicetrees are in Linux, we will remove them and enable OF_UPSTREAM for the relevant boards.
Thanks, Oliver