All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Mark Brown <broonie@kernel.org>, Bard Liao <bardliao@realtek.com>,
	Oder Chiou <oder_chiou@realtek.com>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: Hans de Goede <hdegoede@redhat.com>,
	alsa-devel@alsa-project.org, Carlo Caione <carlo@endlessm.com>,
	Takashi Iwai <tiwai@suse.com>
Subject: [PATCH v2 04/32] ASoC: rt5651: Use standard component set_jack callback
Date: Sun, 25 Feb 2018 11:46:45 +0100	[thread overview]
Message-ID: <20180225104713.4745-5-hdegoede@redhat.com> (raw)
In-Reply-To: <20180225104713.4745-1-hdegoede@redhat.com>

Use the standard component set_jack callback instead of defining a codec
private API for this.

Tested-by: Carlo Caione <carlo@endlessm.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 sound/soc/codecs/rt5651.c             | 6 +++---
 sound/soc/codecs/rt5651.h             | 2 --
 sound/soc/intel/boards/bytcr_rt5651.c | 2 +-
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/sound/soc/codecs/rt5651.c b/sound/soc/codecs/rt5651.c
index 11741cdf722e..3e507f53fc13 100644
--- a/sound/soc/codecs/rt5651.c
+++ b/sound/soc/codecs/rt5651.c
@@ -1593,8 +1593,8 @@ static irqreturn_t rt5651_irq(int irq, void *data)
 	return IRQ_HANDLED;
 }
 
-int rt5651_set_jack_detect(struct snd_soc_component *component,
-			   struct snd_soc_jack *hp_jack)
+static int rt5651_set_jack(struct snd_soc_component *component,
+			   struct snd_soc_jack *hp_jack, void *data)
 {
 	struct snd_soc_dapm_context *dapm = snd_soc_component_get_dapm(component);
 	struct rt5651_priv *rt5651 = snd_soc_component_get_drvdata(component);
@@ -1658,7 +1658,6 @@ int rt5651_set_jack_detect(struct snd_soc_component *component,
 
 	return 0;
 }
-EXPORT_SYMBOL_GPL(rt5651_set_jack_detect);
 
 static int rt5651_probe(struct snd_soc_component *component)
 {
@@ -1762,6 +1761,7 @@ static const struct snd_soc_component_driver soc_component_dev_rt5651 = {
 	.suspend		= rt5651_suspend,
 	.resume			= rt5651_resume,
 	.set_bias_level		= rt5651_set_bias_level,
+	.set_jack		= rt5651_set_jack,
 	.controls		= rt5651_snd_controls,
 	.num_controls		= ARRAY_SIZE(rt5651_snd_controls),
 	.dapm_widgets		= rt5651_dapm_widgets,
diff --git a/sound/soc/codecs/rt5651.h b/sound/soc/codecs/rt5651.h
index 8f128d057ff0..f3158488fc89 100644
--- a/sound/soc/codecs/rt5651.h
+++ b/sound/soc/codecs/rt5651.h
@@ -2080,6 +2080,4 @@ struct rt5651_priv {
 	bool hp_mute;
 };
 
-int rt5651_set_jack_detect(struct snd_soc_component *component,
-			   struct snd_soc_jack *hp_jack);
 #endif /* __RT5651_H__ */
diff --git a/sound/soc/intel/boards/bytcr_rt5651.c b/sound/soc/intel/boards/bytcr_rt5651.c
index 1f8407b17d2b..8008a027cc93 100644
--- a/sound/soc/intel/boards/bytcr_rt5651.c
+++ b/sound/soc/intel/boards/bytcr_rt5651.c
@@ -368,7 +368,7 @@ static int byt_rt5651_init(struct snd_soc_pcm_runtime *runtime)
 		return ret;
 	}
 
-	rt5651_set_jack_detect(codec, &priv->jack);
+	snd_soc_component_set_jack(codec, &priv->jack, NULL);
 
 	return ret;
 }
-- 
2.14.3

  parent reply	other threads:[~2018-02-25 10:47 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-25 10:46 [PATCH v2 00/32] ASoC: rt5651: jack-detect fixes and improvements Hans de Goede
2018-02-25 10:46 ` [PATCH v2 01/32] ASoC: rt5651: Remove unused rt5651_platform_data Hans de Goede
2018-02-25 10:46 ` [PATCH v2 02/32] ASoC: rt5651: Move all jack-detect initialization to rt5651_set_jack_detect Hans de Goede
2018-03-01 18:06   ` Applied "ASoC: rt5651: Move all jack-detect initialization to rt5651_set_jack_detect" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 03/32] ASoC: rt5651: Move 2 functions higher up in rt5651.c Hans de Goede
2018-03-01 18:06   ` Applied "ASoC: rt5651: Move 2 functions higher up in rt5651.c" to the asoc tree Mark Brown
2018-02-25 10:46 ` Hans de Goede [this message]
2018-02-25 10:46 ` [PATCH v2 05/32] ASoC: rt5651: Add rt5651_apply_properties() helper function Hans de Goede
2018-03-01 18:06   ` Applied "ASoC: rt5651: Add rt5651_apply_properties() helper function" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 06/32] ASoC: rt5651: Configure jack-detect source through a device-property Hans de Goede
2018-03-01 19:30   ` Mark Brown
2018-03-01 22:35     ` Hans de Goede
2018-03-02  9:32       ` Hans de Goede
2018-03-02 12:18         ` Mark Brown
2018-03-02 12:58           ` Hans de Goede
2018-03-02 17:41             ` Mark Brown
2018-03-03 21:20               ` Hans de Goede
2018-03-02 22:17             ` Rob Herring
2018-02-25 10:46 ` [PATCH v2 07/32] ASoC: rt5651: Remove is_sys_clk_from_pll() Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Remove is_sys_clk_from_pll()" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 08/32] ASoC: rt5651: Fix bias_level confusion Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Fix bias_level confusion" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 09/32] ASoC: rt5651: Do not modify the LDO voltage control bits from set_bias_level() Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Do not modify the LDO voltage control bits from set_bias_level()" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 10/32] ASoC: rt5651: Do not modify jd and PLL power bits from set_bias_level() Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Do not modify jd and PLL power bits from set_bias_level()" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 11/32] ASoC: rt5651: Remove programming of PWR regs before force_bias_level() call Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Remove programming of PWR regs before force_bias_level() call" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 12/32] ASoC: rt5651: Only configure LDO voltage once at boot Hans de Goede
2018-03-01 19:26   ` Applied "ASoC: rt5651: Only configure LDO voltage once at boot" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 13/32] ASoC: rt5651: Remove "JD Power" dapm supply Hans de Goede
2018-03-01 19:25   ` Applied "ASoC: rt5651: Remove "JD Power" dapm supply" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 14/32] ASoC: rt5651: Enable LDO and micbias1 supplies for jack-type detection Hans de Goede
2018-02-25 10:46 ` [PATCH v2 15/32] ASoC: rt5651: Only configure OVCD once at set_jack time Hans de Goede
2018-03-01 19:25   ` Applied "ASoC: rt5651: Only configure OVCD once at set_jack time" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 16/32] ASoC: rt5651: Always keep OVCD enabled Hans de Goede
2018-03-01 19:25   ` Applied "ASoC: rt5651: Always keep OVCD enabled" to the asoc tree Mark Brown
2018-02-25 10:46 ` [PATCH v2 17/32] ASoC: rt5651: Allow specifying over-current thresholds through device-properties Hans de Goede
2018-03-02 22:18   ` Rob Herring
2018-02-25 10:46 ` [PATCH v2 18/32] ASoC: rt5651: Enable sticky mode for OVCD Hans de Goede
2018-03-07 14:14   ` Applied "ASoC: rt5651: Enable sticky mode for OVCD" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 19/32] ASoC: rt5651: Enable Platform Clock during jack-type detect Hans de Goede
2018-03-07 14:13   ` Applied "ASoC: rt5651: Enable Platform Clock during jack-type detect" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 20/32] ASoC: rt5651: Add rt5651_jack_inserted() helper Hans de Goede
2018-03-07 14:13   ` Applied "ASoC: rt5651: Add rt5651_jack_inserted() helper" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 21/32] ASoC: rt5651: Rewrite jack-type detection Hans de Goede
2018-02-25 10:47 ` [PATCH v2 22/32] ASoC: Intel: bytcr_rt5651: Actually honor the DMIC_EN quirk if specified Hans de Goede
2018-02-25 10:47 ` [PATCH v2 23/32] ASoC: Intel: bytcr_rt5651: Only create jack if we have a jack-detect source Hans de Goede
2018-02-25 10:47 ` [PATCH v2 24/32] ASoC: Intel: bytcr_rt5651: Add quirk micbias OVCD configuration Hans de Goede
2018-02-25 10:47 ` [PATCH v2 25/32] ASoC: Intel: bytcr_rt5651: Configure PLL1 before using it Hans de Goede
2018-03-07 14:22   ` Applied "ASoC: Intel: bytcr_rt5651: Configure PLL1 before using it" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 26/32] ASoC: Intel: bytcr_rt5651: Drop snd_soc_dai_set_bclk_ratio() call Hans de Goede
2018-03-07 14:22   ` Applied "ASoC: Intel: bytcr_rt5651: Drop snd_soc_dai_set_bclk_ratio() call" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 27/32] ASoC: Intel: bytcr_rt5651: Rename IN3_MAP to IN1_HS_IN3_MAP Hans de Goede
2018-03-07 14:22   ` Applied "ASoC: Intel: bytcr_rt5651: Rename IN3_MAP to IN1_HS_IN3_MAP" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 28/32] ASoC: Intel: bytcr_rt5651: Add new IN2_HS_IN3 input map and a quirk using it Hans de Goede
2018-03-07 14:21   ` Applied "ASoC: Intel: bytcr_rt5651: Add new IN2_HS_IN3 input map and a quirk using it" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 29/32] ASoC: Intel: bytcr_rt5651: Add support for Bay Trail CR / SSP0 using boards Hans de Goede
2018-02-25 10:47 ` [PATCH v2 30/32] ASoC: Intel: bytcr_rt5651: Add quirk for the VIOS LTH17 laptop Hans de Goede
2018-03-07 14:21   ` Applied "ASoC: Intel: bytcr_rt5651: Add quirk for the VIOS LTH17 laptop" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 31/32] ASoC: Intel: bytcr_rt5651: Change defaults to enable jack-detect, analog mics Hans de Goede
2018-03-07 14:21   ` Applied "ASoC: Intel: bytcr_rt5651: Change defaults to enable jack-detect, analog mics" to the asoc tree Mark Brown
2018-02-25 10:47 ` [PATCH v2 32/32] ASoC: Intel: bytcr_rt5651: Select RCCLK on init() Hans de Goede
2018-03-07 14:21   ` Applied "ASoC: Intel: bytcr_rt5651: Select RCCLK on init()" to the asoc tree Mark Brown
2018-02-25 10:58 ` [PATCH v2 00/32] ASoC: rt5651: jack-detect fixes and improvements Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180225104713.4745-5-hdegoede@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=bardliao@realtek.com \
    --cc=broonie@kernel.org \
    --cc=carlo@endlessm.com \
    --cc=oder_chiou@realtek.com \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=tiwai@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.