From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maurus Cuelenaere Subject: Re: [PATCH 2/3] ASoC: Add SmartQ sound driver Date: Sun, 04 Jul 2010 11:19:37 +0200 Message-ID: <4C305229.30409@gmail.com> References: <4c2e8856.487e0e0a.3bb5.3ec0@mx.google.com> <20100704090549.GD16825@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-ew0-f51.google.com (mail-ew0-f51.google.com [209.85.215.51]) by alsa0.perex.cz (Postfix) with ESMTP id 4236F24153 for ; Sun, 4 Jul 2010 11:19:39 +0200 (CEST) Received: by ewy21 with SMTP id 21so1367992ewy.38 for ; Sun, 04 Jul 2010 02:19:39 -0700 (PDT) In-Reply-To: <20100704090549.GD16825@opensource.wolfsonmicro.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Mark Brown Cc: alsa-devel@alsa-project.org, linux-samsung-soc@vger.kernel.org, Takashi Iwai , linux-arm-kernel@lists.infradead.org, Liam Girdwood List-Id: alsa-devel@alsa-project.org Op 04-07-10 11:05, Mark Brown schreef: > On Sat, Jul 03, 2010 at 02:46:12AM +0200, Maurus Cuelenaere wrote: >> This adds sound support for the SmartQ board. >> >> The hardware consists of a S3C6410 coupled with a WM8987 over I=B2S. The= WM8750 >> driver is used for driving the WM8987, as they are register compatible. > I've applied this, but... > >> +static struct snd_soc_jack_pin smartq_jack_pins[] =3D { >> + /* Disable speaker when headphone is plugged in */ >> + { >> + .pin =3D "Internal Speaker", >> + .mask =3D SND_JACK_HEADPHONE, >> + .invert =3D true, >> + }, >> +}; > ...it probably also make sense to disable the headphone output when the > headphone is not plugged in to save power? I don't see what power there is to save? The speaker and headphone jack are= connected to the same physical pins, so when disabling the pins for one of= them, the other is also disabled. The only way to disable the speaker is to shutdown its amplifier (using tha= t GPIO pin). -- = Maurus Cuelenaere From mboxrd@z Thu Jan 1 00:00:00 1970 From: mcuelenaere@gmail.com (Maurus Cuelenaere) Date: Sun, 04 Jul 2010 11:19:37 +0200 Subject: [PATCH 2/3] ASoC: Add SmartQ sound driver In-Reply-To: <20100704090549.GD16825@opensource.wolfsonmicro.com> References: <4c2e8856.487e0e0a.3bb5.3ec0@mx.google.com> <20100704090549.GD16825@opensource.wolfsonmicro.com> Message-ID: <4C305229.30409@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Op 04-07-10 11:05, Mark Brown schreef: > On Sat, Jul 03, 2010 at 02:46:12AM +0200, Maurus Cuelenaere wrote: >> This adds sound support for the SmartQ board. >> >> The hardware consists of a S3C6410 coupled with a WM8987 over I?S. The WM8750 >> driver is used for driving the WM8987, as they are register compatible. > I've applied this, but... > >> +static struct snd_soc_jack_pin smartq_jack_pins[] = { >> + /* Disable speaker when headphone is plugged in */ >> + { >> + .pin = "Internal Speaker", >> + .mask = SND_JACK_HEADPHONE, >> + .invert = true, >> + }, >> +}; > ...it probably also make sense to disable the headphone output when the > headphone is not plugged in to save power? I don't see what power there is to save? The speaker and headphone jack are connected to the same physical pins, so when disabling the pins for one of them, the other is also disabled. The only way to disable the speaker is to shutdown its amplifier (using that GPIO pin). -- Maurus Cuelenaere