netifd: propagate error code on netifd_reload()

[project/netifd.git] / vlandev.c

diff --git a/vlandev.c b/vlandev.c
index b93527c..7e46251 100644 (file)
--- a/vlandev.c
+++ b/vlandev.c
@@ -20,14 +20,12 @@
 #include "system.h"
 
 enum {
-       VLANDEV_ATTR_TYPE,
        VLANDEV_ATTR_IFNAME,
        VLANDEV_ATTR_VID,
        __VLANDEV_ATTR_MAX
 };
 
 static const struct blobmsg_policy vlandev_attrs[__VLANDEV_ATTR_MAX] = {
-       [VLANDEV_ATTR_TYPE] = { "type", BLOBMSG_TYPE_STRING },
        [VLANDEV_ATTR_IFNAME] = { "ifname", BLOBMSG_TYPE_STRING },
        [VLANDEV_ATTR_VID] = { "vid", BLOBMSG_TYPE_INT32 },
 };
@@ -40,6 +38,8 @@ static const struct uci_blob_param_list vlandev_attr_list = {
        .next = { &device_attr_list },
 };
 
+static struct device_type vlan8021q_device_type;
+
 struct vlandev_device {
        struct device dev;
        struct device_user parent;
@@ -158,15 +158,10 @@ vlandev_apply_settings(struct vlandev_device *mvdev, struct blob_attr **tb)
        struct vlandev_config *cfg = &mvdev->config;
        struct blob_attr *cur;
 
-       cfg->proto = VLAN_PROTO_8021Q;
+       cfg->proto = (mvdev->dev.type == &vlan8021q_device_type) ?
+               VLAN_PROTO_8021Q : VLAN_PROTO_8021AD;
        cfg->vid = 1;
 
-       if ((cur = tb[VLANDEV_ATTR_TYPE]))
-       {
-               if(!strcmp(blobmsg_data(cur), "8021ad"))
-                       cfg->proto = VLAN_PROTO_8021AD;
-       }
-
        if ((cur = tb[VLANDEV_ATTR_VID]))
                cfg->vid = (uint16_t) blobmsg_get_u32(cur);
 }
@@ -216,7 +211,8 @@ vlandev_reload(struct device *dev, struct blob_attr *attr)
 }
 
 static struct device *
-vlandev_create(const char *name, struct blob_attr *attr)
+vlandev_create(const char *name, struct device_type *devtype,
+              struct blob_attr *attr)
 {
        struct vlandev_device *mvdev;
        struct device *dev = NULL;
@@ -226,7 +222,7 @@ vlandev_create(const char *name, struct blob_attr *attr)
                return NULL;
 
        dev = &mvdev->dev;
-       device_init(dev, &vlandev_device_type, name);
+       device_init(dev, devtype, name);
        dev->config_pending = true;
 
        mvdev->set_state = dev->set_state;
@@ -240,8 +236,18 @@ vlandev_create(const char *name, struct blob_attr *attr)
        return dev;
 }
 
-const struct device_type vlandev_device_type = {
-       .name = "VLANDEV",
+static struct device_type vlan8021ad_device_type = {
+       .name = "8021ad",
+       .config_params = &vlandev_attr_list,
+       .create = vlandev_create,
+       .config_init = vlandev_config_init,
+       .reload = vlandev_reload,
+       .free = vlandev_free,
+       .dump_info = vlandev_dump_info,
+};
+
+static struct device_type vlan8021q_device_type = {
+       .name = "8021q",
        .config_params = &vlandev_attr_list,
        .create = vlandev_create,
        .config_init = vlandev_config_init,
@@ -249,3 +255,9 @@ const struct device_type vlandev_device_type = {
        .free = vlandev_free,
        .dump_info = vlandev_dump_info,
 };
+
+static void __init vlandev_device_type_init(void)
+{
+       device_type_add(&vlan8021ad_device_type);
+       device_type_add(&vlan8021q_device_type);
+}