lkml.org 
[lkml]   [2019]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: next/master boot bisection: next-20190114 on sun8i-h2-plus-libretech-all-h3-cc
From
Date

On 1/14/2019 6:41 PM, kernelci.org bot wrote:
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> * This automated bisection report was sent to you on the basis *
> * that you may be involved with the breaking commit it has *
> * found. No manual investigation has been done to verify it, *
> * and the root cause of the problem may be somewhere else. *
> * Hope this helps! *
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
>
> next/master boot bisection: next-20190114 on sun8i-h2-plus-libretech-all-h3-cc
>
> Summary:
> Start: 8ce4d582f564 Add linux-next specific files for 20190114
> Details: https://kernelci.org/boot/id/5c3c411659b51457a76edd34
> Plain log: https://storage.kernelci.org//next/master/next-20190114/arm/sunxi_defconfig/lab-baylibre/boot-sun8i-h2-plus-libretech-all-h3-cc.txt
> HTML log: https://storage.kernelci.org//next/master/next-20190114/arm/sunxi_defconfig/lab-baylibre/boot-sun8i-h2-plus-libretech-all-h3-cc.html
> Result: 8780cf1142a5 ASoC: soc-core: defer card probe until all component is added to list
>
> Checks:
> revert: PASS
> verify: PASS
>
> Parameters:
> Tree: next
> URL: http://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> Branch: master
> Target: sun8i-h2-plus-libretech-all-h3-cc
> CPU arch: arm
> Lab: lab-baylibre
> Config: sunxi_defconfig
> Test suite: boot
>
> Breaking commit found:
>
> -------------------------------------------------------------------------------
> commit 8780cf1142a59568a3aa77959cbd76b2edb6fd81
> Author: Ajit Pandey <ajitp@codeaurora.org>
> Date: Wed Jan 9 14:17:07 2019 +0530
>
> ASoC: soc-core: defer card probe until all component is added to list
>
> DAI component probe is not called if it is not present
> in component list during sound card registration.
> Check if component is available in component list for
> platform and cpu dai before soundcard registration.
>
> Signed-off-by: Ajit Pandey <ajitp@codeaurora.org>
> Signed-off-by: Rohit kumar <rohitkr@codeaurora.org>
> Signed-off-by: Mark Brown <broonie@kernel.org>
The fix is already posted via
https://lore.kernel.org/patchwork/patch/1031226/.

Mark, Can you pick this up. We will still have to consider intel platform

where there is no soc_component associated with platform_name.

Other solution is to revert this patch and use client specific defer
probe as

done in v1.

https://lore.kernel.org/patchwork/patch/1027560/
https://lore.kernel.org/patchwork/patch/1027561/

> diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
> index 0462b3ec977a..eec92f17dd15 100644
> --- a/sound/soc/soc-core.c
> +++ b/sound/soc/soc-core.c
> @@ -1027,7 +1027,6 @@ static int snd_soc_init_platform(struct snd_soc_card *card,
> struct snd_soc_dai_link *dai_link)
> {
> struct snd_soc_dai_link_component *platform = dai_link->platform;
> -
> /*
> * FIXME
> *
> @@ -1129,6 +1128,14 @@ static int soc_init_dai_link(struct snd_soc_card *card,
> link->name);
> return -EINVAL;
> }
> +
> + /*
> + * Defer card registartion if platform dai component is not added to
> + * component list.
> + */
> + if (!soc_find_component(link->platform->of_node, link->platform->name))
> + return -EPROBE_DEFER;
> +
> /*
> * CPU device may be specified by either name or OF node, but
> * can be left unspecified, and will be matched based on DAI
> @@ -1140,6 +1147,14 @@ static int soc_init_dai_link(struct snd_soc_card *card,
> link->name);
> return -EINVAL;
> }
> +
> + /*
> + * Defer card registartion if cpu dai component is not added to
> + * component list.
> + */
> + if (!soc_find_component(link->cpu_of_node, link->cpu_name))
> + return -EPROBE_DEFER;
> +
> /*
> * At least one of CPU DAI name or CPU device name/node must be
> * specified
> -------------------------------------------------------------------------------
>
>
> Git bisection log:
>
> -------------------------------------------------------------------------------
> git bisect start
> # good: [6b529fb0a3eabf9c4cc3e94c11477250379ce6d8] Merge tag 'for-5.0-rc1-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux
> git bisect good 6b529fb0a3eabf9c4cc3e94c11477250379ce6d8
> # bad: [8ce4d582f5647052b86f364a070aa26268810d7e] Add linux-next specific files for 20190114
> git bisect bad 8ce4d582f5647052b86f364a070aa26268810d7e
> # bad: [bb937d2e6697b49f6e3fa55ec09951ebbc8717ac] Merge remote-tracking branch 'crypto/master'
> git bisect bad bb937d2e6697b49f6e3fa55ec09951ebbc8717ac
> # bad: [04bab95e799a00ffe1b2980e1c786ac9a94f8026] Merge remote-tracking branch 'renesas/next'
> git bisect bad 04bab95e799a00ffe1b2980e1c786ac9a94f8026
> # bad: [5726796d7b631fe9af2202c1ce66232a79984a19] Merge remote-tracking branch 'devicetree-fixes/dt/linus'
> git bisect bad 5726796d7b631fe9af2202c1ce66232a79984a19
> # good: [34aea5d2c46b2581ddf0b78a6119f85c1b3a8d1d] Merge remote-tracking branch 'net/master'
> git bisect good 34aea5d2c46b2581ddf0b78a6119f85c1b3a8d1d
> # bad: [e1faf5c97ee3194c3f3e0b5dc86b3e1490c323d9] Merge remote-tracking branch 'usb-serial-fixes/usb-linus'
> git bisect bad e1faf5c97ee3194c3f3e0b5dc86b3e1490c323d9
> # good: [49ff71f99a8826eda5cee76b69f15600d1cbf4b2] Merge remote-tracking branch 'rdma-fixes/for-rc'
> git bisect good 49ff71f99a8826eda5cee76b69f15600d1cbf4b2
> # bad: [04eb1efcd614d6f067b76a355b3a3599667959dc] ASoC: soc-core: Hold client_mutex around soc_init_dai_link()
> git bisect bad 04eb1efcd614d6f067b76a355b3a3599667959dc
> # good: [a41d9dbf5dac5b6a1283ee8001f22807d18352ea] ASoC: qdsp6: q6asm-dai: Fix a small memory leak
> git bisect good a41d9dbf5dac5b6a1283ee8001f22807d18352ea
> # good: [6175471755075d256c1c654151fc1cad183c1e33] ASoC: ti: davinci-mcasp: Move context save/restore to runtime_pm callbacks
> git bisect good 6175471755075d256c1c654151fc1cad183c1e33
> # good: [44fabd8cdaaa3acb80ad2bb3b5c61ae2136af661] ASoC: atom: fix a missing check of snd_pcm_lib_malloc_pages
> git bisect good 44fabd8cdaaa3acb80ad2bb3b5c61ae2136af661
> # bad: [239b8b34a856777e562373ae0de605536a7ccade] ASoC: Intel: Boards: move the codec PLL configuration to _init
> git bisect bad 239b8b34a856777e562373ae0de605536a7ccade
> # bad: [8780cf1142a59568a3aa77959cbd76b2edb6fd81] ASoC: soc-core: defer card probe until all component is added to list
> git bisect bad 8780cf1142a59568a3aa77959cbd76b2edb6fd81
> # first bad commit: [8780cf1142a59568a3aa77959cbd76b2edb6fd81] ASoC: soc-core: defer card probe until all component is added to list
> -------------------------------------------------------------------------------

--
Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member
of the Code Aurora Forum, hosted by the Linux Foundation.

\
 
 \ /
  Last update: 2019-01-14 14:33    [W:0.073 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site