All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: avorontsov@ru.mvista.com
Cc: sfr@canb.auug.org.au, linux-kernel@vger.kernel.org,
	linuxppc-dev@ozlabs.org, wim@iguana.be, scottwood@freescale.com
Subject: Re: [PATCH 0/8 v4] mpc83xx_wdt rework, support for mpc8610 and mpc8xx
Date: Tue, 3 Jun 2008 17:32:14 -0700	[thread overview]
Message-ID: <20080603173214.0bf71571.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080604001739.GB24548@polina.dev.rtsoft.ru>

On Wed, 4 Jun 2008 04:17:39 +0400
Anton Vorontsov <avorontsov@ru.mvista.com> wrote:

> > Please put the subsystem identifier (eg, "watchdog" and "powerpc")
> > outside the [], for reasons which should be in
> > Documentation/SubmittingPatches, which used to be there but which got
> > lost.  Bascially the text inside [] is for temporary not-for-committing
> > information such as "rfc", "2.6.24-rc4", "resend", etc and should be stripped
> > by the email recipient before merging.
> 
> Yeah, I know. It is just hard to remember all the preferences.
> 
> For example, PowerPC maintainers asking to do patches with "[POWERPC]"
> identifier, this identifier purposely keeps intact for git-log.

Addition of "[powerpc]" if it was absent can be scripted.

However, the retaining of "[powerpc]" (etc) while not retaining "[rfc]"
(etc) is not practical.

Plus putting things into git with "[powerpc]" in the title is wrong. 
The chances are good that anyone who is taking such a patch off the
git-commits list (say, for a backport) will lose that part of the
title.  It should be "powerpc: "

(http://userweb.kernel.org/~akpm/dh.gif)

WARNING: multiple messages have this Message-ID (diff)
From: Andrew Morton <akpm@linux-foundation.org>
To: avorontsov@ru.mvista.com
Cc: galak@kernel.crashing.org, wim@iguana.be, jochen@scram.de,
	sfr@canb.auug.org.au, segher@kernel.crashing.org,
	scottwood@freescale.com, linuxppc-dev@ozlabs.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/8 v4] mpc83xx_wdt rework, support for mpc8610 and mpc8xx
Date: Tue, 3 Jun 2008 17:32:14 -0700	[thread overview]
Message-ID: <20080603173214.0bf71571.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080604001739.GB24548@polina.dev.rtsoft.ru>

On Wed, 4 Jun 2008 04:17:39 +0400
Anton Vorontsov <avorontsov@ru.mvista.com> wrote:

> > Please put the subsystem identifier (eg, "watchdog" and "powerpc")
> > outside the [], for reasons which should be in
> > Documentation/SubmittingPatches, which used to be there but which got
> > lost.  Bascially the text inside [] is for temporary not-for-committing
> > information such as "rfc", "2.6.24-rc4", "resend", etc and should be stripped
> > by the email recipient before merging.
> 
> Yeah, I know. It is just hard to remember all the preferences.
> 
> For example, PowerPC maintainers asking to do patches with "[POWERPC]"
> identifier, this identifier purposely keeps intact for git-log.

Addition of "[powerpc]" if it was absent can be scripted.

However, the retaining of "[powerpc]" (etc) while not retaining "[rfc]"
(etc) is not practical.

Plus putting things into git with "[powerpc]" in the title is wrong. 
The chances are good that anyone who is taking such a patch off the
git-commits list (say, for a backport) will lose that part of the
title.  It should be "powerpc: "

(http://userweb.kernel.org/~akpm/dh.gif)

  reply	other threads:[~2008-06-04  0:32 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-02 17:37 [PATCH 0/8 v4] mpc83xx_wdt rework, support for mpc8610 and mpc8xx Anton Vorontsov
2008-06-02 17:37 ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 1/8] [WATCHDOG] mpc83xx_wdt: fix checkpatch issues Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 19:43   ` Alan Cox
2008-06-02 19:43     ` Alan Cox
2008-06-02 20:49     ` Anton Vorontsov
2008-06-02 20:49       ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 2/8] [WATCHDOG] mpc83xx_wdt: convert to the OF platform driver Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 3/8] [WATCHDOG] mpc83xx_wdt: add support for MPC86xx CPUs Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 4/8] [WATCHDOG] mpc83xx_wdt: rename to mpc8xxx_wdt Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 5/8] [WATCHDOG] mpc8xxx_wdt: various renames, mostly s/mpc83xx/mpc8xxx/g Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 6/8] [WATCHDOG] mpc8xxx_wdt: add support for MPC8xx watchdogs Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 7/8] [POWERPC] fsl_soc: remove mpc83xx_wdt code Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-02 17:38 ` [PATCH 8/8] [POWERPC] 86xx: mpc8610_hpcd: add watchdog node Anton Vorontsov
2008-06-02 17:38   ` Anton Vorontsov
2008-06-03 23:48 ` [PATCH 0/8 v4] mpc83xx_wdt rework, support for mpc8610 and mpc8xx Andrew Morton
2008-06-03 23:48   ` Andrew Morton
2008-06-04  0:17   ` Anton Vorontsov
2008-06-04  0:17     ` Anton Vorontsov
2008-06-04  0:32     ` Andrew Morton [this message]
2008-06-04  0:32       ` Andrew Morton
2008-06-04  4:07       ` Paul Mackerras
2008-06-04  4:07         ` Paul Mackerras
2008-06-04  4:15         ` Andrew Morton
2008-06-04  4:15           ` Andrew Morton
2008-06-04 17:31           ` Randy Dunlap
2008-06-04 17:31             ` Randy Dunlap
2008-06-04 12:27         ` Anton Vorontsov
2008-06-04 12:27           ` Anton Vorontsov
2008-06-07 17:57   ` [PATCH -mm] watchdog: mpc8xxx_wdt: fix build Anton Vorontsov
2008-06-07 17:57     ` Anton Vorontsov
  -- strict thread matches above, loose matches on Subject: below --
2008-05-19 18:37 [PATCH 0/8 v4] mpc83xx_wdt rework, support for mpc8610 and mpc8xx Anton Vorontsov

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=20080603173214.0bf71571.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=avorontsov@ru.mvista.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=scottwood@freescale.com \
    --cc=sfr@canb.auug.org.au \
    --cc=wim@iguana.be \
    /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.