From: Lee Jones <lee.jones@linaro.org> To: Mark Brown <broonie@opensource.wolfsonmicro.com> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, STEricsson_nomadik_linux@list.st.com, linus.walleij@stericsson.com, arnd@arndb.de, olalilja@yahoo.se, ola.o.lilja@stericsson.com, alsa-devel@alsa-project.org, lrg@ti.com Subject: Re: [PATCH 1/6] ASoC: dapm: If one widget fails, do not force all subsequent widgets to fail too Date: Wed, 01 Aug 2012 14:50:32 +0100 [thread overview] Message-ID: <50193428.5000708@linaro.org> (raw) In-Reply-To: <20120801132022.GS11892@opensource.wolfsonmicro.com> On 01/08/12 14:20, Mark Brown wrote: > On Wed, Aug 01, 2012 at 08:19:28AM +0100, Lee Jones wrote: >> On 31/07/12 16:18, Mark Brown wrote: > >>> I'm not going to apply this patch. This isn't a vendor BSP, we >>> shouldn't be putting random hacks like this in core code. > >> BSP kernel or otherwise, it still seems wrong to me to fail and >> entire audio driver just because of a broken link. > > No, really. Random disconnections in the DAPM graph are just endless > pain from a support and debug point of view. This isn't something that > randomly breaks on specific hardware where we'd expect random errors at > runtime, it's something that will never have worked - it seems clear > nobody tested the mainline submission. I am under the impression that it was tested. Perhaps before it was rebased, but tests were completed. Ola was very surprised when I told him there were link failures. The only issue is that I only found out and told him a day before he was due to take his Summer leave. > It's very disappointing to see such an error exist, and even more > disappointing that there's no interest in fixing the driver. This is incorrect. I'm sure the driver will be fixed post-haste when Ola returns back from vacation. If I can find some time I might dabble in the mean-time also. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: lee.jones@linaro.org (Lee Jones) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/6] ASoC: dapm: If one widget fails, do not force all subsequent widgets to fail too Date: Wed, 01 Aug 2012 14:50:32 +0100 [thread overview] Message-ID: <50193428.5000708@linaro.org> (raw) In-Reply-To: <20120801132022.GS11892@opensource.wolfsonmicro.com> On 01/08/12 14:20, Mark Brown wrote: > On Wed, Aug 01, 2012 at 08:19:28AM +0100, Lee Jones wrote: >> On 31/07/12 16:18, Mark Brown wrote: > >>> I'm not going to apply this patch. This isn't a vendor BSP, we >>> shouldn't be putting random hacks like this in core code. > >> BSP kernel or otherwise, it still seems wrong to me to fail and >> entire audio driver just because of a broken link. > > No, really. Random disconnections in the DAPM graph are just endless > pain from a support and debug point of view. This isn't something that > randomly breaks on specific hardware where we'd expect random errors at > runtime, it's something that will never have worked - it seems clear > nobody tested the mainline submission. I am under the impression that it was tested. Perhaps before it was rebased, but tests were completed. Ola was very surprised when I told him there were link failures. The only issue is that I only found out and told him a day before he was due to take his Summer leave. > It's very disappointing to see such an error exist, and even more > disappointing that there's no interest in fixing the driver. This is incorrect. I'm sure the driver will be fixed post-haste when Ola returns back from vacation. If I can find some time I might dabble in the mean-time also. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2012-08-01 13:50 UTC|newest] Thread overview: 145+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-07-31 13:31 [PATCH 1/6] Bugfixes and clean-ups bound for the v3.6 RCs Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 1/6] ASoC: ab8500: Inform SoC Core that we have our own I/O arrangements Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 1/6] ASoC: dapm: If one widget fails, do not force all subsequent widgets to fail too Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:42 ` Mark Brown 2012-07-31 13:42 ` Mark Brown 2012-07-31 14:25 ` Lee Jones 2012-07-31 14:25 ` Lee Jones 2012-07-31 14:25 ` Lee Jones 2012-07-31 14:28 ` Mark Brown 2012-07-31 14:28 ` Mark Brown 2012-07-31 14:28 ` Mark Brown 2012-07-31 14:38 ` Lee Jones 2012-07-31 14:38 ` Lee Jones 2012-07-31 14:38 ` Lee Jones 2012-07-31 14:54 ` Mark Brown 2012-07-31 14:54 ` Mark Brown 2012-07-31 14:54 ` Mark Brown 2012-07-31 15:15 ` Lee Jones 2012-07-31 15:15 ` Lee Jones 2012-07-31 15:15 ` Lee Jones 2012-07-31 15:18 ` Mark Brown 2012-07-31 15:18 ` Mark Brown 2012-07-31 15:18 ` Mark Brown 2012-08-01 7:19 ` Lee Jones 2012-08-01 7:19 ` Lee Jones 2012-08-01 7:19 ` Lee Jones 2012-08-01 13:20 ` Mark Brown 2012-08-01 13:20 ` Mark Brown 2012-08-01 13:20 ` Mark Brown 2012-08-01 13:50 ` Lee Jones [this message] 2012-08-01 13:50 ` Lee Jones 2012-08-01 16:08 ` Mark Brown 2012-08-01 16:08 ` Mark Brown 2012-08-01 16:08 ` Mark Brown 2012-08-01 19:41 ` [alsa-devel] " Mark Brown 2012-08-01 19:41 ` Mark Brown 2012-08-01 19:41 ` Mark Brown 2012-08-02 7:45 ` [alsa-devel] " Lee Jones 2012-08-02 7:45 ` Lee Jones 2012-08-02 17:56 ` Mark Brown 2012-08-02 17:56 ` Mark Brown 2012-08-02 17:56 ` Mark Brown 2012-08-03 8:30 ` [alsa-devel] " Lee Jones 2012-08-03 8:30 ` Lee Jones 2012-08-03 8:30 ` Lee Jones 2012-08-04 0:48 ` [alsa-devel] " Mark Brown 2012-08-04 0:48 ` Mark Brown 2012-08-04 0:48 ` Mark Brown 2012-08-02 5:58 ` Ola Lilja 2012-08-02 5:58 ` Ola Lilja 2012-08-02 5:58 ` Ola Lilja 2012-08-02 9:59 ` Mark Brown 2012-08-02 9:59 ` Mark Brown 2012-08-02 9:59 ` Mark Brown 2012-08-10 11:43 ` Linus Walleij 2012-08-10 11:43 ` Linus Walleij 2012-08-10 11:43 ` Linus Walleij 2012-08-02 12:21 ` Lee Jones 2012-08-02 12:21 ` Lee Jones 2012-08-02 12:21 ` Lee Jones 2012-07-31 13:31 ` [PATCH 2/6] ARM: ux500: Remove unused snowball_of_platform_devs struct Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 2/6] ASoC: ab8500: Inform SoC Core that we have our own I/O arrangements Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 3/6] ARM: ux500: Fix merge error, so such struct 'snd_soc_u8500' Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 16:46 ` Sergei Shtylyov 2012-07-31 16:46 ` Sergei Shtylyov 2012-08-01 7:37 ` Lee Jones 2012-08-01 7:37 ` Lee Jones 2012-08-01 7:37 ` Lee Jones 2012-08-01 8:19 ` Lee Jones 2012-08-01 8:19 ` Lee Jones 2012-08-01 8:19 ` Lee Jones 2012-08-01 8:46 ` [PATCH 3/6 v2] ARM: ux500: Fix merge error, no matching driver name for, 'snd_soc_u8500' Lee Jones 2012-08-01 8:46 ` Lee Jones 2012-08-01 8:46 ` Lee Jones 2012-07-31 13:31 ` [PATCH 3/6] ARM: ux500: Remove unused snowball_of_platform_devs struct Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 13:31 ` [PATCH 4/6] ARM: ux500: Ensure probing of Audio devices when Device Tree is enabled Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 4/6] ARM: ux500: Fix merge error, so such struct 'snd_soc_u8500' Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 20:58 ` Arnd Bergmann 2012-07-31 13:31 ` [PATCH 5/6] ARM: ux500: Enable HIGHMEM on all mop500 platforms Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:56 ` Russell King - ARM Linux 2012-07-31 13:56 ` Russell King - ARM Linux 2012-07-31 13:56 ` Russell King - ARM Linux 2012-07-31 14:29 ` Lee Jones 2012-07-31 14:29 ` Lee Jones 2012-07-31 14:29 ` Lee Jones 2012-07-31 14:37 ` Russell King - ARM Linux 2012-07-31 14:37 ` Russell King - ARM Linux 2012-07-31 14:37 ` Russell King - ARM Linux 2012-07-31 20:50 ` Arnd Bergmann 2012-07-31 20:50 ` Arnd Bergmann 2012-07-31 20:50 ` Arnd Bergmann 2012-07-31 22:01 ` Russell King - ARM Linux 2012-07-31 22:01 ` Russell King - ARM Linux 2012-08-01 7:56 ` Lee Jones 2012-08-01 7:56 ` Lee Jones 2012-08-01 7:56 ` Lee Jones 2012-08-01 8:41 ` Russell King - ARM Linux 2012-08-01 8:41 ` Russell King - ARM Linux 2012-08-01 8:48 ` Lee Jones 2012-08-01 8:48 ` Lee Jones 2012-07-31 13:31 ` [PATCH 5/6] ARM: ux500: Ensure probing of Audio devices when Device Tree is enabled Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 20:54 ` Arnd Bergmann 2012-07-31 20:54 ` Arnd Bergmann 2012-08-01 7:34 ` Lee Jones 2012-08-01 7:34 ` Lee Jones 2012-08-01 7:34 ` Lee Jones 2012-08-01 13:32 ` Arnd Bergmann 2012-08-01 13:32 ` Arnd Bergmann 2012-08-01 13:32 ` Arnd Bergmann 2012-08-01 13:55 ` Lee Jones 2012-08-01 13:55 ` Lee Jones 2012-08-01 13:55 ` Lee Jones 2012-08-01 14:32 ` Arnd Bergmann 2012-08-01 14:32 ` Arnd Bergmann 2012-07-31 13:31 ` [PATCH 6/6] ARM: ux500: Enable HIGHMEM on all mop500 platforms Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:31 ` [PATCH 6/6] ASoC: Ux500: Move MSP pinctrl setup into the MSP driver Lee Jones 2012-07-31 13:31 ` Lee Jones 2012-07-31 13:40 ` [PATCH 1/6] Bugfixes and clean-ups bound for the v3.6 RCs Mark Brown 2012-07-31 13:40 ` Mark Brown 2012-07-31 14:30 ` Lee Jones 2012-07-31 14:30 ` Lee Jones 2012-07-31 14:30 ` Lee Jones
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=50193428.5000708@linaro.org \ --to=lee.jones@linaro.org \ --cc=STEricsson_nomadik_linux@list.st.com \ --cc=alsa-devel@alsa-project.org \ --cc=arnd@arndb.de \ --cc=broonie@opensource.wolfsonmicro.com \ --cc=linus.walleij@stericsson.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lrg@ti.com \ --cc=ola.o.lilja@stericsson.com \ --cc=olalilja@yahoo.se \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.