Date: Sat, 06 Sep 2008 23:24:12 +0100 From: Dieter <freebsd@sopwith.solgatos.com> To: Sean Bruno <sbruno@miralink.com> Cc: freebsd-firewire@freebsd.org Subject: Re: New and improved? patch Message-ID: <200809070624.GAA22550@sopwith.solgatos.com> In-Reply-To: Your message of "Sat, 06 Sep 2008 19:19:33 PDT." <48C33A35.6090203@miralink.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> > - 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.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200809070624.GAA22550>