mt76x0: remove mt76x0_mcu_msg_alloc routine
authorLorenzo Bianconi <lorenzo.bianconi@redhat.com>
Sat, 25 Aug 2018 10:40:48 +0000 (12:40 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 4 Sep 2018 08:09:03 +0000 (11:09 +0300)
Remove mt76x0_mcu_msg_alloc duplicated routine and use mt76u_mcu_msg_alloc
utility function for usb mcu message allocation

Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/mediatek/mt76/mt76x0/mcu.c

index cf3acee..8a258fa 100644 (file)
@@ -59,21 +59,6 @@ static inline void trace_mt76x0_mcu_msg_send_cs(struct mt76_dev *dev,
        trace_mt76x0_mcu_msg_send(dev, skb, csum, need_resp);
 }
 
-static struct sk_buff *
-mt76x0_mcu_msg_alloc(struct mt76x0_dev *dev, const void *data, int len)
-{
-       struct sk_buff *skb;
-
-       WARN_ON(len % 4); /* if length is not divisible by 4 we need to pad */
-
-       skb = alloc_skb(len + MT_DMA_HDR_LEN + 4, GFP_KERNEL);
-       if (skb) {
-               skb_reserve(skb, MT_DMA_HDR_LEN);
-               memcpy(skb_put(skb, len), data, len);
-       }
-       return skb;
-}
-
 static void mt76x0_read_resp_regs(struct mt76x0_dev *dev, int len)
 {
        int i;
@@ -217,7 +202,7 @@ int mt76x0_mcu_function_select(struct mt76x0_dev *dev,
                .value = cpu_to_le32(val),
        };
 
-       skb = mt76x0_mcu_msg_alloc(dev, &msg, sizeof(msg));
+       skb = mt76u_mcu_msg_alloc(&msg, sizeof(msg));
        if (!skb)
                return -ENOMEM;
        return mt76x0_mcu_msg_send(dev, skb, CMD_FUN_SET_OP, func == 5);
@@ -235,7 +220,7 @@ mt76x0_mcu_calibrate(struct mt76x0_dev *dev, enum mcu_calibrate cal, u32 val)
                .value = cpu_to_le32(val),
        };
 
-       skb = mt76x0_mcu_msg_alloc(dev, &msg, sizeof(msg));
+       skb = mt76u_mcu_msg_alloc(&msg, sizeof(msg));
        if (!skb)
                return -ENOMEM;
        return mt76x0_mcu_msg_send(dev, skb, CMD_CALIBRATION_OP, true);