devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Neil Zhang <zhangwm@marvell.com>
Cc: Jason Cooper <jason@lakedaemon.net>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"haojian.zhuang@gmail.com" <haojian.zhuang@gmail.com>,
	Matt Sealey <neko@bakuhatsu.net>,
	"grant.likely@linaro.org" <grant.likely@linaro.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
Date: Sun, 14 Jul 2013 21:29:43 +0200	[thread overview]
Message-ID: <201307142129.43271.arnd@arndb.de> (raw)
In-Reply-To: <175CCF5F49938B4D99B2E3EF7F558EBE3DBC97FDFA@SC-VEXCH4.marvell.com>

On Thursday 11 July 2013, Neil Zhang wrote:
> > > > To do this properly, the drivers are going to have to be compatible
> > > > with the old and the new names, and the binding docs updated to
> > > > reflect the legacy name and the preferred name.
> > >
> > > Properly would be as above. You can stop using stock tickers for new
> > > company names, but anything that has been defined in a device tree
> > > before has to stay that way, and all the manufacturer prefixes to
> > > device names should be the same. What you're proposing is purely
> > > driver bloat and increasing the size of kernel.
> > 
> > *I'm* not proposing to change the name, Neil is.  I'm proposing that
> > iff they chose to do that, don't break sh*t along the way.
> > 
> 
> What's your opinion?

We discussed the topic of fixing bad bindings vs. keeping backwards
compatibility during last week's Linaro Connect.

The main outcome was that we need to have a better review for new bindings
getting merged to avoid this situation in the future, but the general
consensus seems that bindings that are already in use but were never
reviewed properly (which should have caught this) should be changed.
We will likely establish an annotation in the binding soon to mark those
that can not be changed as opposed to those that are not considered stable
yet.

	Arnd

  reply	other threads:[~2013-07-14 19:29 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-09  6:42 [PATCH V3 0/3] bring up pxa988 with DT Neil Zhang
2013-07-09  6:42 ` [PATCH V3 1/3] dts: change Marvell prefix to 'marvell' Neil Zhang
     [not found]   ` <1373352166-10064-2-git-send-email-zhangwm-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
2013-07-09 12:49     ` Jason Cooper
2013-07-10  7:25       ` Neil Zhang
2013-07-10 12:20         ` Jason Cooper
2013-07-10 14:41           ` Daniel Drake
2013-07-10  8:19       ` Haojian Zhuang
2013-07-10 12:24         ` Jason Cooper
2013-07-10 14:08           ` Haojian Zhuang
     [not found]             ` <CAN1soZzMvront1-jcBW6-uRKNs0OZNGnmiOjKGZSM28bViB_8A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-10 21:50               ` Jason Cooper
2013-07-11 23:54                 ` Haojian Zhuang
2013-07-12 15:10                   ` Daniel Drake
2013-07-12 15:57                     ` Jason Cooper
2013-07-12 16:05                       ` Daniel Drake
2013-07-12 16:14                         ` Jason Cooper
     [not found]                     ` <CAMLZHHQGZvDssY-YxQ3ZW8FYhpG-H1-VbokOid33socRSPrMNw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-13  0:35                       ` Haojian Zhuang
     [not found]                         ` <CAN1soZzQz8PaHOwNf-Ec6G=ahhx_P8gaaX4fQnpX-Mqrepk_7g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-13 14:30                           ` Daniel Drake
     [not found]                 ` <20130710215041.GR11908-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2013-07-13 19:56                   ` Olof Johansson
2013-07-15  4:39                     ` Jason Cooper
2013-07-10 20:50       ` Matt Sealey
2013-07-10 21:49         ` Jason Cooper
2013-07-11 11:20           ` Neil Zhang
2013-07-14 19:29             ` Arnd Bergmann [this message]
     [not found]               ` <201307142129.43271.arnd-r2nGTMty4D4@public.gmane.org>
2013-07-17 13:37                 ` Neil Zhang
2013-07-09  6:42 ` [PATCH V3 2/3] clocksource: mmp: support CLOCKSOURCE OF DECLARE Neil Zhang
2013-07-09 16:14   ` Thomas Petazzoni
2013-07-09 21:16     ` Arnd Bergmann
2013-07-10  5:58       ` Haojian Zhuang
2013-07-11 11:23         ` Neil Zhang
2013-07-11 23:51           ` Haojian Zhuang
2013-07-09  6:42 ` [PATCH V3 3/3] ARM: mmp: bring up pxa988 with device tree support Neil Zhang
2013-07-09 22:05   ` Arnd Bergmann
2013-07-11 11:35     ` Neil Zhang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201307142129.43271.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@linaro.org \
    --cc=haojian.zhuang@gmail.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=neko@bakuhatsu.net \
    --cc=zhangwm@marvell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).