- 19 Sep, 2012 6 commits
-
-
Richard Zhao authored
snd_imx_open should return error code returned by snd_dmaengine_pcm_open. Signed-off-by:
Richard Zhao <richard.zhao@freescale.com> Signed-off-by:
Mark Brown <broonie@opensource.wolfsonmicro.com>
-
Richard Zhao authored
It fixed smatch warning: sound/soc/fsl/imx-pcm-dma.c:112 snd_imx_open() error: potential null dereference 'dma_data'. (kzalloc returns null) Signed-off-by:
-
Lars-Peter Clausen authored
For ENUM controls the bitmask is calculated based on the number of items. Currently this is done each time the control is accessed. And while the performance impact of this should be negligible we can easily do better. The roundup_pow_of_two macro performs the same calculation which is currently done manually, but it is also possible to use this macro with compile time constants and so it can be used to initialize static data. So we can use it to initialize the mask field of a ENUM control during its declaration. Signed-off-by:
Lars-Peter Clausen <lars@metafoo.de> Acked-by:
Peter Ujfalusi <peter.ujfalusi@ti.com> Signed-off-by:
-
Timur Tabi authored
PowerPC ASoC drivers frequently use the _BE variants of the SNDRV_PCM_FORMAT macros, so we need to look for those as well. Signed-off-by:
Timur Tabi <timur@freescale.com> Signed-off-by:
-
Timur Tabi authored
Remove a call to dma_unmap_single() from the PowerPC ASoC DMA driver. The buffer is allocated and not actually mapped, so the unmap call doesn't make sense. It was probably left over from some early version of the driver. This bug was unnoticed for so long because the DMA mapping functions normally don't do anything on PowerPC. Signed-off-by:
-
Timur Tabi authored
Use snd_soc_register_card() instead of platform_device_alloc("soc-audio") to register the sound card from the machine drivers. The use of platform_device_alloc is deprecated. Although several other drivers still use platform_device_alloc(), the Freescale drivers were not using it to pass driver data. Instead of fixing the driver data usage, it's better to replace the deprecated code. Signed-off-by:
-
- 17 Sep, 2012 1 commit
-
-
Shawn Guo authored
Rather than including mach/iomux-mx27.h to define gpio numbers and set up the pins, the patch moves all these into machine code and has the gpio numbers passed to driver via platform_data. As the result, we can remove the mach/iomux-mx27.h inclusion from driver. Signed-off-by:
Shawn Guo <shawn.guo@linaro.org> Acked-by:
Javier Martin <javier.martin@vista-silicon.com> Signed-off-by:
-
- 13 Sep, 2012 4 commits
-
-
Lars-Peter Clausen authored
Signed-off-by:
-
Lars-Peter Clausen authored
Signed-off-by:
-
Peter Senna Tschudin authored
Remove useless kfree() and clean up code related to the removal. The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged exists@ position r.p1,r.p2; expression e <= r.x,x,e1; iterator I; statement S; @@ if (x@p1 == NULL) { ... when != I(x,...) S when != e = e1 when != e += e1 when != e -= e1 when != ++e when != --e when != e++ when != e-- when != &e kfree@p2(x); ... return ...; } @ok depends on unchanged exists@ position any r.p1; position r.p2; expression x; @@ ... when != true x@p1 == NULL kfree@p2(x); @depends on !ok && unchanged@ position r.p2; expression x; @@ *kfree@p2(x); // </smpl> Signed-off-by:
Peter Senna Tschudin <peter.senna@gmail.com> Signed-off-by:
-
Mark Brown authored
The core has for a long time had support for marking the register maps of devices dirty when suspending so that they are resynced on resume. Also implement this feature for CODECs using regmap. Signed-off-by:
Liam Girdwood <lrg@ti.com>
-
- 12 Sep, 2012 15 commits
-
-
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
This is better style as we acquire resources we will need before we go into the ASoC card probe. Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Better style as we get all the resources we need prior to starting the ASoC level probe. Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by:
-
- 11 Sep, 2012 10 commits
-
-
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Signed-off-by: -
Fabio Estevam authored
Since commit 98d3088e (SoC: core: Fix check before defaulting to regmap) , it is not necessary to provide codec->control_data anymore. Signed-off-by:
Fabio Estevam <fabio.estevam@freescale.com> Signed-off-by:
-
Fabio Estevam authored
Since commit 98d3088e (SoC: core: Fix check before defaulting to regmap) , it is not necessary to provide codec->control_data anymore. Signed-off-by:
-
Fabio Estevam authored
Since commit 98d3088e (SoC: core: Fix check before defaulting to regmap) , it is not necessary to provide codec->control_data anymore. Signed-off-by:
-
Peter Ujfalusi authored
cpu_dai is not in use in this function and just generates warning at compile time. Signed-off-by:
-
Mark Brown authored
Signed-off-by:
-
Mark Brown authored
This is better style since it has us obtaining all resources before we try the ASoC probe. This change also fixes a potential issue where we don't enable the regulators before trying to confirm the device ID which could cause a failure during probe in some system configurations. Signed-off-by:
-
Mark Brown authored
Signed-off-by:
-
- 10 Sep, 2012 4 commits
-
-
Mark Brown authored
Signed-off-by: -
Bo Shen authored
The core_intercon is added two times, remove the redundant one Signed-off-by:
Bo Shen <voice.shen@atmel.com> Signed-off-by:
-
Mark Brown authored
Signed-off-by: -
Mark Brown authored
Better style as we acquire resources before trying the ASoC card probe. Signed-off-by:
-
