- 08 Oct, 2019 32 commits
-
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Link: https://lore.kernel.org/r/87o8yz90e9.wl-kuninori.morimoto.gx@renesas.comSigned-off-by:
Mark Brown <broonie@kernel.org>
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. *Note* Only Intel skl-pcm has .get_time_info implementation, but ALSA SoC framework doesn't call it so far. To keep its implementation, this patch keeps .get_time_info, but it is still not called. Intel guy need to support it in the future. Signed-off-by:
-
Kuninori Morimoto authored
snd_pcm_ops is no longer needed. Let's use component driver callback. Signed-off-by:
-
Kuninori Morimoto authored
add snd_soc_pcm_lib_ioctl() to bypass to snd_pcm_lib_ioctl() Signed-off-by:
-
Kuninori Morimoto authored
Current snd_soc_component_driver has pcm_new/pcm_free, but, it doesn't have "component" at parameter. Thus, each callback can't know it is called for which component. Each callback currently is getting "component" by using snd_soc_rtdcom_lookup() with driver name. It works today, but, will not work in the future if we support multi CPU/Codec/Platform, because 1 rtd might have multiple same driver name component. To solve this issue, each callback need to be called with component. This patch adds new pcm_construct/pcm_destruct with "component" parameter. Signed-off-by:
-
Kuninori Morimoto authored
Current snd_soc_component_driver has snd_pcm_ops, and each driver can have callback via it (1). But, it is mainly created for ALSA, thus, it doesn't have "component" as parameter for ALSA SoC (1)(2). Thus, each callback can't know it is called for which component. Thus, each callback currently is getting "component" by using snd_soc_rtdcom_lookup() with driver name (3). --- ALSA SoC --- ... if (component->driver->ops && component->driver->ops->open) (1) return component->driver->ops->open(substream); ... --- driver --- (2) static int xxx_open(struct snd_pcm_substream *substream) { struct snd_soc_pcm_runtime *rtd = substream->private_data; (3) struct snd_soc_component *component = snd_soc_rtdcom_lookup(..); ... } It works today, but, will not work in the future if we support multi CPU/Codec/Platform, because 1 rtd might have multiple components which have same driver name. To solve this issue, each callback needs to be called with component. We already have many component driver callback. This patch copies each snd_pcm_ops member under component driver, and having "component" as parameter. --- ALSA SoC --- ... if (component->driver->open) => return component->driver->open(component, substream); ... --- driver --- => static int xxx_open(struct snd_soc_component *component, struct snd_pcm_substream *substream) { ... } *Note* Only Intel skl-pcm has .get_time_info implementation, but ALSA SoC framework doesn't call it so far. To keep its implementation, this patch keeps .get_time_info, but it is still not called. Intel guy need to support it in the future. Signed-off-by: -
Jiaxin Yu authored
Before regmap_reinit_cache we must reset audio regs as default values. So we use reset controller unit(toprgu) to reset audio hw. Signed-off-by:
Jiaxin Yu <jiaxin.yu@mediatek.com> Reviewed-by:
Yingjoe Chen <yingjoe.chen@mediatek.com> Link: https://lore.kernel.org/r/1569580317-21181-5-git-send-email-jiaxin.yu@mediatek.comSigned-off-by:
-
Jiaxin Yu authored
This patch add property "resets" && "reset-names" in examples so that we can use reset controller to reset audio domain regs. Signed-off-by:
-
Peter Ujfalusi authored
The snd_pcm_hw_constraint_minmax() works fine when a single codec is connected to a single CPU DAI, but in multicodec or DPCM setup the constraints placed by the driver will apply to the whole PCM stream (FE included) and thus prevents more than 8 playback channels for example. Signed-off-by:
Peter Ujfalusi <peter.ujfalusi@ti.com> Link: https://lore.kernel.org/r/20191008115720.7135-1-peter.ujfalusi@ti.comSigned-off-by:
-
- 07 Oct, 2019 8 commits
-
-
Pierre-Louis Bossart authored
For cases where an interface can be pin-muxed, we need to assess at probe time which configuration should be used. In cases such as SoundWire, we need to maintain an alternate list of machines and walk through them when the primary detection based on ACPI _HID fails. Signed-off-by:
Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Link: https://lore.kernel.org/r/20190916214251.13130-3-pierre-louis.bossart@linux.intel.comSigned-off-by:
-
Pierre-Louis Bossart authored
When interfaces can be pin-muxed, static information from ACPI might not be enough. Add information on which links needs to be enabled by hardware/firmware for a specific machine driver to be selected. When walking through the list of possible machines, links will be checked, which implies that configurations where multiple links are required need to be checked first. Additional criteria will be needed later, such as which SoundWire Slave devices are actually enabled, but for now this helps detect between basic configurations. Signed-off-by:
-
Dan Murphy authored
According the documentation for snd_soc_update_bits the API will return a 1 if the update was successful with a value change, a 0 if the update was successful with no value change or a negative if the command just failed. So the value of return in the driver needs to be checked for being less then 0 or the caller may indicate failure when the value actually changed. Signed-off-by:
Dan Murphy <dmurphy@ti.com> Link: https://lore.kernel.org/r/20191007171157.17813-1-dmurphy@ti.comSigned-off-by:
-
Dan Murphy authored
Remove unused defines and structure variables that are not referenced by the code. If these are needed for future enhancements then they should be added at that time. Signed-off-by:
-
Dan Murphy authored
Remove the unneeded and incorrect read of the TDM_CFG3 register. The read is done but the value is never used. Signed-off-by:
-
Dan Carpenter authored
There are several problems in the error handling in fsl_mqs_probe(). 1) "ret" isn't initialized on some paths. GCC has a feature which warns about uninitialized variables but the code initializes "ret" to zero at the start of the function so the checking is turned off. 2) "gpr_np" is a pointer so initializing it to zero is confusing and generates a Sparse warning. 3) of_parse_phandle() doesn't return error pointers on error, it returns NULL. 4) If devm_snd_soc_register_component() fails then the function should free the "gpr_np". Fixes: 9e28f653 ("ASoC: fsl_mqs: Add MQS component driver") Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Link: https://lore.kernel.org/r/20191004102208.GB823@mwandaSigned-off-by:
-
YueHaibing authored
Fixes gcc '-Wunused-but-set-variable' warning: sound/soc/fsl/fsl_mqs.c: In function fsl_mqs_hw_params: sound/soc/fsl/fsl_mqs.c:54:6: warning: variable bclk set but not used [-Wunused-but-set-variable] It is never used, so can be removed. Reported-by:
Hulk Robot <hulkci@huawei.com> Signed-off-by:
YueHaibing <yuehaibing@huawei.com> Link: https://lore.kernel.org/r/20191006105522.58560-1-yuehaibing@huawei.comSigned-off-by:
-
YueHaibing authored
If GPIOLIB is not set, building fails: sound/soc/codecs/tas2770.c: In function tas2770_reset: sound/soc/codecs/tas2770.c:38:3: error: implicit declaration of function gpiod_set_value_cansleep; did you mean gpio_set_value_cansleep? [-Werror=implicit-function-declaration] gpiod_set_value_cansleep(tas2770->reset_gpio, 0); ^~~~~~~~~~~~~~~~~~~~~~~~ gpio_set_value_cansleep sound/soc/codecs/tas2770.c: In function tas2770_i2c_probe: sound/soc/codecs/tas2770.c:749:24: error: implicit declaration of function devm_gpiod_get_optional; did you mean devm_regulator_get_optional? [-Werror=implicit-function-declaration] tas2770->reset_gpio = devm_gpiod_get_optional(tas2770->dev, ^~~~~~~~~~~~~~~~~~~~~~~ devm_regulator_get_optional sound/soc/codecs/tas2770.c:751:13: error: GPIOD_OUT_HIGH undeclared (first use in this function); did you mean GPIOF_INIT_HIGH? GPIOD_OUT_HIGH); ^~~~~~~~~~~~~~ GPIOF_INIT_HIGH Reported-by:Ladislav Michl <ladis@linux-mips.org> Signed-off-by:
-
