#include <stdio.h>
#include "netifd.h"
+#include "device.h"
+#include "interface.h"
#include "proto.h"
#include "ubus.h"
/* TODO */
}
+static void
+mark_interface_down(struct interface *iface)
+{
+ release_device(iface->main_dev.dev);
+ iface->state = IFS_DOWN;
+}
+
static int
__set_interface_up(struct interface *iface)
{
ret = claim_device(iface->main_dev.dev);
if (ret)
- goto out;
+ return ret;
iface->state = IFS_SETUP;
ret = iface->proto->handler(iface->proto, PROTO_CMD_SETUP, false);
- if (ret)
- goto release;
+ if (ret) {
+ mark_interface_down(iface);
+ return ret;
+ }
return 0;
-release:
- release_device(iface->main_dev.dev);
-out:
- iface->state = IFS_DOWN;
- return ret;
}
static void
interface_event(iface, IFEV_UP);
break;
case IFPEV_DOWN:
- iface->state = IFS_DOWN;
+ if (iface->state == IFS_DOWN)
+ return;
+
+ mark_interface_down(iface);
break;
}
}
}
void
-interface_remove_link(struct interface *iface, struct device *llif)
+interface_remove_link(struct interface *iface, struct device *dev)
{
- struct device *dev = iface->main_dev.dev;
+ struct device *mdev = iface->main_dev.dev;
- if (dev && dev->hotplug_ops) {
- dev->hotplug_ops->del(dev, llif);
+ if (mdev && mdev->hotplug_ops) {
+ mdev->hotplug_ops->del(mdev, dev);
return;
}
}
int
-interface_add_link(struct interface *iface, struct device *llif)
+interface_add_link(struct interface *iface, struct device *dev)
{
- struct device *dev = iface->main_dev.dev;
+ struct device *mdev = iface->main_dev.dev;
- if (dev && dev->hotplug_ops)
- return dev->hotplug_ops->add(dev, llif);
+ if (mdev && mdev->hotplug_ops)
+ return mdev->hotplug_ops->add(mdev, dev);
if (iface->main_dev.dev)
interface_remove_link(iface, NULL);
- add_device_user(&iface->main_dev, llif);
+ add_device_user(&iface->main_dev, dev);
return 0;
}