On 7/1/08, Grant Likely <grant.likely@xxxxxxxxxxxx> wrote: > From: Grant Likely <grant.likely@xxxxxxxxxxxx> > > ASoC Codec driver for the TLV320AIC26 device. This driver uses the ASoC > v1 API, so I don't expect it to get merged as-is, but I want to get it > out there for review. > --- > > sound/soc/codecs/Kconfig | 4 > sound/soc/codecs/Makefile | 2 > sound/soc/codecs/tlv320aic26.c | 630 ++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 636 insertions(+), 0 deletions(-) > > diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig > index 3903ab7..96c7bfe 100644 > --- a/sound/soc/codecs/Kconfig > +++ b/sound/soc/codecs/Kconfig > @@ -41,6 +41,10 @@ config SND_SOC_CS4270_VD33_ERRATA > bool > depends on SND_SOC_CS4270 > > +config SND_SOC_TLV320AIC26 > + tristate "TI TLB320AIC26 Codec support" > + depends on SND_SOC && SPI > + > config SND_SOC_TLV320AIC3X > tristate > depends on SND_SOC && I2C > diff --git a/sound/soc/codecs/Makefile b/sound/soc/codecs/Makefile > index 4e1314c..ec0cd93 100644 > --- a/sound/soc/codecs/Makefile > +++ b/sound/soc/codecs/Makefile > @@ -5,6 +5,7 @@ snd-soc-wm8753-objs := wm8753.o > snd-soc-wm9712-objs := wm9712.o > snd-soc-wm9713-objs := wm9713.o > snd-soc-cs4270-objs := cs4270.o > +snd-soc-tlv320aic26-objs := tlv320aic26.o > snd-soc-tlv320aic3x-objs := tlv320aic3x.o > > obj-$(CONFIG_SND_SOC_AC97_CODEC) += snd-soc-ac97.o > @@ -14,4 +15,5 @@ obj-$(CONFIG_SND_SOC_WM8753) += snd-soc-wm8753.o > obj-$(CONFIG_SND_SOC_WM9712) += snd-soc-wm9712.o > obj-$(CONFIG_SND_SOC_WM9713) += snd-soc-wm9713.o > obj-$(CONFIG_SND_SOC_CS4270) += snd-soc-cs4270.o > +obj-$(CONFIG_SND_SOC_TLV320AIC26) += snd-soc-tlv320aic26.o > obj-$(CONFIG_SND_SOC_TLV320AIC3X) += snd-soc-tlv320aic3x.o > diff --git a/sound/soc/codecs/tlv320aic26.c b/sound/soc/codecs/tlv320aic26.c > new file mode 100644 > index 0000000..aee1dbc > --- /dev/null > +++ b/sound/soc/codecs/tlv320aic26.c > @@ -0,0 +1,630 @@ > +/* > + * Texas Instruments TLV320AIC26 low power audio CODEC > + * ALSA SoC CODEC driver > + * > + * Copyright (C) 2008 Secret Lab Technologies Ltd. > + */ > + > +#include <linux/module.h> > +#include <linux/moduleparam.h> > +#include <linux/init.h> > +#include <linux/delay.h> > +#include <linux/pm.h> > +#include <linux/device.h> > +#include <linux/sysfs.h> > +#include <linux/spi/spi.h> > +#include <sound/core.h> > +#include <sound/pcm.h> > +#include <sound/pcm_params.h> > +#include <sound/soc.h> > +#include <sound/soc-dapm.h> > +#include <sound/soc-of.h> > +#include <sound/initval.h> > + > +MODULE_DESCRIPTION("ASoC TLV320AIC26 codec driver"); > +MODULE_AUTHOR("Grant Likely <grant.likely@xxxxxxxxxxxx>"); > +MODULE_LICENSE("GPL"); > + > +/* AIC26 Registers */ > +#define AIC26_READ_COMMAND_WORD(addr) ((1 << 15) | (addr << 5)) > +#define AIC26_WRITE_COMMAND_WORD(addr) ((0 << 15) | (addr << 5)) > +#define AIC26_PAGE_ADDR(page, offset) ((page << 6) | offset) > +#define AIC26_NUM_REGS AIC26_PAGE_ADDR(3, 0) > +#define AIC26_REG_CACHE_SIZE (0x20) /* only page 2 cached */ > +#define AIC26_REG_IS_CACHED(addr) ((addr & ~0x1f) == (2 << 6)) > +#define AIC26_REG_CACHE_ADDR(addr) (addr & 0x1f) > + > +/* Page 0: Auxillary data registers */ > +#define AIC26_REG_BAT1 AIC26_PAGE_ADDR(0, 0x05) > +#define AIC26_REG_BAT2 AIC26_PAGE_ADDR(0, 0x06) > +#define AIC26_REG_AUX AIC26_PAGE_ADDR(0, 0x07) > +#define AIC26_REG_TEMP1 AIC26_PAGE_ADDR(0, 0x09) > +#define AIC26_REG_TEMP2 AIC26_PAGE_ADDR(0, 0x0A) > + > +/* Page 1: Auxillary control registers */ > +#define AIC26_REG_AUX_ADC AIC26_PAGE_ADDR(1, 0x00) > +#define AIC26_REG_STATUS AIC26_PAGE_ADDR(1, 0x01) > +#define AIC26_REG_REFERENCE AIC26_PAGE_ADDR(1, 0x03) > +#define AIC26_REG_RESET AIC26_PAGE_ADDR(1, 0x04) > + > +/* Page 2: Audio control registers */ > +#define AIC26_REG_AUDIO_CTRL1 AIC26_PAGE_ADDR(2, 0x00) > +#define AIC26_REG_ADC_GAIN AIC26_PAGE_ADDR(2, 0x01) > +#define AIC26_REG_DAC_GAIN AIC26_PAGE_ADDR(2, 0x02) > +#define AIC26_REG_SIDETONE AIC26_PAGE_ADDR(2, 0x03) > +#define AIC26_REG_AUDIO_CTRL2 AIC26_PAGE_ADDR(2, 0x04) > +#define AIC26_REG_POWER_CTRL AIC26_PAGE_ADDR(2, 0x05) > +#define AIC26_REG_AUDIO_CTRL3 AIC26_PAGE_ADDR(2, 0x06) > + > +#define AIC26_REG_FILTER_COEFF_L_N0 AIC26_PAGE_ADDR(2, 0x07) > +#define AIC26_REG_FILTER_COEFF_L_N1 AIC26_PAGE_ADDR(2, 0x08) > +#define AIC26_REG_FILTER_COEFF_L_N2 AIC26_PAGE_ADDR(2, 0x09) > +#define AIC26_REG_FILTER_COEFF_L_N3 AIC26_PAGE_ADDR(2, 0x0A) > +#define AIC26_REG_FILTER_COEFF_L_N4 AIC26_PAGE_ADDR(2, 0x0B) > +#define AIC26_REG_FILTER_COEFF_L_N5 AIC26_PAGE_ADDR(2, 0x0C) > +#define AIC26_REG_FILTER_COEFF_L_D1 AIC26_PAGE_ADDR(2, 0x0D) > +#define AIC26_REG_FILTER_COEFF_L_D2 AIC26_PAGE_ADDR(2, 0x0E) > +#define AIC26_REG_FILTER_COEFF_L_D4 AIC26_PAGE_ADDR(2, 0x0F) > +#define AIC26_REG_FILTER_COEFF_L_D5 AIC26_PAGE_ADDR(2, 0x10) > +#define AIC26_REG_FILTER_COEFF_R_N0 AIC26_PAGE_ADDR(2, 0x11) > +#define AIC26_REG_FILTER_COEFF_R_N1 AIC26_PAGE_ADDR(2, 0x12) > +#define AIC26_REG_FILTER_COEFF_R_N2 AIC26_PAGE_ADDR(2, 0x13) > +#define AIC26_REG_FILTER_COEFF_R_N3 AIC26_PAGE_ADDR(2, 0x14) > +#define AIC26_REG_FILTER_COEFF_R_N4 AIC26_PAGE_ADDR(2, 0x15) > +#define AIC26_REG_FILTER_COEFF_R_N5 AIC26_PAGE_ADDR(2, 0x16) > +#define AIC26_REG_FILTER_COEFF_R_D1 AIC26_PAGE_ADDR(2, 0x17) > +#define AIC26_REG_FILTER_COEFF_R_D2 AIC26_PAGE_ADDR(2, 0x18) > +#define AIC26_REG_FILTER_COEFF_R_D4 AIC26_PAGE_ADDR(2, 0x19) > +#define AIC26_REG_FILTER_COEFF_R_D5 AIC26_PAGE_ADDR(2, 0x1A) > + > +#define AIC26_REG_PLL_PROG1 AIC26_PAGE_ADDR(2, 0x1B) > +#define AIC26_REG_PLL_PROG2 AIC26_PAGE_ADDR(2, 0x1C) > +#define AIC26_REG_AUDIO_CTRL4 AIC26_PAGE_ADDR(2, 0x1D) > +#define AIC26_REG_AUDIO_CTRL5 AIC26_PAGE_ADDR(2, 0x1E) > + > +#define AIC26_RATES (SNDRV_PCM_RATE_8000 | SNDRV_PCM_RATE_11025 |\ > + SNDRV_PCM_RATE_16000 | SNDRV_PCM_RATE_22050 |\ > + SNDRV_PCM_RATE_32000 | SNDRV_PCM_RATE_44100 |\ > + SNDRV_PCM_RATE_48000) > +#define AIC26_FORMATS (SNDRV_PCM_FMTBIT_S8 | SNDRV_PCM_FMTBIT_S16_BE |\ > + SNDRV_PCM_FMTBIT_S24_BE | SNDRV_PCM_FMTBIT_S32_BE) > + > +/* fsref dividers; used in register 'Audio Control 1' */ > +enum aic26_divisors { > + AIC26_DIV_1 = 0, > + AIC26_DIV_1_5 = 1, > + AIC26_DIV_2 = 2, > + AIC26_DIV_3 = 3, > + AIC26_DIV_4 = 4, > + AIC26_DIV_5 = 5, > + AIC26_DIV_5_5 = 6, > + AIC26_DIV_6 = 7, > +}; > + > +/* Digital data format */ > +enum aic26_datfm { > + AIC26_DATFM_I2S = 0 << 8, > + AIC26_DATFM_DSP = 1 << 8, > + AIC26_DATFM_RIGHTJ = 2 << 8, /* right justified */ > + AIC26_DATFM_LEFTJ = 3 << 8, /* left justified */ > +}; > + > +/* Sample word length in bits; used in register 'Audio Control 1' */ > +enum aic26_wlen { > + AIC26_WLEN_16 = 0 << 10, > + AIC26_WLEN_20 = 1 << 10, > + AIC26_WLEN_24 = 2 << 10, > + AIC26_WLEN_32 = 3 << 10, > +}; > + > +/* AIC26 driver private data */ > +struct aic26 { > + struct spi_device *spi; > + struct snd_soc_codec codec; > + u16 reg_cache[AIC26_REG_CACHE_SIZE]; /* shadow registers */ > + int master; > + int datfm; > + int mclk; > + > + /* Keyclick parameters */ > + int keyclick_amplitude; > + int keyclick_freq; > + int keyclick_len; > +}; > + > +/* --------------------------------------------------------------------- > + * Register access routines > + */ > +static unsigned int aic26_reg_read(struct snd_soc_codec *codec, > + unsigned int reg) > +{ > + struct aic26 *aic26 = codec->private_data; > + u16 *cache = codec->reg_cache; > + u16 cmd, value; > + u8 buffer[2]; > + int rc; > + > + if ((reg < 0) || (reg >= AIC26_NUM_REGS)) { > + WARN_ON_ONCE(1); > + return 0; > + } > + > + /* Do SPI transfer; first 16bits are command; remaining is > + * register contents */ > + cmd = AIC26_READ_COMMAND_WORD(reg); > + buffer[0] = (cmd >> 8) & 0xff; > + buffer[1] = cmd & 0xff; > + rc = spi_write_then_read(aic26->spi, buffer, 2, buffer, 2); > + if (rc) { > + dev_err(&aic26->spi->dev, "AIC26 reg read error\n"); > + return -EIO; > + } > + value = (buffer[0] << 8) | buffer[1]; > + > + /* Update the cache before returning with the value */ > + if (AIC26_REG_IS_CACHED(reg)) > + cache[AIC26_REG_CACHE_ADDR(reg)] = value; > + return value; > +} > + > +static unsigned int aic26_reg_read_cache(struct snd_soc_codec *codec, > + unsigned int reg) > +{ > + u16 *cache = codec->reg_cache; > + > + if ((reg < 0) || (reg >= AIC26_NUM_REGS)) { > + WARN_ON_ONCE(1); > + return 0; > + } > + > + if (AIC26_REG_IS_CACHED(reg)) > + return cache[AIC26_REG_CACHE_ADDR(reg)]; > + > + return aic26_reg_read(codec, reg); > +} > + > +static int aic26_reg_write(struct snd_soc_codec *codec, unsigned int reg, > + unsigned int value) > +{ > + struct aic26 *aic26 = codec->private_data; > + u16 *cache = codec->reg_cache; > + u16 cmd; > + u8 buffer[4]; > + int rc; > + > + if ((reg < 0) || (reg >= AIC26_NUM_REGS)) { > + WARN_ON_ONCE(1); > + return -EINVAL; > + } > + > + /* Do SPI transfer; first 16bits are command; remaining is data > + * to write into register */ > + cmd = AIC26_WRITE_COMMAND_WORD(reg); > + buffer[0] = (cmd >> 8) & 0xff; > + buffer[1] = cmd & 0xff; > + buffer[2] = value >> 8; > + buffer[3] = value; > + rc = spi_write(aic26->spi, buffer, 4); > + if (rc) { > + dev_err(&aic26->spi->dev, "AIC26 reg read error\n"); > + return -EIO; > + } > + > + /* update cache before returning */ > + if (AIC26_REG_IS_CACHED(reg)) > + cache[AIC26_REG_CACHE_ADDR(reg)] = value; > + return 0; > +} > + > +/* --------------------------------------------------------------------- > + * Digital Audio Interface Operations > + */ > +static int aic26_hw_params(struct snd_pcm_substream *substream, > + struct snd_pcm_hw_params *params) > +{ > + struct snd_soc_pcm_runtime *rtd = substream->private_data; > + struct snd_soc_device *socdev = rtd->socdev; > + struct snd_soc_codec *codec = socdev->codec; > + struct aic26 *aic26 = codec->private_data; > + int fsref, divisor, wlen, pval, jval, dval, qval; > + u16 reg; > + > + dev_dbg(&aic26->spi->dev, "aic26_hw_params(substream=%p, params=%p)\n", > + substream, params); > + dev_dbg(&aic26->spi->dev, "rate=%i format=%i\n", params_rate(params), > + params_format(params)); > + > + switch (params_rate(params)) { > + case 8000: fsref = 48000; divisor = AIC26_DIV_6; break; > + case 11025: fsref = 44100; divisor = AIC26_DIV_4; break; > + case 12000: fsref = 48000; divisor = AIC26_DIV_4; break; > + case 16000: fsref = 48000; divisor = AIC26_DIV_3; break; > + case 22050: fsref = 44100; divisor = AIC26_DIV_2; break; > + case 24000: fsref = 48000; divisor = AIC26_DIV_2; break; > + case 32000: fsref = 48000; divisor = AIC26_DIV_1_5; break; > + case 44100: fsref = 44100; divisor = AIC26_DIV_1; break; > + case 48000: fsref = 48000; divisor = AIC26_DIV_1; break; > + default: dev_dbg(&aic26->spi->dev, "bad rate\n"); return -EINVAL; > + } > + > + /* select data word length */ > + switch (params_format(params)) { > + case SNDRV_PCM_FORMAT_S8: wlen = AIC26_WLEN_16; break; > + case SNDRV_PCM_FORMAT_S16_BE: wlen = AIC26_WLEN_16; break; > + case SNDRV_PCM_FORMAT_S24_BE: wlen = AIC26_WLEN_24; break; > + case SNDRV_PCM_FORMAT_S32_BE: wlen = AIC26_WLEN_32; break; > + default: dev_dbg(&aic26->spi->dev, "bad format\n"); return -EINVAL; > + } > + > + /* Configure PLL */ > + pval = 1; > + jval = (fsref == 44100) ? 7 : 8; > + dval = (fsref == 44100) ? 5264 : 1920; > + qval = 0; > + reg = 0x8000 | qval << 11 | pval << 8 | jval << 2; > + aic26_reg_write(codec, AIC26_REG_PLL_PROG1, reg); > + reg = dval << 2; > + aic26_reg_write(codec, AIC26_REG_PLL_PROG2, reg); > + > + /* Power up CODEC */ > + aic26_reg_write(codec, AIC26_REG_POWER_CTRL, 0); > + > + /* Audio Control 3 (master mode, fsref rate) */ > + reg = aic26_reg_read_cache(codec, AIC26_REG_AUDIO_CTRL3); > + reg &= ~0xf800; > + if (aic26->master) > + reg |= 0x0800; > + if (fsref == 48000) > + reg |= 0x2000; > + aic26_reg_write(codec, AIC26_REG_AUDIO_CTRL3, reg); > + > + /* Audio Control 1 (FSref divisor) */ > + reg = aic26_reg_read_cache(codec, AIC26_REG_AUDIO_CTRL1); > + reg &= ~0x0fff; > + reg |= wlen | aic26->datfm | (divisor << 3) | divisor; > + aic26_reg_write(codec, AIC26_REG_AUDIO_CTRL1, reg); > + > + return 0; > +} > + > +/** > + * aic26_mute - Mute control to reduce noise when changing audio format > + */ > +static int aic26_mute(struct snd_soc_codec_dai *dai, int mute) > +{ > + struct snd_soc_codec *codec = dai->codec; > + struct aic26 *aic26 = codec->private_data; > + u16 reg = aic26_reg_read_cache(codec, AIC26_REG_DAC_GAIN); > + > + dev_dbg(&aic26->spi->dev, "aic26_mute(dai=%p, mute=%i)\n", > + dai, mute); > + > + if (mute) > + reg |= 0x8080; > + else > + reg &= ~0x8080; > + aic26_reg_write(codec, AIC26_REG_DAC_GAIN, reg); > + > + return 0; > +} > + > +static int aic26_set_sysclk(struct snd_soc_codec_dai *codec_dai, > + int clk_id, unsigned int freq, int dir) > +{ > + struct snd_soc_codec *codec = codec_dai->codec; > + struct aic26 *aic26 = codec->private_data; > + > + dev_dbg(&aic26->spi->dev, "aic26_set_sysclk(dai=%p, clk_id==%i," > + " freq=%i, dir=%i)\n", > + codec_dai, clk_id, freq, dir); > + > + /* MCLK needs to fall between 2MHz and 50 MHz */ > + if ((freq < 2000000) || (freq > 50000000)) > + return -EINVAL; > + > + aic26->mclk = freq; > + return 0; > +} > + > +static int aic26_set_fmt(struct snd_soc_codec_dai *codec_dai, unsigned int fmt) > +{ > + struct snd_soc_codec *codec = codec_dai->codec; > + struct aic26 *aic26 = codec->private_data; > + > + dev_dbg(&aic26->spi->dev, "aic26_set_fmt(dai=%p, fmt==%i)\n", > + codec_dai, fmt); > + > + /* set master/slave audio interface */ > + switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { > + case SND_SOC_DAIFMT_CBM_CFM: aic26->master = 1; break; > + //case SND_SOC_DAIFMT_CBS_CFS: aic26->master = 0; break; > + default: dev_dbg(&aic26->spi->dev, "bad master\n"); return -EINVAL; > + } > + > + /* interface format */ > + switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) { > + case SND_SOC_DAIFMT_I2S: aic26->datfm = AIC26_DATFM_I2S; break; > + case SND_SOC_DAIFMT_DSP_A: aic26->datfm = AIC26_DATFM_DSP; break; > + case SND_SOC_DAIFMT_RIGHT_J: aic26->datfm = AIC26_DATFM_RIGHTJ; break; > + case SND_SOC_DAIFMT_LEFT_J: aic26->datfm = AIC26_DATFM_LEFTJ; break; > + default: dev_dbg(&aic26->spi->dev, "bad format\n"); return -EINVAL; > + } > + > + return 0; > +} > + > +/* --------------------------------------------------------------------- > + * Digital Audio Interface Definition > + */ > +struct snd_soc_codec_dai aic26_dai = { > + .name = "tlv320aic26", > + .playback = { > + .stream_name = "Playback", > + .channels_min = 2, > + .channels_max = 2, > + .rates = AIC26_RATES, > + .formats = AIC26_FORMATS, > + }, > + .capture = { > + .stream_name = "Capture", > + .channels_min = 2, > + .channels_max = 2, > + .rates = AIC26_RATES, > + .formats = AIC26_FORMATS, > + }, > + .ops = { > + .hw_params = aic26_hw_params, > + }, > + .dai_ops = { > + .digital_mute = aic26_mute, > + .set_sysclk = aic26_set_sysclk, > + .set_fmt = aic26_set_fmt, > + }, > +}; > +EXPORT_SYMBOL_GPL(aic26_dai); > + > +/* --------------------------------------------------------------------- > + * ALSA controls > + */ > +static const char *aic26_capture_src_text[] = {"Mic", "Aux"}; > +static const struct soc_enum aic26_capture_src_enum = > + SOC_ENUM_SINGLE(AIC26_REG_AUDIO_CTRL1, 12,2, aic26_capture_src_text); > + > +static const struct snd_kcontrol_new aic26_snd_controls[] = { > + /* Output */ > + SOC_DOUBLE("PCM Playback Volume", AIC26_REG_DAC_GAIN, 8, 0, 0x7f, 1), > + SOC_DOUBLE("PCM Playback Switch", AIC26_REG_DAC_GAIN, 15, 7, 1, 1), > + SOC_SINGLE("PCM Capture Volume", AIC26_REG_ADC_GAIN, 8, 0x7f, 0), > + SOC_SINGLE("PCM Capture Mute", AIC26_REG_ADC_GAIN, 15, 1, 1), > + SOC_ENUM("Capture Source", aic26_capture_src_enum), > +}; > + > +/* --------------------------------------------------------------------- > + * SoC CODEC portion of driver: probe and release routines > + */ > +static int aic26_probe(struct platform_device *pdev) > +{ > + struct snd_soc_device *socdev = platform_get_drvdata(pdev); > + struct snd_soc_codec *codec; > + struct snd_kcontrol *kcontrol; > + struct aic26 *aic26; > + int i, ret, err; > + > + dev_info(&pdev->dev, "Probing AIC26 SoC CODEC driver\n"); > + dev_dbg(&pdev->dev, "socdev=%p\n", socdev); > + dev_dbg(&pdev->dev, "codec_data=%p\n", socdev->codec_data); > + > + /* Fetch the relevant aic26 private data here (it's already been > + * stored in the .codec pointer) */ > + aic26 = socdev->codec_data; > + if (aic26 == NULL) { > + dev_err(&pdev->dev, "aic26: missing codec pointer\n"); > + return -ENODEV; > + } > + codec = &aic26->codec; > + socdev->codec = codec; > + > + dev_dbg(&pdev->dev, "Registering PCMs, dev=%p, socdev->dev=%p\n", > + &pdev->dev, socdev->dev); > + /* register pcms */ > + ret = snd_soc_new_pcms(socdev, SNDRV_DEFAULT_IDX1, SNDRV_DEFAULT_STR1); > + if (ret < 0) { > + dev_err(&pdev->dev, "aic26: failed to create pcms\n"); > + return -ENODEV; > + } > + > + /* register controls */ > + dev_dbg(&pdev->dev, "Registering controls\n"); > + for (i = 0; i < ARRAY_SIZE(aic26_snd_controls); i++) { > + kcontrol = snd_soc_cnew(&aic26_snd_controls[i], codec, NULL); > + err = snd_ctl_add(codec->card, kcontrol); > + WARN_ON(err < 0); > + } > + > + /* CODEC is setup, we can register the card now */ > + dev_dbg(&pdev->dev, "Registering card\n"); > + ret = snd_soc_register_card(socdev); > + if (ret < 0) { > + dev_err(&pdev->dev, "aic26: failed to register card\n"); > + goto card_err; > + } > + return 0; > + > + card_err: > + snd_soc_free_pcms(socdev); > + return ret; > +} > + > +static int aic26_remove(struct platform_device *pdev) > +{ > + struct snd_soc_device *socdev = platform_get_drvdata(pdev); > + snd_soc_free_pcms(socdev); > + return 0; > +} > + > +struct snd_soc_codec_device aic26_soc_codec_dev = { > + .probe = aic26_probe, > + .remove = aic26_remove, > +}; > + > +/* --------------------------------------------------------------------- > + * SPI device portion of driver: sysfs files for debugging > + */ > + > +static ssize_t aic26_regs_show(struct device *dev, > + struct device_attribute *attr, char *buf) > +{ > + struct aic26 *aic26 = dev_get_drvdata(dev); > + char *idx = buf; > + int cache_flag, addr, page, i, reg; > + > + cache_flag = (strcmp(attr->attr.name, "regs_cache") == 0); > + > + for (page = 0; page < 3; page++) { > + for (i = 0; i < 0x20; i++) { > + addr = AIC26_PAGE_ADDR(page, i); > + if (i % 8 == 0) > + idx += sprintf(idx, "%i:%.2i:", page,i); > + if (cache_flag) > + reg = aic26_reg_read_cache(&aic26->codec, addr); > + else > + reg = aic26_reg_read(&aic26->codec, addr); > + idx += sprintf(idx, " %.4x", reg); > + if (i % 8 == 7) > + idx += sprintf(idx, "\n"); > + } > + } > + return idx - buf; > +} > + > +static ssize_t aic26_keyclick_show(struct device *dev, > + struct device_attribute *attr, char *buf) > +{ > + struct aic26 *aic26 = dev_get_drvdata(dev); > + int val, amp, freq, len; > + > + val = aic26_reg_read_cache(&aic26->codec, AIC26_REG_AUDIO_CTRL2); > + amp = (val >> 12) & 0x7; > + freq = (125 << ((val >> 8) & 0x7)) >> 1; > + len = 2 * (1 +((val >> 8) & 0xf)); > + > + return sprintf(buf, "amp=%x freq=%iHz len=%iclks\n", amp, freq, len); > +} > + > +/* Any write to the keyclick attribute will trigger the keyclick */ > +static ssize_t aic26_keyclick_set(struct device *dev, > + struct device_attribute *attr, > + const char *buf, size_t count) > +{ > + struct aic26 *aic26 = dev_get_drvdata(dev); > + int val; > + > + val = aic26_reg_read_cache(&aic26->codec, AIC26_REG_AUDIO_CTRL2); > + val |= 0x8000; > + aic26_reg_write(&aic26->codec, AIC26_REG_AUDIO_CTRL2, val); > + > + return count; > +} > + > +DEVICE_ATTR(regs, 0644, aic26_regs_show, NULL); > +DEVICE_ATTR(regs_cache, 0644, aic26_regs_show, NULL); > +DEVICE_ATTR(keyclick, 0644, aic26_keyclick_show, aic26_keyclick_set); > + > +/* --------------------------------------------------------------------- > + * SPI device portion of driver: probe and release routines and SPI > + * driver registration. > + */ > +static int aic26_spi_probe(struct spi_device *spi) > +{ > + struct aic26 *aic26; > + int rc, i, reg; > + > + dev_dbg(&spi->dev, "probing tlv320aic26 spi device\n"); > + > + /* Allocate driver data */ > + aic26 = kzalloc(sizeof *aic26, GFP_KERNEL); > + if (!aic26) > + return -ENOMEM; > + > + /* Initialize the driver data */ > + aic26->spi = spi; > + dev_set_drvdata(&spi->dev, aic26); > + > + /* Setup what we can in the codec structure so that the register > + * access functions will work as expected. More will be filled > + * out when it is probed by the SoC CODEC part of this driver */ > + aic26->codec.private_data = aic26; > + aic26->codec.name = "aic26"; > + aic26->codec.owner = THIS_MODULE; > + aic26->codec.dai = &aic26_dai; > + aic26->codec.num_dai = 1; > + aic26->codec.read = aic26_reg_read; > + aic26->codec.write = aic26_reg_write; > + aic26->master = 1; > + mutex_init(&aic26->codec.mutex); > + INIT_LIST_HEAD(&aic26->codec.dapm_widgets); > + INIT_LIST_HEAD(&aic26->codec.dapm_paths); > + aic26->codec.reg_cache_size = sizeof(aic26->reg_cache); > + aic26->codec.reg_cache = aic26->reg_cache; > + > + /* Reset the codec to power on defaults */ > + aic26_reg_write(&aic26->codec, AIC26_REG_RESET, 0xBB00); > + > + /* Power up CODEC */ > + aic26_reg_write(&aic26->codec, AIC26_REG_POWER_CTRL, 0); > + > + /* Audio Control 3 (master mode, fsref rate) */ > + reg = aic26_reg_read(&aic26->codec, AIC26_REG_AUDIO_CTRL3); > + reg &= ~0xf800; > + reg |= 0x0800; /* set master mode */ > + aic26_reg_write(&aic26->codec, AIC26_REG_AUDIO_CTRL3, reg); > + > + /* Fill page 2 register cache */ > + for (i = 0; i < ARRAY_SIZE(aic26->reg_cache); i++) > + aic26_reg_read(&aic26->codec, AIC26_PAGE_ADDR(2, i)); > + > + /* Register the sysfs files for debugging */ > + /* Create SysFS files */ > + rc = device_create_file(&spi->dev, &dev_attr_regs); > + rc |= device_create_file(&spi->dev, &dev_attr_regs_cache); > + rc |= device_create_file(&spi->dev, &dev_attr_keyclick); > + if (rc) > + dev_info(&spi->dev, "error creating sysfs files\n"); > + > + /* Tell the of_soc helper about this codec */ > + of_snd_soc_register_codec(&aic26_soc_codec_dev, aic26, &aic26_dai, > + spi->dev.archdata.of_node); I've been using asoc-v2 so it may not be possible to make this arch independent in asoc-v1. Maybe try porting this to asoc-v2 and see if a bunch of the registration complexity disappears. Most of soc-of.c should be unnecessary. Liam, when is asoc-v2 going into mainline? > + > + dev_dbg(&spi->dev, "SPI device initialized\n"); > + return 0; > +} > + > +static int aic26_spi_remove(struct spi_device *spi) > +{ > + struct aic26 *aic26 = dev_get_drvdata(&spi->dev); > + > + kfree(aic26); > + > + return 0; > +} > + > +static struct spi_driver aic26_spi = { > + .driver = { > + .name = "tlv320aic26", > + .owner = THIS_MODULE, > + }, > + .probe = aic26_spi_probe, > + .remove = aic26_spi_remove, > +}; > + > +static int __init aic26_init(void) > +{ > + return spi_register_driver(&aic26_spi); > +} > +module_init(aic26_init); > + > +static void __exit aic26_exit(void) > +{ > + spi_unregister_driver(&aic26_spi); > +} > +module_exit(aic26_exit); > > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@xxxxxxxxxxxxxxxx > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > -- Jon Smirl jonsmirl@xxxxxxxxx _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel