Hi Andy Answer inline From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Sent: Tuesday, August 13, 2024 1:12 AM 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] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781 On Tue, Aug 06, 2024 at 12: 17: 17PM +0800, Shenghao Ding wrote: > Rename dai_driver name to unify the name between TAS2563 and > TAS2781. I am not sure this is a good change. Theoretically one with the platform that has a certain codec 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. Report Suspicious <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uBdnVV-BGyfThcmOspiPk_6mgviDIpMpU9lNaStHOlAnmlORtYg0KTnMhfSSI5acx0kk2CbMnnsjoWU-o8U$> ZjQcmQRYFpfptBannerEnd On Tue, Aug 06, 2024 at 12:17:17PM +0800, Shenghao Ding wrote: > Rename dai_driver name to unify the name between TAS2563 and > TAS2781. I am not sure this is a good change. Theoretically one with the platform that has a certain codec may connect the other one. How that will be distinguished? [DING] As you know, this driver code can support both tas2563 and tas2781. But some tas2563 customers feel confused and uncomfortable, when they found the dai_driver name is still tas2781. So, update the name for them If there is no issue for such configurations, please mention this fact in the commit message. [DING] Accept -- With Best Regards, Andy Shevchenko