All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Drivers:RTLWIFI:rtl8723ae: add the function name to the printouts
@ 2017-12-12 13:34 venkat.prashanth2498
  2018-01-08 16:01 ` Kalle Valo
  0 siblings, 1 reply; 2+ messages in thread
From: venkat.prashanth2498 @ 2017-12-12 13:34 UTC (permalink / raw)
  To: Larry.Finger, chaoming_li; +Cc: linux-wireless, Venkat Prashanth B U

From: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>

This is a patch to rtl8723ae/trx.c file that adds the
function name to the printouts.

Signed-off-by: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>
---
 drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c
index 2f7c144..96c5385 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/trx.c
@@ -389,7 +389,7 @@ void rtl8723e_tx_fill_desc(struct ieee80211_hw *hw,

 	if (pci_dma_mapping_error(rtlpci->pdev, mapping)) {
 		RT_TRACE(rtlpriv, COMP_SEND, DBG_TRACE,
-			 "DMA mapping error");
+			 "%s():DMA mapping error", __func__);
 		return;
 	}
 	if (mac->opmode == NL80211_IFTYPE_STATION) {
@@ -557,7 +557,7 @@ void rtl8723e_tx_fill_cmddesc(struct ieee80211_hw *hw,

 	if (pci_dma_mapping_error(rtlpci->pdev, mapping)) {
 		RT_TRACE(rtlpriv, COMP_SEND, DBG_TRACE,
-			 "DMA mapping error");
+			 "%s():DMA mapping error", __func__);
 		return;
 	}
 	CLEAR_PCI_TX_DESC_CONTENT(pdesc, TX_DESC_SIZE);
--
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: Drivers:RTLWIFI:rtl8723ae: add the function name to the printouts
  2017-12-12 13:34 [PATCH] Drivers:RTLWIFI:rtl8723ae: add the function name to the printouts venkat.prashanth2498
@ 2018-01-08 16:01 ` Kalle Valo
  0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2018-01-08 16:01 UTC (permalink / raw)
  To: venkat.prashanth2498
  Cc: Larry.Finger, chaoming_li, linux-wireless, Venkat Prashanth B U

venkat.prashanth2498@gmail.com wrote:

> From: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>
> 
> This is a patch to rtl8723ae/trx.c file that adds the
> function name to the printouts.
> 
> Signed-off-by: Venkat Prashanth B U <venkat.prashanth2498@gmail.com>

The title is wrong:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#commit_title_is_wrong

Also I would like to see an ack from Larry or other active developers before I
take something cosmetic like this.

-- 
https://patchwork.kernel.org/patch/10107381/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-01-08 16:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-12 13:34 [PATCH] Drivers:RTLWIFI:rtl8723ae: add the function name to the printouts venkat.prashanth2498
2018-01-08 16:01 ` Kalle Valo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.