[PATCH 00/17] arm: semihosting: Cleanups and new features
trini at konsulko.com
Fri Mar 4 19:46:37 CET 2022
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 at 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 at 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.
> > From skimming over this series here, it looks quite nice, I especially love
> > promoting the semihosting load to a proper filesystem. The only drawback
> > seems to be that this will break existing scripts used by people in their
> > deployments, which rely on the smhload command. I don't really know if
> > there are actually any users doing so, but I guess we will find out.
> Yeah, I'm not sure what the stance on shell compatibility is. As I understand
> it, for the C API only in-tree users matter. However, shell commands are more
> of an "external" API, used by distro boot scripts. Simon/Tom, what's the
> process for this?
Well, in this case we can be a bit more cavalier than typical because of
the very limited user base. So long as we have good documentation
around it including something around how to transition to the new
commands, it should be fine to do so, when the code is otherwise ready.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 659 bytes
Desc: not available
More information about the U-Boot