Re: [PATCH] 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]

To: Shenghao Ding <shenghao-ding@xxxxxx>
Subject: Re: [PATCH] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781
From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Date: Mon, 12 Aug 2024 20:12:05 +0300
Cc: broonie@xxxxxxxxxx, lgirdwood@xxxxxxxxx, perex@xxxxxxxx, pierre-louis.bossart@xxxxxxxxxxxxxxx, 13916275206@xxxxxxx, zhourui@xxxxxxxxxx, alsa-devel@xxxxxxxxxxxxxxxx, i-salazar@xxxxxx, linux-kernel@xxxxxxxxxxxxxxx, j-chadha@xxxxxx, liam.r.girdwood@xxxxxxxxx, jaden-yue@xxxxxx, yung-chuan.liao@xxxxxxxxxxxxxxx, dipa@xxxxxx, yuhsuan@xxxxxxxxxx, henry.lo@xxxxxx, tiwai@xxxxxxx, baojun.xu@xxxxxx, soyer@xxxxxx, Baojun.Xu@xxxxxxx, judyhsiao@xxxxxxxxxx, navada@xxxxxx, cujomalainey@xxxxxxxxxx, aanya@xxxxxx, nayeem.mahmud@xxxxxx, savyasanchi.shukla@xxxxxxxxxxxxx, flaviopr@xxxxxxxxxxxxx, jesse-ji@xxxxxx, darren.ye@xxxxxxxxxxxx, antheas.dk@xxxxxxxxx, Jerry2.Huang@xxxxxxxxxxxxxxxxxx
In-reply-to: <20240806041719.1477-1-shenghao-ding@ti.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
References: <20240806041719.1477-1-shenghao-ding@ti.com>

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?

If there is no issue for such configurations, please mention this fact in the
commit message.

-- 
With Best Regards,
Andy Shevchenko

Follow-Ups:
- RE: [EXTERNAL] Re: [PATCH] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781
  - From: Ding, Shenghao

References:
- [PATCH] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781
  - From: Shenghao Ding

Prev by Date: Re: [PATCH 00/12] ASoC: Intel: Remove skylake driver
Next by Date: [PATCH 0/2] Add a driver for the Iron Device SMA1307 Amp
Previous by thread: [PATCH] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781
Next by thread: RE: [EXTERNAL] Re: [PATCH] ASoc: tas2781: Rename dai_driver name to unify the name between TAS2563 and TAS2781
Index(es):
- Date
- Thread

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