Bug 85100

Summary: NOTES: ICH audio device support statement is ambiguous
Product: Documentation Reporter: Arthur Hartwig <arthur.hartwig>
Component: Books & ArticlesAssignee: Alexander Motin <mav>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Arthur Hartwig 2005-08-19 00:40:27 UTC 
The ICH6 (possibly other members of the ICH family) provides two audio
devices: an AC97 audio and the Intel High Definition Audio (also called
azalia). The FreeBSD/i386 5.4-RELEASE Hardware Notes (and possible the
Hardware Notes for other platforms) ambiguously states that the ICH6 audio
device is supported without stating which ICH6 audio device. My examination
of the dev/sound/pci/ich.c driver suggests that the support is for the AC97
audio device and that there is no support for the High Definition Audio device.

I discovered this when I tried to get the sound working on my ICH7 based
motherboard (Gigabyte Gi-8I945P-G) Noting that the ICH6 and ICH7 High
definition audio devices seemed very similar (apart from the PCI ids) I
tweaked the ich sound driver to also recognise the ICH7 HDA device but the
driver attach function reported another problem. When I went looking into
that I discovered the AC97 audio device on the IC6 and ICH7 and that the
ich sound driver supported the AC97 audio. Since the BIOS on my motherboard
disables the AC97 audio device I am left with no audio support.

Fix: 

Suggest the hardware release notes on all relevant platforms be clarified,
the file conf/NOTES be modified by replacing

# snd_ich:              Intel ICH PCI and some more audio controllers
#                       embedded in a chipset.

with something like
# snd_ich:		The AC97 audio device in Intel ICH PCI and 
#			some other chipsets. This does not support
#			the Intel High Definition Audio device.

and the file dev/sound/pci/ich.c be changed to include a statement that its
a driver for the AC97 audio device not the Intel High Definition Audio device.
Comment 1 Joel Dahl freebsd_committer freebsd_triage 2006-11-06 23:17:32 UTC 
Responsible Changed
From-To: freebsd-doc->joel

Grab.
Comment 2 Jeroen Ruigrok van der Werven 2007-05-17 14:21:44 UTC 
Hi Joel,

This PR needs to be updated to reflect that NOTES in CURRENT contains
references to snd_hda and snd_ich.
Also, both manual pages exist. Perhaps a Xref to snd_hda(4) from snd_ich(4) is
needed due to the fact the chipset is typically known as ICH and people will
use that as their first vector of approach.

Nonetheless, it still means that on STABLE the NOTES and snd_ich(4)
documentation needs changes to reflect STABLE does NOT support HDA yet.

-- 
Jeroen Ruigrok van der Werven <asmodai(-at-)in-nomine.org> / asmodai
ã¤ã§ã«ã¼ã³ ã©ã¦ãã­ã㯠ã´ã¡ã³ ãã« ã¦ã§ã«ã´ã§ã³
http://www.in-nomine.org/ | http://www.rangaku.org/
Alas! The world is full of enormous lights and mysteries, and man shuts
them from himself with one small hand!
Comment 3 Joel Dahl freebsd_committer freebsd_triage 2008-09-15 17:30:47 UTC 
Responsible Changed
From-To: joel->freebsd-doc

Over to docs.
Comment 4 Tom Rhodes freebsd_committer freebsd_triage 2009-01-07 09:25:57 UTC 
State Changed
From-To: open->patched

Alexander has recently patched this up in CURRENT, assign 
to him.  Thanks Alexander!
Comment 5 Tom Rhodes freebsd_committer freebsd_triage 2009-01-07 09:25:57 UTC 
Responsible Changed
From-To: freebsd-doc->mav

Alexander has recently patched this up in CURRENT, assign 
to him.  Thanks Alexander!
Comment 6 Alexander Motin freebsd_committer freebsd_triage 2009-01-07 21:25:19 UTC 
State Changed
From-To: patched->closed

Changes merged to 7-STABLE.