[alsa-devel] [PATCH] ASoC: intel: remove unused variable

Takashi Iwai tiwai at suse.de
Tue Mar 1 08:39:58 CET 2016


On Tue, 01 Mar 2016 06:49:27 +0100,
Sudip Mukherjee wrote:
> 
> On Tue, Mar 01, 2016 at 11:59:12AM +0900, Mark Brown wrote:
> > On Mon, Feb 29, 2016 at 10:53:46PM +0530, Sudip Mukherjee wrote:
> > > The variable cmd_id is only assigned some value and is never used.
> > > 
> > > Signed-off-by: Sudip Mukherjee <sudip.mukherjee at codethink.co.uk>
> > 
> > Your signoff doesn't correspond to your e-mail address.  While I'm
> > fairly sure that you are in fact the same person so it's OK this time
> > please try to ensure that they match up so that it's clear that the
> > signoffs are all in order.
> 
> But it has been like this way since I started contributing.
> git log --author=sudip will show you.
> 
> From: address was always sudipm.mukherjee at gmail.com
> but Signed-off-by: was sudip at vectorindia.org and now
> Signed-off-by: has changed to sudip.mukherjee at codethink.co.uk
> and you already have many such patches from me. (well, with my
> vectorindia.org email)
> 
> Can i not keep using my gmail as my From: ?
> The advantage of using gmail as my From: is that the discussion and all
> threads related to my patches will be in the same place even if my work
> email is changed.
> 
> And AFAIK, the From: name and the Signed-off-by: name should match else
> the tools used by Greg and others to generate the stats will break.
> 
> Please reconsider and let me use my gmail in my From:

Using a different author address from the sign-off is unusual, but
there are a few people preferring it, indeed, AFAIK.


Takashi


More information about the Alsa-devel mailing list