d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove unneeded variable and casting from newdev().

Browse Source 
Remove the `udev' variable, which has a different type than the original
function argument and si_drv0. The `udev' name is also misleading,
because it is not the number returned by dev2udev(). Rename this
argument to `unit'. It is the same number as returned by dev2unit().
This commit is contained in:
Ed Schouten 2009-04-15 20:15:36 +00:00
parent d7cbfc1b18
commit bce79dbb29
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=191116
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sys/kern/kern_conf.c
View File

@ -513,23 +513,21 @@ notify_destroy(struct cdev *dev)
}
static struct cdev *
newdev(struct cdevsw *csw, int y, struct cdev *si)
newdev(struct cdevsw *csw, int unit, struct cdev *si)
{
struct cdev *si2;
dev_t udev;
mtx_assert(&devmtx, MA_OWNED);
udev = y;
if (csw->d_flags & D_NEEDMINOR) {
/* We may want to return an existing device */
LIST_FOREACH(si2, &csw->d_devs, si_list) {
if (dev2unit(si2) == udev) {
if (dev2unit(si2) == unit) {
dev_free_devlocked(si);
return (si2);
}
}
}
si->si_drv0 = udev;
si->si_drv0 = unit;
si->si_devsw = csw;
LIST_INSERT_HEAD(&csw->d_devs, si, si_list);
return (si);