From: Simon Horman <horms@verge.net.au>
To: Kuninori Morimoto <kuninori.morimoto.gx@gmail.com>
Cc: Mark Brown <broonie@kernel.org>,
Morimoto <kuninori.morimoto.gx@renesas.com>,
Magnus <magnus.damm@gmail.com>,
Linux-SH <linux-sh@vger.kernel.org>,
Linux-ALSA <alsa-devel@alsa-project.org>
Subject: Re: BockW 4th sound doesn't work on latest linus/master
Date: Wed, 29 Jan 2014 06:29:44 +0000 [thread overview]
Message-ID: <20140129062942.GF23833@verge.net.au> (raw)
In-Reply-To: <8761p48ry3.wl%kuninori.morimoto.gx@gmail.com>
On Mon, Jan 27, 2014 at 11:51:37PM -0800, Kuninori Morimoto wrote:
>
> Hi Simon, Mark again
>
> > I noticed that bockw 4th sound device doesn't work on
> > latest linus/master.
> >
> > My git bisect said the trigger is this patch
> >
> > e14ee5deab24200e4b70fe31a8c806f0acd3d37c
> > (ARM: shmobile: bockw: Do not enable CONFIG_DEVTMPFS defconfig)
> >
> > This patch removed below from defconfig
> >
> > -CONFIG_DEVTMPFS=y
> > -CONFIG_DEVTMPFS_MOUNT=y
> >
> > But, why this has relationship to sound ??
> >
> > #0 - #3 sound device have no problem.
> > But #4 sound device will be broken by this patch
> > Do you have any idea ?
> >
> > --- my kernel log -------
> > ...
> > ALSA device list:
> > #0: SSI56-AK4643
> > #1: SSI3-AK4554(playback)
> > #2: SSI4-AK4554(capture)
> > #3: SSI7-AK4554(playback)
> > #4: SSI8-AK4554(capture)
> >
> > --- test result of 4th device (bad case) ---
> >
> > # /home/arecord -c 2 -t wav -d 10 -r 48000 -f S16_LE -D hw:4 /tmp/hoge
> > ALSA lib pcm_hw.c:1433:(_snd_pcm_hw_open) Invalid value for card
> > arecord: main:608: audio open error: No such file or directory
> >
> > --- test result of 4th device (good case) ---
> >
> > # /home/arecord -c 2 -t wav -d 10 -r 48000 -f S16_LE -D hw:4 /tmp/hoge
> > Recording WAVE '/tmp/hoge' : Signed 16 bit Little Endian, Rate 48000 Hz, Stereo
>
> My rootfs with no DEVTMPFS didn't have /dev/snd/controlC4, /dev/snd/pcmC4D0c
> It start works if I created these node.
Makes sense.
Could you post a patch to revert the offending defconfig change.
Geert tells me that DEVTMPFS is needed for udev on Debian Jesse
and is also common in other arm defconfigs. So I think it should be
enabled.
next prev parent reply other threads:[~2014-01-29 6:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-28 7:13 BockW 4th sound doesn't work on latest linus/master Kuninori Morimoto
2014-01-28 7:51 ` Kuninori Morimoto
2014-01-29 6:29 ` Simon Horman [this message]
2014-01-29 7:17 ` Kuninori Morimoto
2014-01-29 12:38 ` Simon Horman
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=20140129062942.GF23833@verge.net.au \
--to=horms@verge.net.au \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=kuninori.morimoto.gx@gmail.com \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.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).