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

From: Takashi Iwai
Date: Tue Mar 01 2016 - 02:40:09 EST


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@xxxxxxxxxxxxxxx>
> >
> > 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@xxxxxxxxx
> but Signed-off-by: was sudip@xxxxxxxxxxxxxxx and now
> Signed-off-by: has changed to sudip.mukherjee@xxxxxxxxxxxxxxx
> 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