Re: [PATCH v7 1/2] spi: add support for DLN-2 USB-SPI adapter

From: Laurentiu Palcu
Date: Fri Nov 28 2014 - 13:13:37 EST


On Fri, Nov 28, 2014 at 04:01:56PM +0000, Mark Brown wrote:
> On Fri, Nov 28, 2014 at 05:31:25PM +0200, Laurentiu Palcu wrote:
> > On Fri, Nov 28, 2014 at 03:14:35PM +0000, Mark Brown wrote:
> > > On Fri, Nov 28, 2014 at 04:57:04PM +0200, Laurentiu Palcu wrote:
>
> > > > The other patch in this series was already applied by Lee on
> > > > for-mfd-next branch:
>
> > > What this means is that the series you're sending actually contains one
> > > patch, not two. The point of numbering patches within a series is so
> > > people can tell what order to apply them in, nothing else. Things can
> > > be lumped together and split during the upstreaming process, the only
> > > thing that matterrs is what you're sending at a given time.
>
> > The order of the patches in this series can be interchanged. However, if
> > you wish, I can resend the entire series again.
>
> No, please don't - you're missing the point. The point is that now Lee
> has applied the other patch you're not sending two patches any more,
> you're sending a single patch. This isn't patch 1/2, it's just a single
> patch by itself now.
I have to admit Linux kernel mailing list workflow is a little new to
me...

All I did was send new revisions of the 1/2 patch using the
--in-reply-to option of git-send-email, with the message-id of the
original version. I used to do this for other projects I contributed to,
so I didn't have to re-send the entire series for just one patch that
needs to be changed. Mail clients, at least the ones I use (mutt,
evolution), order the mails nicely and you have all the history in one
thread. So, one can see the cover letter of the series and the other
original patches in the series, together with the new revisions of the
changed patch. Of course, if more patches needed change, I would re-send a new
series.

So, Mark, can you please instruct me (a link to a page explaining the
workflow would be fine too) what is the exact workflow you are/were
expecting?

thanks in advance,
laurentiu

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/