[alsa-devel] [PATCH 09/17] ASoC: Tegra I2S: Remove dependency on pdev->id

Stephen Warren swarren at nvidia.com
Wed Nov 23 18:54:21 CET 2011


Mark Brown wrote at Wednesday, November 23, 2011 4:04 AM:
> On Tue, Nov 22, 2011 at 06:21:17PM -0700, Stephen Warren wrote:
> 
> > +	i2s->dai = tegra_i2s_dai_template;
> > +	i2s->dai.name = pdev->name;
> > +
> 
> This should really be part of the same commit as the arch/arm side
> change in order to avoid bisection breaks as from the point of view of
> non-DT systems it's moving the number into the name.

Sorry, I don't quite follow here; the original hard-coded dai structures
already had the ID in the name, and hence exactly match the platform
device names tegra-i2s.0 and tegra-i2s.1:

static struct snd_soc_dai_driver tegra_i2s_dai[] = {
	{
		.name = DRV_NAME ".0",
...
	},
	{
		.name = DRV_NAME ".1",

Or were you referring the AUXDATA fix for tegra-i2s.1? That's not an
issue since nothing uses the 2nd I2C controller on Tegra in mainline
yet.

-- 
nvpublic



More information about the Alsa-devel mailing list