W dniu 19.10.2011 13:44, Mauro Carvalho Chehab pisze:
Em 19-10-2011 09:41, Javier Martinez Canillas escreveu:
On Tue, Oct 18, 2011 at 7:20 PM, Piotr Chmura<[email protected]>  wrote:
On Tue, 18 Oct 2011 11:52:17 -0400
Devin Heitmueller<[email protected]>  wrote:

On Tue, Oct 18, 2011 at 5:10 AM, Piotr Chmura<[email protected]>  wrote:
Thanks for comments for all of you.

[PATCH 1-12/14] Following your guidelines i exported all changes from hg one by 
one. This way we will have all history in kernel tree.
I moved driver to staging/media and removed Kconfig/Makefile changes in parent 
directory in first patch.
Hello Piotr,

Not that I want to create more work for you, but it would appear that
your patches stripped off all the Signed-off-by lines for both myself
and Pierrick Hascoet (the developer from the hardware vendor).  You
have replaced them with "cc:" lines, which breaks the chain of
"Developer's Certificate of Origin".

When you take somebody else's patches, you need to preserve any
existing Signed-off-by lines, adding your own at the bottom of the
list.

In other words, the first patch should be:

Signed-off-by: Pierrick Hascoet<[email protected]>
Signed-off-by: Devin Heitmueller<[email protected]>
Signed-off-by: Piotr Chmura<[email protected]>

instead of:

Signed-off-by: Piotr Chmura<[email protected]>
Cc: Pierrick Hascoet<[email protected]>
Cc: Devin Heitmueller<[email protected]>

Devin

--
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ok, i'll resend them again.

Should I replay to every patch with something like [RESEND PATCH nn/mm]..., 
right ?

Peter
Hi Peter,

A common convention is to add the version of your patch in the subject like:

[PATCH v2 0/14] staging/media/as102: new driver submission

That way people can know that is actually a resend of a new patch-set
and not a resend of the last one.
Thanks, I'll do it this way next time.

Yes. Also, it seems that you've submitted only 12 patches of this 14 patch
series. Where are the other two missing patches?
13 and 14 were written by me, so they didn't suffer "signed-off-by" mistake. Do I need resend them too ?

Peter

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to