Commit b84eab08 authored by Janusz Krzysztofik's avatar Janusz Krzysztofik Committed by Mark Brown

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: default avatarJanusz Krzysztofik <jkrzyszt@tis.icnet.pl>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 4ce2f2fe
...@@ -132,7 +132,7 @@ enum v253_vls { ...@@ -132,7 +132,7 @@ enum v253_vls {
static int cx20442_pm_to_v253_vls(u8 value) static int cx20442_pm_to_v253_vls(u8 value)
{ {
switch(value & ~(1 << CX20442_AGC)) { switch (value & ~(1 << CX20442_AGC)) {
case 0: case 0:
return V253_VLS_T; return V253_VLS_T;
case (1 << CX20442_SPKOUT): case (1 << CX20442_SPKOUT):
...@@ -152,7 +152,7 @@ static int cx20442_pm_to_v253_vls(u8 value) ...@@ -152,7 +152,7 @@ static int cx20442_pm_to_v253_vls(u8 value)
} }
static int cx20442_pm_to_v253_vsp(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_SPKOUT):
case (1 << CX20442_MIC): case (1 << CX20442_MIC):
case (1 << CX20442_SPKOUT) | (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, ...@@ -171,6 +171,9 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
if (reg >= codec->reg_cache_size) if (reg >= codec->reg_cache_size)
return -EINVAL; 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) if (!codec->hw_write || !codec->control_data)
return -EIO; return -EIO;
...@@ -182,7 +185,7 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg, ...@@ -182,7 +185,7 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
return vls; return vls;
vsp = cx20442_pm_to_v253_vsp(value); vsp = cx20442_pm_to_v253_vsp(value);
if (vsp < 0 ) if (vsp < 0)
return vsp; return vsp;
if ((vls == V253_VLS_T) || if ((vls == V253_VLS_T) ||
...@@ -232,7 +235,7 @@ static int cx20442_codec_probe(struct platform_device *pdev) ...@@ -232,7 +235,7 @@ static int cx20442_codec_probe(struct platform_device *pdev)
struct snd_soc_codec *codec; struct snd_soc_codec *codec;
int ret; int ret;
if(!cx20442_codec) { if (!cx20442_codec) {
dev_err(&pdev->dev, "cx20442 not yet discovered\n"); dev_err(&pdev->dev, "cx20442 not yet discovered\n");
return -ENODEV; return -ENODEV;
} }
...@@ -310,13 +313,13 @@ static int cx20442_register(struct cx20442_priv *cx20442) ...@@ -310,13 +313,13 @@ static int cx20442_register(struct cx20442_priv *cx20442)
ret = snd_soc_register_codec(codec); ret = snd_soc_register_codec(codec);
if (ret != 0) { 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; goto err;
} }
ret = snd_soc_register_dai(&cx20442_dai); ret = snd_soc_register_dai(&cx20442_dai);
if (ret != 0) { 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; goto err_codec;
} }
...@@ -392,4 +395,4 @@ module_exit(cx20442_exit); ...@@ -392,4 +395,4 @@ module_exit(cx20442_exit);
MODULE_DESCRIPTION("ASoC CX20442-11 voice modem codec driver"); MODULE_DESCRIPTION("ASoC CX20442-11 voice modem codec driver");
MODULE_AUTHOR("Janusz Krzysztofik"); MODULE_AUTHOR("Janusz Krzysztofik");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_ALIAS("platform:cx20442-codec"); MODULE_ALIAS("platform:cx20442");
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment