linux-kernel - Re: Nokia N900 - audio TPA6130A2 problems

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 25 Jul 2015 15:17:13 +0200
From:	Lars-Peter Clausen <lars@...afoo.de>
To:	Pali Rohár <pali.rohar@...il.com>,
	Jarkko Nikula <jarkko.nikula@...mer.com>,
	Peter Ujfalusi <peter.ujfalusi@...com>
CC:	linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org,
	Pavel Machek <pavel@....cz>,
	Sebastian Reichel <sre@...nel.org>,
	Aaro Koskinen <aaro.koskinen@....fi>,
	Tony Lindgren <tony@...mide.com>, Nishanth Menon <nm@...com>
Subject: Re: Nokia N900 - audio TPA6130A2 problems

On 07/25/2015 12:28 PM, Pali Rohár wrote:
> Hello,
>
> sometimes after rebooting Nokia N900 initializing alsa audio fails.
> Here output from dmesg log when it happen:
>
> [    6.925140] tpa6130a2 2-0060: Write failed
> [    6.929534] tpa6130a2 2-0060: Failed to initialize chip
> [    6.935272] tpa6130a2: probe of 2-0060 failed with error -121
> [    7.624237] rx51-audio n900-audio: Failed to add TPA6130A2 controls
> [    7.635101] rx51-audio n900-audio: ASoC: failed to init TLV320AIC34: -19
> [    7.645874] rx51-audio n900-audio: ASoC: failed to instantiate card -19
> [    7.665740] rx51-audio n900-audio: snd_soc_register_card failed (-19)
> [    8.063049] ALSA device list:
> [    8.070343]   No soundcards found.
>
> Any idea what to do?

Looks like the chip is not responding. Try to add a small delay after 
powerup to give the device to be fully ready, something like the following:

--- a/sound/soc/codecs/tpa6130a2.c
+++ b/sound/soc/codecs/tpa6130a2.c
@@ -152,6 +152,8 @@ static int tpa6130a2_power(u8 power)
  		if (data->power_gpio >= 0)
  			gpio_set_value(data->power_gpio, 1);

+		msleep(5);
+
  		data->power_state = 1;
  		ret = tpa6130a2_initialize();
  		if (ret < 0) {

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/