Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 29 Mar 2022 12:30:57 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Sameer Pujar <spujar@nvidia.com>
Cc:     robh+dt@kernel.org, krzk+dt@kernel.org, thierry.reding@gmail.com,
        lgirdwood@gmail.com, perex@perex.cz, tiwai@suse.com,
        jonathanh@nvidia.com, catalin.marinas@arm.com, will@kernel.org,
        alsa-devel@alsa-project.org, devicetree@vger.kernel.org,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/6] ASoC: tegra: Add Tegra186 based ASRC driver
Message-ID: <YkLt8U5MnG+0gFxl@sirena.org.uk>
References: <1648447526-14523-1-git-send-email-spujar@nvidia.com>
 <1648447526-14523-3-git-send-email-spujar@nvidia.com>
 <YkHX3/8BbXo4obWI@sirena.org.uk>
 <4e8bd876-48ed-8fd8-7b7b-989b45b54f1e@nvidia.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="V7EwDDVgYNffjf6e"
Content-Disposition: inline
In-Reply-To: <4e8bd876-48ed-8fd8-7b7b-989b45b54f1e@nvidia.com>
Precedence: bulk


--V7EwDDVgYNffjf6e
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Mar 29, 2022 at 02:02:34PM +0530, Sameer Pujar wrote:
> On 28-03-2022 21:14, Mark Brown wrote:
> > On Mon, Mar 28, 2022 at 11:35:22AM +0530, Sameer Pujar wrote:

> > > +	/* Source of ratio provider */
> > > +	SOC_ENUM_EXT("Ratio1 Source", src_select1,
> > > +		     tegra186_asrc_get_ratio_source,
> > > +		     tegra186_asrc_put_ratio_source),

> > ...the sources?  Or does it need to be configured before either side is
> > ready in which case this might be the best we can do for now.

> The ratio needs to be updated before start of the stream and this
> programming via controls is required only when the ratio source is SW.
> When ratio detector module is used (support is not yet added), the
> ratio is automatically updated by HW.

OK, that's not ideal but should be fine for now.  Ideally we'd have the
rate detector support introduced in the same release.

--V7EwDDVgYNffjf6e
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmJC7fAACgkQJNaLcl1U
h9ANSAf+IOqA/07bEMbKWE9/DpOYo8OUux8tdLQWz4Ll7egQt8NHkvEo388AqOTH
t/y6I6zjXwXkALN/IguSFatfz11OkdNuL454wozODL795TEtdJQKTbPuTIM+NGWZ
i4tqNr/nXk9jWWBeeSK7esZIS+a3c/W6e2b54XH1+P7nCYAJoMNW0RtKWIvXWBa/
lzZACNRH6kojanb4bC/BZMIuOVoqOWxFWC5j6dVdO24Z0OWHkPZ11FN+8dp+f3nY
YkvXv2WyEsg2syC6OzZVBn/BQcWgc3p+Yks54AEsNyUdHfyefggOAQ/R3Rj4oZp/
njbYdmDQ1gNUsMD9M3IX0xhoEGkbtQ==
=dnPU
-----END PGP SIGNATURE-----

--V7EwDDVgYNffjf6e--