Re: [RESEND PATCHv2] of: add vendor prefix for Altera Corp.

[Dinh Nguyen <dinguyen@altera.com>]

On Mon, 2013-08-05 at 15:50 -0500, Kumar Gala wrote:
> On Aug 5, 2013, at 3:48 PM, <dinguyen@altera.com> <dinguyen@altera.com> wrote:
> 
> > From: Dinh Nguyen <dinguyen@altera.com>
> > 
> > Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
> > Cc: Grant Likely <grant.likely@linaro.org>
> > Cc: Rob Herring <rob.herring@calxeda.com>
> > Cc: Pawel Moll <pawel.moll@arm.com>
> > Cc: Mark Rutland <mark.rutland@arm.com>
> > Cc: Stephen Warren <swarren@wwwdotorg.org>
> > Cc: Ian Campbell <ian.campbell@citrix.com>
> > Cc: devicetree@vger.kernel.org
> > 
> > v2:
> > - Changed altr -> ALTR
> > ---
> > .../devicetree/bindings/vendor-prefixes.txt        |    1 +
> > 1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > index 366ce9b..6c2b273 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > @@ -7,6 +7,7 @@ ad	Avionic Design GmbH
> > adi	Analog Devices, Inc.
> > aeroflexgaisler	Aeroflex Gaisler AB
> > ak	Asahi Kasei Corp.
> > +ALTR	Altera Corp.
> 
> why all caps?  we seem to have avoid doing that for almost everyone?

CC: David Gibson <david@gibson.dropbear.id.au>

I can't find the archived email, but this is the excerpt from v1:

On Sat, Jul 27, 2013 at 10:41:25PM -0600, Grant Likely wrote:
> On Fri, 26 Jul 2013 14:05:26 -0600, Stephen Warren
<swarren@wwwdotorg.org> wrote:
> > On 07/26/2013 01:39 PM, dinguyen@altera.com wrote:
> > 
> > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt
b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > 
> > > +altr     Altera Corp.
> > 
> > In the context of Marvel(?) wanting to change between a longer and
> > stock-ticker vendor name, there was some discussion re: whether we
had
> > decided not to use stock tickers any more. Is there still consensus
that
> > we should? If so, I have no problem with this patch, but I wanted to
> > make sure everyone was on the same page.
> 
> Stock ticker names are just fine by me. But once set they should not
> change.

On Tue, 2013-07-30 at 16:10 +1000, David Gibson wrote:
>The old recommendation from OF working group was that vendor
>prefixes could be either ticker symbols in all caps (e.g. "AAPL"), or
>some other string in lower case.

>We've mostly only used the second option.
------

So I'm think we're going with all caps for ALTR. We've already been
using altr, so the 2nd option is colliding with the 1st option.


Dinh

> 
> - k
> 
> > amcc	Applied Micro Circuits Corporation (APM, formally AMCC)
> > apm	Applied Micro Circuits Corporation (APM)
> > arm	ARM Ltd.
> > -- 
> > 1.7.9.5
> > 
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe devicetree" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> --
> Employee of Qualcomm Innovation Center, Inc.
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
> 
>