From nobody@FreeBSD.org  Mon Oct 23 15:27:14 2006
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id A544016A417
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 23 Oct 2006 15:27:14 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [216.136.204.117])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 1D65D43D79
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 23 Oct 2006 15:27:10 +0000 (GMT)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.13.1/8.13.1) with ESMTP id k9NFR9I4066971
	for <freebsd-gnats-submit@FreeBSD.org>; Mon, 23 Oct 2006 15:27:09 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.13.1/8.13.1/Submit) id k9NFR91d066970;
	Mon, 23 Oct 2006 15:27:09 GMT
	(envelope-from nobody)
Message-Id: <200610231527.k9NFR91d066970@www.freebsd.org>
Date: Mon, 23 Oct 2006 15:27:09 GMT
From: KAWATA Masahiko<kawata@mta.biglobe.ne.jp>
To: freebsd-gnats-submit@FreeBSD.org
Subject: Please add inverted amplifier sense quirks for NEC VersaPro VY10FBH-L / VJ12FCH-X

>Number:         104715
>Category:       kern
>Synopsis:       [sound] [patch] Please add inverted amplifier sense quirks for NEC VersaPro VY10FBH-L / VJ12FCH-X
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    ariff
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Oct 23 15:30:18 GMT 2006
>Closed-Date:    Sun Dec 24 06:01:36 GMT 2006
>Last-Modified:  Sun Dec 24 06:01:36 GMT 2006
>Originator:     KAWATA Masahiko
>Release:        FreeBSD messi 6.1-STABLE
>Organization:
>Environment:
FreeBSD messi 6.1-STABLE FreeBSD 6.1-STABLE #5: Wed Jul 12 01:34:02 JST 2006  kawata@messi:/usr/src/sys/i386/compile/MESSI  i386

>Description:
My note PC's need to turn on inverted external amplifier sense flag.
Please add inverted amplifier sense quirks for these machines.

NEC VersaPro VY10FBH-L
pcm0@pci0:31:5: class=0x040100 card=0x82131033 chip=0x24c58086 rev=0x03 hdr=0x00    vendor   = 'Intel Corporation'
    device   = '82801DB/DBL/DBM (ICH4/ICH4-L/ICH4-M) AC'97 Audio Controller'
    class    = multimedia
    subclass = audio

NEC VersaPro VJ12FCH-X
pcm0@pci0:31:5: class=0x040100 card=0x82be1033 chip=0x24c58086 rev=0x03 hdr=0x00    vendor   = 'Intel Corporation'
    device   = '82801DB/DBL/DBM (ICH4/ICH4-L/ICH4-M) AC'97 Audio Controller'
    class    = multimedia
    subclass = audio

>How-To-Repeat:
# kldload snd_ich
% mpg123 anything.mp3
>Fix:
--- /usr/src/sys/dev/sound/pci/ich.c.orig       Sat Oct 14 02:16:55 2006
+++ /usr/src/sys/dev/sound/pci/ich.c    Sat Oct 14 02:12:38 2006
@@ -850,6 +850,7 @@
        case 0x8197104d:        /* Sony S1XP */
        case 0x81c0104d:        /* Sony VAIO type T */
        case 0x81c5104d:        /* Sony VAIO VGN B1VP/B1XP */
+       case 0x82131033:        /* NEC VersaPro VJ10F/BH */
+       case 0x82be1033:        /* NEC VersaPro VJ12F/CH */
        case 0x3089103c:        /* Compaq Presario B3800 */
                ac97_setflags(sc->codec, ac97_getflags(sc->codec) | AC97_F_EAPD_INV);
                break;

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->freebsd-multimedia 
Responsible-Changed-By: linimon 
Responsible-Changed-When: Mon Oct 23 20:43:50 UTC 2006 
Responsible-Changed-Why:  
Over to maintainer(s). 

http://www.freebsd.org/cgi/query-pr.cgi?pr=104715 
Responsible-Changed-From-To: freebsd-multimedia->ariff 
Responsible-Changed-By: ariff 
Responsible-Changed-When: Tue Oct 24 20:03:26 UTC 2006 
Responsible-Changed-Why:  
I'll handle this. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=104715 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/104715: commit references a PR
Date: Sun, 26 Nov 2006 12:24:38 +0000 (UTC)

 ariff       2006-11-26 12:24:06 UTC
 
   FreeBSD src repository
 
   Modified files:
     sys/sys              param.h 
     .                    UPDATING 
     sys/dev/sound/pci    atiixp.c cmi.c es137x.c ich.c via8233.c 
     sys/dev/sound/pci/hda hda_reg.h hdac.c hdac_private.h 
     sys/dev/sound/pcm    ac97.c ac97.h buffer.c buffer.h channel.c 
                          channel.h dsp.c fake.c feeder.c feeder.h 
                          feeder_fmt.c feeder_rate.c 
                          feeder_volume.c mixer.c sndstat.c sound.c 
                          sound.h vchan.c 
     sys/dev/sound/usb    uaudio.c 
   Log:
   Welcome to Once-a-year Sound Mega-Commit. Enjoy numerous updates and fixes
   in every sense.
   
   General
   -------
   
   - Multichannel safe, endian safe, format safe
      * Large part of critical pcm filters such as vchan.c, feeder_rate.c,
        feeder_volume.c, feeder_fmt.c and feeder.c has been rewritten so that
        using them does not cause the pcm data to be converted to 16bit little
        endian.
      * Macrosses for accessing pcm data safely are defined within sound.h in
        the form of PCM_READ_* / PCM_WRITE_*
      * Currently, most of them are probably limited for mono/stereo handling,
        but the future addition of true multichannel will be much easier.
   
   - Low latency operation
     * Well, this require lot more works to do not just within sound driver,
       but we're heading towards right direction. Buffer/block sizing within
       channel.c is rewritten to calculate precise allocation for various
       combination of sample/data/rate size. As a result, applying correct
       SNDCTL_DSP_POLICY value will achive expected latency behaviour simmilar
       to what commercial 4front driver do.
     * Signal handling fix. ctrl+c of "cat /dev/zero > /dev/dsp" does not
       result long delay.
     * Eliminate sound truncation if the sound data is too small.
       DIY:
         1) Download / extract
            http://people.freebsd.org/~ariff/lowlatency/shortfiles.tar.gz
         2) Do a comparison between "cat state*.au > /dev/dsp" and
            "for x in state*.au ; do cat $x > /dev/dsp ; done"
            - there should be no "perceivable" differences.
       Double close for PR kern/31445.
   
     CAVEAT: Low latency come with (unbearable) price especially for poorly
             written applications. Applications that trying to act smarter
             by requesting (wrong) blocksize/blockcount will suffer the most.
             Fixup samples/patches can be found at:
             http://people.freebsd.org/~ariff/ports/
   
   - Switch minimum/maximum sampling rate limit to "1" and "2016000" (48k * 42)
     due to closer compatibility with 4front driver.
     Discussed with: marcus@ (long time ago?)
   
   - All driver specific sysctls in the form of "hw.snd.pcm%d.*" have been
     moved to their own dev sysctl nodes, notably:
     hw.snd.pcm%d.vchans -> dev.pcm.%d.vchans
     Bump __FreeBSD_version.
   
   Driver specific
   ---------------
   
   - Ditto for sysctls.
   
   - snd_atiixp, snd_es137x, snd_via8233, snd_hda
     * Numerous cleanups and fixes.
     * _EXPERIMENTAL_ polling mode support using simple callout_* mechanisme.
      This was intended for pure debugging and latency measurement, but proven
      good enough in few unexpected and rare cases (such as problematic shared
      IRQ with GIANT devices - USB). Polling can be enabled/disabled through
      dev.pcm.0.polling. Disabled by default.
   
   - snd_ich
     * Fix possible overflow during speed calibration. Delay final
       initialization (pcm_setstatus) after calibration finished.
       PR: kern/100169
       Tested by: Kevin Overman <oberman@es.net>
     * Inverted EAPD for few Nec VersaPro.
       PR: kern/104715
       Submitted by: KAWATA Masahiko <kawata@mta.biglobe.ne.jp>
   
   Thanks to various people, notably Joel Dahl, Yuriy Tsibizov, Kevin Oberman,
   those at #freebsd-azalia @ freenode and others for testing.
   
   Joel Dahl will do the manpage update.
   
   Revision  Changes    Path
   1.466     +13 -0     src/UPDATING
   1.8       +354 -196  src/sys/dev/sound/pci/atiixp.c
   1.38      +3 -3      src/sys/dev/sound/pci/cmi.c
   1.61      +622 -371  src/sys/dev/sound/pci/es137x.c
   1.2       +24 -24    src/sys/dev/sound/pci/hda/hda_reg.h
   1.9       +456 -171  src/sys/dev/sound/pci/hda/hdac.c
   1.3       +10 -1     src/sys/dev/sound/pci/hda/hdac_private.h
   1.68      +228 -103  src/sys/dev/sound/pci/ich.c
   1.26      +558 -275  src/sys/dev/sound/pci/via8233.c
   1.64      +86 -18    src/sys/dev/sound/pcm/ac97.c
   1.19      +0 -2      src/sys/dev/sound/pcm/ac97.h
   1.29      +60 -26    src/sys/dev/sound/pcm/buffer.c
   1.13      +3 -0      src/sys/dev/sound/pcm/buffer.h
   1.112     +721 -311  src/sys/dev/sound/pcm/channel.c
   1.34      +39 -1     src/sys/dev/sound/pcm/channel.h
   1.100     +16 -17    src/sys/dev/sound/pcm/dsp.c
   1.17      +2 -1      src/sys/dev/sound/pcm/fake.c
   1.38      +452 -134  src/sys/dev/sound/pcm/feeder.c
   1.14      +26 -3     src/sys/dev/sound/pcm/feeder.h
   1.21      +623 -203  src/sys/dev/sound/pcm/feeder_fmt.c
   1.19      +320 -546  src/sys/dev/sound/pcm/feeder_rate.c
   1.3       +178 -33   src/sys/dev/sound/pcm/feeder_volume.c
   1.56      +5 -3      src/sys/dev/sound/pcm/mixer.c
   1.23      +11 -10    src/sys/dev/sound/pcm/sndstat.c
   1.107     +40 -48    src/sys/dev/sound/pcm/sound.c
   1.73      +280 -7    src/sys/dev/sound/pcm/sound.h
   1.26      +458 -131  src/sys/dev/sound/pcm/vchan.c
   1.23      +1 -1      src/sys/dev/sound/usb/uaudio.c
   1.280     +1 -1      src/sys/sys/param.h
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->patched 
State-Changed-By: ariff 
State-Changed-When: Fri Dec 1 15:35:46 UTC 2006 
State-Changed-Why:  


http://www.freebsd.org/cgi/query-pr.cgi?pr=104715 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/104715: commit references a PR
Date: Sun, 24 Dec 2006 05:44:27 +0000 (UTC)

 ariff       2006-12-24 05:44:11 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_6)
     sys/dev/sound/pci    ich.c 
   Log:
   MFC:
   
   Add inverted amplifier sense quirk for HP Compaq nx4300 and few
   NEC VersaPro.
   
   PR:             kern/106104, kern/104715
   Submitted by:   Shun-ichi Kobayashi <koba@mbox.kudpc.kyoto-u.ac.jp>
                   KAWATA Masahiko <kawata@mta.biglobe.ne.jp>
   
   Add support for Nvidia Nforce MCP04 AC97 controller.
   
   PR:             kern/106829
   Submitted by:   Frdric Petit <fredantispam@free.fr>
   
   Revision  Changes    Path
   1.53.2.9  +6 -0      src/sys/dev/sound/pci/ich.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/104715: commit references a PR
Date: Sun, 24 Dec 2006 05:44:59 +0000 (UTC)

 ariff       2006-12-24 05:44:47 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_6_2)
     sys/dev/sound/pci    ich.c 
   Log:
   MFC:
   
   Add inverted amplifier sense quirk for HP Compaq nx4300 and few
   NEC VersaPro.
   
   PR:             kern/106104, kern/104715
   Submitted by:   Shun-ichi Kobayashi <koba@mbox.kudpc.kyoto-u.ac.jp>
                   KAWATA Masahiko <kawata@mta.biglobe.ne.jp>
   
   Add support for Nvidia Nforce MCP04 AC97 controller.
   
   PR:             kern/106829
   Submitted by:   Frdric Petit <fredantispam@free.fr>
   
   Approved by:    re (kensmith)
   
   Revision      Changes    Path
   1.53.2.8.2.1  +6 -0      src/sys/dev/sound/pci/ich.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: ariff 
State-Changed-When: Sun Dec 24 06:00:40 UTC 2006 
State-Changed-Why:  
Fix committed to both RELENG_6 and 6_2. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=104715 
>Unformatted:
 >
 Subject: Please add inverted amplifier sense quirks for NEC VersaPro VY10FBH-L / VJ12FCH-X
 X-REMOTE_ADDR-Is-Open-Proxy: Maybe
 X-Send-Pr-Version: www-3.0
 
