Browse Source

qcacmn: qdf: Replace instances of unadorned %p

Replace instances of unadorned %p in qdf.

Change-Id: Idd264d99d9f275364a30d8b1c0b5f1a0501a7f0f
CRs-Fixed: 2111274
Jeff Johnson 7 years ago
parent
commit
f48993dd63
4 changed files with 10 additions and 10 deletions
  1. 1 1
      qdf/linux/src/i_osdep.h
  2. 1 1
      qdf/linux/src/qdf_lro.c
  3. 3 3
      qdf/linux/src/qdf_mem.c
  4. 5 5
      qdf/linux/src/qdf_nbuf.c

+ 1 - 1
qdf/linux/src/i_osdep.h

@@ -66,7 +66,7 @@ typedef struct timer_list os_timer_t;
 	do { \
 		if (!spin_is_locked(x)) { \
 			WARN_ON(1); \
-			printk(KERN_EMERG " %s:%d unlock addr=%p, %s \n", __func__,  __LINE__, x, \
+			printk(KERN_EMERG " %s:%d unlock addr=%pK, %s \n", __func__,  __LINE__, x, \
 			       !spin_is_locked(x) ? "Not locked" : "");	\
 		} \
 		spin_unlock_bh(x); \

+ 1 - 1
qdf/linux/src/qdf_lro.c

@@ -191,7 +191,7 @@ void qdf_lro_deinit(qdf_lro_ctx_t lro_ctx)
 {
 	if (likely(lro_ctx)) {
 		QDF_TRACE(QDF_MODULE_ID_QDF, QDF_TRACE_LEVEL_ERROR,
-			 "LRO instance %p is being freed", lro_ctx);
+			 "LRO instance %pK is being freed", lro_ctx);
 		qdf_mem_free(lro_ctx);
 	}
 }

+ 3 - 3
qdf/linux/src/qdf_mem.c

@@ -1458,7 +1458,7 @@ void qdf_mem_copy(void *dst_addr, const void *src_addr, uint32_t num_bytes)
 
 	if ((dst_addr == NULL) || (src_addr == NULL)) {
 		QDF_TRACE(QDF_MODULE_ID_QDF, QDF_TRACE_LEVEL_ERROR,
-			  "%s called with NULL parameter, source:%p destination:%p",
+			  "%s called with NULL parameter, source:%pK destination:%pK",
 			  __func__, src_addr, dst_addr);
 		QDF_ASSERT(0);
 		return;
@@ -1532,7 +1532,7 @@ void qdf_mem_move(void *dst_addr, const void *src_addr, uint32_t num_bytes)
 
 	if ((dst_addr == NULL) || (src_addr == NULL)) {
 		QDF_TRACE(QDF_MODULE_ID_QDF, QDF_TRACE_LEVEL_ERROR,
-			  "%s called with NULL parameter, source:%p destination:%p",
+			  "%s called with NULL parameter, source:%pK destination:%pK",
 			  __func__, src_addr, dst_addr);
 		QDF_ASSERT(0);
 		return;
@@ -1725,7 +1725,7 @@ void qdf_ether_addr_copy(void *dst_addr, const void *src_addr)
 {
 	if ((dst_addr == NULL) || (src_addr == NULL)) {
 		QDF_TRACE(QDF_MODULE_ID_QDF, QDF_TRACE_LEVEL_ERROR,
-			  "%s called with NULL parameter, source:%p destination:%p",
+			  "%s called with NULL parameter, source:%pK destination:%pK",
 			  __func__, src_addr, dst_addr);
 		QDF_ASSERT(0);
 		return;

+ 5 - 5
qdf/linux/src/qdf_nbuf.c

@@ -1575,7 +1575,7 @@ void qdf_net_buf_debug_add_node(qdf_nbuf_t net_buf, size_t size,
 	p_node = qdf_net_buf_debug_look_up(net_buf);
 
 	if (p_node) {
-		qdf_print("Double allocation of skb ! Already allocated from %p %s %d current alloc from %p %s %d",
+		qdf_print("Double allocation of skb ! Already allocated from %pK %s %d current alloc from %pK %s %d",
 			  p_node->net_buf, p_node->file_name, p_node->line_num,
 			  net_buf, file_name, line_num);
 		qdf_nbuf_track_free(new_node);
@@ -1645,7 +1645,7 @@ done:
 	spin_unlock_irqrestore(&g_qdf_net_buf_track_lock[i], irq_flag);
 
 	if (!found) {
-		qdf_print("Unallocated buffer ! Double free of net_buf %p ?",
+		qdf_print("Unallocated buffer ! Double free of net_buf %pK ?",
 			  net_buf);
 		QDF_ASSERT(0);
 	} else {
@@ -1893,7 +1893,7 @@ static inline void __qdf_nbuf_fill_tso_cmn_seg_info(
 	curr_seg->seg.total_len = curr_seg->seg.tso_frags[0].length;
 	curr_seg->seg.tso_frags[0].paddr = tso_cmn_info->eit_hdr_dma_map_addr;
 
-	TSO_DEBUG("%s %d eit hdr %p eit_hdr_len %d tcp_seq_num %u tso_info->total_len %u\n",
+	TSO_DEBUG("%s %d eit hdr %pK eit_hdr_len %d tcp_seq_num %u tso_info->total_len %u\n",
 		   __func__, __LINE__, tso_cmn_info->eit_hdr,
 		   tso_cmn_info->eit_hdr_len,
 		   curr_seg->seg.tso_flags.tcp_seq_num,
@@ -2005,7 +2005,7 @@ uint32_t __qdf_nbuf_get_tso_info(qdf_device_t osdev, struct sk_buff *skb,
 					tso_frag_paddr;
 			}
 
-			TSO_DEBUG("%s[%d] frag %d frag len %d total_len %u vaddr %p\n",
+			TSO_DEBUG("%s[%d] frag %d frag len %d total_len %u vaddr %pK\n",
 					__func__, __LINE__,
 					i,
 					tso_frag_len,
@@ -2193,7 +2193,7 @@ uint32_t __qdf_nbuf_get_tso_num_seg(struct sk_buff *skb)
 	 * region which cannot be accessed by Target
 	 */
 	if (virt_to_phys(skb->data) < 0x50000040) {
-		TSO_DEBUG("%s %d: Invalid Address nr_frags = %d, paddr = %p \n",
+		TSO_DEBUG("%s %d: Invalid Address nr_frags = %d, paddr = %pK \n",
 				__func__, __LINE__, skb_shinfo(skb)->nr_frags,
 				virt_to_phys(skb->data));
 		goto fail;