[alsa-devel] [PATCH 1/6 v3] ASoC: fsi: modify format area definition on flags
There is no necessity that each bit in this area has the meaning. This patch modify it to sequence number
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com --- v2 -> v3
o don't use << 0 shift
2/6 - 6/6 doesn't depend on 1/6 patch
include/sound/sh_fsi.h | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/sound/sh_fsi.h b/include/sound/sh_fsi.h index c022736..3ecdb96 100644 --- a/include/sound/sh_fsi.h +++ b/include/sound/sh_fsi.h @@ -55,12 +55,12 @@ #define SH_FSI_GET_IFMT(x) ((x >> 8) & SH_FSI_FMT_MASK) #define SH_FSI_GET_OFMT(x) ((x >> 0) & SH_FSI_FMT_MASK)
-#define SH_FSI_FMT_MONO (1 << 0) -#define SH_FSI_FMT_MONO_DELAY (1 << 1) -#define SH_FSI_FMT_PCM (1 << 2) -#define SH_FSI_FMT_I2S (1 << 3) -#define SH_FSI_FMT_TDM (1 << 4) -#define SH_FSI_FMT_TDM_DELAY (1 << 5) +#define SH_FSI_FMT_MONO 0 +#define SH_FSI_FMT_MONO_DELAY 1 +#define SH_FSI_FMT_PCM 2 +#define SH_FSI_FMT_I2S 3 +#define SH_FSI_FMT_TDM 4 +#define SH_FSI_FMT_TDM_DELAY 5
#define SH_FSI_IFMT_TDM_CH(x) \ (SH_FSI_IFMT(TDM) | SH_FSI_SET_CH_I(x))
On Tue, 2010-07-13 at 18:13 +0900, Kuninori Morimoto wrote:
There is no necessity that each bit in this area has the meaning. This patch modify it to sequence number
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com
v2 -> v3
o don't use << 0 shift
2/6 - 6/6 doesn't depend on 1/6 patch
All
Acked-by: Liam Girdwood lrg@slimlogic.co.uk
On Tue, Jul 13, 2010 at 11:50:38AM +0100, Liam Girdwood wrote:
All
Acked-by: Liam Girdwood lrg@slimlogic.co.uk
Applied, thanks.
participants (3)
-
Kuninori Morimoto
-
Liam Girdwood
-
Mark Brown