From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: From: Kalle Valo Subject: Re: [PATCH] wcn36xx: Fix logging macro with unnecessary semicolon References: <1383684000.4387.42.camel@joe-AO722> <1383760556.7940.28.camel@joe-AO722> <1383853221.9263.14.camel@joe-AO722> Date: Fri, 8 Nov 2013 09:06:19 +0200 In-Reply-To: <1383853221.9263.14.camel@joe-AO722> (Joe Perches's message of "Thu, 7 Nov 2013 11:40:21 -0800") Message-ID: <87bo1vcqz8.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Joe Perches Cc: "linux-kernel@vger.kernel.org" , wil6210@qca.qualcomm.com, netdev@vger.kernel.org, ath5k-devel@lists.ath5k.org, linux-wireless , "John W. Linville" , ath10k@lists.infradead.org, "Luis R. Rodriguez" , ath9k-devel , wcn36xx , Eugene Krasnikov Joe Perches writes: >> have a >> second round of convincing ath guys to change printing code. How does >> that sound? > > Luis? Kalle? Other Qualcomm/Atheros folk? I didn't quite get what you are asking from me. But for me most important is that the current debugging facilities from user's point of view don't change. I don't care if the code is in ath10k.ko or ath.ko, we are talking about ~100 lines of code anyway. > One of the nominal benefits of separating the ath_ > macros by subsystem was perf/tracing. Nominal? -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k