Commit 3c89724e authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown

ASoC: cx20442: fix regression by adding back .read/.write

commit 39b5a0f8 ("ASoC: cx20442: don't use reg_cache")
removed .read/.write from driver, but it might breaks non-regmap
driver, because ALSA SoC framework might call it.

To fix this regression, this patch back .read/.write.
and also this patch uses cx20442 internal reg_cache
which is needed for .read/.write.
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 39b5a0f8
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
struct cx20442_priv { struct cx20442_priv {
struct tty_struct *tty; struct tty_struct *tty;
struct regulator *por; struct regulator *por;
u8 reg_cache;
}; };
#define CX20442_PM 0x0 #define CX20442_PM 0x0
...@@ -88,6 +89,17 @@ static const struct snd_soc_dapm_route cx20442_audio_map[] = { ...@@ -88,6 +89,17 @@ static const struct snd_soc_dapm_route cx20442_audio_map[] = {
{"ADC", NULL, "Input Mixer"}, {"ADC", NULL, "Input Mixer"},
}; };
static unsigned int cx20442_read_reg_cache(struct snd_soc_codec *codec,
unsigned int reg)
{
struct cx20442_priv *cx20442 = snd_soc_codec_get_drvdata(codec);
if (reg >= 1)
return -EINVAL;
return cx20442->reg_cache;
}
enum v253_vls { enum v253_vls {
V253_VLS_NONE = 0, V253_VLS_NONE = 0,
V253_VLS_T, V253_VLS_T,
...@@ -112,8 +124,6 @@ enum v253_vls { ...@@ -112,8 +124,6 @@ enum v253_vls {
V253_VLS_TEST, V253_VLS_TEST,
}; };
#if 0
/* FIXME : these function will be re-used */
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)) {
...@@ -147,11 +157,10 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg, ...@@ -147,11 +157,10 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
unsigned int value) unsigned int value)
{ {
struct cx20442_priv *cx20442 = snd_soc_codec_get_drvdata(codec); struct cx20442_priv *cx20442 = snd_soc_codec_get_drvdata(codec);
u8 *reg_cache = codec->reg_cache;
int vls, vsp, old, len; int vls, vsp, old, len;
char buf[18]; char buf[18];
if (reg >= codec->driver->reg_cache_size) if (reg >= 1)
return -EINVAL; return -EINVAL;
/* tty and write pointers required for talking to the modem /* tty and write pointers required for talking to the modem
...@@ -159,8 +168,8 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg, ...@@ -159,8 +168,8 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
if (!cx20442->tty || !cx20442->tty->ops->write) if (!cx20442->tty || !cx20442->tty->ops->write)
return -EIO; return -EIO;
old = reg_cache[reg]; old = cx20442->reg_cache;
reg_cache[reg] = value; cx20442->reg_cache = value;
vls = cx20442_pm_to_v253_vls(value); vls = cx20442_pm_to_v253_vls(value);
if (vls < 0) if (vls < 0)
...@@ -190,7 +199,6 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg, ...@@ -190,7 +199,6 @@ static int cx20442_write(struct snd_soc_codec *codec, unsigned int reg,
return 0; return 0;
} }
#endif
/* /*
* Line discpline related code * Line discpline related code
...@@ -384,12 +392,12 @@ static int cx20442_codec_remove(struct snd_soc_codec *codec) ...@@ -384,12 +392,12 @@ static int cx20442_codec_remove(struct snd_soc_codec *codec)
return 0; return 0;
} }
static const u8 cx20442_reg;
static const struct snd_soc_codec_driver cx20442_codec_dev = { static const struct snd_soc_codec_driver cx20442_codec_dev = {
.probe = cx20442_codec_probe, .probe = cx20442_codec_probe,
.remove = cx20442_codec_remove, .remove = cx20442_codec_remove,
.set_bias_level = cx20442_set_bias_level, .set_bias_level = cx20442_set_bias_level,
.read = cx20442_read_reg_cache,
.write = cx20442_write,
.component_driver = { .component_driver = {
.dapm_widgets = cx20442_dapm_widgets, .dapm_widgets = cx20442_dapm_widgets,
......
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