Re: [PATCH 7/8] tty/serial: add arm64 semihosting earlycon

From: Nicolas Pitre
Date: Sun Mar 23 2014 - 17:49:03 EST


On Sun, 23 Mar 2014, Arnd Bergmann wrote:

> On Sunday 23 March 2014 16:04:41 Nicolas Pitre wrote:
> > On Fri, 21 Mar 2014, Rob Herring wrote:
> >
> > > From: Rob Herring <robh@xxxxxxxxxx>
> > >
> > > Add earlycon support for the arm64 semihosting debug serial interface.
> > > This allows enabling a debug console when early_params are processed.
> > > This is based on the arm64 earlyprintk smh support and is intended to
> > > replace it.
> > >
> > > This is named arm rather than arm64 in hopes it will be used for both,
> > > but only arm64 is supported ATM.
> >
> > It can't be used for both as the semihosting calls themselves need
> > assembly code and this is not compatible with the equivalent ARM32
> > calls. So I'd suggest naming this "arm64" to avoid potential confusion.
>
> How about using an compile-time conditional to put both inline assemblies
> in there? While the driver is rather trivial to start with, I don't see
> a point in duplicating it either.

Sure. But it can be trivially renamed at that point.

Or, even better would be to add the ARM calls right away. And of course
ARM in Thumb mode has yet another call convention.

The printascii and printch code in arch/arm/kernel/debug.S for
CONFIG_DEBUG_SEMIHOSTING can be copied almost as is.


Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/