Re: [PATCH 2/2] ARM: DTS: am437x-sk-evm: Switch using simple-audio-card for audio support

Path	csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From	Felipe Balbi <balbi@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] ARM: DTS: am437x-sk-evm: Switch using simple-audio-card for audio support
Date	Thu, 02 Jul 2015 15:30:01 +0200
Message-ID	<pHMIV-3df-1@gated-at.bofh.it> (permalink)
References	<pHLjQ-2bH-3@gated-at.bofh.it> <pHLjQ-2bH-1@gated-at.bofh.it> <pHLMS-2Cb-43@gated-at.bofh.it> <pHMfU-2Nx-1@gated-at.bofh.it>
X-Original-To	Peter Ujfalusi <peter.ujfalusi@ti.com>
Reply-To	<balbi@ti.com>
MIME-Version	1.0
Content-Type	multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="vtJ+CqYNzKB4ukR4"
Content-Disposition	inline
User-Agent	Mutt/1.5.23 (2014-03-12)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	71
Organization	linux.* mail to news gateway
X-Original-Cc	<balbi@ti.com>, <tony@atomide.com>, <linux-omap@vger.kernel.org>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>
X-Original-Date	Thu, 2 Jul 2015 08:19:31 -0500
X-Original-Message-ID	<20150702131931.GH5837@saruman.tx.rr.com>
X-Original-References	<1435838286-30491-1-git-send-email-peter.ujfalusi@ti.com> <1435838286-30491-3-git-send-email-peter.ujfalusi@ti.com> <20150702122337.GE5837@saruman.tx.rr.com> <55953421.30009@ti.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	aioe.org linux.kernel:1175969

Show key headers only | View raw

[Multipart message — attachments visible in raw view] - view raw

On Thu, Jul 02, 2015 at 03:52:49PM +0300, Peter Ujfalusi wrote:
> On 07/02/2015 03:23 PM, Felipe Balbi wrote:
> > On Thu, Jul 02, 2015 at 02:58:06PM +0300, Peter Ujfalusi wrote:
> >> The sound support consist only Headset output on the board and can be
> >> handled by "simple-audio-card"
> >>
> >> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> > 
> > Still works fine:
> > 
> > Tested-by: Felipe Balbi <balbi@ti.com>
> > 
> > one question though, should Line In be listed below ?
> 
> Does the board have Line-in? I don't have the board, but I know this patch

yeah, it does :-)

> works on it ;)
> We only had playback path before also.

indeed, should go for -next only :-s

> > Also, unrelated to
> > $subject, but if I wait for a few seconds between runs of speaker-test
> > (iow, if I allow mcasp to idle) then I hear a slight chopping sound when
> > starting, perhaps we're unmuting too early ?
> 
> Is it only with this patch (via simple-card) or was it the same via the
> davinci-evm (ti,da830-evm-audio)?

/me goes test again

also without this patch :-)

-- 
balbi

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/2] ARM: DTS: am437x-sk-evm: Switch using  simple-audio-card for audio support Felipe Balbi <balbi@ti.com> - 2015-07-02 14:30 +0200
  Re: [PATCH 2/2] ARM: DTS: am437x-sk-evm: Switch using  simple-audio-card for audio support Peter Ujfalusi <peter.ujfalusi@ti.com> - 2015-07-02 15:00 +0200
    Re: [PATCH 2/2] ARM: DTS: am437x-sk-evm: Switch using  simple-audio-card for audio support Felipe Balbi <balbi@ti.com> - 2015-07-02 15:30 +0200

csiph-web