On Sunday 23 March 2014 16:04:41 Nicolas Pitre wrote:

On Fri, 21 Mar 2014, Rob Herring wrote:

quoted

From: Rob Herring <robh@kernel.org>

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.

	Arnd