At Tue, 15 Sep 2009 00:44:58 +0200, pavel.hofman@xxxxxxxxxxx wrote: > > From: Pavel Hofman <pavel.hofman@xxxxxxxxxxx> > > > Signed-off-by: Pavel Hofman <pavel.hofman@xxxxxxxxxxx> 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. thanks, Takashi > > diff --git a/include/ak4114.h b/include/ak4114.h > index d293d36..3ce69fd 100644 > --- a/include/ak4114.h > +++ b/include/ak4114.h > @@ -95,13 +95,13 @@ > > /* AK4114_REG_IO0 */ > #define AK4114_TX1E (1<<7) /* TX1 Output Enable (1 = enable) */ > -#define AK4114_OPS12 (1<<2) /* Output Though Data Selector for TX1 pin */ > -#define AK4114_OPS11 (1<<1) /* Output Though Data Selector for TX1 pin */ > -#define AK4114_OPS10 (1<<0) /* Output Though Data Selector for TX1 pin */ > +#define AK4114_OPS12 (1<<6) /* Output Data Selector for TX1 pin */ > +#define AK4114_OPS11 (1<<5) /* Output Data Selector for TX1 pin */ > +#define AK4114_OPS10 (1<<4) /* Output Data Selector for TX1 pin */ > #define AK4114_TX0E (1<<3) /* TX0 Output Enable (1 = enable) */ > -#define AK4114_OPS02 (1<<2) /* Output Though Data Selector for TX0 pin */ > -#define AK4114_OPS01 (1<<1) /* Output Though Data Selector for TX0 pin */ > -#define AK4114_OPS00 (1<<0) /* Output Though Data Selector for TX0 pin */ > +#define AK4114_OPS02 (1<<2) /* Output Data Selector for TX0 pin */ > +#define AK4114_OPS01 (1<<1) /* Output Data Selector for TX0 pin */ > +#define AK4114_OPS00 (1<<0) /* Output Data Selector for TX0 pin */ > > /* AK4114_REG_IO1 */ > #define AK4114_EFH1 (1<<7) /* Interrupt 0 pin Hold */ > -- > 1.5.6.3 > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel