From mboxrd@z Thu Jan  1 00:00:00 1970
From: Takashi Iwai <tiwai@suse.de>
Subject: Re: [PATCH] ALSA: hda - Add
	digital	BEEP	generator	support for Realtek codecs.
Date: Tue, 10 Feb 2009 15:53:58 +0100
Message-ID: <s5hfxim2vi1.wl%tiwai@suse.de>
References: <200902031428523216360006MA62@nm06mta.auone-net.jp>
	<s5hbptjbna3.wl%tiwai@suse.de>
	<200902040837524161510006MA49@nm06mta.auone-net.jp>
	<s5hocxia1tx.wl%tiwai@suse.de>
	<200902041420132766610006MA66@nm06mta.auone-net.jp>
	<s5hljsm8doj.wl%tiwai@suse.de> <s5hprhv7ct2.wl%tiwai@suse.de>
	<200902071040534227980006MA83@nm06mta.auone-net.jp>
	<s5hprhuwjgu.wl%tiwai@suse.de>
Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <alsa-devel-bounces@alsa-project.org>
Received: from mx1.suse.de (ns.suse.de [195.135.220.2])
	by alsa0.perex.cz (Postfix) with ESMTP id E25CC103814
	for <alsa-devel@alsa-project.org>; Tue, 10 Feb 2009 15:54:00 +0100 (CET)
In-Reply-To: <s5hprhuwjgu.wl%tiwai@suse.de>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Sender: alsa-devel-bounces@alsa-project.org
Errors-To: alsa-devel-bounces@alsa-project.org
To: Kusanagi Kouichi <slash@ma.neweb.ne.jp>
Cc: alsa-devel@alsa-project.org
List-Id: alsa-devel@alsa-project.org

At Sat, 07 Feb 2009 12:54:25 +0100,
I wrote:
> 
> At Sat, 7 Feb 2009 19:40:52 +0900,
> Kusanagi Kouichi wrote:
> > 
> > On Fri, Feb 06, 2009 at 05:25:13PM +0100, Takashi Iwai wrote:
> > > At Wed, 04 Feb 2009 15:44:12 +0100,
> > > I wrote:
> > > > 
> > > > At Wed, 4 Feb 2009 23:20:12 +0900,
> > > > Kusanagi Kouichi wrote:
> > > > > 
> > > > > On Wed, Feb 04, 2009 at 12:17:14PM +0100, Takashi Iwai wrote:
> > > > > > Fair enough.
> > > > > > 
> > > > > > Though, there is still one problem -- not all models enabled the
> > > > > > analog beep path.  So, the next step is to create beep controls in all
> > > > > > possible situations.
> > > > > 
> > > > > I'll try later.
> > > > 
> > > > This can be a separate (additional) patch.  Let's get the first patch
> > > > into the main tree so that one can work on further issues.
> > > 
> > > ... and now fixed on sound git tree.
> > 
> > Some models for alc268 still seems to lack a beep control.
> > Is it OK to add a beep control unconditionally?
> 
> The question is whether it works.
> In your previous patch, patch_alc268() is untouched and has no
> digital beep hook yet.
> 
> I'll check a machine with ALC268 later.

As it seems working, I fixed the issue on sound git tree now.


Takashi