mt76x0: remove mt76x0_vendor_single_wr routine
authorLorenzo Bianconi <lorenzo.bianconi@redhat.com>
Sat, 25 Aug 2018 10:40:47 +0000 (12:40 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 4 Sep 2018 08:08:59 +0000 (11:08 +0300)
Remove mt76x0_vendor_single_wr routine and use mt76u_single_wr
utility function for mcu firmware loading

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
drivers/net/wireless/mediatek/mt76/mt76x0/usb.c
drivers/net/wireless/mediatek/mt76/mt76x0/usb.h

index c8ea3126e7dc176f349160cdceacb0d15ad49b07..cf3acee12dc3cf35d4d08cdbcdc73d6d9dd96d46 100644 (file)
@@ -416,15 +416,12 @@ static int __mt76x0_dma_fw(struct mt76x0_dev *dev,
        memcpy(buf.buf + sizeof(reg), data, len);
        memset(buf.buf + sizeof(reg) + len, 0, 8);
 
-       ret = mt76x0_vendor_single_wr(dev, MT_VEND_WRITE_FCE,
-                                      MT_FCE_DMA_ADDR, dst_addr);
-       if (ret)
-               return ret;
+       mt76u_single_wr(&dev->mt76, MT_VEND_WRITE_FCE, MT_FCE_DMA_ADDR,
+                       dst_addr);
+
        len = roundup(len, 4);
-       ret = mt76x0_vendor_single_wr(dev, MT_VEND_WRITE_FCE,
-                                      MT_FCE_DMA_LEN, len << 16);
-       if (ret)
-               return ret;
+       mt76u_single_wr(&dev->mt76, MT_VEND_WRITE_FCE, MT_FCE_DMA_LEN,
+                       len << 16);
 
        buf.len = MT_DMA_HDR_LEN + len + 4;
        ret = mt76x0_usb_submit_buf(dev, USB_DIR_OUT, MT_EP_OUT_INBAND_CMD,
index a0052ef3626673343fabdfac5fa3e29398c3faaf..a6cb0c3ce5a678fb790ef17acce2629e9e4c958a 100644 (file)
@@ -131,25 +131,6 @@ int mt76x0_vendor_request(struct mt76x0_dev *dev, const u8 req,
        return ret;
 }
 
-int mt76x0_vendor_single_wr(struct mt76x0_dev *dev, const u8 req,
-                            const u16 offset, const u32 val)
-{
-       struct mt76x0_dev *mdev = dev;
-       int ret;
-
-       mutex_lock(&mdev->usb_ctrl_mtx);
-
-       ret = mt76x0_vendor_request(dev, req, USB_DIR_OUT,
-                                   val & 0xffff, offset, NULL, 0);
-       if (!ret)
-               ret = mt76x0_vendor_request(dev, req, USB_DIR_OUT,
-                                           val >> 16, offset + 2, NULL, 0);
-
-       mutex_unlock(&mdev->usb_ctrl_mtx);
-
-       return ret;
-}
-
 void mt76x0_addr_wr(struct mt76x0_dev *dev, const u32 offset, const u8 *addr)
 {
        mt76_wr(dev, offset, get_unaligned_le32(addr));
index d23ca45a294eea2b6503a87537a96897d7ff2450..931c36554ff4a75bdca4cfb6f2456ce3b6c32ded 100644 (file)
@@ -54,7 +54,5 @@ void mt76x0_complete_urb(struct urb *urb);
 int mt76x0_vendor_request(struct mt76x0_dev *dev, const u8 req,
                           const u8 direction, const u16 val, const u16 offset,
                           void *buf, const size_t buflen);
-int mt76x0_vendor_single_wr(struct mt76x0_dev *dev, const u8 req,
-                            const u16 offset, const u32 val);
 
 #endif