From owner-freebsd-firewire@FreeBSD.ORG  Sun Sep  7 06:28:51 2008
Return-Path: <owner-freebsd-firewire@FreeBSD.ORG>
Delivered-To: freebsd-firewire@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id CA2001065675
	for <freebsd-firewire@freebsd.org>;
	Sun,  7 Sep 2008 06:28:51 +0000 (UTC)
	(envelope-from freebsd@sopwith.solgatos.com)
Received: from parsely.rain.com (parsely.rain.com [199.26.172.196])
	by mx1.freebsd.org (Postfix) with ESMTP id 5FD8C8FC27
	for <freebsd-firewire@freebsd.org>;
	Sun,  7 Sep 2008 06:28:51 +0000 (UTC)
	(envelope-from freebsd@sopwith.solgatos.com)
Received: from sopwith.solgatos.com (uucp@localhost)
	by parsely.rain.com (8.11.4/8.11.4) with UUCP id m876SjN09234;
	Sat, 6 Sep 2008 23:28:45 -0700 (PDT)
	(envelope-from freebsd@sopwith.solgatos.com)
Received: from localhost by sopwith.solgatos.com (8.8.8/6.24)
	id GAA22550; Sun, 7 Sep 2008 06:24:12 GMT
Message-Id: <200809070624.GAA22550@sopwith.solgatos.com>
To: Sean Bruno <sbruno@miralink.com>
In-reply-to: Your message of "Sat, 06 Sep 2008 19:19:33 PDT."
	<48C33A35.6090203@miralink.com> 
Date: Sat, 06 Sep 2008 23:24:12 +0100
From: Dieter <freebsd@sopwith.solgatos.com>
Cc: freebsd-firewire@freebsd.org
Subject: Re: New and improved? patch 
X-BeenThere: freebsd-firewire@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Firewire support in FreeBSD <freebsd-firewire.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-firewire>, 
	<mailto:freebsd-firewire-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-firewire>
List-Post: <mailto:freebsd-firewire@freebsd.org>
List-Help: <mailto:freebsd-firewire-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-firewire>, 
	<mailto:freebsd-firewire-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 07 Sep 2008 06:28:51 -0000

> > -               for (i = 0; i < 4; i++) {
> > -                       snprintf(name, sizeof(name), "%s.%d", devbase, i);
> > -                       if ((*fd = open(name, O_RDWR)) >= 0)
> > -                               break;
> > -               }
> > +               *fd = open(devname, O_RDWR);
> >
> >   
> 
> > Looking at various firewire man pages, I don't find any explanation of
> > the various /dev filenames, such as what the .%d part was/is for.  So I
> > have no clue why this code was changed.  Did I miss a discussion?
> >   
> I'm going to have to put a big "I have no idea" here.  This predates my 
> attempts
> at stabilization.  Let's examine it further in the driver code.  Perhaps 
> that will explain it's use.

I've been working on getting your new fwcontrol to compile and run
on NetBSD.  Looks like FreeBSD used to use /dev/fw0.0 but now
uses fw0, which...is...a symlink to fw0.0.  I only have .0 no .[123]
NetBSD still uses fw0.0, MAKEDEV doesn't create the symlink.  So now
you know why I noticed that fwcontrol doesn't tell you what filename
it couldn't open.  :-)  Still no idea what the trailing digit is for,
or why the fwcontrol code was changed.