Many rcv functions from net/dsa/tag_*.c have a useless out_drop goto
label which simply returns NULL. Kill it in favor of the obvious.

Signed-off-by: Vivien Didelot <vivien.dide...@savoirfairelinux.com>
---
 net/dsa/tag_brcm.c    | 11 ++++-------
 net/dsa/tag_dsa.c     | 13 +++++--------
 net/dsa/tag_edsa.c    | 13 +++++--------
 net/dsa/tag_mtk.c     |  9 +++------
 net/dsa/tag_qca.c     | 11 ++++-------
 net/dsa/tag_trailer.c |  9 +++------
 6 files changed, 24 insertions(+), 42 deletions(-)

diff --git a/net/dsa/tag_brcm.c b/net/dsa/tag_brcm.c
index c8bff4d7543d..10fa4c0ca46b 100644
--- a/net/dsa/tag_brcm.c
+++ b/net/dsa/tag_brcm.c
@@ -102,27 +102,27 @@ static struct sk_buff *brcm_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
        ds = dst->cpu_dp->ds;
 
        if (unlikely(!pskb_may_pull(skb, BRCM_TAG_LEN)))
-               goto out_drop;
+               return NULL;
 
        /* skb->data points to the EtherType, the tag is right before it */
        brcm_tag = skb->data - 2;
 
        /* The opcode should never be different than 0b000 */
        if (unlikely((brcm_tag[0] >> BRCM_OPCODE_SHIFT) & BRCM_OPCODE_MASK))
-               goto out_drop;
+               return NULL;
 
        /* We should never see a reserved reason code without knowing how to
         * handle it
         */
        if (unlikely(brcm_tag[2] & BRCM_EG_RC_RSVD))
-               goto out_drop;
+               return NULL;
 
        /* Locate which port this is coming from */
        source_port = brcm_tag[3] & BRCM_EG_PID_MASK;
 
        /* Validate port against switch setup, either the port is totally */
        if (source_port >= ds->num_ports || !ds->ports[source_port].netdev)
-               goto out_drop;
+               return NULL;
 
        /* Remove Broadcom tag and update checksum */
        skb_pull_rcsum(skb, BRCM_TAG_LEN);
@@ -135,9 +135,6 @@ static struct sk_buff *brcm_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
        skb->dev = ds->ports[source_port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops brcm_netdev_ops = {
diff --git a/net/dsa/tag_dsa.c b/net/dsa/tag_dsa.c
index 41f78e42f9ba..9f5fecaa4a93 100644
--- a/net/dsa/tag_dsa.c
+++ b/net/dsa/tag_dsa.c
@@ -77,7 +77,7 @@ static struct sk_buff *dsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
        int source_port;
 
        if (unlikely(!pskb_may_pull(skb, DSA_HLEN)))
-               goto out_drop;
+               return NULL;
 
        /*
         * The ethertype field is part of the DSA header.
@@ -88,7 +88,7 @@ static struct sk_buff *dsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
         * Check that frame type is either TO_CPU or FORWARD.
         */
        if ((dsa_header[0] & 0xc0) != 0x00 && (dsa_header[0] & 0xc0) != 0xc0)
-               goto out_drop;
+               return NULL;
 
        /*
         * Determine source device and port.
@@ -101,14 +101,14 @@ static struct sk_buff *dsa_rcv(struct sk_buff *skb, 
struct net_device *dev)
         * port is a registered DSA port.
         */
        if (source_device >= DSA_MAX_SWITCHES)
-               goto out_drop;
+               return NULL;
 
        ds = dst->ds[source_device];
        if (!ds)
-               goto out_drop;
+               return NULL;
 
        if (source_port >= ds->num_ports || !ds->ports[source_port].netdev)
-               goto out_drop;
+               return NULL;
 
        /*
         * Convert the DSA header to an 802.1q header if the 'tagged'
@@ -159,9 +159,6 @@ static struct sk_buff *dsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
        skb->dev = ds->ports[source_port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops dsa_netdev_ops = {
diff --git a/net/dsa/tag_edsa.c b/net/dsa/tag_edsa.c
index 36389fc8fe06..a9a06e19abeb 100644
--- a/net/dsa/tag_edsa.c
+++ b/net/dsa/tag_edsa.c
@@ -90,7 +90,7 @@ static struct sk_buff *edsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
        int source_port;
 
        if (unlikely(!pskb_may_pull(skb, EDSA_HLEN)))
-               goto out_drop;
+               return NULL;
 
        /*
         * Skip the two null bytes after the ethertype.
@@ -101,7 +101,7 @@ static struct sk_buff *edsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
         * Check that frame type is either TO_CPU or FORWARD.
         */
        if ((edsa_header[0] & 0xc0) != 0x00 && (edsa_header[0] & 0xc0) != 0xc0)
-               goto out_drop;
+               return NULL;
 
        /*
         * Determine source device and port.
@@ -114,14 +114,14 @@ static struct sk_buff *edsa_rcv(struct sk_buff *skb, 
struct net_device *dev)
         * port is a registered DSA port.
         */
        if (source_device >= DSA_MAX_SWITCHES)
-               goto out_drop;
+               return NULL;
 
        ds = dst->ds[source_device];
        if (!ds)
-               goto out_drop;
+               return NULL;
 
        if (source_port >= ds->num_ports || !ds->ports[source_port].netdev)
-               goto out_drop;
+               return NULL;
 
        /*
         * If the 'tagged' bit is set, convert the DSA tag to a 802.1q
@@ -178,9 +178,6 @@ static struct sk_buff *edsa_rcv(struct sk_buff *skb, struct 
net_device *dev)
        skb->dev = ds->ports[source_port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops edsa_netdev_ops = {
diff --git a/net/dsa/tag_mtk.c b/net/dsa/tag_mtk.c
index 7c9764c8d61b..327dd7b596df 100644
--- a/net/dsa/tag_mtk.c
+++ b/net/dsa/tag_mtk.c
@@ -55,7 +55,7 @@ static struct sk_buff *mtk_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
        __be16 *phdr, hdr;
 
        if (unlikely(!pskb_may_pull(skb, MTK_HDR_LEN)))
-               goto out_drop;
+               return NULL;
 
        /* The MTK header is added by the switch between src addr
         * and ethertype at this point, skb->data points to 2 bytes
@@ -77,19 +77,16 @@ static struct sk_buff *mtk_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
         */
        ds = dst->ds[0];
        if (!ds)
-               goto out_drop;
+               return NULL;
 
        /* Get source port information */
        port = (hdr & MTK_HDR_RECV_SOURCE_PORT_MASK);
        if (!ds->ports[port].netdev)
-               goto out_drop;
+               return NULL;
 
        skb->dev = ds->ports[port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops mtk_netdev_ops = {
diff --git a/net/dsa/tag_qca.c b/net/dsa/tag_qca.c
index d38bd5cbfad8..8080ad8f2abb 100644
--- a/net/dsa/tag_qca.c
+++ b/net/dsa/tag_qca.c
@@ -75,7 +75,7 @@ static struct sk_buff *qca_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
        __be16 *phdr, hdr;
 
        if (unlikely(!pskb_may_pull(skb, QCA_HDR_LEN)))
-               goto out_drop;
+               return NULL;
 
        /* The QCA header is added by the switch between src addr and Ethertype
         * At this point, skb->data points to ethertype so header should be
@@ -87,7 +87,7 @@ static struct sk_buff *qca_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
        /* Make sure the version is correct */
        ver = (hdr & QCA_HDR_RECV_VERSION_MASK) >> QCA_HDR_RECV_VERSION_S;
        if (unlikely(ver != QCA_HDR_VERSION))
-               goto out_drop;
+               return NULL;
 
        /* Remove QCA tag and recalculate checksum */
        skb_pull_rcsum(skb, QCA_HDR_LEN);
@@ -99,20 +99,17 @@ static struct sk_buff *qca_tag_rcv(struct sk_buff *skb, 
struct net_device *dev)
         */
        ds = dst->cpu_dp->ds;
        if (!ds)
-               goto out_drop;
+               return NULL;
 
        /* Get source port information */
        port = (hdr & QCA_HDR_RECV_SOURCE_PORT_MASK);
        if (!ds->ports[port].netdev)
-               goto out_drop;
+               return NULL;
 
        /* Update skb & forward the frame accordingly */
        skb->dev = ds->ports[port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops qca_netdev_ops = {
diff --git a/net/dsa/tag_trailer.c b/net/dsa/tag_trailer.c
index e1fd78197fc8..7422b1329712 100644
--- a/net/dsa/tag_trailer.c
+++ b/net/dsa/tag_trailer.c
@@ -68,25 +68,22 @@ static struct sk_buff *trailer_rcv(struct sk_buff *skb, 
struct net_device *dev)
        ds = dst->cpu_dp->ds;
 
        if (skb_linearize(skb))
-               goto out_drop;
+               return NULL;
 
        trailer = skb_tail_pointer(skb) - 4;
        if (trailer[0] != 0x80 || (trailer[1] & 0xf8) != 0x00 ||
            (trailer[2] & 0xef) != 0x00 || trailer[3] != 0x00)
-               goto out_drop;
+               return NULL;
 
        source_port = trailer[1] & 7;
        if (source_port >= ds->num_ports || !ds->ports[source_port].netdev)
-               goto out_drop;
+               return NULL;
 
        pskb_trim_rcsum(skb, skb->len - 4);
 
        skb->dev = ds->ports[source_port].netdev;
 
        return skb;
-
-out_drop:
-       return NULL;
 }
 
 const struct dsa_device_ops trailer_netdev_ops = {
-- 
2.13.0

Reply via email to