All of lore.kernel.org
 help / color / mirror / Atom feed
From: Neil Zhang <zhangwm@marvell.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Jason Cooper <jason@lakedaemon.net>,
	Matt Sealey <neko@bakuhatsu.net>,
	"grant.likely@linaro.org" <grant.likely@linaro.org>,
	"haojian.zhuang@gmail.com" <haojian.zhuang@gmail.com>,
	"devicetree-discuss@lists.ozlabs.org" 
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.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: Wed, 17 Jul 2013 06:37:57 -0700	[thread overview]
Message-ID: <175CCF5F49938B4D99B2E3EF7F558EBE3DBCDAF7C4@SC-VEXCH4.marvell.com> (raw)
In-Reply-To: <201307142129.43271.arnd@arndb.de>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb2312", Size: 2083 bytes --]

Arnd

> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd@arndb.de]
> Sent: 2013Äê7ÔÂ15ÈÕ 3:30
> To: Neil Zhang
> Cc: Jason Cooper; Matt Sealey; grant.likely@linaro.org;
> haojian.zhuang@gmail.com; devicetree-discuss@lists.ozlabs.org;
> linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
> 
> 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.

So would you mind if I leave it there (discard this patch) and still use the current 'mvrl' prefix for this patch set.

> 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


Best Regards,
Neil Zhang
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

WARNING: multiple messages have this Message-ID (diff)
From: Neil Zhang <zhangwm-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Cc: Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>,
	"devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org"
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Matt Sealey <neko-HhXTZounMxbZATc7fWT8Dg@public.gmane.org>,
	"grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org"
	<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: RE: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
Date: Wed, 17 Jul 2013 06:37:57 -0700	[thread overview]
Message-ID: <175CCF5F49938B4D99B2E3EF7F558EBE3DBCDAF7C4@SC-VEXCH4.marvell.com> (raw)
In-Reply-To: <201307142129.43271.arnd-r2nGTMty4D4@public.gmane.org>

Arnd

> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd@arndb.de]
> Sent: 2013年7月15日 3:30
> To: Neil Zhang
> Cc: Jason Cooper; Matt Sealey; grant.likely@linaro.org;
> haojian.zhuang@gmail.com; devicetree-discuss@lists.ozlabs.org;
> linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
> 
> 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.

So would you mind if I leave it there (discard this patch) and still use the current 'mvrl' prefix for this patch set.

> 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


Best Regards,
Neil Zhang
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

WARNING: multiple messages have this Message-ID (diff)
From: zhangwm@marvell.com (Neil Zhang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
Date: Wed, 17 Jul 2013 06:37:57 -0700	[thread overview]
Message-ID: <175CCF5F49938B4D99B2E3EF7F558EBE3DBCDAF7C4@SC-VEXCH4.marvell.com> (raw)
In-Reply-To: <201307142129.43271.arnd@arndb.de>

Arnd

> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd at arndb.de]
> Sent: 2013?7?15? 3:30
> To: Neil Zhang
> Cc: Jason Cooper; Matt Sealey; grant.likely at linaro.org;
> haojian.zhuang at gmail.com; devicetree-discuss at lists.ozlabs.org;
> linux-kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH V3 1/3] dts: change Marvell prefix to 'marvell'
> 
> 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.

So would you mind if I leave it there (discard this patch) and still use the current 'mvrl' prefix for this patch set.

> 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


Best Regards,
Neil Zhang

  reply	other threads:[~2013-07-17 13:39 UTC|newest]

Thread overview: 96+ 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 ` Neil Zhang
2013-07-09  6:42 ` Neil Zhang
2013-07-09  6:42 ` [PATCH V3 1/3] dts: change Marvell prefix to 'marvell' Neil Zhang
2013-07-09  6:42   ` Neil Zhang
2013-07-09  6:42   ` Neil Zhang
2013-07-09 12:49   ` Jason Cooper
2013-07-09 12:49     ` Jason Cooper
2013-07-09 12:49     ` Jason Cooper
2013-07-10  7:25     ` Neil Zhang
2013-07-10  7:25       ` Neil Zhang
2013-07-10  7:25       ` Neil Zhang
2013-07-10 12:20       ` Jason Cooper
2013-07-10 12:20         ` Jason Cooper
2013-07-10 12:20         ` Jason Cooper
2013-07-10 14:41         ` Daniel Drake
2013-07-10 14:41           ` Daniel Drake
2013-07-10 14:41           ` Daniel Drake
2013-07-10  8:19     ` Haojian Zhuang
2013-07-10  8:19       ` Haojian Zhuang
2013-07-10  8:19       ` Haojian Zhuang
2013-07-10 12:24       ` Jason Cooper
2013-07-10 12:24         ` Jason Cooper
2013-07-10 12:24         ` Jason Cooper
2013-07-10 14:08         ` Haojian Zhuang
2013-07-10 14:08           ` Haojian Zhuang
2013-07-10 14:08           ` Haojian Zhuang
2013-07-10 21:50           ` Jason Cooper
2013-07-10 21:50             ` Jason Cooper
2013-07-10 21:50             ` Jason Cooper
2013-07-11 23:54             ` Haojian Zhuang
2013-07-11 23:54               ` Haojian Zhuang
2013-07-11 23:54               ` Haojian Zhuang
2013-07-12 15:10               ` Daniel Drake
2013-07-12 15:10                 ` Daniel Drake
2013-07-12 15:10                 ` Daniel Drake
2013-07-12 15:57                 ` Jason Cooper
2013-07-12 15:57                   ` Jason Cooper
2013-07-12 15:57                   ` Jason Cooper
2013-07-12 16:05                   ` Daniel Drake
2013-07-12 16:05                     ` Daniel Drake
2013-07-12 16:05                     ` Daniel Drake
2013-07-12 16:14                     ` Jason Cooper
2013-07-12 16:14                       ` Jason Cooper
2013-07-12 16:14                       ` Jason Cooper
2013-07-13  0:35                 ` Haojian Zhuang
2013-07-13  0:35                   ` Haojian Zhuang
2013-07-13  0:35                   ` Haojian Zhuang
2013-07-13 14:30                   ` Daniel Drake
2013-07-13 14:30                     ` Daniel Drake
2013-07-13 14:30                     ` Daniel Drake
2013-07-13 19:56             ` Olof Johansson
2013-07-13 19:56               ` Olof Johansson
2013-07-13 19:56               ` Olof Johansson
2013-07-15  4:39               ` Jason Cooper
2013-07-15  4:39                 ` Jason Cooper
2013-07-15  4:39                 ` Jason Cooper
2013-07-10 20:50     ` Matt Sealey
2013-07-10 20:50       ` Matt Sealey
2013-07-10 21:49       ` Jason Cooper
2013-07-10 21:49         ` Jason Cooper
2013-07-11 11:20         ` Neil Zhang
2013-07-11 11:20           ` Neil Zhang
2013-07-11 11:20           ` Neil Zhang
2013-07-14 19:29           ` Arnd Bergmann
2013-07-14 19:29             ` Arnd Bergmann
2013-07-14 19:29             ` Arnd Bergmann
2013-07-17 13:37             ` Neil Zhang [this message]
2013-07-17 13:37               ` Neil Zhang
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  6:42   ` Neil Zhang
2013-07-09  6:42   ` Neil Zhang
2013-07-09 16:14   ` Thomas Petazzoni
2013-07-09 16:14     ` Thomas Petazzoni
2013-07-09 16:14     ` Thomas Petazzoni
2013-07-09 21:16     ` Arnd Bergmann
2013-07-09 21:16       ` Arnd Bergmann
2013-07-09 21:16       ` Arnd Bergmann
2013-07-10  5:58       ` Haojian Zhuang
2013-07-10  5:58         ` Haojian Zhuang
2013-07-10  5:58         ` Haojian Zhuang
2013-07-11 11:23         ` Neil Zhang
2013-07-11 11:23           ` Neil Zhang
2013-07-11 11:23           ` Neil Zhang
2013-07-11 23:51           ` Haojian Zhuang
2013-07-11 23:51             ` Haojian Zhuang
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  6:42   ` Neil Zhang
2013-07-09  6:42   ` Neil Zhang
2013-07-09 22:05   ` Arnd Bergmann
2013-07-09 22:05     ` Arnd Bergmann
2013-07-11 11:35     ` Neil Zhang
2013-07-11 11:35       ` Neil Zhang
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=175CCF5F49938B4D99B2E3EF7F558EBE3DBCDAF7C4@SC-VEXCH4.marvell.com \
    --to=zhangwm@marvell.com \
    --cc=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 \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.