Message ID | 20170908011256.7706-4-pierre-louis.bossart@linux.intel.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
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 --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),
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(-)