Hi Stephen,
On Friday 24 August 2012 21:20:15 Stephen Warren wrote:
> From: Stephen Warren <swarren@xxxxxxxxxx>
>
> Both the schematics and practical testing show that the HP detect GPIO
> is high when the headphones are plugged in. Hence, the snd_soc_jack_gpio
> should not specify to invert the signal.
>
> Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx>
> Cc: Marc Dietrich <marvin24@xxxxxx>
> Cc: Leon Romanovsky <leon@xxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx> # v3.4 v3.5
I tested it and evtest gives back the right stuff with your patch. Can't test
more because I have no headphone/mic here (I just inserted a screwdriver,
grr). On 3.1 kernels, we also have no pin inversion, so unless Leon or Andrey
(cc'ed) have different opinions, this can go through. Just give them a week...
Thanks!
Marc
> ---
> sound/soc/tegra/tegra_alc5632.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/sound/soc/tegra/tegra_alc5632.c
> b/sound/soc/tegra/tegra_alc5632.c index e463529..76cb1b3 100644
> --- a/sound/soc/tegra/tegra_alc5632.c
> +++ b/sound/soc/tegra/tegra_alc5632.c
> @@ -89,7 +89,6 @@ static struct snd_soc_jack_gpio tegra_alc5632_hp_jack_gpio
> = { .name = "Headset detection",
> .report = SND_JACK_HEADSET,
> .debounce_time = 150,
> - .invert = 1,
> };
>
> static const struct snd_soc_dapm_widget tegra_alc5632_dapm_widgets[] = {
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
