From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759277AbeAIQk2 (ORCPT + 1 other); Tue, 9 Jan 2018 11:40:28 -0500 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:51856 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755613AbeAIQkY (ORCPT ); Tue, 9 Jan 2018 11:40:24 -0500 Date: Tue, 9 Jan 2018 16:39:43 +0000 From: Mark Brown To: Matthias Kaehlcke Cc: Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Heiko Stuebner , alsa-devel@alsa-project.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Brian Norris , Dylan Reid Subject: Re: [PATCH] ASoC: rockchip: i2s: Support mono capture Message-ID: <20180109163943.GC11471@sirena.org.uk> References: <20180105221242.22083-1-mka@chromium.org> <20180109000115.GA184703@google.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="9Ek0hoCL9XbhcSqy" Content-Disposition: inline In-Reply-To: <20180109000115.GA184703@google.com> X-Cookie: Check here if tax deductible. User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: --9Ek0hoCL9XbhcSqy Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Jan 08, 2018 at 04:01:15PM -0800, Matthias Kaehlcke wrote: > Due to different issues in our audio recording path I couldn't test > this properly, and only looked at the .wav data in audacity, which > appeared to look ok. After putting some of the missing pieces > together and I noticed that mono capture doesn't work as intended, > the recorded audio is slowed down. I think this is because the I2S > controller keeps generating a two-channel stream, however the sound > system interprets it as a monophonic stream because of the channel > configuration. That seems odd - I'd expect underruns somewhere? > Not sure if there is a clean solution for this if we can't tell the > hardware to generate a single-channel stream. Any suggestions or do > we have to revert the patch? It sounds like a very standard problem with I2S devices, it's a fundamentally stereo format after all. If you can't get it to discard the second channel there's probably not much doing. --9Ek0hoCL9XbhcSqy Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlpU8E4ACgkQJNaLcl1U h9CNggf+M899bG3zeS/CrKb2ovGdRSbS0DqZCMjSO1u8VaLwabjfhEw20uDjkKXh OnjOKE0AQVkCvSJS4hCdbw1xg+ObkCbkjcq2rGlR1ln4ANjfhvz9VO9xTFe5NvG4 zOyczymwwuLKWJX4p7Gsim+wsLwdUnMc4zcvP/QWkAHLcPNrMtYbed0AK7qDc0Ou CuoBOCKc7idtJjXpIj6nQitUpFzaj6l4+WTSMWflNlLl/MS9XfuKwGHP1NmNCVJ3 k/JVto6+HheqiwNSrQVABEl8J+IUWmeqT79E+UOq8wC5JIB1zhIXerGRB40sYTUb zxwNvegTcVjXRdCmtn+CaTx66tOXyA== =l2Wj -----END PGP SIGNATURE----- --9Ek0hoCL9XbhcSqy--