[PATCH] Handle reg-shift property for of_serial ports

Arnd Bergmann arnd at arndb.de
Sat Jul 7 22:10:14 EST 2007


On Saturday 07 July 2007, David Woodhouse wrote:
> 
> The MV64660 has reg-shift==2 for its otherwise 16550-compatible uarts.
> While the bootwrapper copes with this, of_serial.c doesn't. (The udbg
> code doesn't either, but I'll fix that later).
> 
> Signed-off-by: David Woodhouse <dwmw2 at infradead.org>

Acked-by: Arnd Bergmann <arnd at arndb.de>

Given the existence of the boards, it looks correct to do this.
However, I wonder if it was correct for the MV64660 to claim
compatibily witn ns16550 if the programming model is not exactly
the same. The official OF serial port bindings don't mention the
reg-shift property, so it maybe would have been better to have
a different value for the "compatible" property, in order not
to confuse existing operating systems that implement the standard.

Also, what about etherboot and yaboot? Do they handle this
correctly?

	Arnd <><



More information about the Linuxppc-dev mailing list