From: Ben Dooks <ben-linux@fluff.org>
To: Jassi Brar <jassi.brar@samsung.com>
Cc: alsa-devel@alsa-project.org, broonie@opensource.wolfsonmicro.com,
ben-linux@fluff.org
Subject: Re: [PATCH 10/20] ASoC: S3C: I2Sv2: Move register definitions closer to driver
Date: Tue, 27 Apr 2010 03:55:16 +0100 [thread overview]
Message-ID: <20100427025516.GA24020@trinity.fluff.org> (raw)
In-Reply-To: <1268207349-9979-10-git-send-email-jassi.brar@samsung.com>
On Wed, Mar 10, 2010 at 04:48:59PM +0900, Jassi Brar wrote:
> The header for I2Sv2 (linux/arch/arm/plat-s3c/include/plat/regs-s3c2412-iis.h)
> contains only controller specific definitions and nothing SoC specific. So, it
> could be moved to sound/soc/s3c24xx/
> That will also help avoid delays in accepting patches that touch ASOC and ARCH
> parts.
>
> For now, we simply copy to regs-i2s-v2.h without any material changes, and render
> the regs-s3c2412-iis.h useless in the kernel.
> A separate patch will be submitted to S3C ARCH tree to remove the regs-s3c2412-iis.h
could you please reformat the header to wrap nicely?
I'd ack a patch moving this in one go since there shouldn't be any
updates to this file going via my tree.
> Signed-off-by: Jassi Brar <jassi.brar@samsung.com>
> ---
> sound/soc/s3c24xx/regs-i2s-v2.h | 79 +++++++++++++++++++++++++++++++++++++++
> sound/soc/s3c24xx/s3c-i2s-v2.c | 3 +-
> sound/soc/s3c24xx/s3c2412-i2s.c | 3 +-
> sound/soc/s3c24xx/s3c64xx-i2s.c | 2 +-
> 4 files changed, 82 insertions(+), 5 deletions(-)
> create mode 100644 sound/soc/s3c24xx/regs-i2s-v2.h
>
> diff --git a/sound/soc/s3c24xx/regs-i2s-v2.h b/sound/soc/s3c24xx/regs-i2s-v2.h
> new file mode 100644
> index 0000000..8e1a585
> --- /dev/null
> +++ b/sound/soc/s3c24xx/regs-i2s-v2.h
> @@ -0,0 +1,79 @@
> +/* linux/sound/soc/s3c24xx/regs-i2s-v2.h
> + *
> + * Copyright 2007 Simtec Electronics <linux@simtec.co.uk>
> + * http://armlinux.simtec.co.uk/
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + * Register definitions for controllers that share the I2Sv2 part
> +*/
> +
> +#ifndef __REGS_IISV2_H
> +#define __REGS_IISV2_H
> +
> +#define S3C2412_IISCON (0x00)
> +#define S3C2412_IISMOD (0x04)
> +#define S3C2412_IISFIC (0x08)
> +#define S3C2412_IISPSR (0x0C)
> +#define S3C2412_IISTXD (0x10)
> +#define S3C2412_IISRXD (0x14)
> +
> +#define S3C2412_IISCON_LRINDEX (1 << 11)
> +#define S3C2412_IISCON_TXFIFO_EMPTY (1 << 10)
> +#define S3C2412_IISCON_RXFIFO_EMPTY (1 << 9)
> +#define S3C2412_IISCON_TXFIFO_FULL (1 << 8)
> +#define S3C2412_IISCON_RXFIFO_FULL (1 << 7)
> +#define S3C2412_IISCON_TXDMA_PAUSE (1 << 6)
> +#define S3C2412_IISCON_RXDMA_PAUSE (1 << 5)
> +#define S3C2412_IISCON_TXCH_PAUSE (1 << 4)
> +#define S3C2412_IISCON_RXCH_PAUSE (1 << 3)
> +#define S3C2412_IISCON_TXDMA_ACTIVE (1 << 2)
> +#define S3C2412_IISCON_RXDMA_ACTIVE (1 << 1)
> +#define S3C2412_IISCON_IIS_ACTIVE (1 << 0)
> +
> +#define S3C64XX_IISMOD_BLC_16BIT (0 << 13)
> +#define S3C64XX_IISMOD_BLC_8BIT (1 << 13)
> +#define S3C64XX_IISMOD_BLC_24BIT (2 << 13)
> +#define S3C64XX_IISMOD_BLC_MASK (3 << 13)
> +
> +#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_MODE_TXONLY (0 << 8)
> +#define S3C2412_IISMOD_MODE_RXONLY (1 << 8)
> +#define S3C2412_IISMOD_MODE_TXRX (2 << 8)
> +#define S3C2412_IISMOD_MODE_MASK (3 << 8)
> +#define S3C2412_IISMOD_LR_LLOW (0 << 7)
> +#define S3C2412_IISMOD_LR_RLOW (1 << 7)
> +#define S3C2412_IISMOD_SDF_IIS (0 << 5)
> +#define S3C2412_IISMOD_SDF_MSB (1 << 5)
> +#define S3C2412_IISMOD_SDF_LSB (2 << 5)
> +#define S3C2412_IISMOD_SDF_MASK (3 << 5)
> +#define S3C2412_IISMOD_RCLK_256FS (0 << 3)
> +#define S3C2412_IISMOD_RCLK_512FS (1 << 3)
> +#define S3C2412_IISMOD_RCLK_384FS (2 << 3)
> +#define S3C2412_IISMOD_RCLK_768FS (3 << 3)
> +#define S3C2412_IISMOD_RCLK_MASK (3 << 3)
> +#define S3C2412_IISMOD_BCLK_32FS (0 << 1)
> +#define S3C2412_IISMOD_BCLK_48FS (1 << 1)
> +#define S3C2412_IISMOD_BCLK_16FS (2 << 1)
> +#define S3C2412_IISMOD_BCLK_24FS (3 << 1)
> +#define S3C2412_IISMOD_BCLK_MASK (3 << 1)
> +#define S3C2412_IISMOD_8BIT (1 << 0)
> +
> +#define S3C64XX_IISMOD_CDCLKCON (1 << 12)
> +
> +#define S3C2412_IISPSR_PSREN (1 << 15)
> +
> +#define S3C2412_IISFIC_TXFLUSH (1 << 15)
> +#define S3C2412_IISFIC_RXFLUSH (1 << 7)
> +#define S3C2412_IISFIC_TXCOUNT(x) (((x) >> 8) & 0xf)
> +#define S3C2412_IISFIC_RXCOUNT(x) (((x) >> 0) & 0xf)
> +
> +#endif /* __REGS_IISV2_H */
> diff --git a/sound/soc/s3c24xx/s3c-i2s-v2.c b/sound/soc/s3c24xx/s3c-i2s-v2.c
> index 0d655ab..b690078 100644
> --- a/sound/soc/s3c24xx/s3c-i2s-v2.c
> +++ b/sound/soc/s3c24xx/s3c-i2s-v2.c
> @@ -24,10 +24,9 @@
> #include <sound/pcm_params.h>
> #include <sound/soc.h>
>
> -#include <plat/regs-s3c2412-iis.h>
> -
> #include <mach/dma.h>
>
> +#include "regs-i2s-v2.h"
> #include "s3c-i2s-v2.h"
> #include "s3c-dma.h"
>
> diff --git a/sound/soc/s3c24xx/s3c2412-i2s.c b/sound/soc/s3c24xx/s3c2412-i2s.c
> index a5b21f6..42fb663 100644
> --- a/sound/soc/s3c24xx/s3c2412-i2s.c
> +++ b/sound/soc/s3c24xx/s3c2412-i2s.c
> @@ -32,12 +32,11 @@
> #include <sound/soc.h>
> #include <mach/hardware.h>
>
> -#include <plat/regs-s3c2412-iis.h>
> -
> #include <mach/regs-gpio.h>
> #include <mach/dma.h>
>
> #include "s3c-dma.h"
> +#include "regs-i2s-v2.h"
> #include "s3c2412-i2s.h"
>
> #define S3C2412_I2S_DEBUG 0
> diff --git a/sound/soc/s3c24xx/s3c64xx-i2s.c b/sound/soc/s3c24xx/s3c64xx-i2s.c
> index 6552894..9bcc99d 100644
> --- a/sound/soc/s3c24xx/s3c64xx-i2s.c
> +++ b/sound/soc/s3c24xx/s3c64xx-i2s.c
> @@ -18,7 +18,6 @@
>
> #include <sound/soc.h>
>
> -#include <plat/regs-s3c2412-iis.h>
> #include <plat/gpio-bank-d.h>
> #include <plat/gpio-bank-e.h>
> #include <plat/gpio-cfg.h>
> @@ -27,6 +26,7 @@
> #include <mach/dma.h>
>
> #include "s3c-dma.h"
> +#include "regs-i2s-v2.h"
> #include "s3c64xx-i2s.h"
>
> /* The value should be set to maximum of the total number
> --
> 1.6.2.5
>
--
--
Ben
Q: What's a light-year?
A: One-third less calories than a regular year.
next prev parent reply other threads:[~2010-04-27 2:55 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-10 7:48 [PATCH 01/20] ASoC: S3C: I2Sv2: Add missing semicolon Jassi Brar
2010-03-10 7:48 ` [PATCH 02/20] ASoC: S3C: I2Sv2: Reject immidiate register value Jassi Brar
2010-03-10 7:48 ` [PATCH 03/20] ASoC: S3C64XX: I2S: Make BCLK independent of sample size Jassi Brar
2010-03-10 7:48 ` [PATCH 04/20] ASoC: S3C: I2Sv2: Remove S3C_IIS_V2_SUPPORTED define Jassi Brar
2010-03-10 7:48 ` [PATCH 05/20] ASoC: S3C: I2Sv2: Unify clock source IDs Jassi Brar
2010-03-10 7:48 ` [PATCH 06/20] ASoC: s3c-i2s-v2 remove unnecessary headers Jassi Brar
2010-03-10 7:48 ` [PATCH 07/20] ASoC: s3c64xx-i2s remove unncessary headers Jassi Brar
2010-03-10 7:48 ` [PATCH 08/20] ASoC: S3C64XX: I2S: Move RATE and FMT defines to header Jassi Brar
2010-03-10 7:48 ` [PATCH 09/20] ASoC: S3C: I2Sv2: Segregate hw_params callback Jassi Brar
2010-03-10 7:48 ` [PATCH 10/20] ASoC: S3C: I2Sv2: Move register definitions closer to driver Jassi Brar
2010-03-10 7:49 ` [PATCH 11/20] ASoC: SAMSUNG: I2S: Add bit definitions Jassi Brar
2010-03-10 7:49 ` [PATCH 12/20] ASoC: S3C2412: I2S: Debug IMS field Jassi Brar
2010-03-10 7:49 ` [PATCH 13/20] ASoC: S3C2412: I2S: Return correct source clock Jassi Brar
2010-03-10 7:49 ` [PATCH 14/20] ASoC: S3C: I2Sv2: Discard redundant field iis_clk Jassi Brar
2010-03-10 7:49 ` [PATCH 15/20] ASoC: S3C: I2Sv2: Unify i2s_get_clock callback Jassi Brar
2010-03-10 7:49 ` [PATCH 16/20] ASoC: S3C64XX: I2S: Use s3c2412 defines Jassi Brar
2010-03-10 7:49 ` [PATCH 17/20] ASoC: S3C: I2Sv2: Define new field for controller features Jassi Brar
2010-03-10 7:49 ` [PATCH 18/20] ASoC: S3C: I2S: Move set_sysclk callback to common code Jassi Brar
2010-03-10 7:49 ` [PATCH 19/20] ASoC: S3C64XX: IISv4: Add CPU driver Jassi Brar
2010-03-10 7:49 ` [PATCH 20/20] ASoC: SMDK64XX: Switch to IISv4 " Jassi Brar
2010-03-10 11:40 ` Mark Brown
2010-03-10 13:05 ` jassi brar
2010-03-10 11:39 ` [PATCH 19/20] ASoC: S3C64XX: IISv4: Add " Mark Brown
2010-03-10 12:57 ` jassi brar
2010-03-10 11:09 ` [PATCH 17/20] ASoC: S3C: I2Sv2: Define new field for controller features Mark Brown
2010-03-10 11:11 ` Mark Brown
2010-03-10 11:00 ` [PATCH 11/20] ASoC: SAMSUNG: I2S: Add bit definitions Mark Brown
2010-03-10 12:24 ` jassi brar
2010-03-10 12:38 ` Mark Brown
2010-03-10 12:49 ` jassi brar
2010-03-10 12:56 ` Mark Brown
2010-03-10 13:03 ` jassi brar
2010-03-10 14:11 ` Mark Brown
2010-04-27 2:55 ` Ben Dooks [this message]
2010-03-10 14:14 ` [PATCH 09/20] ASoC: S3C: I2Sv2: Segregate hw_params callback Mark Brown
2010-03-10 18:46 ` Mark Brown
2010-03-10 10:51 ` [PATCH 08/20] ASoC: S3C64XX: I2S: Move RATE and FMT defines to header Mark Brown
2010-03-10 12:52 ` jassi brar
2010-03-10 12:59 ` Mark Brown
2010-03-10 13:01 ` [PATCH 07/20] ASoC: s3c64xx-i2s remove unncessary headers Mark Brown
2010-03-10 13:01 ` [PATCH 06/20] ASoC: s3c-i2s-v2 remove unnecessary headers Mark Brown
2010-03-10 13:01 ` [PATCH 05/20] ASoC: S3C: I2Sv2: Unify clock source IDs Mark Brown
2010-03-10 10:49 ` [PATCH 04/20] ASoC: S3C: I2Sv2: Remove S3C_IIS_V2_SUPPORTED define Mark Brown
2010-03-10 12:38 ` jassi brar
2010-03-10 12:51 ` Mark Brown
2010-03-10 12:31 ` [PATCH 03/20] ASoC: S3C64XX: I2S: Make BCLK independent of sample size Mark Brown
2010-03-10 12:46 ` jassi brar
2010-03-10 13:00 ` Mark Brown
2010-03-10 10:32 ` [PATCH 02/20] ASoC: S3C: I2Sv2: Reject immidiate register value Mark Brown
2010-03-10 12:27 ` jassi brar
2010-03-10 12:48 ` Mark Brown
2010-03-10 12:32 ` [PATCH 01/20] ASoC: S3C: I2Sv2: Add missing semicolon Mark Brown
2010-03-10 13:00 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100427025516.GA24020@trinity.fluff.org \
--to=ben-linux@fluff.org \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=jassi.brar@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).