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

[Mark Rutland <mark.rutland@arm.com>]

On Mon, Aug 05, 2013 at 10:02:53PM +0100, Dinh Nguyen wrote:
> 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.

This would leave ALTR as the oddball out of all the prefixes currently
in existence (of which several are *lower case* stock ticker symbols).

Grant, do you have any view here? I'm happy to allow uppercase if you're
happy with it.

> 
> >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.

Why is using the string you're using already (atlr) a problem?

Thanks,
Mark.

> 
> 
> 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
> > 
> > 
> 
> 
> 
>