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

Ben Dooks ben.dooks at codethink.co.uk
Tue May 22 08:00:23 EDT 2012


On 18/05/12 03:12, Rob Herring wrote:
> On 05/17/2012 06:35 PM, Jason Cooper wrote:
>> On Tue, May 15, 2012 at 08:31:06AM -0500, Rob Herring wrote:
>>> 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



More information about the linux-arm-kernel mailing list