Re: [PATCH 0/4] Add DT support for Network Space v2 and parents

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 15, 2012 at 08:50:37PM +0200, Simon Guinot wrote:
> On Mon, Oct 15, 2012 at 01:01:44PM -0400, Jason Cooper wrote:
> > I'm pretty sure this isn't the first version of this patch series ;-)
> > No need to resend, but could you attach a version number and changelog?
> 
> Yes, you are right. For the second patch, it is the second version.
> But all the other patches are at their first version. Then the whole
> patch series is more a v1 than a v2. I am always embarrassed when the
> patches versions are not the same within a series...

Yes, it makes my skin crawl when I send something out like that as well.

My rule of thumb is that if the patch subject matter is still identical
to the previous version, then I only increment that patch.  If suggested
changes cause a reshuffle of the whole series, then I just increment the
version number across the board.

In this case, incrementing the first patch would have been fine.

> If you agree, I will reply against the second patch in order to
> underline the changes.

Perfect.

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux