linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.

  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).