Re: [PATCH 66/73] staging/lustre: remove lots of dead code

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

 



On Sunday 27 September 2015 16:46:06 green@xxxxxxxxxxxxxx wrote:
> From: Oleg Drokin <green@xxxxxxxxxxxxxx>
> 
> This is a large patch to remove all dead code from obdclass and
> ptlrpc, purely removing functions and declarations now, so
> despite the size should be easy enough to verify.
> 
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Oleg Drokin <green@xxxxxxxxxxxxxx>

Thanks for all the work to split out the smaller patches from my
larger code removal!

There is one small point about the attribution though: your split
out patches now have 'From: Oleg Drokin <green@xxxxxxxxxxxxxx>' along
wtih 'Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>', which is
inconsistent. For the larger chunks of patches that you took from me
and that retain my 'Signed-off-by', it would be nice if you could
set the author field to me for consistency as well.

For the smaller patches for which writing the changelog was likely
more work than doing the patch, it's fine if you claim authorship,
but if you do that then please change the 'Signed-off-by' to
'Reported-by' or 'Suggested-by' for me, so the author field
matches the first Signed-off-by.

	Arnd
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux