net: dsa: simplify the return rtl8366_vlan_prepare()
authorZheng Yongjun <zhengyongjun3@huawei.com>
Wed, 9 Dec 2020 09:26:21 +0000 (17:26 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Dec 2020 01:05:37 +0000 (17:05 -0800)
Simplify the return expression.

Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/rtl8366.c

index 307466b..83d481e 100644 (file)
@@ -384,7 +384,6 @@ int rtl8366_vlan_prepare(struct dsa_switch *ds, int port,
 {
        struct realtek_smi *smi = ds->priv;
        u16 vid;
-       int ret;
 
        for (vid = vlan->vid_begin; vid < vlan->vid_end; vid++)
                if (!smi->ops->is_vlan_valid(smi, vid))
@@ -397,11 +396,7 @@ int rtl8366_vlan_prepare(struct dsa_switch *ds, int port,
         * FIXME: what's with this 4k business?
         * Just rtl8366_enable_vlan() seems inconclusive.
         */
-       ret = rtl8366_enable_vlan4k(smi, true);
-       if (ret)
-               return ret;
-
-       return 0;
+       return rtl8366_enable_vlan4k(smi, true);
 }
 EXPORT_SYMBOL_GPL(rtl8366_vlan_prepare);