[PATCH] small cleanup udev-remove.c
Here is a small cleanup. It replaces the malloc in udev-remove.c with a struct, like we do in udev-add.c
This commit is contained in:
parent
ca593541e6
commit
808423c90c
|
@ -118,30 +118,26 @@ static int delete_node(struct udevice *dev)
|
||||||
*/
|
*/
|
||||||
int udev_remove_device(char *path, char *subsystem)
|
int udev_remove_device(char *path, char *subsystem)
|
||||||
{
|
{
|
||||||
struct udevice *dev;
|
struct udevice dev;
|
||||||
char *temp;
|
char *temp;
|
||||||
int retval;
|
int retval;
|
||||||
|
|
||||||
dev = malloc(sizeof(*dev));
|
memset(&dev, 0, sizeof(dev));
|
||||||
if (dev == NULL)
|
|
||||||
return -ENOMEM;
|
|
||||||
memset(dev, 0, sizeof(*dev));
|
|
||||||
|
|
||||||
retval = udevdb_get_dev(path, dev);
|
retval = udevdb_get_dev(path, &dev);
|
||||||
if (retval) {
|
if (retval) {
|
||||||
dbg("'%s' not found in database, falling back on default name", path);
|
dbg("'%s' not found in database, falling back on default name", path);
|
||||||
temp = strrchr(path, '/');
|
temp = strrchr(path, '/');
|
||||||
if (temp == NULL)
|
if (temp == NULL)
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
strncpy(dev->name, &temp[1], sizeof(dev->name));
|
strncpy(dev.name, &temp[1], sizeof(dev.name));
|
||||||
}
|
}
|
||||||
|
|
||||||
dbg("name is '%s'", dev->name);
|
dbg("name is '%s'", dev.name);
|
||||||
udevdb_delete_dev(path);
|
udevdb_delete_dev(path);
|
||||||
|
|
||||||
sysbus_send_remove(dev->name, path);
|
sysbus_send_remove(dev.name, path);
|
||||||
|
|
||||||
retval = delete_node(dev);
|
retval = delete_node(&dev);
|
||||||
free(dev);
|
|
||||||
return retval;
|
return retval;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue