On 09/06/2012 04:20 AM, Lukasz Majewski wrote: > Hi Stephen, > >> On 08/24/2012 02:13 AM, Lukasz Majewski wrote: >>> Custom definitions of le_XX_to_int functions have been replaced with >>> standard ones, defined at <compiler.h> >>> >>> Signed-off-by: Chang Hyun Park <chchch.p...@samsung.com> >>> Signed-off-by: Lukasz Majewski <l.majew...@samsung.com> >>> Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com> >> >> Did Chang Hyun Park write this? If so, shouldn't git have generated a >> "From" header for him, to reflect his authorship upstream. I'm not >> sure how Kyungmin Park's S-o-b plays into this, since he's not >> sending the patch. > > My work (for those patches) was based on Mr. Chang Hyum Park patches. > However I had to modify his work for patches submission. > > Therefore I've added Mr. Chang in a first place to credit his effort to > this patch series creation. > > Shall I do it in a different way?
To be honest, I'm not really sure. When I've done this, I've kept the original author in the git "author" field to show this, and then listed what I changed in the commit description. Whether that's the right way to go perhaps depends on the size of your changes. If your changes are so large that it doesn't make sense to do that, perhaps his S-o-b should be removed and replaced with a simple note "based on work by Chang Hyun Park <chchch.p...@samsung.com>"? _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
