Bug 116680

Summary: [patch] fwohci(4): Documentation mistake: CXD1947 _not_ supported, see fwohci_pci.c
Product: Documentation Reporter: Jens Köhler <mail>
Component: Books & ArticlesAssignee: Christian Brueffer <brueffer>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Jens Köhler 2007-09-26 23:30:01 UTC
FreeBSD probably NEVER supported CXD1947! What a pity.

See your own patch:

==== //depot/projects/smpng/sys/dev/firewire/fwohci_pci.c#35 (text+ko) ====

@@ -31,7 +31,7 @@
  * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  *
- * $FreeBSD: src/sys/dev/firewire/fwohci_pci.c,v 1.52 2005/05/20 12:37:16 marius Exp $
+ * $FreeBSD: src/sys/dev/firewire/fwohci_pci.c,v 1.53 2005/07/17 19:59:07 imp Exp $
  */

 #define BOUNCE_BUFFER_TEST	0
@@ -169,8 +169,8 @@
 		return BUS_PROBE_DEFAULT;
 	}
 	if (id == (FW_VENDORID_SONY | FW_DEVICE_CXD1947)) {
-		device_set_desc(dev, "Sony i.LINK (CXD1947)");
-		return BUS_PROBE_DEFAULT;
+		device_printf(dev, "Sony i.LINK (CXD1947) not supported");
+		return ENXIO;
 	}
 	if (id == (FW_VENDORID_SONY | FW_DEVICE_CXD3222)) {
 		device_set_desc(dev, "Sony i.LINK (CXD3222)");

Fix: 

write some cool driver
How-To-Repeat: install on adequate hardware
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2007-09-26 23:50:55 UTC
State Changed
From-To: open->feedback

So where is the part of the documentation that has the bug, so we can 
address it?
Comment 2 Mark Linimon freebsd_committer freebsd_triage 2008-03-03 07:02:45 UTC
State Changed
From-To: feedback->analyzed

OK, after some investigation I've figured out where the manpage lives. 
OTOH I don't have any manpage-fu so all I can do is generate the patch 
(which seems obvious). 

Is anyone from docs interested in testing and comitting? 

cvs diff: Diffing . 
Index: fwohci.4 
=================================================================== 
RCS file: /home/FreeBSD/ncvs/src/share/man/man4/fwohci.4,v 
retrieving revision 1.16 
diff -r1.16 fwohci.4 
102,103d101 
< Sony i.LINK (CXD1947) 
< .It 



Comment 3 Mark Linimon freebsd_committer freebsd_triage 2008-03-03 07:02:45 UTC
Responsible Changed
From-To: freebsd-bugs->freebsd-doc
Comment 4 dfilter service freebsd_committer freebsd_triage 2008-03-03 08:55:59 UTC
brueffer    2008-03-03 08:55:50 UTC

  FreeBSD src repository

  Modified files:
    share/man/man4       fwohci.4 
  Log:
  The Sony i.LINK (CXD1947) is _not_ supported.
  
  PR:             116680
  Submitted by:   Jens Köhler <mail@chatty.de>
  MFC after:      3 days
  
  Revision  Changes    Path
  1.17      +1 -3      src/share/man/man4/fwohci.4
_______________________________________________
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"
Comment 5 Christian Brueffer freebsd_committer freebsd_triage 2008-03-03 08:56:10 UTC
State Changed
From-To: analyzed->patched

Fixed in CURRENT, MFC pending. Thanks! 


Comment 6 Christian Brueffer freebsd_committer freebsd_triage 2008-03-03 08:56:10 UTC
Responsible Changed
From-To: freebsd-doc->brueffer

MFC reminder.
Comment 7 Gavin Atkinson freebsd_committer freebsd_triage 2008-06-02 10:47:18 UTC
State Changed
From-To: patched->closed

This doc bug is fixed in HEAD, RELENG_7 and RELENG_6.