From: John Stultz <john.stultz@linaro.org>
To: Takashi Iwai <tiwai@suse.de>,
Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: Mark Brown <broonie@kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Bard Liao <yung-chuan.liao@linux.intel.com>,
Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>,
Srini Kandagatla <srinivas.kandagatla@linaro.org>,
Vinod Koul <vinod.koul@linaro.org>,
Amit Pundir <amit.pundir@linaro.org>
Subject: Re: [GIT PULL] sound fixes for 5.8-rc1
Date: Thu, 11 Jun 2020 17:49:29 -0700 [thread overview]
Message-ID: <CALAqxLWEfRJ_Jr0iDgmvqop5Etz5xve89Zy3OeUbnKUGtqjCqQ@mail.gmail.com> (raw)
In-Reply-To: <CANcMJZAG4QqJ2Cxt+1RLsa8Z4oR=2y8zyD_sqy3sm_8MXyhg2g@mail.gmail.com>
On Thu, Jun 11, 2020 at 5:13 PM John Stultz <john.stultz@linaro.org> wrote:
>
> On Thu, Jun 11, 2020 at 6:39 AM Takashi Iwai <tiwai@suse.de> wrote:
> > sound fixes for 5.8-rc1
> >
> > Here are last-minute fixes gathered before merge window close;
> > a few fixes are for the core while the rest majority are driver
> > fixes.
> >
> > * PCM locking annotation fixes and the possible self-lock fix
> > * ASoC DPCM regression fixes with multi-CPU DAI
>
> Just as a heads up, we just recently got HDMI audio working on the
> Dragonboard 845c (Vinod has patches he's sending out here in the next
> few days), but they suddenly stopped working today with the following
> error:
> [ 13.110725] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia1 mapping ok
> [ 13.119343] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia2 mapping ok
> [ 13.127969] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia3 mapping ok
> [ 13.135891] msm-snd-sdm845 soc@0:sound: Compress ASoC:
> snd-soc-dummy-dai <-> MultiMedia4 mapping ok
> [ 13.145042] msm-snd-sdm845 soc@0:sound: CPU DAI QUAT_MI2S_RX for
> rtd HDMI Playback does not support capture
> [ 13.154873] msm-snd-sdm845 soc@0:sound: ASoC: can't create pcm HDMI
> Playback :-22
> [ 13.165634] snd-malloc: invalid device type 0
> [ 13.170057] snd-malloc: invalid device type 0
> [ 13.174888] msm-snd-sdm845 soc@0:sound: Sound card registration failed
> [ 13.181574] msm-snd-sdm845: probe of soc@0:sound failed with error -22
>
> I've bisected it down to the following commit from this pull req:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=b73287f0b0745961b14e5ebcce92cc8ed24d4d52
>
> Without this patch:
> [ 13.056906] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia1 mapping ok
> [ 13.075465] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia2 mapping ok
> [ 13.092949] msm-snd-sdm845 soc@0:sound: snd-soc-dummy-dai <->
> MultiMedia3 mapping ok
> [ 13.109704] msm-snd-sdm845 soc@0:sound: Compress ASoC:
> snd-soc-dummy-dai <-> MultiMedia4 mapping ok
> [ 13.125584] msm-snd-sdm845 soc@0:sound: i2s-hifi <-> QUAT_MI2S_RX mapping ok
> [ 13.125621] msm-snd-sdm845 soc@0:sound: multicodec <-> SLIMBUS_0_RX
> mapping ok
> [ 13.141682] msm-snd-sdm845 soc@0:sound: wcd934x_tx1 <->
> SLIMBUS_0_TX mapping ok
> ...
>
> I've not managed to dig in with much analysis yet (possibly something
> off with the current patches we have), but wanted to raise the issue
> in case others start to see it too.
I don't know the backgroun again, but would something like the
following make sense?
https://git.linaro.org/people/john.stultz/android-dev.git/commit/?h=dev/db845c-mainline-WIP&id=7e49b248db77b5ed29b2aa278268e77650c75482
It avoids failing completely if playback or capture isn't found.
thanks
-john
next prev parent reply other threads:[~2020-06-12 0:50 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-11 11:33 [GIT PULL] sound fixes for 5.8-rc1 Takashi Iwai
2020-06-11 19:50 ` pr-tracker-bot
2020-06-12 0:13 ` John Stultz
2020-06-12 0:49 ` John Stultz [this message]
2020-06-12 11:41 ` Srinivas Kandagatla
2020-06-12 12:35 ` Mark Brown
2020-06-12 23:59 ` John Stultz
2020-06-12 12:19 ` Mark Brown
2020-06-12 13:54 ` Pierre-Louis Bossart
2020-06-12 14:46 ` Mark Brown
2020-06-12 14:56 ` Pierre-Louis Bossart
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=CALAqxLWEfRJ_Jr0iDgmvqop5Etz5xve89Zy3OeUbnKUGtqjCqQ@mail.gmail.com \
--to=john.stultz@linaro.org \
--cc=amit.pundir@linaro.org \
--cc=broonie@kernel.org \
--cc=guennadi.liakhovetski@linux.intel.com \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pierre-louis.bossart@linux.intel.com \
--cc=srinivas.kandagatla@linaro.org \
--cc=tiwai@suse.de \
--cc=vinod.koul@linaro.org \
--cc=yung-chuan.liao@linux.intel.com \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).