[PATCH 6/8] arm: mach-armada: add support for Armada XP board with device tree

On 18/05/12 03:12, Rob Herring wrote:

On 05/17/2012 06:35 PM, Jason Cooper wrote:

quoted

On Tue, May 15, 2012 at 08:31:06AM -0500, Rob Herring wrote:

quoted

marvell is the documented vendor string, not mrvl.

When Arnd and I were adding the initial kirkwood support for devicetree,
the common wisdom was to use the stock ticker symbol when available.
Hence, mrvl, iom, and dlink (only on taiwanese exchange, which uses
numbers)...

Yes, that's true. But consistency is more important. If we have cases of
both, then one needs to be deprecated.

I'd go for "should" instead of "needs". I would say it would be nice
to settle on marvell.

There are already instances of "mrvl" in the kernel, see

	drivers/i2c/busses/i2c-pxa.c
	drivers/tty/serial/pxa.c
	drivers/rtc/rtc-mv.c
	drivers/rtc/rtc-sa1100.c

Do people mind if the original compatibilities are left in to avoid
having to rename .dts file contents, or are we happy to go with just
"marvell"?

Also, it seems that some of the marvell bluetooth firmware is defined
to be mrvl prefixed.

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius