From owner-freebsd-doc@FreeBSD.ORG Thu Jan 19 16:51:16 2012 Return-Path: Delivered-To: freebsd-doc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5AB6F1065673; Thu, 19 Jan 2012 16:51:16 +0000 (UTC) (envelope-from simon@freebsd.org) Received: from emx.nitro.dk (emx.nitro.dk [IPv6:2a01:4f8:120:7384::102]) by mx1.freebsd.org (Postfix) with ESMTP id D0D7A8FC0C; Thu, 19 Jan 2012 16:51:15 +0000 (UTC) Received: from mailscan.leto.nitro.dk (mailscan.leto.nitro.dk [127.0.1.4]) by emx.nitro.dk (Postfix) with ESMTP id 0E8C41AE1CB; Thu, 19 Jan 2012 16:51:15 +0000 (UTC) Received: from emx.nitro.dk ([127.0.1.2]) by mailscan.leto.nitro.dk (mailscan.leto.nitro.dk [127.0.1.4]) (amavisd-new, port 10024) with LMTP id kVlkGOwweVI9; Thu, 19 Jan 2012 16:51:09 +0000 (UTC) Received: from [192.168.4.28] (unknown [89.100.56.44]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by emx.nitro.dk (Postfix) with ESMTPSA id 9CF3B1AE1BE; Thu, 19 Jan 2012 16:51:08 +0000 (UTC) Mime-Version: 1.0 (Apple Message framework v1251.1) Content-Type: text/plain; charset=iso-8859-1 From: "Simon L. B. Nielsen" In-Reply-To: <201201191009.45027.jhb@freebsd.org> Date: Thu, 19 Jan 2012 16:51:07 +0000 Content-Transfer-Encoding: quoted-printable Message-Id: <1287A4E8-5289-42AB-B53A-521155C1AA8B@freebsd.org> References: <201201191009.45027.jhb@freebsd.org> To: John Baldwin X-Mailer: Apple Mail (2.1251.1) Cc: freebsd-doc@freebsd.org, Adam Kirchhoff Subject: Re: FreeBSD 9.0-RELEASE Hardware Notes is wrong. X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 19 Jan 2012 16:51:16 -0000 On 19 Jan 2012, at 15:09, John Baldwin wrote: > On Wednesday, January 18, 2012 8:01:17 pm Benjamin Kaduk wrote: >> On Wed, 18 Jan 2012, Adam Kirchhoff wrote: >>=20 >>> http://www.freebsd.org/releases/9.0R/hardware.html >>>=20 >>> This section is correct: >>>=20 >>> [i386,amd64] The snd_emu10kx(4) driver supports the following sound = cards: >>>> Creative Sound Blaster Live! (EMU10K1 Chipset). Both PCM and MIDI >>>> interfaces are available. >>>> Creative Sound Blaster Audigy (CA0100 and CA0101 Chipset). PCM and = two >>>> MIDI interfaces available. >>>> Creative Sound Blaster Audigy 2 and Creative Sound Blaster Audigy 4 >>>> (CA0102 Chipset). PCM support is limited to 48kHz/16 bit stereo = (192kHz/24 >>>> bit part of this chipset is not supported). >>>> Creative Sound Blaster Audigy 2 Value (CA0108 Chipset). PCM support = is >>>> limited to 48kHz/16 bit stereo (192kHz/24 bit part of this chipset = is not >>>> supported). There is no MIDI support for this card. >>>=20 >>>=20 >>>=20 >>> This section is wrong: >>>=20 >>>=20 >>> The snd_emu10kx(4) driver does support the following sound cards = (although >>>> they have names similar to some supported ones): >>>> Creative Sound Blaster Live! 24-Bit, identified by FreeBSD as >>>> Creative Sound Blaster Audigy LS / ES, identified by FreeBSD as >>>> All other Creative sound cards with -DAT chipsets. >>>> All Creative X-Fi series sound cards. >>>=20 >>>=20 >>>=20 >>> None of those cards are supported by snd_emu10kx (according to the = man >>> page, and in my own personal experience). I believe a "does support" = should >>> read "does not support". >>=20 >> Looks like the second one should be "does not support", yes. >>=20 >> Thanks for spotting it; hopefully I can prod a committer into making = the=20 >> change. >=20 > The snd_emu10kx(4) manpage from which this is derived is correct. = This=20 > appears to be a problem with the code that parses the manpages to = generate the=20 > hardware release notes. Maybe it is confused by the markup: >=20 > The > .Nm > driver does > .Em not > support the following sound cards (although they have names > similar to some supported ones): The parser script basically implemented the mdoc commands which were = needed to generate valid output at the time, so it sounds entirely = likely that ".Em foo" just wasn't implemented. > This change fixes it. If simon ok's it I'll commit it. It does = result in > a new header line for malo(4) making it into the hardware notes (it = may be that > the malo(4) manpage needs to format it's hardware table differently = anyway). It's been a long time since I looked at the code and I don't have time = to do any tests right now, but the patch looks sane enough. If it = produces the expected output the patch is fine with me. > Index: release/doc/share/misc/man2hwnotes.pl > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > --- man2hwnotes.pl (revision 230267) > +++ man2hwnotes.pl (working copy) > @@ -324,6 +324,11 @@ sub parse { > } elsif (/^Fx/) { > dlog(3, "Got Fx command"); > parabuf_addline(\%mdocvars, "FreeBSD"); > + } elsif (/^Em (.+)$/) { > + my ($txt, $punct_str) =3D split_punct_chars($1); > + > + parabuf_addline(\%mdocvars, > + = normalize("$txt$punct_str")); > } else { > # Ignore all other commands. > dlog(3, "Ignoring unknown command $cmd"); --=20 Simon L. B. Nielsen