Re: Device-tree and Marvell vendor prefix

On Thu, Jun 19, 2014 at 09:22:17PM +0200, Robert Jarzmik wrote:

Hi Haojian, Grant and Rob,

I'm submitting various patches for Marvell PXA27x architecture to support
device-tree.

In a review from Guennadi, he made me a comment that the official prefix for
Marvell is "marvell" (according to
Documentation/devicetree/bindings/vendor-prefixes.txt).

As a matter of fact, there are 2 prefixes used :
 - "marvell" : 264 uses in *.c files
 - "mrvl" : 64 uses in *.c files

Yes, that's unfortunate.  When we first started converting the Marvell
legacy platforms (kirwkood, etc) to DT, there was discussion on which
one to use.  We chose to stay with 'marvell', despite 'mrvl' being the
official stock ticker symbol.  This was to maintain reverse
compatibility with the binding for the marvell ethernet driver,
mv643xx_eth, which had been is use on powerpc for years before us.

So, my preference is to use 'marvell'.  I've attempted to correct any
new bindings I see that use 'mrvl', but apparently I didn't catch them
all. :(

So which one should I use ?

I say 'marvell', but apparently there are others who think differently.

A check of the bindings docs and arch/*/boot/dts/ would be more telling
than the C files.

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html