* [PATCH v2 0/2] ASoC: Use DMI name in sound card long name
@ 2017-01-03 4:58 mengdong.lin
2017-01-03 4:58 ` [PATCH v2 1/2] ASoC: core: Add API to use " mengdong.lin
2017-01-03 5:00 ` [PATCH v2 2/2] ASoC: Intel: Use DMI name for sound card long name in Broadwell machine driver mengdong.lin
0 siblings, 2 replies; 11+ messages in thread
From: mengdong.lin @ 2017-01-03 4:58 UTC (permalink / raw)
To: alsa-devel, broonie
Cc: Mengdong Lin, tiwai, mengdong.lin, liam.r.girdwood, vinod.koul,
pierre-louis.bossart
From: Mengdong Lin <mengdong.lin@linux.intel.com>
This series is based on Liam's idea to use DMI name as the sound card
short name in the audio workshop. Now we add DMI info to the card long
name:
- Not change the card name (short name), keep it simple. So the card ID
won't change as well.
- Use DMI name to make the card long name.
Machine drivers can use a new API to get this feature.
In user space, when the sound server ask UCM to open a sound card, UCM
will try to find the best machine-specific configuration file based on
the card long name, and only if not avaiable, fallback to the default
configuration file that matches the card name (short name).
Please see patch:
ucm: Automatically load the best config file based on the card long name
History:
v1 from RFC: Give up using a table to map DMI info to a simple string,
since we cannot maintain such a table for each machine. Just do
verbatim copy of DMI info and let user space decide how to use the
DMI info in card long name to find the best UCM config file.
If device driver adds a flavor string, just append it after the
DMI info fields to the card long name.
v2: Exclude card name and only keep DMI name in the long name, to
save the space since the long name cannot exceed 80 characters.
Trim special characters like SPACE in the long name, because UCM
configuration files & directories may use the same long name and
Autoconf cannot handle special characters.
Liam Girdwood (1):
ASoC: core: Add API to use DMI name in sound card long name
Mengdong Lin (1):
ASoC: Intel: Use DMI name for sound card long name in Broadwell
machine driver
include/sound/soc.h | 4 ++
sound/soc/intel/boards/broadwell.c | 2 +
sound/soc/soc-core.c | 99 ++++++++++++++++++++++++++++++++++++++
3 files changed, 105 insertions(+)
--
2.5.0
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 4:58 [PATCH v2 0/2] ASoC: Use DMI name in sound card long name mengdong.lin @ 2017-01-03 4:58 ` mengdong.lin 2017-01-03 14:40 ` Liam Girdwood 2017-01-03 5:00 ` [PATCH v2 2/2] ASoC: Intel: Use DMI name for sound card long name in Broadwell machine driver mengdong.lin 1 sibling, 1 reply; 11+ messages in thread From: mengdong.lin @ 2017-01-03 4:58 UTC (permalink / raw) To: alsa-devel, broonie Cc: Mengdong Lin, tiwai, mengdong.lin, liam.r.girdwood, vinod.koul, pierre-louis.bossart From: Liam Girdwood <liam.r.girdwood@linux.intel.com> Intel DSP platform drivers are used by many different devices but are difficult for userspace to differentiate. This patch adds an API to allow the DMI name to be used in the sound card long name, thereby helping userspace load the correct UCM configuration. Usually machine drivers uses their own name as the sound card name (short name), and leave the long name and driver name blank. This API will use the DMI info like vendor, product and board to make up the card long name. If the machine driver has already explicitly set the long name, this API will do nothing. This patch also allows for further differentiation as many devices that share the same DMI name i.e. Minnowboards, UP boards may be configured with different codecs or firmwares. The API supports flavoring the DMI name into the card longname to provide the extra differentiation required for these devices. For Use Case Manager (UCM) in the user space, changing card long name by this API is backward compatible, since the card name does not change. For a given sound card, even if there is no device-specific UCM configuration file that uses the card long name, UCM will fall back to load the default configuration file that uses the card name. Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com> Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> diff --git a/include/sound/soc.h b/include/sound/soc.h index 795e6c4..e4f1844 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -497,6 +497,8 @@ void snd_soc_runtime_deactivate(struct snd_soc_pcm_runtime *rtd, int stream); int snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, unsigned int dai_fmt); +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour); + /* Utility functions to get clock rates from various things */ int snd_soc_calc_frame_size(int sample_size, int channels, int tdm_slots); int snd_soc_params_to_frame_size(struct snd_pcm_hw_params *params); @@ -1094,6 +1096,8 @@ struct snd_soc_card { const char *name; const char *long_name; const char *driver_name; + char dmi_longname[80]; + struct device *dev; struct snd_card *snd_card; struct module *owner; diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index aaab26a..3dbb5aa 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -34,6 +34,7 @@ #include <linux/ctype.h> #include <linux/slab.h> #include <linux/of.h> +#include <linux/dmi.h> #include <sound/core.h> #include <sound/jack.h> #include <sound/pcm.h> @@ -1886,6 +1887,104 @@ int snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, } EXPORT_SYMBOL_GPL(snd_soc_runtime_set_dai_fmt); + +/* Only keep number and alphabet characters and a few separator characters. + * DMI info often has SPACE and we must trim them, since Use Case Manager in + * the user space expects the device-specific configuration files/directories + * use the same name as the card long name but Autoconf cannot support SPACE + * in file or directory name. + */ +static void trim_special_characters(char *name) +{ + int i, j = 0; + + for (i = 0; name[i]; i++) { + if (isalnum(name[i]) || (name[i] == '.') + || (name[i] == '-') || (name[i] == '_')) + name[j++] = name[i]; + } + + name[j] = '\0'; +} + +/** + * snd_soc_set_dmi_name() - Register DMI names to card + * @card: The card to register DMI names + * @flavour: The flavour "differentiator" for the card amongst its peers. + * + * Intel DSP platform drivers are used by many different devices but are + * difficult for userspace to differentiate, since machine drivers ususally + * use their own name as the card name (short name) and leave the card long + * name blank. This function will allow DMI info to be used as the sound + * card long name, thereby helping userspace load the correct UCM (Use Case + * Manager) configuration. Character '.' are used to separate different DMI + * fields like "vendor.product.board". + * + * Possible card long names may be: + * DellInc..XPS139343.0310JH + * IntelCorp..BroadwellClientPlatform.WilsonBeachSDS + * ASUSTeKCOMPUTERINC..T100TA.T100TA + * Circuitco.MinnowboardMaxD0PLATFORM.MinnowBoardMAX + * (Please note DMI info can also include '.' after some abbreviation like + * "Inc." so you may see double '.' sometimes) + * + * This function also supports flavoring the card longname to provide + * the extra differentiation, like "vendor.product.board.flavor". + * + * We only keep number and alphabet characters and a few separator characters + * in the card long name since UCM in the user space uses the card long names + * as card configuration directory names and AudoConf cannot support special + * charactors like SPACE. + * + * Returns 0 on success, otherwise a negative error code. + */ +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour) +{ + const char *vendor, *product, *board; + char dmi_longname[80]; + + if (card->long_name) + return 0; /* long name already set by driver or from DMI */ + + vendor = dmi_get_system_info(DMI_BOARD_VENDOR); + if (!vendor) { + dev_warn(card->dev, "ASoC: no DMI vendor name!\n"); + return 0; + } + + product = dmi_get_system_info(DMI_PRODUCT_NAME); + board = dmi_get_system_info(DMI_BOARD_NAME); + if (!board && !product) { + /* fall back to using legacy name */ + dev_warn(card->dev, "ASoC: no DMI board/product name!\n"); + return 0; + } + + /* make up dmi long name as: vendor.product.board */ + if (product && board) + snprintf(dmi_longname, sizeof(card->snd_card->longname), + "%s.%s.%s", vendor, product, board); + else + snprintf(dmi_longname, sizeof(card->snd_card->longname), + "%s.%s", vendor, product ? product : board); + + /* Add flavour to dmi long name */ + if (flavour) + snprintf(card->dmi_longname, sizeof(card->snd_card->longname), + "%s.%s", dmi_longname, flavour); + else + strncpy(card->dmi_longname, dmi_longname, + sizeof(card->snd_card->longname) - 1); + + trim_special_characters(card->dmi_longname); + + /* set long name */ + card->long_name = card->dmi_longname; + + return 0; +} +EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name); + static int snd_soc_instantiate_card(struct snd_soc_card *card) { struct snd_soc_codec *codec; -- 2.5.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 4:58 ` [PATCH v2 1/2] ASoC: core: Add API to use " mengdong.lin @ 2017-01-03 14:40 ` Liam Girdwood 2017-01-03 15:44 ` Lin, Mengdong 0 siblings, 1 reply; 11+ messages in thread From: Liam Girdwood @ 2017-01-03 14:40 UTC (permalink / raw) To: mengdong.lin Cc: alsa-devel, tiwai, mengdong.lin, vinod.koul, broonie, pierre-louis.bossart On Tue, 2017-01-03 at 12:58 +0800, mengdong.lin@linux.intel.com wrote: > From: Liam Girdwood <liam.r.girdwood@linux.intel.com> > > Intel DSP platform drivers are used by many different devices but are > difficult for userspace to differentiate. This patch adds an API to allow > the DMI name to be used in the sound card long name, thereby helping > userspace load the correct UCM configuration. Usually machine drivers > uses their own name as the sound card name (short name), and leave the > long name and driver name blank. This API will use the DMI info like > vendor, product and board to make up the card long name. If the machine > driver has already explicitly set the long name, this API will do nothing. > > This patch also allows for further differentiation as many devices that > share the same DMI name i.e. Minnowboards, UP boards may be configured > with different codecs or firmwares. The API supports flavoring the DMI > name into the card longname to provide the extra differentiation required > for these devices. > > For Use Case Manager (UCM) in the user space, changing card long name by > this API is backward compatible, since the card name does not change. For > a given sound card, even if there is no device-specific UCM configuration > file that uses the card long name, UCM will fall back to load the default > configuration file that uses the card name. > > Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com> > Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> > > diff --git a/include/sound/soc.h b/include/sound/soc.h > index 795e6c4..e4f1844 100644 > --- a/include/sound/soc.h > +++ b/include/sound/soc.h > @@ -497,6 +497,8 @@ void snd_soc_runtime_deactivate(struct snd_soc_pcm_runtime *rtd, int stream); > int snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, > unsigned int dai_fmt); > > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour); > + > /* Utility functions to get clock rates from various things */ > int snd_soc_calc_frame_size(int sample_size, int channels, int tdm_slots); > int snd_soc_params_to_frame_size(struct snd_pcm_hw_params *params); > @@ -1094,6 +1096,8 @@ struct snd_soc_card { > const char *name; > const char *long_name; > const char *driver_name; > + char dmi_longname[80]; > + > struct device *dev; > struct snd_card *snd_card; > struct module *owner; > diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c > index aaab26a..3dbb5aa 100644 > --- a/sound/soc/soc-core.c > +++ b/sound/soc/soc-core.c > @@ -34,6 +34,7 @@ > #include <linux/ctype.h> > #include <linux/slab.h> > #include <linux/of.h> > +#include <linux/dmi.h> > #include <sound/core.h> > #include <sound/jack.h> > #include <sound/pcm.h> > @@ -1886,6 +1887,104 @@ int snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, > } > EXPORT_SYMBOL_GPL(snd_soc_runtime_set_dai_fmt); > > + > +/* Only keep number and alphabet characters and a few separator characters. > + * DMI info often has SPACE and we must trim them, since Use Case Manager in > + * the user space expects the device-specific configuration files/directories > + * use the same name as the card long name but Autoconf cannot support SPACE > + * in file or directory name. > + */ > +static void trim_special_characters(char *name) > +{ > + int i, j = 0; > + > + for (i = 0; name[i]; i++) { > + if (isalnum(name[i]) || (name[i] == '.') > + || (name[i] == '-') || (name[i] == '_')) > + name[j++] = name[i]; > + } > + > + name[j] = '\0'; > +} > + > +/** > + * snd_soc_set_dmi_name() - Register DMI names to card > + * @card: The card to register DMI names > + * @flavour: The flavour "differentiator" for the card amongst its peers. > + * > + * Intel DSP platform drivers are used by many different devices but are > + * difficult for userspace to differentiate, since machine drivers ususally > + * use their own name as the card name (short name) and leave the card long > + * name blank. This function will allow DMI info to be used as the sound > + * card long name, thereby helping userspace load the correct UCM (Use Case > + * Manager) configuration. Character '.' are used to separate different DMI > + * fields like "vendor.product.board". > + * > + * Possible card long names may be: > + * DellInc..XPS139343.0310JH > + * IntelCorp..BroadwellClientPlatform.WilsonBeachSDS > + * ASUSTeKCOMPUTERINC..T100TA.T100TA > + * Circuitco.MinnowboardMaxD0PLATFORM.MinnowBoardMAX > + * (Please note DMI info can also include '.' after some abbreviation like > + * "Inc." so you may see double '.' sometimes) > + * > + * This function also supports flavoring the card longname to provide > + * the extra differentiation, like "vendor.product.board.flavor". I dont think we need vendor (it just makes it too long). The product or board name should be unique enough for us to load the correct files. Liam > + * > + * We only keep number and alphabet characters and a few separator characters > + * in the card long name since UCM in the user space uses the card long names > + * as card configuration directory names and AudoConf cannot support special > + * charactors like SPACE. > + * > + * Returns 0 on success, otherwise a negative error code. > + */ > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour) > +{ > + const char *vendor, *product, *board; > + char dmi_longname[80]; > + > + if (card->long_name) > + return 0; /* long name already set by driver or from DMI */ > + > + vendor = dmi_get_system_info(DMI_BOARD_VENDOR); > + if (!vendor) { > + dev_warn(card->dev, "ASoC: no DMI vendor name!\n"); > + return 0; > + } > + > + product = dmi_get_system_info(DMI_PRODUCT_NAME); > + board = dmi_get_system_info(DMI_BOARD_NAME); > + if (!board && !product) { > + /* fall back to using legacy name */ > + dev_warn(card->dev, "ASoC: no DMI board/product name!\n"); > + return 0; > + } > + > + /* make up dmi long name as: vendor.product.board */ > + if (product && board) > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > + "%s.%s.%s", vendor, product, board); > + else > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > + "%s.%s", vendor, product ? product : board); > + > + /* Add flavour to dmi long name */ > + if (flavour) > + snprintf(card->dmi_longname, sizeof(card->snd_card->longname), > + "%s.%s", dmi_longname, flavour); > + else > + strncpy(card->dmi_longname, dmi_longname, > + sizeof(card->snd_card->longname) - 1); > + > + trim_special_characters(card->dmi_longname); > + > + /* set long name */ > + card->long_name = card->dmi_longname; > + > + return 0; > +} > +EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name); > + > static int snd_soc_instantiate_card(struct snd_soc_card *card) > { > struct snd_soc_codec *codec; ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 14:40 ` Liam Girdwood @ 2017-01-03 15:44 ` Lin, Mengdong 2017-01-03 19:28 ` Liam Girdwood 0 siblings, 1 reply; 11+ messages in thread From: Lin, Mengdong @ 2017-01-03 15:44 UTC (permalink / raw) To: Liam Girdwood, mengdong.lin Cc: tiwai, Koul, Vinod, alsa-devel, broonie, Bossart, Pierre-louis > -----Original Message----- > From: Liam Girdwood [mailto:liam.r.girdwood@linux.intel.com] > Sent: Tuesday, January 3, 2017 10:40 PM > To: mengdong.lin@linux.intel.com > Cc: alsa-devel@alsa-project.org; broonie@kernel.org; tiwai@suse.de; Lin, > Mengdong <mengdong.lin@intel.com>; Koul, Vinod <vinod.koul@intel.com>; > Bossart, Pierre-louis <pierre-louis.bossart@intel.com> > Subject: Re: [alsa-devel] [PATCH v2 1/2] ASoC: core: Add API to use DMI name > in sound card long name > > On Tue, 2017-01-03 at 12:58 +0800, mengdong.lin@linux.intel.com wrote: > > From: Liam Girdwood <liam.r.girdwood@linux.intel.com> > > > > Intel DSP platform drivers are used by many different devices but are > > difficult for userspace to differentiate. This patch adds an API to > > allow the DMI name to be used in the sound card long name, thereby > > helping userspace load the correct UCM configuration. Usually machine > > drivers uses their own name as the sound card name (short name), and > > leave the long name and driver name blank. This API will use the DMI > > info like vendor, product and board to make up the card long name. If > > the machine driver has already explicitly set the long name, this API will do > nothing. > > > > This patch also allows for further differentiation as many devices > > that share the same DMI name i.e. Minnowboards, UP boards may be > > configured with different codecs or firmwares. The API supports > > flavoring the DMI name into the card longname to provide the extra > > differentiation required for these devices. > > > > For Use Case Manager (UCM) in the user space, changing card long name > > by this API is backward compatible, since the card name does not > > change. For a given sound card, even if there is no device-specific > > UCM configuration file that uses the card long name, UCM will fall > > back to load the default configuration file that uses the card name. > > > > Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com> > > Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> > > > > diff --git a/include/sound/soc.h b/include/sound/soc.h index > > 795e6c4..e4f1844 100644 > > --- a/include/sound/soc.h > > +++ b/include/sound/soc.h > > @@ -497,6 +497,8 @@ void snd_soc_runtime_deactivate(struct > > snd_soc_pcm_runtime *rtd, int stream); int > snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, > > unsigned int dai_fmt); > > > > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char > > +*flavour); > > + > > /* Utility functions to get clock rates from various things */ int > > snd_soc_calc_frame_size(int sample_size, int channels, int tdm_slots); > > int snd_soc_params_to_frame_size(struct snd_pcm_hw_params *params); > @@ > > -1094,6 +1096,8 @@ struct snd_soc_card { > > const char *name; > > const char *long_name; > > const char *driver_name; > > + char dmi_longname[80]; > > + > > struct device *dev; > > struct snd_card *snd_card; > > struct module *owner; > > diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index > > aaab26a..3dbb5aa 100644 > > --- a/sound/soc/soc-core.c > > +++ b/sound/soc/soc-core.c > > @@ -34,6 +34,7 @@ > > #include <linux/ctype.h> > > #include <linux/slab.h> > > #include <linux/of.h> > > +#include <linux/dmi.h> > > #include <sound/core.h> > > #include <sound/jack.h> > > #include <sound/pcm.h> > > @@ -1886,6 +1887,104 @@ int snd_soc_runtime_set_dai_fmt(struct > > snd_soc_pcm_runtime *rtd, } > > EXPORT_SYMBOL_GPL(snd_soc_runtime_set_dai_fmt); > > > > + > > +/* Only keep number and alphabet characters and a few separator > characters. > > + * DMI info often has SPACE and we must trim them, since Use Case > > +Manager in > > + * the user space expects the device-specific configuration > > +files/directories > > + * use the same name as the card long name but Autoconf cannot > > +support SPACE > > + * in file or directory name. > > + */ > > +static void trim_special_characters(char *name) { > > + int i, j = 0; > > + > > + for (i = 0; name[i]; i++) { > > + if (isalnum(name[i]) || (name[i] == '.') > > + || (name[i] == '-') || (name[i] == '_')) > > + name[j++] = name[i]; > > + } > > + > > + name[j] = '\0'; > > +} > > + > > +/** > > + * snd_soc_set_dmi_name() - Register DMI names to card > > + * @card: The card to register DMI names > > + * @flavour: The flavour "differentiator" for the card amongst its peers. > > + * > > + * Intel DSP platform drivers are used by many different devices but > > +are > > + * difficult for userspace to differentiate, since machine drivers > > +ususally > > + * use their own name as the card name (short name) and leave the > > +card long > > + * name blank. This function will allow DMI info to be used as the > > +sound > > + * card long name, thereby helping userspace load the correct UCM > > +(Use Case > > + * Manager) configuration. Character '.' are used to separate > > +different DMI > > + * fields like "vendor.product.board". > > + * > > + * Possible card long names may be: > > + * DellInc..XPS139343.0310JH > > + * IntelCorp..BroadwellClientPlatform.WilsonBeachSDS > > + * ASUSTeKCOMPUTERINC..T100TA.T100TA > > + * Circuitco.MinnowboardMaxD0PLATFORM.MinnowBoardMAX > > + * (Please note DMI info can also include '.' after some abbreviation > > +like > > + * "Inc." so you may see double '.' sometimes) > > + * > > + * This function also supports flavoring the card longname to provide > > + * the extra differentiation, like "vendor.product.board.flavor". > > I dont think we need vendor (it just makes it too long). The product or board > name should be unique enough for us to load the correct files. > > Liam Yes, from the sample machines I checked, product or board name are unique. But I feel there might be the risk that two vendors happen to use the same product or board name, e.g. "T100TA". And if we want to add the vendor field later, we may need to rename existing UCM configuration files and directories, which may be not so easy if those machines are not around us. Thanks Mengdong > > > + * > > + * We only keep number and alphabet characters and a few separator > > +characters > > + * in the card long name since UCM in the user space uses the card > > +long names > > + * as card configuration directory names and AudoConf cannot support > > +special > > + * charactors like SPACE. > > + * > > + * Returns 0 on success, otherwise a negative error code. > > + */ > > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char > > +*flavour) { > > + const char *vendor, *product, *board; > > + char dmi_longname[80]; > > + > > + if (card->long_name) > > + return 0; /* long name already set by driver or from DMI */ > > + > > + vendor = dmi_get_system_info(DMI_BOARD_VENDOR); > > + if (!vendor) { > > + dev_warn(card->dev, "ASoC: no DMI vendor name!\n"); > > + return 0; > > + } > > + > > + product = dmi_get_system_info(DMI_PRODUCT_NAME); > > + board = dmi_get_system_info(DMI_BOARD_NAME); > > + if (!board && !product) { > > + /* fall back to using legacy name */ > > + dev_warn(card->dev, "ASoC: no DMI board/product > name!\n"); > > + return 0; > > + } > > + > > + /* make up dmi long name as: vendor.product.board */ > > + if (product && board) > > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > > + "%s.%s.%s", vendor, product, board); > > + else > > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > > + "%s.%s", vendor, product ? product : board); > > + > > + /* Add flavour to dmi long name */ > > + if (flavour) > > + snprintf(card->dmi_longname, sizeof(card->snd_card- > >longname), > > + "%s.%s", dmi_longname, flavour); > > + else > > + strncpy(card->dmi_longname, dmi_longname, > > + sizeof(card->snd_card->longname) - 1); > > + > > + trim_special_characters(card->dmi_longname); > > + > > + /* set long name */ > > + card->long_name = card->dmi_longname; > > + > > + return 0; > > +} > > +EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name); > > + > > static int snd_soc_instantiate_card(struct snd_soc_card *card) { > > struct snd_soc_codec *codec; > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 15:44 ` Lin, Mengdong @ 2017-01-03 19:28 ` Liam Girdwood 2017-01-03 20:09 ` Pierre-Louis Bossart 0 siblings, 1 reply; 11+ messages in thread From: Liam Girdwood @ 2017-01-03 19:28 UTC (permalink / raw) To: Lin, Mengdong Cc: alsa-devel, mengdong.lin, tiwai, Koul, Vinod, broonie, Bossart, Pierre-louis On Tue, 2017-01-03 at 15:44 +0000, Lin, Mengdong wrote: > > -----Original Message----- > > From: Liam Girdwood [mailto:liam.r.girdwood@linux.intel.com] > > Sent: Tuesday, January 3, 2017 10:40 PM > > To: mengdong.lin@linux.intel.com > > Cc: alsa-devel@alsa-project.org; broonie@kernel.org; tiwai@suse.de; Lin, > > Mengdong <mengdong.lin@intel.com>; Koul, Vinod <vinod.koul@intel.com>; > > Bossart, Pierre-louis <pierre-louis.bossart@intel.com> > > Subject: Re: [alsa-devel] [PATCH v2 1/2] ASoC: core: Add API to use DMI name > > in sound card long name > > > > On Tue, 2017-01-03 at 12:58 +0800, mengdong.lin@linux.intel.com wrote: > > > From: Liam Girdwood <liam.r.girdwood@linux.intel.com> > > > > > > Intel DSP platform drivers are used by many different devices but are > > > difficult for userspace to differentiate. This patch adds an API to > > > allow the DMI name to be used in the sound card long name, thereby > > > helping userspace load the correct UCM configuration. Usually machine > > > drivers uses their own name as the sound card name (short name), and > > > leave the long name and driver name blank. This API will use the DMI > > > info like vendor, product and board to make up the card long name. If > > > the machine driver has already explicitly set the long name, this API will do > > nothing. > > > > > > This patch also allows for further differentiation as many devices > > > that share the same DMI name i.e. Minnowboards, UP boards may be > > > configured with different codecs or firmwares. The API supports > > > flavoring the DMI name into the card longname to provide the extra > > > differentiation required for these devices. > > > > > > For Use Case Manager (UCM) in the user space, changing card long name > > > by this API is backward compatible, since the card name does not > > > change. For a given sound card, even if there is no device-specific > > > UCM configuration file that uses the card long name, UCM will fall > > > back to load the default configuration file that uses the card name. > > > > > > Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com> > > > Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> > > > > > > diff --git a/include/sound/soc.h b/include/sound/soc.h index > > > 795e6c4..e4f1844 100644 > > > --- a/include/sound/soc.h > > > +++ b/include/sound/soc.h > > > @@ -497,6 +497,8 @@ void snd_soc_runtime_deactivate(struct > > > snd_soc_pcm_runtime *rtd, int stream); int > > snd_soc_runtime_set_dai_fmt(struct snd_soc_pcm_runtime *rtd, > > > unsigned int dai_fmt); > > > > > > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char > > > +*flavour); > > > + > > > /* Utility functions to get clock rates from various things */ int > > > snd_soc_calc_frame_size(int sample_size, int channels, int tdm_slots); > > > int snd_soc_params_to_frame_size(struct snd_pcm_hw_params *params); > > @@ > > > -1094,6 +1096,8 @@ struct snd_soc_card { > > > const char *name; > > > const char *long_name; > > > const char *driver_name; > > > + char dmi_longname[80]; > > > + > > > struct device *dev; > > > struct snd_card *snd_card; > > > struct module *owner; > > > diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index > > > aaab26a..3dbb5aa 100644 > > > --- a/sound/soc/soc-core.c > > > +++ b/sound/soc/soc-core.c > > > @@ -34,6 +34,7 @@ > > > #include <linux/ctype.h> > > > #include <linux/slab.h> > > > #include <linux/of.h> > > > +#include <linux/dmi.h> > > > #include <sound/core.h> > > > #include <sound/jack.h> > > > #include <sound/pcm.h> > > > @@ -1886,6 +1887,104 @@ int snd_soc_runtime_set_dai_fmt(struct > > > snd_soc_pcm_runtime *rtd, } > > > EXPORT_SYMBOL_GPL(snd_soc_runtime_set_dai_fmt); > > > > > > + > > > +/* Only keep number and alphabet characters and a few separator > > characters. > > > + * DMI info often has SPACE and we must trim them, since Use Case > > > +Manager in > > > + * the user space expects the device-specific configuration > > > +files/directories > > > + * use the same name as the card long name but Autoconf cannot > > > +support SPACE > > > + * in file or directory name. > > > + */ > > > +static void trim_special_characters(char *name) { > > > + int i, j = 0; > > > + > > > + for (i = 0; name[i]; i++) { > > > + if (isalnum(name[i]) || (name[i] == '.') > > > + || (name[i] == '-') || (name[i] == '_')) > > > + name[j++] = name[i]; > > > + } > > > + > > > + name[j] = '\0'; > > > +} > > > + > > > +/** > > > + * snd_soc_set_dmi_name() - Register DMI names to card > > > + * @card: The card to register DMI names > > > + * @flavour: The flavour "differentiator" for the card amongst its peers. > > > + * > > > + * Intel DSP platform drivers are used by many different devices but > > > +are > > > + * difficult for userspace to differentiate, since machine drivers > > > +ususally > > > + * use their own name as the card name (short name) and leave the > > > +card long > > > + * name blank. This function will allow DMI info to be used as the > > > +sound > > > + * card long name, thereby helping userspace load the correct UCM > > > +(Use Case > > > + * Manager) configuration. Character '.' are used to separate > > > +different DMI > > > + * fields like "vendor.product.board". > > > + * > > > + * Possible card long names may be: > > > + * DellInc..XPS139343.0310JH > > > + * IntelCorp..BroadwellClientPlatform.WilsonBeachSDS > > > + * ASUSTeKCOMPUTERINC..T100TA.T100TA > > > + * Circuitco.MinnowboardMaxD0PLATFORM.MinnowBoardMAX > > > + * (Please note DMI info can also include '.' after some abbreviation > > > +like > > > + * "Inc." so you may see double '.' sometimes) > > > + * > > > + * This function also supports flavoring the card longname to provide > > > + * the extra differentiation, like "vendor.product.board.flavor". > > > > I dont think we need vendor (it just makes it too long). The product or board > > name should be unique enough for us to load the correct files. > > > > Liam > > Yes, from the sample machines I checked, product or board name are > unique. But I feel there might be the risk that two vendors happen to > use the same product or board name, e.g. "T100TA". I think this would be problematic from a legal/marketing position from two different vendors so it would be unlikely. Liam > > And if we want to add the vendor field later, we may need to rename > existing UCM configuration files and directories, which may be not so > easy if those machines are not around us. > > Thanks > Mengdong > > > > > > + * > > > + * We only keep number and alphabet characters and a few separator > > > +characters > > > + * in the card long name since UCM in the user space uses the card > > > +long names > > > + * as card configuration directory names and AudoConf cannot support > > > +special > > > + * charactors like SPACE. > > > + * > > > + * Returns 0 on success, otherwise a negative error code. > > > + */ > > > +int snd_soc_set_dmi_name(struct snd_soc_card *card, const char > > > +*flavour) { > > > + const char *vendor, *product, *board; > > > + char dmi_longname[80]; > > > + > > > + if (card->long_name) > > > + return 0; /* long name already set by driver or from DMI */ > > > + > > > + vendor = dmi_get_system_info(DMI_BOARD_VENDOR); > > > + if (!vendor) { > > > + dev_warn(card->dev, "ASoC: no DMI vendor name!\n"); > > > + return 0; > > > + } > > > + > > > + product = dmi_get_system_info(DMI_PRODUCT_NAME); > > > + board = dmi_get_system_info(DMI_BOARD_NAME); > > > + if (!board && !product) { > > > + /* fall back to using legacy name */ > > > + dev_warn(card->dev, "ASoC: no DMI board/product > > name!\n"); > > > + return 0; > > > + } > > > + > > > + /* make up dmi long name as: vendor.product.board */ > > > + if (product && board) > > > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > > > + "%s.%s.%s", vendor, product, board); > > > + else > > > + snprintf(dmi_longname, sizeof(card->snd_card->longname), > > > + "%s.%s", vendor, product ? product : board); > > > + > > > + /* Add flavour to dmi long name */ > > > + if (flavour) > > > + snprintf(card->dmi_longname, sizeof(card->snd_card- > > >longname), > > > + "%s.%s", dmi_longname, flavour); > > > + else > > > + strncpy(card->dmi_longname, dmi_longname, > > > + sizeof(card->snd_card->longname) - 1); > > > + > > > + trim_special_characters(card->dmi_longname); > > > + > > > + /* set long name */ > > > + card->long_name = card->dmi_longname; > > > + > > > + return 0; > > > +} > > > +EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name); > > > + > > > static int snd_soc_instantiate_card(struct snd_soc_card *card) { > > > struct snd_soc_codec *codec; > > > > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 19:28 ` Liam Girdwood @ 2017-01-03 20:09 ` Pierre-Louis Bossart 2017-01-04 7:03 ` Mengdong Lin 0 siblings, 1 reply; 11+ messages in thread From: Pierre-Louis Bossart @ 2017-01-03 20:09 UTC (permalink / raw) To: Liam Girdwood, Lin, Mengdong Cc: tiwai, Koul, Vinod, alsa-devel, mengdong.lin, broonie >>> I dont think we need vendor (it just makes it too long). The product or board >>> name should be unique enough for us to load the correct files. >>> >>> Liam >> Yes, from the sample machines I checked, product or board name are >> unique. But I feel there might be the risk that two vendors happen to >> use the same product or board name, e.g. "T100TA". > I think this would be problematic from a legal/marketing position from > two different vendors so it would be unlikely. Please don't remove the manufacturer name for now... Some vendors use product names that are indeed unique and easy enough to remember, but others like Lenovo use "1952W5R" or "20C3001VHH" (real examples you can Google to see I am not making this up). The board name can also be "SKG18 t". I really have no appetite for a UCM directory called 20C3001VHH.SKG18t :-) I would also be ready to bet that smaller manufacturers in the Chinese ecosystem use similar product names at the DMI level. removing the vendor name would likely result in mistakes. You really want the manufacturer name to make the maintenance of these files easier on the rest of us. Sometimes the DMI version field is actually more self-explanatory, e.g. "ThinkPad 60" or "ThinkPad 10" for the two Lenovo examples, maybe this is something we ought to look at. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-03 20:09 ` Pierre-Louis Bossart @ 2017-01-04 7:03 ` Mengdong Lin 2017-01-04 21:47 ` Liam Girdwood 0 siblings, 1 reply; 11+ messages in thread From: Mengdong Lin @ 2017-01-04 7:03 UTC (permalink / raw) To: Pierre-Louis Bossart, Liam Girdwood, Lin, Mengdong Cc: tiwai, Koul, Vinod, alsa-devel, broonie On 01/04/2017 04:09 AM, Pierre-Louis Bossart wrote: > >>>> I dont think we need vendor (it just makes it too long). The product >>>> or board >>>> name should be unique enough for us to load the correct files. >>>> >>>> Liam >>> Yes, from the sample machines I checked, product or board name are >>> unique. But I feel there might be the risk that two vendors happen to >>> use the same product or board name, e.g. "T100TA". >> I think this would be problematic from a legal/marketing position from >> two different vendors so it would be unlikely. > > Please don't remove the manufacturer name for now... > Some vendors use product names that are indeed unique and easy enough to > remember, but others like Lenovo use "1952W5R" or "20C3001VHH" (real > examples you can Google to see I am not making this up). The board name > can also be "SKG18 t". I really have no appetite for a UCM directory > called 20C3001VHH.SKG18t :-) > I would also be ready to bet that smaller manufacturers in the Chinese > ecosystem use similar product names at the DMI level. removing the > vendor name would likely result in mistakes. > You really want the manufacturer name to make the maintenance of these > files easier on the rest of us. Sometimes the DMI version field is > actually more self-explanatory, e.g. "ThinkPad 60" or "ThinkPad 10" for > the two Lenovo examples, maybe this is something we ought to look at. > > So we'll keep the manufacturer name. And let's put the "Product version" as well if available, Liam? I also observed Lenovo puts the the user-friendly name in the product version like "Thinkpad S5 Yoga 15", while it product name is "20DQA00KCD", no self-explanatory. It seems different manufacturer tend to put self-explanatory name in different fields: Lenovo thinkpad: product version Dell: product name ASUS: both product name and board name Intel: board name Do we want some table to specify the field we want for major vendors? So we may just pick the most valuable DMI field for the card long name, for those manufacturers to save the space. But it may not scale if the manufacturer change their preference or use different way for different product series, since I only checked a few machines by a few vendors. Also I wonder if we find the key word like "Dell, HP or ASUS", may we not use verbatim copy on the vendor name but just use these key words? This can also save some space. Thanks Mengdong > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-04 7:03 ` Mengdong Lin @ 2017-01-04 21:47 ` Liam Girdwood 2017-01-06 6:07 ` Mengdong Lin 0 siblings, 1 reply; 11+ messages in thread From: Liam Girdwood @ 2017-01-04 21:47 UTC (permalink / raw) To: Mengdong Lin Cc: alsa-devel, tiwai, Lin, Mengdong, Koul, Vinod, broonie, Pierre-Louis Bossart On Wed, 2017-01-04 at 15:03 +0800, Mengdong Lin wrote: > > On 01/04/2017 04:09 AM, Pierre-Louis Bossart wrote: > > > >>>> I dont think we need vendor (it just makes it too long). The product > >>>> or board > >>>> name should be unique enough for us to load the correct files. > >>>> > >>>> Liam > >>> Yes, from the sample machines I checked, product or board name are > >>> unique. But I feel there might be the risk that two vendors happen to > >>> use the same product or board name, e.g. "T100TA". > >> I think this would be problematic from a legal/marketing position from > >> two different vendors so it would be unlikely. > > > > Please don't remove the manufacturer name for now... > > Some vendors use product names that are indeed unique and easy enough to > > remember, but others like Lenovo use "1952W5R" or "20C3001VHH" (real > > examples you can Google to see I am not making this up). The board name > > can also be "SKG18 t". I really have no appetite for a UCM directory > > called 20C3001VHH.SKG18t :-) > > I would also be ready to bet that smaller manufacturers in the Chinese > > ecosystem use similar product names at the DMI level. removing the > > vendor name would likely result in mistakes. > > You really want the manufacturer name to make the maintenance of these > > files easier on the rest of us. Sometimes the DMI version field is > > actually more self-explanatory, e.g. "ThinkPad 60" or "ThinkPad 10" for > > the two Lenovo examples, maybe this is something we ought to look at. > > > > > > So we'll keep the manufacturer name. > > And let's put the "Product version" as well if available, Liam? Yes, lets keep the manufacturer name. I agree with Pierre in that we dont want meaningless directories names. Creating a directory structures like 1) manufacturer/product/board OR 2) manufacturer/product OR 3) manufacturer/board (2 or 3 if board or product is NULL) are fine by me and make finding the correct card simpler. You could even replace the . with / in your longname to save the string conversion later on. Liam > > I also observed Lenovo puts the the user-friendly name in the product > version like "Thinkpad S5 Yoga 15", while it product name is > "20DQA00KCD", no self-explanatory. > > It seems different manufacturer tend to put self-explanatory name in > different fields: > Lenovo thinkpad: product version > Dell: product name > ASUS: both product name and board name > Intel: board name > > Do we want some table to specify the field we want for major vendors? So > we may just pick the most valuable DMI field for the card long name, for > those manufacturers to save the space. But it may not scale if the > manufacturer change their preference or use different way for different > product series, since I only checked a few machines by a few vendors. > > Also I wonder if we find the key word like "Dell, HP or ASUS", may we > not use verbatim copy on the vendor name but just use these key words? > This can also save some space. > > Thanks > Mengdong > > > > > > > _______________________________________________ > > Alsa-devel mailing list > > Alsa-devel@alsa-project.org > > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-04 21:47 ` Liam Girdwood @ 2017-01-06 6:07 ` Mengdong Lin 2017-01-09 11:13 ` Liam Girdwood 0 siblings, 1 reply; 11+ messages in thread From: Mengdong Lin @ 2017-01-06 6:07 UTC (permalink / raw) To: Liam Girdwood Cc: alsa-devel, tiwai, Lin, Mengdong, Koul, Vinod, broonie, Pierre-Louis Bossart On 01/05/2017 05:47 AM, Liam Girdwood wrote: > On Wed, 2017-01-04 at 15:03 +0800, Mengdong Lin wrote: >> >> On 01/04/2017 04:09 AM, Pierre-Louis Bossart wrote: >> So we'll keep the manufacturer name. >> >> And let's put the "Product version" as well if available, Liam? > > Yes, lets keep the manufacturer name. I agree with Pierre in that we > dont want meaningless directories names. Okay. Thanks for your confirmation! > Creating a directory structures > like > > 1) manufacturer/product/board OR > 2) manufacturer/product OR > 3) manufacturer/board > > (2 or 3 if board or product is NULL) > > are fine by me and make finding the correct card simpler. You could even > replace the . with / in your longname to save the string conversion > later on. > > Liam May we keep the flat directory structure for UCM files? For the same manufacturer, its DMI vendor name could change, e.g. "Intel Inc." or "Intel Corporation". In addition, there may be changes caused by using upper case or low case characters. Since we don't want to maintain a mapping table and just do verbatim copy of DMI info, it's hard to group all the products/boards of one manufacturer into one UCM directory. So I feel it may be better to keep the simple flat directory structure. Thanks Mengdong ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] ASoC: core: Add API to use DMI name in sound card long name 2017-01-06 6:07 ` Mengdong Lin @ 2017-01-09 11:13 ` Liam Girdwood 0 siblings, 0 replies; 11+ messages in thread From: Liam Girdwood @ 2017-01-09 11:13 UTC (permalink / raw) To: Mengdong Lin Cc: alsa-devel, tiwai, Lin, Mengdong, Koul, Vinod, broonie, Pierre-Louis Bossart On Fri, 2017-01-06 at 14:07 +0800, Mengdong Lin wrote: > > On 01/05/2017 05:47 AM, Liam Girdwood wrote: > > On Wed, 2017-01-04 at 15:03 +0800, Mengdong Lin wrote: > >> > >> On 01/04/2017 04:09 AM, Pierre-Louis Bossart wrote: > > >> So we'll keep the manufacturer name. > >> > >> And let's put the "Product version" as well if available, Liam? > > > > Yes, lets keep the manufacturer name. I agree with Pierre in that we > > dont want meaningless directories names. > > Okay. Thanks for your confirmation! > > > Creating a directory structures > > like > > > > 1) manufacturer/product/board OR > > 2) manufacturer/product OR > > 3) manufacturer/board > > > > (2 or 3 if board or product is NULL) > > > > are fine by me and make finding the correct card simpler. You could even > > replace the . with / in your longname to save the string conversion > > later on. > > > > Liam > > May we keep the flat directory structure for UCM files? This may lead to really long file names and a very crowded master directory. > > For the same manufacturer, its DMI vendor name could change, e.g. "Intel > Inc." or "Intel Corporation". In addition, there may be changes caused > by using upper case or low case characters. Since we don't want to > maintain a mapping table and just do verbatim copy of DMI info, it's > hard to group all the products/boards of one manufacturer into one UCM > directory. So I feel it may be better to keep the simple flat directory > structure. This wont matter if we have > 1 directory for Intel or Dell etc, as it's obvious in the naming. No mapping needed. Liam > > Thanks > Mengdong > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2 2/2] ASoC: Intel: Use DMI name for sound card long name in Broadwell machine driver 2017-01-03 4:58 [PATCH v2 0/2] ASoC: Use DMI name in sound card long name mengdong.lin 2017-01-03 4:58 ` [PATCH v2 1/2] ASoC: core: Add API to use " mengdong.lin @ 2017-01-03 5:00 ` mengdong.lin 1 sibling, 0 replies; 11+ messages in thread From: mengdong.lin @ 2017-01-03 5:00 UTC (permalink / raw) To: alsa-devel, broonie Cc: Mengdong Lin, tiwai, mengdong.lin, liam.r.girdwood, vinod.koul, pierre-louis.bossart From: Mengdong Lin <mengdong.lin@linux.intel.com> Intel Broadwell machine driver will call API snd_soc_set_dmi_name() to use DMI info to make the sound card long name. For example, here are the changed long name for two Broadwell-based machines: Dell XPS-13(2015): DellInc..XPS139343.0310JH Intel WilsonBeach: Intel Corp..BroadwellClientplatform.WilsonBeachSDS They still share the same card name "broadwell-rt286". Signed-off-by: Mengdong Lin <mengdong.lin@linux.intel.com> diff --git a/sound/soc/intel/boards/broadwell.c b/sound/soc/intel/boards/broadwell.c index 5bb64d1..cda73bb 100644 --- a/sound/soc/intel/boards/broadwell.c +++ b/sound/soc/intel/boards/broadwell.c @@ -266,6 +266,8 @@ static int broadwell_audio_probe(struct platform_device *pdev) { broadwell_rt286.dev = &pdev->dev; + snd_soc_set_dmi_name(&broadwell_rt286, NULL); + return devm_snd_soc_register_card(&pdev->dev, &broadwell_rt286); } -- 2.5.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-01-09 11:13 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-03 4:58 [PATCH v2 0/2] ASoC: Use DMI name in sound card long name mengdong.lin 2017-01-03 4:58 ` [PATCH v2 1/2] ASoC: core: Add API to use " mengdong.lin 2017-01-03 14:40 ` Liam Girdwood 2017-01-03 15:44 ` Lin, Mengdong 2017-01-03 19:28 ` Liam Girdwood 2017-01-03 20:09 ` Pierre-Louis Bossart 2017-01-04 7:03 ` Mengdong Lin 2017-01-04 21:47 ` Liam Girdwood 2017-01-06 6:07 ` Mengdong Lin 2017-01-09 11:13 ` Liam Girdwood 2017-01-03 5:00 ` [PATCH v2 2/2] ASoC: Intel: Use DMI name for sound card long name in Broadwell machine driver mengdong.lin
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.