>-----Original Message----- >From: Takashi Iwai [mailto:tiwai@xxxxxxx] >Sent: Friday, February 23, 2018 10:22 PM >To: Ughreja, Rakesh A <rakesh.a.ughreja@xxxxxxxxx> >Cc: alsa-devel@xxxxxxxxxxxxxxxx; broonie@xxxxxxxxxx; >liam.r.girdwood@xxxxxxxxxxxxxxx; Koul, Vinod <vinod.koul@xxxxxxxxx>; Patches >Audio <patches.audio@xxxxxxxxx>; pierre-louis.bossart@xxxxxxxxxxxxxxx >Subject: Re: [PATCH v1 5/9] ALSA: hda - split snd_hda_codec_new >function > >On Fri, 23 Feb 2018 09:12:26 +0100, >Rakesh Ughreja wrote: >> >> +int snd_hda_codec_device_new(struct hda_bus *bus, struct snd_card *card, >> + unsigned int codec_addr, struct hda_codec *codec) >> +{ >> char component[31]; > >Unused after the patch? We still use the component variable to construct the name, so I think we still need it. Following code, + sprintf(component, "hdaudioC%dD%d", card->number, codec_addr); + err = snd_hdac_device_init(&codec->core, &bus->core, component, + codec_addr); Regards, Rakesh _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
