From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751699AbcFSJjk (ORCPT ); Sun, 19 Jun 2016 05:39:40 -0400 Received: from www381.your-server.de ([78.46.137.84]:53882 "EHLO www381.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751631AbcFSJjh (ORCPT ); Sun, 19 Jun 2016 05:39:37 -0400 Subject: Re: [PATCH 4/4] ASoC: tpa6130a2: Add DAPM support To: Sebastian Reichel , Helen Koike References: <2aed60529eea206f2192a7236f2d80386a73f2ee.1466218805.git.helen.koike@collabora.co.uk> <20160618232131.GF5510@earth> Cc: lgirdwood@gmail.com, broonie@kernel.org, peter.ujfalusi@ti.com, jarkko.nikula@bitmer.com, k.kozlowski@samsung.com, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, perex@perex.cz, tiwai@suse.com From: Lars-Peter Clausen Message-ID: <57666843.2020604@metafoo.de> Date: Sun, 19 Jun 2016 11:39:15 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.8.0 MIME-Version: 1.0 In-Reply-To: <20160618232131.GF5510@earth> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ECqn3XmgfkV6ebxCSNkUbkHE6Xcw6eBDp" X-Authenticated-Sender: lars@metafoo.de Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --ECqn3XmgfkV6ebxCSNkUbkHE6Xcw6eBDp Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 06/19/2016 01:21 AM, Sebastian Reichel wrote: > Hi, >=20 > On Sat, Jun 18, 2016 at 12:26:31AM -0300, Helen Koike wrote: >> Add DAPM support and updated rx51 accordingly. >> As a consequence: >> - the exported function tpa6130a2_stereo_enable is not needed anymore >> - the mutex is dealt in the DAPM >> - the power state is tracked by the DAPM >=20 > This breaks headphone on N900. So far I only checked, that > tpa6130a2_power_event() is not called. I guess the DAPM > graph is wrong. Can you try with + {"TPA6130A2 HPLEFT", NULL, "LLOUT"}, + {"TPA6130A2 HPRIGHT", NULL, "RLOUT"} being replaced with + {"TPA6130A2 LEFTIN", NULL, "LLOUT"}, + {"TPA6130A2 RIGHTIN", NULL, "RLOUT"}, Thanks, - Lars --ECqn3XmgfkV6ebxCSNkUbkHE6Xcw6eBDp Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJXZmhKAAoJEOj3aLScRbOZ1+EP/jwQj/N9qQZA7KJ1xETgzWn8 MVhRgzAok3jw+Xnf3gQTLd5CphAuQYlT+V0/muzuvV1ohJXyojgehRKB+MwwMdI7 bOrhvXwW5WeLnWvlaAc0Ux/Ae2jetQsnqkh7cpXojPKHTL+n7obsgeJD3I59rQYI B9Nbf1bllx+JEfp8bHqkMUL3Xo4eZSPQS63OqpBYDIkqXx6A1p2Dw7O0Rn1EGtEZ Idq4dl11gkcwUFgg9FBVzfCcIYjC3e7GuxYOCY5IIR6IHltrDD47im/umDoyLKMR Oee8VCO5vq5N04agemgLVLaHymosTki/CQYONRxofmzmY+PYO5qgQL0ZUMLJoosE K9lxJP81yBpo7Yh2PZWtm4RVQ6ukQZkjBtoDOocrmS8VxK4LhIk6wIZC8QaOl4eS UdPb1/roYBw13U7hT6LObaYP1VADezdAi8EVYuQ3LD+QD68bRgxS62djUDaOiAJ7 HHsDDzfDxFm5cA2J5mcxXzGzS7riUIHz3oowAbwG+vgEOXRCyPf/U7rdEA30EiFm 3iOPXscctHpJLK1zgCeVH+6WlKATsis5mz6ZkFTl31aYQoqvLZmi9pNh1X0fpsh+ Pf4G09ewMR49du5OVnpovCadIbaIxoyR6mHkwZyXTbCPl3AsYLQdiMxlh19CuGfg BTw1Nuw05GWAcM+fkTSF =Q/7I -----END PGP SIGNATURE----- --ECqn3XmgfkV6ebxCSNkUbkHE6Xcw6eBDp--