diff --git a/qdf/inc/qdf_trace.h b/qdf/inc/qdf_trace.h
index 88066b49e0..f1eed699a2 100644
--- a/qdf/inc/qdf_trace.h
+++ b/qdf/inc/qdf_trace.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014-2018 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2014-2019 The Linux Foundation. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -632,8 +632,18 @@ void qdf_dp_set_no_of_record(uint32_t val);
 #define QDF_DP_TRACE_RECORD_INFO_LIVE (0x1)
 #define QDF_DP_TRACE_RECORD_INFO_THROTTLED (0x1 << 1)
 
-bool qdf_dp_trace_log_pkt(uint8_t session_id, struct sk_buff *skb,
-				enum qdf_proto_dir dir, uint8_t pdev_id);
+/**
+ * qdf_dp_trace_log_pkt() - log packet type enabled through iwpriv
+ * @vdev_id: vdev_id
+ * @skb: skb pointer
+ * @dir: direction
+ * @pdev_id: pdev_id
+ *
+ * Return: true: some protocol was logged, false: no protocol was logged.
+ */
+bool qdf_dp_trace_log_pkt(uint8_t vdev_id, struct sk_buff *skb,
+			  enum qdf_proto_dir dir, uint8_t pdev_id);
+
 void qdf_dp_trace_init(bool live_mode_config, uint8_t thresh,
 				uint16_t time_limit, uint8_t verbosity,
 				uint8_t proto_bitmap);
@@ -835,8 +845,8 @@ void qdf_dp_trace_record_event(enum QDF_DP_TRACE_ID code, uint8_t vdev_id,
 			       enum qdf_proto_subtype subtype);
 #else
 static inline
-bool qdf_dp_trace_log_pkt(uint8_t session_id, struct sk_buff *skb,
-				enum qdf_proto_dir dir, uint8_t pdev_id)
+bool qdf_dp_trace_log_pkt(uint8_t vdev_id, struct sk_buff *skb,
+			  enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	return false;
 }
diff --git a/qdf/linux/src/qdf_trace.c b/qdf/linux/src/qdf_trace.c
index be68d9b6c9..ca56dd8385 100644
--- a/qdf/linux/src/qdf_trace.c
+++ b/qdf/linux/src/qdf_trace.c
@@ -1550,14 +1550,15 @@ static void qdf_dp_add_record(enum QDF_DP_TRACE_ID code, uint8_t pdev_id,
 
 /**
  * qdf_log_icmpv6_pkt() - log ICMPv6 packet
- * @session_id: vdev_id
+ * @vdev_id: ID of the vdev
  * @skb: skb pointer
  * @dir: direction
+ * @pdev_id: ID of the pdev
  *
  * Return: true/false
  */
-static bool qdf_log_icmpv6_pkt(uint8_t session_id, struct sk_buff *skb,
-			    enum qdf_proto_dir dir, uint8_t pdev_id)
+static bool qdf_log_icmpv6_pkt(uint8_t vdev_id, struct sk_buff *skb,
+			       enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	enum qdf_proto_subtype subtype;
 
@@ -1576,7 +1577,7 @@ static bool qdf_log_icmpv6_pkt(uint8_t session_id, struct sk_buff *skb,
 
 		DPTRACE(qdf_dp_trace_proto_pkt(
 			QDF_DP_TRACE_ICMPv6_PACKET_RECORD,
-			session_id, (skb->data + QDF_NBUF_SRC_MAC_OFFSET),
+			vdev_id, (skb->data + QDF_NBUF_SRC_MAC_OFFSET),
 			(skb->data + QDF_NBUF_DEST_MAC_OFFSET),
 			QDF_PROTO_TYPE_ICMPv6, subtype, dir, pdev_id, false));
 
@@ -1610,14 +1611,15 @@ static bool qdf_log_icmpv6_pkt(uint8_t session_id, struct sk_buff *skb,
 
 /**
  * qdf_log_icmp_pkt() - log ICMP packet
- * @session_id: vdev_id
+ * @vdev_id: ID of the vdev
  * @skb: skb pointer
  * @dir: direction
+ * @pdev_id: ID of the pdev
  *
  * Return: true/false
  */
-static bool qdf_log_icmp_pkt(uint8_t session_id, struct sk_buff *skb,
-			    enum qdf_proto_dir dir, uint8_t pdev_id)
+static bool qdf_log_icmp_pkt(uint8_t vdev_id, struct sk_buff *skb,
+			     enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	enum qdf_proto_subtype proto_subtype;
 
@@ -1633,7 +1635,7 @@ static bool qdf_log_icmp_pkt(uint8_t session_id, struct sk_buff *skb,
 			QDF_NBUF_CB_RX_DP_TRACE(skb) = 1;
 
 		DPTRACE(qdf_dp_trace_proto_pkt(QDF_DP_TRACE_ICMP_PACKET_RECORD,
-					       session_id,
+					       vdev_id,
 					       skb->data +
 					       QDF_NBUF_SRC_MAC_OFFSET,
 					       skb->data +
@@ -1654,14 +1656,14 @@ static bool qdf_log_icmp_pkt(uint8_t session_id, struct sk_buff *skb,
 
 /**
  * qdf_log_eapol_pkt() - log EAPOL packet
- * @session_id: vdev_id
+ * @vdev_id: ID of the vdev
  * @skb: skb pointer
  * @dir: direction
- * @pdev_id: pdev_id
+ * @pdev_id: ID of the pdev
  *
  * Return: true/false
  */
-static bool qdf_log_eapol_pkt(uint8_t session_id, struct sk_buff *skb,
+static bool qdf_log_eapol_pkt(uint8_t vdev_id, struct sk_buff *skb,
 			      enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	enum qdf_proto_subtype subtype;
@@ -1679,7 +1681,7 @@ static bool qdf_log_eapol_pkt(uint8_t session_id, struct sk_buff *skb,
 			QDF_NBUF_CB_RX_DP_TRACE(skb) = 1;
 
 		DPTRACE(qdf_dp_trace_proto_pkt(QDF_DP_TRACE_EAPOL_PACKET_RECORD,
-					       session_id,
+					       vdev_id,
 					       skb->data +
 					       QDF_NBUF_SRC_MAC_OFFSET,
 					       skb->data +
@@ -1711,14 +1713,14 @@ static bool qdf_log_eapol_pkt(uint8_t session_id, struct sk_buff *skb,
 
 /**
  * qdf_log_dhcp_pkt() - log DHCP packet
- * @session_id: vdev_id
+ * @vdev_id: ID of the vdev
  * @skb: skb pointer
  * @dir: direction
- * @pdev_id: pdev_id
+ * @pdev_id: ID of the pdev
  *
  * Return: true/false
  */
-static bool qdf_log_dhcp_pkt(uint8_t session_id, struct sk_buff *skb,
+static bool qdf_log_dhcp_pkt(uint8_t vdev_id, struct sk_buff *skb,
 			     enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	enum qdf_proto_subtype subtype = QDF_PROTO_INVALID;
@@ -1736,7 +1738,7 @@ static bool qdf_log_dhcp_pkt(uint8_t session_id, struct sk_buff *skb,
 			QDF_NBUF_CB_RX_DP_TRACE(skb) = 1;
 
 		DPTRACE(qdf_dp_trace_proto_pkt(QDF_DP_TRACE_DHCP_PACKET_RECORD,
-					       session_id,
+					       vdev_id,
 					       skb->data +
 					       QDF_NBUF_SRC_MAC_OFFSET,
 					       skb->data +
@@ -1772,14 +1774,14 @@ static bool qdf_log_dhcp_pkt(uint8_t session_id, struct sk_buff *skb,
 
 /**
  * qdf_log_arp_pkt() - log ARP packet
- * @session_id: vdev_id
+ * @vdev_id: ID of the vdev
  * @skb: skb pointer
  * @dir: direction
- * @pdev_id: pdev_id
+ * @pdev_id: ID of the pdev
  *
  * Return: true/false
  */
-static bool qdf_log_arp_pkt(uint8_t session_id, struct sk_buff *skb,
+static bool qdf_log_arp_pkt(uint8_t vdev_id, struct sk_buff *skb,
 			    enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	enum qdf_proto_subtype proto_subtype;
@@ -1797,7 +1799,7 @@ static bool qdf_log_arp_pkt(uint8_t session_id, struct sk_buff *skb,
 			QDF_NBUF_CB_RX_DP_TRACE(skb) = 1;
 
 		DPTRACE(qdf_dp_trace_proto_pkt(QDF_DP_TRACE_ARP_PACKET_RECORD,
-					       session_id,
+					       vdev_id,
 					       skb->data +
 					       QDF_NBUF_SRC_MAC_OFFSET,
 					       skb->data +
@@ -1817,29 +1819,20 @@ static bool qdf_log_arp_pkt(uint8_t session_id, struct sk_buff *skb,
 }
 
 
-/**
- * qdf_dp_trace_log_pkt() - log packet type enabled through iwpriv
- * @session_id: vdev_id
- * @skb: skb pointer
- * @dir: direction
- * @pdev_id: pdev_id
- *
- * Return: true: some protocol was logged, false: no protocol was logged.
- */
-bool qdf_dp_trace_log_pkt(uint8_t session_id, struct sk_buff *skb,
+bool qdf_dp_trace_log_pkt(uint8_t vdev_id, struct sk_buff *skb,
 			  enum qdf_proto_dir dir, uint8_t pdev_id)
 {
 	if (!qdf_dp_get_proto_bitmap())
 		return false;
-	if (qdf_log_arp_pkt(session_id, skb, dir, pdev_id))
+	if (qdf_log_arp_pkt(vdev_id, skb, dir, pdev_id))
 		return true;
-	if (qdf_log_dhcp_pkt(session_id, skb, dir, pdev_id))
+	if (qdf_log_dhcp_pkt(vdev_id, skb, dir, pdev_id))
 		return true;
-	if (qdf_log_eapol_pkt(session_id, skb, dir, pdev_id))
+	if (qdf_log_eapol_pkt(vdev_id, skb, dir, pdev_id))
 		return true;
-	if (qdf_log_icmp_pkt(session_id, skb, dir, pdev_id))
+	if (qdf_log_icmp_pkt(vdev_id, skb, dir, pdev_id))
 		return true;
-	if (qdf_log_icmpv6_pkt(session_id, skb, dir, pdev_id))
+	if (qdf_log_icmpv6_pkt(vdev_id, skb, dir, pdev_id))
 		return true;
 	return false;
 }