[alsa-devel] [PATCH 1/6] AK4114 - fix output selector bits, fix spelling

Takashi Iwai tiwai at suse.de
Tue Sep 15 14:31:34 CEST 2009


At Tue, 15 Sep 2009 13:51:16 +0200,
Pavel Hofman wrote:
> 
> Takashi Iwai wrote:
> > At Tue, 15 Sep 2009 00:44:58 +0200,
> > pavel.hofman at ivitera.com wrote:
> >> From: Pavel Hofman <pavel.hofman at ivitera.com>
> >>
> >>
> >> Signed-off-by: Pavel Hofman <pavel.hofman at ivitera.com>
> > 
> > Could you give a bit more changelog content?
> > The summary line isn't enough for non-trivial changes.
> > In this case, you changed AK4114_OPS1[012] values without any
> > explanation.  Please give a bit more information, what these changes
> > mean, what impact they have, etc.
> 
> Sure, I was not sure how the rules apply in git. The first line in
> rebase/patch edit is a summary statement and next lines are the detailed
> changelog? Thanks a lot for the info.

Yes, the first line is the summary, then a blank line.
The rest are changelogs, and your sign-off.

Don't forget to put changelogs to each commit.  A longer description
is much much better than no description :)


thanks,

Takashi


More information about the Alsa-devel mailing list