> >         /* was a break -SG- */
> >                                 return(0);
> 
>   Uh - doesn't a return(0) indicate an error?  This routine looks like it
> returns 1 if everything is OK.

Of course with few exceptions functions should return their error status.
I trust that Michael is correct so I thought I would make the point that
people (i.e. driver authors) should be returning error states e.g.

	return (-1);	/* error */
	return (0);	/* no error */

This is the standard way to do things in NetBSD.

Chris.