Re: [PATCH 1/31] HDA patch_via.c: Remove IS_VT17xx_VENDORID macro

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

 



At Thu, 8 Oct 2009 14:22:07 +0800,
Li Bo wrote:
> 
> On Thu, Oct 8, 2009 at 2:16 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> > At Thu, 8 Oct 2009 14:11:42 +0800,
> > Li Bo wrote:
> >>
> >> On Thu, Oct 8, 2009 at 2:03 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> >> > At Thu, 8 Oct 2009 13:27:49 +0800,
> >> > Li Bo wrote:
> >> >>
> >> >> [ALSA] HDA VIA: Remove unused IS_VT17xx_VENDORID macro
> >> >>
> >> >> IS_VT17*_VENDORID macros are used nowhere, so clean them up.
> >> >>
> >> >> Signed-off-by: Lydia Wang <lydiawang@xxxxxxxxxxxxxx>
> >> >
> >> > Li, could you add your sign-off, too, when you handle the patches?
> >> > Otherwise I can't merge them.
> >> >
> >> >
> >> > thanks,
> >> >
> >> > Takashi
> >> >
> >>
> >> Is that OK signed-off with LoganLi@xxxxxxxxxxxxxx with this gmail
> >> liboat@xxxxxxxxx? LoganLi@xxxxxxxxxxxxxx is my company's email.
> >> If not, I'll will resend them tomorrow when I go back to company.
> >
> > I don't care which address you use (and I don't know whether it
> > matters your employer).  It's fully your choice.
> >
> > But, if the signed address is different from the mail sender, add a
> > line "From: YOUR NAME <your@mail>" at the beginning of the patch
> > explicitly.
> >
> >
> > thanks,
> >
> > Takashi
> >
> I just randomly use my gmail. I will resend them tomorrow, thank you for
> teaching me:)

Well, I was unclear in the text above.

You can send patches with the sign-off of your company account even
from your gmail account.  Just add "From:" line with your company
account in the patch text so that I can pick up the right address.


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux