{
        struct ath10k *ar = file->private_data;
        char *buf;
-       unsigned int len = 0, buf_len = 4096;
+       size_t len = 0, buf_len = 4096;
        const char *name;
        ssize_t ret_cnt;
        bool enabled;
                                    size_t count, loff_t *ppos)
 {
        const char *buf = file->private_data;
-       unsigned int len = strlen(buf);
+       size_t len = strlen(buf);
 
        return simple_read_from_buffer(user_buf, count, ppos, buf, len);
 }
                                                size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       int ret, len, buf_len;
+       int ret;
+       size_t len = 0, buf_len = 500;
        char *buf;
 
-       buf_len = 500;
        buf = kmalloc(buf_len, GFP_KERNEL);
        if (!buf)
                return -ENOMEM;
 
        spin_lock_bh(&ar->data_lock);
 
-       len = 0;
        len += scnprintf(buf + len, buf_len - len,
                         "fw_crash_counter\t\t%d\n", ar->stats.fw_crash_counter);
        len += scnprintf(buf + len, buf_len - len,
                                   size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       unsigned int len;
+       size_t len;
        char buf[50];
 
        len = scnprintf(buf, sizeof(buf), "0x%08x\n", ar->chip_id);
        struct ath10k_ce_crash_hdr *ce_hdr;
        struct ath10k_dump_file_data *dump_data;
        struct ath10k_tlv_dump_data *dump_tlv;
-       int hdr_len = sizeof(*dump_data);
-       unsigned int len, sofar = 0;
+       size_t hdr_len = sizeof(*dump_data);
+       size_t len, sofar = 0;
        unsigned char *buf;
 
        len = hdr_len;
 {
        struct ath10k *ar = file->private_data;
        u8 buf[32];
-       unsigned int len = 0;
+       size_t len = 0;
        u32 reg_addr;
 
        mutex_lock(&ar->conf_mutex);
 {
        struct ath10k *ar = file->private_data;
        u8 buf[48];
-       unsigned int len;
+       size_t len;
        u32 reg_addr, reg_val;
        int ret;
 
 {
        struct ath10k *ar = file->private_data;
        char buf[32];
-       unsigned int len;
+       size_t len;
 
        len = scnprintf(buf, sizeof(buf), "%lu\n", ar->debug.htt_stats_mask);
 
        struct ath10k *ar = file->private_data;
        char buf[64];
        u8 amsdu, ampdu;
-       unsigned int len;
+       size_t len;
 
        mutex_lock(&ar->conf_mutex);
 
                                     size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       unsigned int len;
+       size_t len;
        char buf[96];
 
        len = scnprintf(buf, sizeof(buf), "0x%16llx %u\n",
                                      size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       int len = 0;
+       size_t len;
        char buf[32];
 
-       len = scnprintf(buf, sizeof(buf) - len, "%d\n",
-                       ar->ani_enabled);
+       len = scnprintf(buf, sizeof(buf), "%d\n", ar->ani_enabled);
 
        return simple_read_from_buffer(user_buf, count, ppos, buf, len);
 }
                                         size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       unsigned int len;
+       size_t len;
        char buf[32];
 
-       len = scnprintf(buf, sizeof(buf), "%d\n",
-                       ar->debug.nf_cal_period);
+       len = scnprintf(buf, sizeof(buf), "%d\n", ar->debug.nf_cal_period);
 
        return simple_read_from_buffer(user_buf, count, ppos, buf, len);
 }
 }
 
 static void ath10k_tpc_stats_print(struct ath10k_tpc_stats *tpc_stats,
-                                  unsigned int j, char *buf, unsigned int *len)
+                                  unsigned int j, char *buf, size_t *len)
 {
-       unsigned int i, buf_len;
+       int i;
+       size_t buf_len;
        static const char table_str[][5] = { "CDD",
                                             "STBC",
                                             "TXBF" };
                                  struct ath10k_tpc_stats *tpc_stats,
                                  char *buf)
 {
-       unsigned int len, j, buf_len;
+       int j;
+       size_t len, buf_len;
 
        len = 0;
        buf_len = ATH10K_TPC_CONFIG_BUF_SIZE;
                                     size_t count, loff_t *ppos)
 {
        const char *buf = file->private_data;
-       unsigned int len = strlen(buf);
+       size_t len = strlen(buf);
 
        return simple_read_from_buffer(user_buf, count, ppos, buf, len);
 }
                                              size_t count, loff_t *ppos)
 {
        struct ath10k *ar = file->private_data;
-       unsigned int len = 0, buf_len = 4096;
+       size_t len = 0, buf_len = 4096;
        ssize_t ret_cnt;
        char *buf;
 
                     const void *buf, size_t len)
 {
        char linebuf[256];
-       unsigned int linebuflen;
+       size_t linebuflen;
        const void *ptr;
 
        if (ath10k_debug_mask & mask) {