Commit 648c5382 authored by Daniel Mack's avatar Daniel Mack Committed by Mark Brown

ASoC: tas5086: move two variables into private struct

We need to access the charge_period and start_mid_z values from other
places later, so move them to the private struct.
Signed-off-by: default avatarDaniel Mack <zonque@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent 15c03dd4
...@@ -244,6 +244,8 @@ struct tas5086_private { ...@@ -244,6 +244,8 @@ struct tas5086_private {
unsigned int mclk, sclk; unsigned int mclk, sclk;
unsigned int format; unsigned int format;
bool deemph; bool deemph;
unsigned int charge_period;
unsigned int pwm_start_mid_z;
/* Current sample rate for de-emphasis control */ /* Current sample rate for de-emphasis control */
int rate; int rate;
/* GPIO driving Reset pin, if any */ /* GPIO driving Reset pin, if any */
...@@ -720,13 +722,15 @@ static const int tas5086_charge_period[] = { ...@@ -720,13 +722,15 @@ static const int tas5086_charge_period[] = {
static int tas5086_probe(struct snd_soc_codec *codec) static int tas5086_probe(struct snd_soc_codec *codec)
{ {
struct tas5086_private *priv = snd_soc_codec_get_drvdata(codec); struct tas5086_private *priv = snd_soc_codec_get_drvdata(codec);
int charge_period = 1300000; /* hardware default is 1300 ms */
u8 pwm_start_mid_z = 0;
int i, ret; int i, ret;
priv->pwm_start_mid_z = 0;
priv->charge_period = 1300000; /* hardware default is 1300 ms */
if (of_match_device(of_match_ptr(tas5086_dt_ids), codec->dev)) { if (of_match_device(of_match_ptr(tas5086_dt_ids), codec->dev)) {
struct device_node *of_node = codec->dev->of_node; struct device_node *of_node = codec->dev->of_node;
of_property_read_u32(of_node, "ti,charge-period", &charge_period); of_property_read_u32(of_node, "ti,charge-period",
&priv->charge_period);
for (i = 0; i < 6; i++) { for (i = 0; i < 6; i++) {
char name[25]; char name[25];
...@@ -735,7 +739,7 @@ static int tas5086_probe(struct snd_soc_codec *codec) ...@@ -735,7 +739,7 @@ static int tas5086_probe(struct snd_soc_codec *codec)
"ti,mid-z-channel-%d", i + 1); "ti,mid-z-channel-%d", i + 1);
if (of_get_property(of_node, name, NULL) != NULL) if (of_get_property(of_node, name, NULL) != NULL)
pwm_start_mid_z |= 1 << i; priv->pwm_start_mid_z |= 1 << i;
} }
} }
...@@ -744,25 +748,25 @@ static int tas5086_probe(struct snd_soc_codec *codec) ...@@ -744,25 +748,25 @@ static int tas5086_probe(struct snd_soc_codec *codec)
* configure 'part 1' of the PWM starts to use Mid-Z, and tell * configure 'part 1' of the PWM starts to use Mid-Z, and tell
* all configured mid-z channels to start start under 'part 1'. * all configured mid-z channels to start start under 'part 1'.
*/ */
if (pwm_start_mid_z) if (priv->pwm_start_mid_z)
regmap_write(priv->regmap, TAS5086_PWM_START, regmap_write(priv->regmap, TAS5086_PWM_START,
TAS5086_PWM_START_MIDZ_FOR_START_1 | TAS5086_PWM_START_MIDZ_FOR_START_1 |
pwm_start_mid_z); priv->pwm_start_mid_z);
/* lookup and set split-capacitor charge period */ /* lookup and set split-capacitor charge period */
if (charge_period == 0) { if (priv->charge_period == 0) {
regmap_write(priv->regmap, TAS5086_SPLIT_CAP_CHARGE, 0); regmap_write(priv->regmap, TAS5086_SPLIT_CAP_CHARGE, 0);
} else { } else {
i = index_in_array(tas5086_charge_period, i = index_in_array(tas5086_charge_period,
ARRAY_SIZE(tas5086_charge_period), ARRAY_SIZE(tas5086_charge_period),
charge_period); priv->charge_period);
if (i >= 0) if (i >= 0)
regmap_write(priv->regmap, TAS5086_SPLIT_CAP_CHARGE, regmap_write(priv->regmap, TAS5086_SPLIT_CAP_CHARGE,
i + 0x08); i + 0x08);
else else
dev_warn(codec->dev, dev_warn(codec->dev,
"Invalid split-cap charge period of %d ns.\n", "Invalid split-cap charge period of %d ns.\n",
charge_period); priv->charge_period);
} }
/* enable factory trim */ /* enable factory trim */
......
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