diff mbox

[3/3] ASoC: Intel: cht_bsw_rt5672: fix card name

Message ID 20170908011256.7706-4-pierre-louis.bossart@linux.intel.com (mailing list archive)
State Accepted
Headers show

Commit Message

Pierre-Louis Bossart Sept. 8, 2017, 1:12 a.m. UTC
For now reason the current card name is a ridiculous
'cherrytrailcraudio'. This isn't very useful or self-explanatory,
change to driver name cht-bsw-rt5672.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
 sound/soc/intel/boards/cht_bsw_rt5672.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Takashi Iwai Sept. 8, 2017, 8:16 a.m. UTC | #1
On Fri, 08 Sep 2017 03:12:56 +0200,
Pierre-Louis Bossart wrote:
> 
> For now reason the current card name is a ridiculous
> 'cherrytrailcraudio'. This isn't very useful or self-explanatory,
> change to driver name cht-bsw-rt5672.

One bonus by this change would be to avoid inconsistent names by the
driver's name string size.  Since it's 16 bytes including \0, the
name is shortened as cherrytrailcrau or such in some fields.  With the
new name this shouldn't happen.

In anyway,
  Reviewed-by: Takashi Iwai <tiwai@suse.de>


thanks,

Takashi
diff mbox

Patch

diff --git a/sound/soc/intel/boards/cht_bsw_rt5672.c b/sound/soc/intel/boards/cht_bsw_rt5672.c
index 6d077de..b13d6222 100644
--- a/sound/soc/intel/boards/cht_bsw_rt5672.c
+++ b/sound/soc/intel/boards/cht_bsw_rt5672.c
@@ -384,7 +384,7 @@  static int cht_resume_post(struct snd_soc_card *card)
 
 /* SoC card */
 static struct snd_soc_card snd_soc_card_cht = {
-	.name = "cherrytrailcraudio",
+	.name = "cht-bsw-rt5672",
 	.owner = THIS_MODULE,
 	.dai_link = cht_dailink,
 	.num_links = ARRAY_SIZE(cht_dailink),