[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2a4f9f583b5e4495b9fb50a446c2c949@ti.com>
Date: Fri, 16 Aug 2024 13:41:46 +0000
From: "Ding, Shenghao" <shenghao-ding@...com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
CC: "broonie@...nel.org" <broonie@...nel.org>,
"lgirdwood@...il.com"
<lgirdwood@...il.com>,
"perex@...ex.cz" <perex@...ex.cz>,
"pierre-louis.bossart@...ux.intel.com"
<pierre-louis.bossart@...ux.intel.com>,
"13916275206@....com"
<13916275206@....com>,
"zhourui@...qin.com" <zhourui@...qin.com>,
"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
"Salazar, Ivan"
<i-salazar@...com>,
"linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>,
"Chadha, Jasjot Singh" <j-chadha@...com>,
"liam.r.girdwood@...el.com" <liam.r.girdwood@...el.com>,
"Yue, Jaden"
<jaden-yue@...com>,
"yung-chuan.liao@...ux.intel.com"
<yung-chuan.liao@...ux.intel.com>,
"Rao, Dipa" <dipa@...com>, "yuhsuan@...gle.com" <yuhsuan@...gle.com>,
"Lo, Henry" <henry.lo@...com>, "tiwai@...e.de" <tiwai@...e.de>,
"Xu, Baojun" <baojun.xu@...com>,
"Baojun.Xu@....com" <Baojun.Xu@....com>,
"judyhsiao@...gle.com"
<judyhsiao@...gle.com>,
"Navada Kanyana, Mukund" <navada@...com>,
"cujomalainey@...gle.com" <cujomalainey@...gle.com>,
"Kutty, Aanya"
<aanya@...com>,
"Mahmud, Nayeem" <nayeem.mahmud@...com>,
"savyasanchi.shukla@...radyne.com" <savyasanchi.shukla@...radyne.com>,
"flaviopr@...rosoft.com" <flaviopr@...rosoft.com>,
"Ji, Jesse"
<jesse-ji@...com>,
"darren.ye@...iatek.com" <darren.ye@...iatek.com>,
"antheas.dk@...il.com" <antheas.dk@...il.com>,
"Jerry2.Huang@...uturecenter.com" <Jerry2.Huang@...uturecenter.com>
Subject: RE: [EXTERNAL] Re: [PATCH v1] ASoc: tas2781: Rename dai_driver name
to unify the name between TAS2563 and TAS2781
Hi Andy
> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> Sent: Thursday, August 15, 2024 8:04 PM
> To: Ding, Shenghao <shenghao-ding@...com>
> Cc: broonie@...nel.org; lgirdwood@...il.com; perex@...ex.cz; pierre-
> louis.bossart@...ux.intel.com; 13916275206@....com; zhourui@...qin.com;
> alsa-devel@...a-project.org; Salazar, Ivan <i-salazar@...com>; linux-
> kernel@...r.kernel.org; Chadha, Jasjot Singh <j-chadha@...com>;
> liam.r.girdwood@...el.com; Yue, Jaden <jaden-yue@...com>; yung-
> chuan.liao@...ux.intel.com; Rao, Dipa <dipa@...com>; yuhsuan@...gle.com;
> Lo, Henry <henry.lo@...com>; tiwai@...e.de; Xu, Baojun <baojun.xu@...com>;
> soyer@....hu; Baojun.Xu@....com; judyhsiao@...gle.com; Navada Kanyana,
> Mukund <navada@...com>; cujomalainey@...gle.com; Kutty, Aanya
> <aanya@...com>; Mahmud, Nayeem <nayeem.mahmud@...com>;
> savyasanchi.shukla@...radyne.com; flaviopr@...rosoft.com; Ji, Jesse <jesse-
> ji@...com>; darren.ye@...iatek.com; antheas.dk@...il.com;
> Jerry2.Huang@...uturecenter.com
> Subject: Re: [EXTERNAL] Re: [PATCH v1] ASoc: tas2781: Rename dai_driver
> name to unify the name between TAS2563 and TAS2781
>
> On Thu, Aug 15, 2024 at 03: 02: 01AM +0000, Ding, Shenghao wrote: > > From:
> Andy Shevchenko <andriy. shevchenko@ linux. intel. com> > > Sent: Monday,
> August 12, 2024 11: 52 PM > > On Sat, Aug 03, 2024 at 11: 27: 14AM +0800,
> Shenghao 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!uBdnVVfn3ofU5Y7otlzPNHz8qK-
> 2IJyfkpc722d3LxLdT3TGK8e6EW9B1-j1y1PtAwEKJhr-ue39zDQ-QZA$>
> Report Suspicious
>
> ZjQcmQRYFpfptBannerEnd
> On Thu, Aug 15, 2024 at 03:02:01AM +0000, Ding, Shenghao wrote:
> > > From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> > > Sent: Monday, August 12, 2024 11:52 PM On Sat, Aug 03, 2024 at
> > > 11:27:14AM +0800, Shenghao Ding wrote:
>
> ...
>
> > > > + 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?
>
> 1. It does.
> 2. No, I meant strscpy().
I have tried 2-parameter strscpy(), and occurred compiling issue.
The first parameter of strscpy, *dst, must be array, but in this code
prog_name and conf_name are points to the memories applied by
devm_kcalloc.
>
> --
> With Best Regards,
> Andy Shevchenko
>
Powered by blists - more mailing lists