Commit 1b347b68 authored by Jacob Siverskog's avatar Jacob Siverskog Committed by Mark Brown
Browse files

ASoC: pcm179x: Split into core and SPI parts 



The pcm179x family supports both SPI and I2C for configuration. This
patch splits the driver into core and SPI parts, in preparation for
I2C support.

Signed-off-by: default avatarJacob Siverskog <jacob@teenage.engineering>
Reviewed-by: default avatarJohan Hovold <johan@kernel.org>
Acked-by: default avatarMichael Trimarchi <michael@amarulasolutions.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 92e963f5

sound/soc/codecs/Kconfig

+9 −2
Original line number Diff line number Diff line
@@ -87,7 +87,7 @@ config SND_SOC_ALL_CODECS 
	select SND_SOC_ML26124 if I2C

	select SND_SOC_NAU8825 if I2C

	select SND_SOC_PCM1681 if I2C

	select SND_SOC_PCM179X if SPI_MASTER

	select SND_SOC_PCM179X_SPI if SPI_MASTER

	select SND_SOC_PCM3008

	select SND_SOC_PCM3168A_I2C if I2C

	select SND_SOC_PCM3168A_SPI if SPI_MASTER
@@ -527,8 +527,15 @@ config SND_SOC_PCM1681 
	depends on I2C



config SND_SOC_PCM179X

	tristate "Texas Instruments PCM179X CODEC"

	tristate



config SND_SOC_PCM179X_SPI

	tristate "Texas Instruments PCM179X CODEC (SPI)"

	depends on SPI_MASTER

	select SND_SOC_PCM179X

	help

	  Enable support for Texas Instruments PCM179x CODEC.

	  Select this if your PCM179x is connected via an SPI bus.



config SND_SOC_PCM3008

       tristate

sound/soc/codecs/Makefile

+2 −0
Original line number Diff line number Diff line
@@ -81,6 +81,7 @@ snd-soc-ml26124-objs := ml26124.o 
snd-soc-nau8825-objs := nau8825.o

snd-soc-pcm1681-objs := pcm1681.o

snd-soc-pcm179x-codec-objs := pcm179x.o

snd-soc-pcm179x-spi-objs := pcm179x-spi.o

snd-soc-pcm3008-objs := pcm3008.o

snd-soc-pcm3168a-objs := pcm3168a.o

snd-soc-pcm3168a-i2c-objs := pcm3168a-i2c.o
@@ -285,6 +286,7 @@ obj-$(CONFIG_SND_SOC_ML26124) += snd-soc-ml26124.o 
obj-$(CONFIG_SND_SOC_NAU8825)   += snd-soc-nau8825.o

obj-$(CONFIG_SND_SOC_PCM1681)	+= snd-soc-pcm1681.o

obj-$(CONFIG_SND_SOC_PCM179X)	+= snd-soc-pcm179x-codec.o

obj-$(CONFIG_SND_SOC_PCM179X_SPI)	+= snd-soc-pcm179x-spi.o

obj-$(CONFIG_SND_SOC_PCM3008)	+= snd-soc-pcm3008.o

obj-$(CONFIG_SND_SOC_PCM3168A)	+= snd-soc-pcm3168a.o

obj-$(CONFIG_SND_SOC_PCM3168A_I2C)	+= snd-soc-pcm3168a-i2c.o

sound/soc/codecs/pcm179x-spi.c

0 → 100644
+72 −0
Original line number Diff line number Diff line 
/*

 * PCM179X ASoC SPI driver

 *

 * Copyright (c) Amarula Solutions B.V. 2013

 *

 *     Michael Trimarchi <michael@amarulasolutions.com>

 *

 * This program is free software; you can redistribute it and/or

 * modify it under the terms of the GNU General Public License

 * as published by the Free Software Foundation; either version 2

 * of the License, or (at your option) any later version.

 *

 * This program is distributed in the hope that it will be useful,

 * but WITHOUT ANY WARRANTY; without even the implied warranty of

 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

 * GNU General Public License for more details.

 */



#include <linux/module.h>

#include <linux/of.h>

#include <linux/spi/spi.h>

#include <linux/regmap.h>



#include "pcm179x.h"



static int pcm179x_spi_probe(struct spi_device *spi)

{

	struct regmap *regmap;

	int ret;



	regmap = devm_regmap_init_spi(spi, &pcm179x_regmap_config);

	if (IS_ERR(regmap)) {

		ret = PTR_ERR(regmap);

		dev_err(&spi->dev, "Failed to allocate regmap: %d\n", ret);

		return ret;

	}



	return pcm179x_common_init(&spi->dev, regmap);

}



static int pcm179x_spi_remove(struct spi_device *spi)

{

	return pcm179x_common_exit(&spi->dev);

}



static const struct of_device_id pcm179x_of_match[] = {

	{ .compatible = "ti,pcm1792a", },

	{ }

};

MODULE_DEVICE_TABLE(of, pcm179x_of_match);



static const struct spi_device_id pcm179x_spi_ids[] = {

	{ "pcm179x", 0 },

	{ },

};

MODULE_DEVICE_TABLE(spi, pcm179x_spi_ids);



static struct spi_driver pcm179x_spi_driver = {

	.driver = {

		.name = "pcm179x",

		.of_match_table = of_match_ptr(pcm179x_of_match),

	},

	.id_table = pcm179x_spi_ids,

	.probe = pcm179x_spi_probe,

	.remove = pcm179x_spi_remove,

};



module_spi_driver(pcm179x_spi_driver);



MODULE_DESCRIPTION("ASoC PCM179X SPI driver");

MODULE_AUTHOR("Michael Trimarchi <michael@amarulasolutions.com>");

MODULE_LICENSE("GPL");

sound/soc/codecs/pcm179x.c

+11 −41
Original line number Diff line number Diff line
@@ -20,7 +20,6 @@ 
#include <linux/slab.h>

#include <linux/kernel.h>

#include <linux/device.h>

#include <linux/spi/spi.h>



#include <sound/core.h>

#include <sound/pcm.h>
@@ -29,7 +28,6 @@ 
#include <sound/soc.h>

#include <sound/tlv.h>

#include <linux/of.h>

#include <linux/of_device.h>



#include "pcm179x.h"
@@ -194,13 +192,7 @@ static struct snd_soc_dai_driver pcm179x_dai = { 
	.ops = &pcm179x_dai_ops,

};



static const struct of_device_id pcm179x_of_match[] = {

	{ .compatible = "ti,pcm1792a", },

	{ }

};

MODULE_DEVICE_TABLE(of, pcm179x_of_match);



static const struct regmap_config pcm179x_regmap = {

const struct regmap_config pcm179x_regmap_config = {

	.reg_bits		= 8,

	.val_bits		= 8,

	.max_register		= 23,
@@ -209,6 +201,7 @@ static const struct regmap_config pcm179x_regmap = { 
	.writeable_reg		= pcm179x_writeable_reg,

	.readable_reg		= pcm179x_accessible_reg,

};

EXPORT_SYMBOL_GPL(pcm179x_regmap_config);



static struct snd_soc_codec_driver soc_codec_dev_pcm179x = {

	.controls		= pcm179x_controls,
@@ -219,52 +212,29 @@ static struct snd_soc_codec_driver soc_codec_dev_pcm179x = { 
	.num_dapm_routes	= ARRAY_SIZE(pcm179x_dapm_routes),

};



static int pcm179x_spi_probe(struct spi_device *spi)

int pcm179x_common_init(struct device *dev, struct regmap *regmap)

{

	struct pcm179x_private *pcm179x;

	int ret;



	pcm179x = devm_kzalloc(&spi->dev, sizeof(struct pcm179x_private),

	pcm179x = devm_kzalloc(dev, sizeof(struct pcm179x_private),

				GFP_KERNEL);

	if (!pcm179x)

		return -ENOMEM;



	spi_set_drvdata(spi, pcm179x);



	pcm179x->regmap = devm_regmap_init_spi(spi, &pcm179x_regmap);

	if (IS_ERR(pcm179x->regmap)) {

		ret = PTR_ERR(pcm179x->regmap);

		dev_err(&spi->dev, "Failed to register regmap: %d\n", ret);

		return ret;

	}

	pcm179x->regmap = regmap;

	dev_set_drvdata(dev, pcm179x);



	return snd_soc_register_codec(&spi->dev,

	return snd_soc_register_codec(dev,

			&soc_codec_dev_pcm179x, &pcm179x_dai, 1);

}

EXPORT_SYMBOL_GPL(pcm179x_common_init);



static int pcm179x_spi_remove(struct spi_device *spi)

int pcm179x_common_exit(struct device *dev)

{

	snd_soc_unregister_codec(&spi->dev);

	snd_soc_unregister_codec(dev);

	return 0;

}



static const struct spi_device_id pcm179x_spi_ids[] = {

	{ "pcm179x", 0 },

	{ },

};

MODULE_DEVICE_TABLE(spi, pcm179x_spi_ids);



static struct spi_driver pcm179x_codec_driver = {

	.driver = {

		.name = "pcm179x",

		.of_match_table = of_match_ptr(pcm179x_of_match),

	},

	.id_table = pcm179x_spi_ids,

	.probe = pcm179x_spi_probe,

	.remove = pcm179x_spi_remove,

};



module_spi_driver(pcm179x_codec_driver);

EXPORT_SYMBOL_GPL(pcm179x_common_exit);



MODULE_DESCRIPTION("ASoC PCM179X driver");

MODULE_AUTHOR("Michael Trimarchi <michael@amarulasolutions.com>");

sound/soc/codecs/pcm179x.h

+5 −0
Original line number Diff line number Diff line
@@ -24,4 +24,9 @@ 
#define PCM1792A_FORMATS (SNDRV_PCM_FMTBIT_S32_LE | SNDRV_PCM_FMTBIT_S24_LE | \

			  SNDRV_PCM_FMTBIT_S16_LE)



extern const struct regmap_config pcm179x_regmap_config;



int pcm179x_common_init(struct device *dev, struct regmap *regmap);

int pcm179x_common_exit(struct device *dev);



#endif