Re: New and improved? patch

> > -               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.
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-firewire
To unsubscribe, send any mail to "[email protected]"