[PATCH 00/20] arm: Initial support for Analog Devices SC5xx boards

[Tom Rini]

On Thu, Aug 29, 2024 at 03:50:56PM +0000, Gaskell, Oliver wrote:
> 
> 
> > -----Original Message-----
> > From: Tom Rini <trini at konsulko.com>
> > Sent: Thursday, August 29, 2024 4:18 PM
> > To: Gaskell, Oliver <Oliver.Gaskell at analog.com>
> > Cc: Nathan Barrett-Morrison <nathan.morrison at timesys.com>; Greg Malysa
> > <greg.malysa at timesys.com>; Ian Roberts <ian.roberts at timesys.com>;
> > Bimpikas, Vasileios <Vasileios.Bimpikas at analog.com>; Agarwal, Utsav
> > <Utsav.Agarwal at analog.com>; Artamonovs, Arturs
> > <Arturs.Artamonovs at analog.com>; Rayagonda Kokatanur
> > <rayagonda.kokatanur at broadcom.com>; u-boot at lists.denx.de; Linux Factory
> > <adsp-linux at analog.com>; Trevor Woerner <twoerner at 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 at konsulko.com>
> > > > Sent: Thursday, August 29, 2024 3:52 PM
> > > > To: Gaskell, Oliver <Oliver.Gaskell at analog.com>
> > > > Cc: Nathan Barrett-Morrison <nathan.morrison at timesys.com>; Greg Malysa
> > > > <greg.malysa at timesys.com>; Ian Roberts <ian.roberts at timesys.com>;
> > > > Bimpikas, Vasileios <Vasileios.Bimpikas at analog.com>; Agarwal, Utsav
> > > > <Utsav.Agarwal at analog.com>; Artamonovs, Arturs
> > > > <Arturs.Artamonovs at analog.com>; Rayagonda Kokatanur
> > > > <rayagonda.kokatanur at broadcom.com>; u-boot at lists.denx.de; Linux
> > Factory
> > > > <adsp-linux at analog.com>; Trevor Woerner <twoerner at 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.

OK, 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/20240829/ffbde0ab/attachment.sig>