Re: [PATCH 00/17] arm: semihosting: Cleanups and new features

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 2511 bytes --]

On Thu, Mar 10, 2022 at 12:06:48PM -0500, Sean Anderson wrote:
> 
> 
> On 3/10/22 12:01 PM, Andre Przywara wrote:
> > On Thu, 10 Mar 2022 11:48:50 -0500
> > Sean Anderson <sean.anderson@seco.com> wrote:
> > 
> > Hi,
> > 
> >> On 3/4/22 1:46 PM, Tom Rini wrote:
> >> > On Fri, Mar 04, 2022 at 12:19:27PM -0500, Sean Anderson wrote:  
> >> >> Hi Andre,
> >> >> 
> >> >> On 3/4/22 6:47 AM, Andre Przywara wrote:  
> >> >> > On Fri, 4 Mar 2022 02:06:03 +0100
> >> >> > Linus Walleij <linus.walleij@linaro.org> wrote:
> >> >> > 
> >> >> > Hi,
> >> >> > 
> >> >> > Linus, thanks for the heads up, much appreciated!
> >> >> >   
> >> >> >> On Thu, Mar 3, 2022 at 9:44 PM Sean Anderson <sean.anderson@seco.com> wrote:
> >> >> >>   
> >> >> >> > The main device affected by these changes is vexpress64, so I'd appreciate
> >> >> >> > if Linus (or anyone else) could try booting.    
> >> >> > 
> >> >> > As it happens, I am sitting on a series updating vexpress_aemv8a_semi, and
> >> >> > letting it use OF_CONTROL. I will send this ASAP.
> >> >> > As it turned out, the automatic semihosting boot command is actually broken
> >> >> > in v2022.01, the fix is the first patch in my series. This clashes with
> >> >> > this series, but it's easy to resolve.  
> >> >> 
> >> >> I, for one, appreciate that the boot command is no longer all one line after
> >> >> your series.
> >> >> 
> >> >> Simon/Tom, which series would you like to be based off the other?  
> >> > 
> >> > I think it's however it's easiest for you two to collaborate.  
> >> 
> >> I think it will be cleaner for me to base my series on his, but I just want to
> >> avoid having to rebase a lot :)
> > 
> > Yeah, it should just be this one line, though, right?
> > Tom, would you consider taking patch 01/11 now, as this fixes a
> > regression? The Sean could at least rebase on top of this. Rebasing on top
> > of the rest should be more straight-forward, because it's not one
> > super-long line anymore.
> 
> The patch which makes the boot command multi-line is [PATCH 07/11] vexpress64: fvp:
> add distro_boot support. I'm not sure how much that patch depends on the others,
> but presumably you would need to apply 5 and 6 as well.

Um, so for clarity, should I take:
https://patchwork.ozlabs.org/project/uboot/patch/20220304163018.3167841-2-andre.przywara@arm.com/
or if it's more than just that, how about a regression-fixes-only series
for me to take for this release?  Thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]