All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Mark Brown
	<broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
Cc: Laxman Dewangan
	<ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"lrg-l0cyMroinI0@public.gmane.org"
	<lrg-l0cyMroinI0@public.gmane.org>
Subject: Re: [PATCH 2/2] regulator: palmas: add input supply names
Date: Wed, 20 Mar 2013 10:56:16 -0600	[thread overview]
Message-ID: <5149EA30.8020302@wwwdotorg.org> (raw)
In-Reply-To: <20130320165227.GU28775-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>

On 03/20/2013 10:52 AM, Mark Brown wrote:
> On Wed, Mar 20, 2013 at 10:42:26AM -0600, Stephen Warren wrote:
>> On 03/20/2013 10:38 AM, Mark Brown wrote:
>>> On Wed, Mar 20, 2013 at 10:07:17AM -0600, Stephen Warren
>>> wrote:
>>>> On 03/20/2013 06:54 AM, Laxman Dewangan wrote:
> 
>>>>> I generally send a patches on single git-send command git 
>>>>> send-emal --to=ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org 0000.patch 0001.patch
> 
>>>> Don't do that; send *.patch. If you don't, then patches 2..n 
>>>> don't end up being "in-reply-to" patch 1, so they won't show
>>>> up as a single email thread.
> 
>>> The two should be equivalent (though the glob is much easier
>>> to type)? The result of the globbing ought to be what Laxman is
>>> typing by hand assuming that he's getting the order correct.
> 
>> If you send *.patch at once, git send-email adds an in-reply-to
>> header to the email which sets up the threading. If you send the
>> patches 1-by-1, this header isn't added, since git send-email
>> doesn't have a clue what message ID it chose for patch 1. Unless
>> you pass the --in-reply-to command-line option, that is.
> 
> Indeed - I think you may have quoted the wrong bit of Laxman's
> mail there?  The bit you're replying to is doing it as a single
> command, though he did talk about doing this in multiple commands
> elsewhere in his mail.

Oh right yes. I'd seen 0000.patch and not that there were multiple
entries in the command-line that listed (I assume) all the files.
Sorry for the noise.

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren@wwwdotorg.org>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Stephen Warren <swarren@nvidia.com>,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
	"lrg@ti.com" <lrg@ti.com>
Subject: Re: [PATCH 2/2] regulator: palmas: add input supply names
Date: Wed, 20 Mar 2013 10:56:16 -0600	[thread overview]
Message-ID: <5149EA30.8020302@wwwdotorg.org> (raw)
In-Reply-To: <20130320165227.GU28775@opensource.wolfsonmicro.com>

On 03/20/2013 10:52 AM, Mark Brown wrote:
> On Wed, Mar 20, 2013 at 10:42:26AM -0600, Stephen Warren wrote:
>> On 03/20/2013 10:38 AM, Mark Brown wrote:
>>> On Wed, Mar 20, 2013 at 10:07:17AM -0600, Stephen Warren
>>> wrote:
>>>> On 03/20/2013 06:54 AM, Laxman Dewangan wrote:
> 
>>>>> I generally send a patches on single git-send command git 
>>>>> send-emal --to=ldewangan@nvidia.com 0000.patch 0001.patch
> 
>>>> Don't do that; send *.patch. If you don't, then patches 2..n 
>>>> don't end up being "in-reply-to" patch 1, so they won't show
>>>> up as a single email thread.
> 
>>> The two should be equivalent (though the glob is much easier
>>> to type)? The result of the globbing ought to be what Laxman is
>>> typing by hand assuming that he's getting the order correct.
> 
>> If you send *.patch at once, git send-email adds an in-reply-to
>> header to the email which sets up the threading. If you send the
>> patches 1-by-1, this header isn't added, since git send-email
>> doesn't have a clue what message ID it chose for patch 1. Unless
>> you pass the --in-reply-to command-line option, that is.
> 
> Indeed - I think you may have quoted the wrong bit of Laxman's
> mail there?  The bit you're replying to is doing it as a single
> command, though he did talk about doing this in multiple commands
> elsewhere in his mail.

Oh right yes. I'd seen 0000.patch and not that there were multiple
entries in the command-line that listed (I assume) all the files.
Sorry for the noise.

  parent reply	other threads:[~2013-03-20 16:56 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-18  9:29 [PATCH 2/2] regulator: palmas: add input supply names Laxman Dewangan
2013-03-18  9:29 ` Laxman Dewangan
2013-03-18  9:29 ` [PATCH 1/2] regulator: palmas: rename probe/remove callback functions Laxman Dewangan
2013-03-18  9:29   ` Laxman Dewangan
     [not found] ` <20130320123128.GM28775@opensource.wolfsonmicro.com>
     [not found]   ` <20130320123128.GM28775-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2013-03-20 12:54     ` [PATCH 2/2] regulator: palmas: add input supply names Laxman Dewangan
2013-03-20 12:54       ` Laxman Dewangan
     [not found]       ` <20130320132531.GO28775@opensource.wolfsonmicro.com>
     [not found]         ` <20130320132531.GO28775-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2013-03-20 13:32           ` Laxman Dewangan
2013-03-20 13:32             ` Laxman Dewangan
     [not found]       ` <5149B19E.90400-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-20 16:07         ` Stephen Warren
2013-03-20 16:07           ` Stephen Warren
     [not found]           ` <20130320163833.GT28775@opensource.wolfsonmicro.com>
     [not found]             ` <20130320163833.GT28775-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2013-03-20 16:42               ` Stephen Warren
2013-03-20 16:42                 ` Stephen Warren
     [not found]                 ` <20130320165227.GU28775@opensource.wolfsonmicro.com>
     [not found]                   ` <20130320165227.GU28775-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2013-03-20 16:56                     ` Stephen Warren [this message]
2013-03-20 16:56                       ` Stephen Warren

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=5149EA30.8020302@wwwdotorg.org \
    --to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
    --cc=broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org \
    --cc=ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=lrg-l0cyMroinI0@public.gmane.org \
    --cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    /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.