Return-path: Received: from perches-mx.perches.com ([206.117.179.246]:36394 "EHLO labridge.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756090Ab1LASsj (ORCPT ); Thu, 1 Dec 2011 13:48:39 -0500 Message-ID: <1322765318.30801.9.camel@joe2Laptop> (sfid-20111201_194857_517512_8D3EC29D) Subject: Re: [PATCH 1/4] nfc: Use standard logging styles From: Joe Perches To: Samuel Ortiz Cc: Lauro Ramos Venancio , Aloisio Almeida Jr , "David S. Miller" , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 01 Dec 2011 10:48:38 -0800 In-Reply-To: <20111201155951.GF22609@sortiz-mobl> References: <6c89a76ddb76926febe36fe5be514cb186e39a76.1322595343.git.joe@perches.com> <20111201155951.GF22609@sortiz-mobl> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, 2011-12-01 at 16:59 +0100, Samuel Ortiz wrote: > Hi Joe, Hello Samuel. > On Tue, Nov 29, 2011 at 11:37:32AM -0800, Joe Perches wrote: > > Using the normal logging styles is preferred over > > subsystem specific styles when the subsystem does > > not take a specific struct. > > Convert nfc_ specific messages to pr_ > > Add newlines to uses. > I would be fine with moving to pr_debug, but I'd like the pr_fmt() routine to > include the function name as well. If you really want it I can add it back but if you are not aware, dynamic debug can add this by using: echo "module nfc +fp" > /dynamic_debug/control echo "module nci +fp" > /dynamic_debug/control If you're getting dmesg logs from customers that's a bit harder to track though.