On Thu, Aug 29, 2024 at 03:50:56PM +0000, Gaskell, Oliver wrote:
>
>
> > -----Original Message-----
> > From: Tom Rini <tr...@konsulko.com>
> > Sent: Thursday, August 29, 2024 4:18 PM
> > To: Gaskell, Oliver <oliver.gask...@analog.com>
> > Cc: Nathan Barrett-Morrison <nathan.morri...@timesys.com>; Greg Malysa
> > <greg.mal...@timesys.com>; Ian Roberts <ian.robe...@timesys.com>;
> > Bimpikas, Vasileios <vasileios.bimpi...@analog.com>; Agarwal, Utsav
> > <utsav.agar...@analog.com>; Artamonovs, Arturs
> > <arturs.artamon...@analog.com>; Rayagonda Kokatanur
> > <rayagonda.kokata...@broadcom.com>; u-boot@lists.denx.de; Linux Factory
> > <adsp-li...@analog.com>; Trevor Woerner <twoer...@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 <tr...@konsulko.com>
> > > > Sent: Thursday, August 29, 2024 3:52 PM
> > > > To: Gaskell, Oliver <oliver.gask...@analog.com>
> > > > Cc: Nathan Barrett-Morrison <nathan.morri...@timesys.com>; Greg Malysa
> > > > <greg.mal...@timesys.com>; Ian Roberts <ian.robe...@timesys.com>;
> > > > Bimpikas, Vasileios <vasileios.bimpi...@analog.com>; Agarwal, Utsav
> > > > <utsav.agar...@analog.com>; Artamonovs, Arturs
> > > > <arturs.artamon...@analog.com>; Rayagonda Kokatanur
> > > > <rayagonda.kokata...@broadcom.com>; u-boot@lists.denx.de; Linux
> > Factory
> > > > <adsp-li...@analog.com>; Trevor Woerner <twoer...@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
signature.asc
Description: PGP signature
