RE: [EXTERNAL] Re: [PATCH v1] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andy
> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Sent: Monday, August 12, 2024 11:52 PM
> To: Ding, Shenghao <shenghao-ding@xxxxxx>
> Cc: broonie@xxxxxxxxxx; lgirdwood@xxxxxxxxx; perex@xxxxxxxx; pierre-
> louis.bossart@xxxxxxxxxxxxxxx; 13916275206@xxxxxxx; zhourui@xxxxxxxxxx;
> alsa-devel@xxxxxxxxxxxxxxxx; Salazar, Ivan <i-salazar@xxxxxx>; linux-
> kernel@xxxxxxxxxxxxxxx; Chadha, Jasjot Singh <j-chadha@xxxxxx>;
> liam.r.girdwood@xxxxxxxxx; Yue, Jaden <jaden-yue@xxxxxx>; yung-
> chuan.liao@xxxxxxxxxxxxxxx; Rao, Dipa <dipa@xxxxxx>; yuhsuan@xxxxxxxxxx;
> Lo, Henry <henry.lo@xxxxxx>; tiwai@xxxxxxx; Xu, Baojun <baojun.xu@xxxxxx>;
> soyer@xxxxxx; Baojun.Xu@xxxxxxx; judyhsiao@xxxxxxxxxx; Navada Kanyana,
> Mukund <navada@xxxxxx>; cujomalainey@xxxxxxxxxx; Kutty, Aanya
> <aanya@xxxxxx>; Mahmud, Nayeem <nayeem.mahmud@xxxxxx>;
> savyasanchi.shukla@xxxxxxxxxxxxx; flaviopr@xxxxxxxxxxxxx; Ji, Jesse <jesse-
> ji@xxxxxx>; darren.ye@xxxxxxxxxxxx; antheas.dk@xxxxxxxxx;
> Jerry2.Huang@xxxxxxxxxxxxxxxxxx
> Subject: [EXTERNAL] Re: [PATCH v1] ASoc: tas2781: Rename dai_driver name
> to unify the name between TAS2563 and TAS2781
> 
> On Sat, Aug 03, 2024 at 11: 27: 14AM +0800, Shenghao Ding wrote: > Rename
> dai_driver name to unify the name between TAS2563 and > TAS2781, remove
> unnecessary line feed, strscpy replace scnprintf. .. . > + strscpy(name, "Speaker
> Profile ZjQcmQRYFpfptBannerStart This message was sent from outside of
> Texas Instruments.
> Do not click links or open attachments unless you recognize the source of this
> email and know the content is safe.
> <https://us-phishalarm-
> ewt.proofpoint.com/EWT/v1/G3vK!uBdnVVxnvcFVRgmk1Nnj19hgRvdKBLGzO0
> fOFtIPHgaMy7sMNbJz9Iq6JNvpJGah386MvWqQ0VSy_XxhVlo$>
> Report Suspicious
> 
> ZjQcmQRYFpfptBannerEnd
> On Sat, Aug 03, 2024 at 11:27:14AM +0800, Shenghao Ding wrote:
> > Rename dai_driver name to unify the name between TAS2563 and TAS2781,
> > remove unnecessary line feed, strscpy replace scnprintf.
> 
> ...
> 
> > +	strscpy(name, "Speaker Profile Id",
> SNDRV_CTL_ELEM_ID_NAME_MAXLEN);
> 
> > +	strscpy(prog_name, "Speaker Program Id",
> > +		SNDRV_CTL_ELEM_ID_NAME_MAXLEN);
> 
> > +	strscpy(conf_name, "Speaker Config Id",
> > +SNDRV_CTL_ELEM_ID_NAME_MAXLEN);
> 
> Why not 2-parameter strscpy()?
strscpy seemed not support 2 params. Do you mean strcpy? 
> 
> --
> With Best Regards,
> Andy Shevchenko
> 





[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux