[alsa-devel] [PATCH v2] ASoC: hdmi-codec: fix non static symbol warnings

Mark Brown broonie at kernel.org
Tue Mar 14 20:46:00 CET 2017


On Tue, Mar 14, 2017 at 01:01:57PM +0900, Takashi Sakamoto wrote:
> On Mar 14 2017 01:50, Mark Brown wrote:

> > Did you write this or did Wei Yongjun?  If they did then the patch
> > should show them as the author.

> I'm sure that you've rejected such a patch in which sender is not commit
> author. With this memory, I sent this patch with my name as author
> regardless of original author.

No, I've never done that.  What I complain about is people sending
patches that they wrote themselves but which have different authorship
information in the file, usually because they've used a different e-mail
address.  This is a problem because it makes it look like it's a patch
that needs at least two signoffs as it wasn't written by the person
sending it so slows things down.  If there are actually two signoffs
there's no problem at all as that's what's expected.

If you look at git you'll see I fairly regularly apply patches which
were forwarded by someone else.

> What fashion is suitable for you? Please give me enough instruction to your
> fashion in this case, Mark. I'm puzzled.

Please follow the process covered in SubmittingPatches - if you're
forwarding on a patch someone else wrote you need to preserve their
authorship and add your own signoff after theirs.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20170314/c6cec0f9/attachment.sig>


More information about the Alsa-devel mailing list