ASoC: S3C2412: I2S: Debug IMS field
authorJassi Brar <jassi.brar@samsung.com>
Tue, 27 Apr 2010 06:56:27 +0000 (15:56 +0900)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Wed, 5 May 2010 14:11:29 +0000 (15:11 +0100)
The IMS field of s3c2412/13 is essentially the same as that of s3c64xx.
That is, the IISMOD[11] bit decides Master/Slave mode and IISMOD[10] bit
selects source clock for signal generation.
For that reason, remove improper defines for IISMOD[11:10] field mask
and define two 1bit fields that can be set independent of each other.
As a consequence, corresponding fields for PLAT_S3C64XX too get to use
these new defines.

Signed-off-by: Jassi Brar <jassi.brar@samsung.com>
Acked-by: Ben Dooks <ben-linux@fluff.org>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
sound/soc/s3c24xx/regs-i2s-v2.h
sound/soc/s3c24xx/s3c-i2s-v2.c
sound/soc/s3c24xx/s3c2412-i2s.c

index f3d90289e7be6dc3fe08e36a4f10029df1b503f6..484858a5f33b57c032069997d8e6c9a4a68ff6fa 100644 (file)
 #define S3C64XX_IISMOD_IMS_PCLK                (0 << 10)
 #define S3C64XX_IISMOD_IMS_SYSMUX      (1 << 10)
 
-#define S3C2412_IISMOD_MASTER_INTERNAL (0 << 10)
-#define S3C2412_IISMOD_MASTER_EXTERNAL (1 << 10)
-#define S3C2412_IISMOD_SLAVE           (2 << 10)
-#define S3C2412_IISMOD_MASTER_MASK     (3 << 10)
+#define S3C2412_IISMOD_IMS_SYSMUX      (1 << 10)
+#define S3C2412_IISMOD_SLAVE           (1 << 11)
 #define S3C2412_IISMOD_MODE_TXONLY     (0 << 8)
 #define S3C2412_IISMOD_MODE_RXONLY     (1 << 8)
 #define S3C2412_IISMOD_MODE_TXRX       (2 << 8)
index cb3652ed057d6aad47ca37237bcbb23e13a50b0b..b8803a182cc1f7ec76054499afeec62acc3850a1 100644 (file)
@@ -265,35 +265,14 @@ static int s3c2412_i2s_set_fmt(struct snd_soc_dai *cpu_dai,
        iismod = readl(i2s->regs + S3C2412_IISMOD);
        pr_debug("hw_params r: IISMOD: %x \n", iismod);
 
-#if defined(CONFIG_CPU_S3C2412) || defined(CONFIG_CPU_S3C2413)
-#define IISMOD_MASTER_MASK S3C2412_IISMOD_MASTER_MASK
-#define IISMOD_SLAVE S3C2412_IISMOD_SLAVE
-#define IISMOD_MASTER S3C2412_IISMOD_MASTER_INTERNAL
-#endif
-
-#if defined(CONFIG_PLAT_S3C64XX)
-/* From Rev1.1 datasheet, we have two master and two slave modes:
- * IMS[11:10]:
- *     00 = master mode, fed from PCLK
- *     01 = master mode, fed from CLKAUDIO
- *     10 = slave mode, using PCLK
- *     11 = slave mode, using I2SCLK
- */
-#define IISMOD_MASTER_MASK (1 << 11)
-#define IISMOD_SLAVE (1 << 11)
-#define IISMOD_MASTER (0 << 11)
-#endif
-
        switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) {
        case SND_SOC_DAIFMT_CBM_CFM:
                i2s->master = 0;
-               iismod &= ~IISMOD_MASTER_MASK;
-               iismod |= IISMOD_SLAVE;
+               iismod |= S3C2412_IISMOD_SLAVE;
                break;
        case SND_SOC_DAIFMT_CBS_CFS:
                i2s->master = 1;
-               iismod &= ~IISMOD_MASTER_MASK;
-               iismod |= IISMOD_MASTER;
+               iismod &= ~S3C2412_IISMOD_SLAVE;
                break;
        default:
                pr_err("unknwon master/slave format\n");
index 7134465693e3023b442bfd9521e3c92a4d8abb92..cf0c15482c34f087ffb04bc935505845a10d51c6 100644 (file)
@@ -78,14 +78,10 @@ static int s3c2412_i2s_set_sysclk(struct snd_soc_dai *cpu_dai,
 
        switch (clk_id) {
        case S3C2412_CLKSRC_PCLK:
-               s3c2412_i2s.master = 1;
-               iismod &= ~S3C2412_IISMOD_MASTER_MASK;
-               iismod |= S3C2412_IISMOD_MASTER_INTERNAL;
+               iismod &= ~S3C2412_IISMOD_IMS_SYSMUX;
                break;
        case S3C2412_CLKSRC_I2SCLK:
-               s3c2412_i2s.master = 0;
-               iismod &= ~S3C2412_IISMOD_MASTER_MASK;
-               iismod |= S3C2412_IISMOD_MASTER_EXTERNAL;
+               iismod |= S3C2412_IISMOD_IMS_SYSMUX;
                break;
        default:
                return -EINVAL;