From: Mark Brown Date: Mon, 30 Jan 2012 21:51:45 +0000 (+0000) Subject: ASoC: max9768: Fix up review comments in max9768 driver X-Git-Tag: v3.4-rc3~5^2~23^2~25^2~185 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=69edea8a50fb88edd0f23eecfb89ef513b68eaee;p=profile%2Fivi%2Fkernel-adaptation-intel-automotive.git ASoC: max9768: Fix up review comments in max9768 driver The Mute Switch should be called Playback Switch to match the volume and the regmap core has been updated so we don't need to mark all the registers as non-volatile. Signed-off-by: Mark Brown Acked-by: Wolfram Sang --- diff --git a/sound/soc/codecs/max9768.c b/sound/soc/codecs/max9768.c index 6348995..79e9901 100644 --- a/sound/soc/codecs/max9768.c +++ b/sound/soc/codecs/max9768.c @@ -115,7 +115,7 @@ static const struct snd_kcontrol_new max9768_volume[] = { }; static const struct snd_kcontrol_new max9768_mute[] = { - SOC_SINGLE_BOOL_EXT("Mute Switch", 0, max9768_get_gpio, max9768_set_gpio), + SOC_SINGLE_BOOL_EXT("Playback Switch", 0, max9768_get_gpio, max9768_set_gpio), }; static int max9768_probe(struct snd_soc_codec *codec) @@ -150,18 +150,12 @@ static struct snd_soc_codec_driver max9768_codec_driver = { .num_controls = ARRAY_SIZE(max9768_volume), }; -static bool max9768_always_false(struct device *dev, unsigned int reg) -{ - return false; -} - static const struct regmap_config max9768_i2c_regmap_config = { .reg_bits = 2, .val_bits = 6, .max_register = 3, .reg_defaults = max9768_default_regs, .num_reg_defaults = ARRAY_SIZE(max9768_default_regs), - .volatile_reg = max9768_always_false, .cache_type = REGCACHE_RBTREE, };