aboutsummaryrefslogtreecommitdiff
path: root/sound/soc/codecs/cx20442.c
diff options
context:
space:
mode:
authorJanusz Krzysztofik <jkrzyszt@tis.icnet.pl>2009-07-28 20:24:12 +0200
committerMark Brown <broonie@opensource.wolfsonmicro.com>2009-07-28 22:15:23 +0100
commitb84eab08a67913581515a1184f1deedf1d54dc5d (patch)
tree4c80dddb17c3c5bd782c8e30472433b98b4f3022 /sound/soc/codecs/cx20442.c
parent4ce2f2fe61002ab35c9333d52c95b5ff9e949f50 (diff)
ASoC: CX20442: fix issues pointed out by subsystem maintainer
The patch fixes some checkpatch identified issues and adds a comment about line discipline interaction to my driver code, as requested by Mark on my inital submission (thank you Mark for applying my imperfect patch anyway). It also fixes MODULE_ALIAS mismatch as used in my machine driver. Signed-off-by: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/codecs/cx20442.c')
-rw-r--r--sound/soc/codecs/cx20442.c17
1 files changed, 10 insertions, 7 deletions
diff --git a/sound/soc/codecs/cx20442.c b/sound/soc/codecs/cx20442.c
index f64483c75b5..7bbb77baa23 100644
--- a/sound/soc/codecs/cx20442.c
+++ b/sound/soc/codecs/cx20442.c
@@ -132,7 +132,7 @@ enum v253_vls {
static int cx20442_pm_to_v253_vls(u8 value)
{
- switch(value & ~(1 << CX20442_AGC)) {
+ switch (value & ~(1 << CX20442_AGC)) {
case 0:
return V253_VLS_T;
case (1 << CX20442_SPKOUT):
@@ -152,7 +152,7 @@ static int cx20442_pm_to_v253_vls(u8 value)
}
static int cx20442_pm_to_v253_vsp(u8 value)
{
- switch(value & ~(1 << CX20442_AGC)) {
+ switch (value & ~(1 << CX20442_AGC)) {
case (1 << CX20442_SPKOUT):
case (1 << CX20442_MIC):
case (1 << CX20442_SPKOUT) | (1 << CX20442_MIC):
@@ -171,6 +171,9 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
if (reg >= codec->reg_cache_size)
return -EINVAL;
+ /* hw_write and control_data pointers required for talking to the modem
+ * are expected to be set by the machine driver's line discipline
+ * initialization code */
if (!codec->hw_write || !codec->control_data)
return -EIO;
@@ -182,7 +185,7 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
return vls;
vsp = cx20442_pm_to_v253_vsp(value);
- if (vsp < 0 )
+ if (vsp < 0)
return vsp;
if ((vls == V253_VLS_T) ||
@@ -232,7 +235,7 @@ static int cx20442_codec_probe(struct platform_device *pdev)
struct snd_soc_codec *codec;
int ret;
- if(!cx20442_codec) {
+ if (!cx20442_codec) {
dev_err(&pdev->dev, "cx20442 not yet discovered\n");
return -ENODEV;
}
@@ -310,13 +313,13 @@ static int cx20442_register(struct cx20442_priv *cx20442)
ret = snd_soc_register_codec(codec);
if (ret != 0) {
- //dev_err(&dev->dev, "Failed to register codec: %d\n", ret);
+ dev_err(&codec->dev, "Failed to register codec: %d\n", ret);
goto err;
}
ret = snd_soc_register_dai(&cx20442_dai);
if (ret != 0) {
- //dev_err(&dev->dev, "Failed to register DAI: %d\n", ret);
+ dev_err(&codec->dev, "Failed to register DAI: %d\n", ret);
goto err_codec;
}
@@ -392,4 +395,4 @@ module_exit(cx20442_exit);
MODULE_DESCRIPTION("ASoC CX20442-11 voice modem codec driver");
MODULE_AUTHOR("Janusz Krzysztofik");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:cx20442-codec");
+MODULE_ALIAS("platform:cx20442");