December 2016 - Alsa-devel - mailman.alsa-project.org

Re: [alsa-devel] [PATCH v2] ASoC: omap-mcbsp: Add PM QoS support for McBSP to prevent glitches
by kbuild test robot 01 Dec '16

01 Dec '16

Hi Matt, [auto build test ERROR on asoc/for-next] [also build test ERROR on v4.9-rc7 next-20161130] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Matt-Ranostay/ASoC-omap-mcbsp-Add-… base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next config: arm-allmodconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/m… -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=arm All errors (new ones prefixed by >>): sound/soc/omap/mcbsp.c: In function 'omap_mcbsp_start': >> sound/soc/omap/mcbsp.c:653:28: error: 'struct omap_mcbsp' has no member named 'pm_qos_req'; did you mean 'dma_req'? pm_qos_add_request(&mcbsp->pm_qos_req, ^~ sound/soc/omap/mcbsp.c: In function 'omap_mcbsp_stop': sound/soc/omap/mcbsp.c:745:30: error: 'struct omap_mcbsp' has no member named 'pm_qos_req'; did you mean 'dma_req'? pm_qos_remove_request(&mcbsp->pm_qos_req); ^~ sound/soc/omap/mcbsp.c: In function 'omap_mcbsp_cleanup': sound/soc/omap/mcbsp.c:1113:34: error: 'struct omap_mcbsp' has no member named 'pm_qos_req'; did you mean 'dma_req'? if (pm_qos_request_active(&mcbsp->pm_qos_req)) ^~ sound/soc/omap/mcbsp.c:1114:31: error: 'struct omap_mcbsp' has no member named 'pm_qos_req'; did you mean 'dma_req'? pm_qos_remove_request(&mcbsp->pm_qos_req); ^~ vim +653 sound/soc/omap/mcbsp.c 647 int enable_srg = 0; 648 int latency = mcbsp->pdata->buffer_size * 23; 649 u16 w; 650 651 /* Prevent omap hardware from hitting off between fifo fills */ 652 if (latency) > 653 pm_qos_add_request(&mcbsp->pm_qos_req, 654 PM_QOS_CPU_DMA_LATENCY, latency); 655 656 if (mcbsp->st_data) --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[alsa-devel] [PATCH V2 0/5] ASoC/arizona: Ensure pin searches use widget name prefix
by Richard Fitzgerald 01 Dec '16

01 Dec '16

The name of a codec pin can have an optional prefix string, which is defined by the audio machine driver. The snd_soc_dapm_x_pin functions take the fully-specified name including the prefix and so the existing code would fail to find the pin if the audio machine driver had added a prefix. This patch chain adds new helper functions that take a non-prefixed name for a specific ASoC component and internally add that component's prefix. The other patches update the arizona drivers to use these new functions. Richard Fitzgerald (5): ASoC: core: Add component pin control functions ASoC: arizona: Use component pin control functions regulator: arizona-micsupp: Use SoC component pin control functions extcon: arizona: Use SoC component pin control functions Input: arizona-haptics - Use SoC component pin control functions drivers/extcon/extcon-arizona.c | 8 +- drivers/input/misc/arizona-haptics.c | 13 ++- drivers/regulator/arizona-micsupp.c | 6 +- include/sound/soc.h | 20 ++++ sound/soc/codecs/arizona.c | 12 ++- sound/soc/codecs/cs47l24.c | 3 +- sound/soc/codecs/wm5102.c | 3 +- sound/soc/codecs/wm5110.c | 3 +- sound/soc/codecs/wm8997.c | 3 +- sound/soc/codecs/wm8998.c | 3 +- sound/soc/soc-utils.c | 199 +++++++++++++++++++++++++++++++++++ 11 files changed, 255 insertions(+), 18 deletions(-) -- 1.9.1

4 13

[alsa-devel] [PATCH v4] clkdev: add devm_of_clk_get()
by Kuninori Morimoto 01 Dec '16

01 Dec '16

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Current Linux has of_clk_get(), but doesn't have devm_of_clk_get(). This patch adds it. It is implemeted in clk-devres.c to share devm_clk_release(). Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- v3 -> v4 - git log explain why it is implemeted in clk-devres - it is related to CONFIG_HAVE_CLK drivers/clk/clk-devres.c | 21 +++++++++++++++++++++ include/linux/clk.h | 27 +++++++++++++++++++++++---- 2 files changed, 44 insertions(+), 4 deletions(-) diff --git a/drivers/clk/clk-devres.c b/drivers/clk/clk-devres.c index 8f57154..2449b25 100644 --- a/drivers/clk/clk-devres.c +++ b/drivers/clk/clk-devres.c @@ -53,3 +53,24 @@ void devm_clk_put(struct device *dev, struct clk *clk) WARN_ON(ret); } EXPORT_SYMBOL(devm_clk_put); + +struct clk *devm_of_clk_get(struct device *dev, + struct device_node *np, int index) +{ + struct clk **ptr, *clk; + + ptr = devres_alloc(devm_clk_release, sizeof(*ptr), GFP_KERNEL); + if (!ptr) + return ERR_PTR(-ENOMEM); + + clk = of_clk_get(np, index); + if (!IS_ERR(clk)) { + *ptr = clk; + devres_add(dev, ptr); + } else { + devres_free(ptr); + } + + return clk; +} +EXPORT_SYMBOL(devm_of_clk_get); diff --git a/include/linux/clk.h b/include/linux/clk.h index 123c027..7f50c5f 100644 --- a/include/linux/clk.h +++ b/include/linux/clk.h @@ -17,8 +17,9 @@ #include <linux/notifier.h> struct device; - struct clk; +struct device_node; +struct of_phandle_args; /** * DOC: clk notifier callback types @@ -249,6 +250,21 @@ static inline void clk_unprepare(struct clk *clk) struct clk *devm_clk_get(struct device *dev, const char *id); /** + * devm_clk_get - lookup and obtain a managed reference to a clock producer. + * @dev: device for clock "consumer" + * @np: pointer to clock consumer node + * @index: clock index + * + * This function parses the clocks, and uses them to look up the + * struct clk from the registered list of clock providers by using + * @np and @index. + * + * The clock will automatically be freed when the device is unbound + * from the bus. + */ +struct clk *devm_of_clk_get(struct device *dev, struct device_node *np, int index); + +/** * clk_enable - inform the system when the clock source should be running. * @clk: clock source * @@ -432,6 +448,12 @@ static inline struct clk *devm_clk_get(struct device *dev, const char *id) return NULL; } +static inline struct clk *devm_of_clk_get(struct device *dev, + struct device_node *np, int index) +{ + return NULL; +} + static inline void clk_put(struct clk *clk) {} static inline void devm_clk_put(struct device *dev, struct clk *clk) {} @@ -501,9 +523,6 @@ static inline void clk_disable_unprepare(struct clk *clk) clk_unprepare(clk); } -struct device_node; -struct of_phandle_args; - #if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) struct clk *of_clk_get(struct device_node *np, int index); struct clk *of_clk_get_by_name(struct device_node *np, const char *name); -- 1.9.1

2 2

[alsa-devel] [PATCH v2] clkdev: add devm_of_clk_get()
by Kuninori Morimoto 01 Dec '16

01 Dec '16

This is based on devm_clk_get() Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- v1 -> v2 - added "static" and "inline" on header drivers/clk/clkdev.c | 26 ++++++++++++++++++++++++++ include/linux/clk.h | 7 +++++++ 2 files changed, 33 insertions(+) diff --git a/drivers/clk/clkdev.c b/drivers/clk/clkdev.c index 89cc700..93a613b 100644 --- a/drivers/clk/clkdev.c +++ b/drivers/clk/clkdev.c @@ -55,6 +55,32 @@ struct clk *of_clk_get(struct device_node *np, int index) } EXPORT_SYMBOL(of_clk_get); +static void devm_of_clk_release(struct device *dev, void *res) +{ + clk_put(*(struct clk **)res); +} + +struct clk *devm_of_clk_get(struct device *dev, + struct device_node *np, int index) +{ + struct clk **ptr, *clk; + + ptr = devres_alloc(devm_of_clk_release, sizeof(*ptr), GFP_KERNEL); + if (!ptr) + return ERR_PTR(-ENOMEM); + + clk = of_clk_get(np, index); + if (!IS_ERR(clk)) { + *ptr = clk; + devres_add(dev, ptr); + } else { + devres_free(ptr); + } + + return clk; +} +EXPORT_SYMBOL(devm_of_clk_get); + static struct clk *__of_clk_get_by_name(struct device_node *np, const char *dev_id, const char *name) diff --git a/include/linux/clk.h b/include/linux/clk.h index a89ba4e..33cd540 100644 --- a/include/linux/clk.h +++ b/include/linux/clk.h @@ -502,6 +502,8 @@ struct of_phandle_args; #if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) struct clk *of_clk_get(struct device_node *np, int index); +struct clk *devm_of_clk_get(struct device *dev, + struct device_node *np, int index); struct clk *of_clk_get_by_name(struct device_node *np, const char *name); struct clk *of_clk_get_from_provider(struct of_phandle_args *clkspec); #else @@ -509,6 +511,11 @@ static inline struct clk *of_clk_get(struct device_node *np, int index) { return ERR_PTR(-ENOENT); } +static inline struct clk *devm_of_clk_get(struct device *dev, + struct device_node *np, int index) +{ + return ERR_PTR(-ENOENT); +} static inline struct clk *of_clk_get_by_name(struct device_node *np, const char *name) { -- 1.9.1

4 15