From: Tony Lindgren <tony@atomide.com>
To: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Cc: Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
alsa-devel@alsa-project.org, linux-omap@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCHv2 0/6] Motorola Droid 4 Audio Support
Date: Mon, 17 Jul 2017 22:48:33 -0700 [thread overview]
Message-ID: <20170718054833.GL10026@atomide.com> (raw)
In-Reply-To: <20170717141347.t2642xbkol3g36rv@earth>
* Sebastian Reichel <sebastian.reichel@collabora.co.uk> [170717 07:14]:
> Hi,
>
> On Mon, Jul 17, 2017 at 03:17:10AM -0700, Tony Lindgren wrote:
> > * Sebastian Reichel <sebastian.reichel@collabora.co.uk> [170717 03:13]:
> > > On Mon, Jul 17, 2017 at 02:29:04AM -0700, Tony Lindgren wrote:
> > > > * Sebastian Reichel <sebastian.reichel@collabora.co.uk> [170712 08:19]:
> > > > > * Switch from simple-audio-card to audio-graph-card
> > > >
> > > > Gave this a quick try against v4.13-rc1 with SND_AUDIO_GRAPH_CARD
> > > > enabled as a loadable module. However loading it oopses for me,
> > > > see below. Maybe some dependencies are missing?
> > >
> > > It works for me on top of v4.13-rc1 (my kernel is monolithic).
> > > Looking at the stacktrace it seems to be a bug in audio graph
> > > card and not in the codec driver.
> >
> > OK I do also have:
> >
> > CONFIG_DEBUG_LOCKDEP=y
> > CONFIG_DEBUG_ATOMIC_SLEEP=y
>
> I added those and I do not get a stacktrace for anything sound
> related and audio works. I noticed one issue in dapm routing,
> that I accidently added. That will be fixed in PATCHv3, but its
> in untestable path anyways (EXT capture, probably FM radio is
> connected to EXT).
Hmm maybe that was not with DEBUG_ATOMIC_SLEEP then? I also
verified it does not happen unless your dts patch
"ARM: dts: omap4-droid4: add soundcard" is applied.
I doubt that having it as loadable module makes any difference
here as it's the might_sleep() in __mutex_lock_common() producing
the BUG.
Regards,
Tony
next prev parent reply other threads:[~2017-07-18 5:48 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-12 15:18 [PATCHv2 0/6] Motorola Droid 4 Audio Support Sebastian Reichel
2017-07-12 15:18 ` [PATCHv2 1/6] ALSA: pcm: Export soc_dpcm_runtime_update Sebastian Reichel
2017-07-12 15:18 ` [PATCHv2 2/6] dt-bindings: sound: add motorola,cpcap-audio-codec Sebastian Reichel
2017-07-12 15:18 ` [PATCHv2 3/6] ASoC: codec: cpcap: new codec Sebastian Reichel
2017-07-12 15:37 ` Takashi Iwai
2017-07-17 9:29 ` Tony Lindgren
2017-07-17 14:18 ` Sebastian Reichel
2017-07-17 15:21 ` Mark Brown
2017-07-12 15:18 ` [PATCHv2 4/6] ARM: dts: motorola-cpcap-mapphone: add audio-codec Sebastian Reichel
2017-07-12 15:18 ` [PATCHv2 5/6] ARM: dts: motorola-cpcap-mapphone: set initial mode for vaudio Sebastian Reichel
2017-07-12 15:18 ` [PATCHv2 6/6] ARM: dts: omap4-droid4: add soundcard Sebastian Reichel
2017-07-17 9:29 ` [PATCHv2 0/6] Motorola Droid 4 Audio Support Tony Lindgren
2017-07-17 10:12 ` Sebastian Reichel
2017-07-17 10:17 ` Tony Lindgren
[not found] ` <20170717101709.GI10026-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-07-17 14:13 ` Sebastian Reichel
2017-07-18 5:48 ` Tony Lindgren [this message]
[not found] ` <20170718054833.GL10026-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-07-18 9:29 ` Sebastian Reichel
2017-07-18 9:37 ` Takashi Iwai
2017-07-27 9:39 ` Tony Lindgren
2017-07-27 10:23 ` Mark Brown
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=20170718054833.GL10026@atomide.com \
--to=tony@atomide.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=perex@perex.cz \
--cc=robh+dt@kernel.org \
--cc=sebastian.reichel@collabora.co.uk \
--cc=tiwai@suse.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).