Mark Brown is the maintainer of sound tree. He is included in cc: broonie@xxxxxxxxxx :) -----Original Message----- From: Bastien Nocera [mailto:hadess@xxxxxxxxxx] Sent: Wednesday, July 20, 2016 4:57 PM To: Koul, Vinod <vinod.koul@xxxxxxxxx> Cc: alsa-devel@xxxxxxxxxxxxxxxx; broonie@xxxxxxxxxx; liam.r.girdwood@xxxxxxxxxxxxxxx; Patches Audio <patches.audio@xxxxxxxxx>; Stephen Just <stephenjust@xxxxxxxxx>; Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>; apterix@xxxxxxxxx; bugzilla@xxxxxxxxxx; Mokashi, SachinX <sachinx.mokashi@xxxxxxxxx> Subject: Re: [PATCH 3/3] ASoC: Intel: Add surface3 entry in CHT-RT5645 machine On Wed, 2016-07-20 at 09:28 +0530, Vinod Koul wrote: > On Tue, Jul 19, 2016 at 04:56:24PM +0200, Bastien Nocera wrote: > > > + /* fixup codec name based on HID */ > > > + i2c_name = sst_acpi_find_name_from_hid(mach->id); > > > + if (i2c_name != NULL) { > > > + snprintf(cht_rt5640_codec_name, > > > sizeof(cht_rt5640_codec_name), > > > + "%s%s", "i2c-", i2c_name); > > > + cht_dailink[dai_index].codec_name = > > > cht_rt5640_codec_name; > > > > FWIW, I get: > > sound/soc/intel/boards/cht_bsw_rt5645.c: In function > > 'snd_cht_mc_probe': > > sound/soc/intel/boards/cht_bsw_rt5645.c:391:37: warning: > > 'dai_index' may be used uninitialized in this function [-Wmaybe- > > uninitialized] > > cht_dailink[dai_index].codec_name = cht_rt5640_codec_name; > > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~ > > Yeah I notice them a bit later. The fixes are already in mark's-next. Care to be a bit more specific. Kind of hard to know which Mark you're talking about when there's no one of that name in the CC: list above. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
