Hello John,

* John Baldwin <j...@freebsd.org> wrote:
> When I performed similar changes in lpt(4) I stored the flags in si_drv2 
> instead of the minor number.  You might want to do the same here so the 
> drivers are closer.

I guess in the very far future we'll hopefully just get rid of
dev2unit() and allow the struct field that stores the unit number
(si_drv0) to be used by the drivers directly. An advantage of the unit
number is that it's an integer type, unlike si_drv1 and si_drv2.

I guess it's good to just leave it the way it is right now, because it
was the least intrusive way to fix it like this and best of all, if I
ever get the chance to remove dev2unit(), I'm forced to let it use
si_drv0 instead of keeping evil casts from/to pointers.

-- 
 Ed Schouten <e...@80386.nl>
 WWW: http://80386.nl/

Attachment: pgp6GQ9vfxCVZ.pgp
Description: PGP signature

Reply via email to