On Thu, 10 Mar 2022 12:16:16 -0500 Tom Rini <tr...@konsulko.com> wrote:
> 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.ander...@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.wall...@linaro.org> wrote: > > >> >> > > > >> >> > Hi, > > >> >> > > > >> >> > Linus, thanks for the heads up, much appreciated! > > >> >> > > > >> >> >> On Thu, Mar 3, 2022 at 9:44 PM Sean Anderson > > >> >> >> <sean.ander...@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.przyw...@arm.com/ Yes, please, it's really just this one patch that fixes the regression. This is also the reason I refrained from sending a separate series, plus the other patches (trivially) depend on it. Thanks! Andre > or if it's more than just that, how about a regression-fixes-only series > for me to take for this release? Thanks! >
