From patchwork Thu Jul 21 17:43:07 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Felipe Balbi X-Patchwork-Id: 996242 Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by demeter1.kernel.org (8.14.4/8.14.4) with ESMTP id p6LHhUKA030914 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Thu, 21 Jul 2011 17:43:51 GMT Received: from canuck.infradead.org ([2001:4978:20e::1]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1QjxHK-0002d3-HR; Thu, 21 Jul 2011 17:43:18 +0000 Received: from localhost ([127.0.0.1] helo=canuck.infradead.org) by canuck.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1QjxHK-0001hk-5R; Thu, 21 Jul 2011 17:43:18 +0000 Received: from na3sys009aog116.obsmtp.com ([74.125.149.240]) by canuck.infradead.org with smtps (Exim 4.76 #1 (Red Hat Linux)) id 1QjxHF-0001hR-Tb for linux-arm-kernel@lists.infradead.org; Thu, 21 Jul 2011 17:43:15 +0000 Received: from mail-ew0-f53.google.com ([209.85.215.53]) (using TLSv1) by na3sys009aob116.postini.com ([74.125.148.12]) with SMTP ID DSNKTihlLluEPrX6jq8IVnEc0RUl8BDURWEW@postini.com; Thu, 21 Jul 2011 10:43:13 PDT Received: by mail-ew0-f53.google.com with SMTP id 8so1567888ewy.26 for ; Thu, 21 Jul 2011 10:43:10 -0700 (PDT) Received: by 10.14.17.210 with SMTP id j58mr257561eej.170.1311270190246; Thu, 21 Jul 2011 10:43:10 -0700 (PDT) Received: from localhost (cs181221225.pp.htv.fi [82.181.221.225]) by mx.google.com with ESMTPS id v8sm1253749eej.50.2011.07.21.10.43.08 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 21 Jul 2011 10:43:09 -0700 (PDT) Date: Thu, 21 Jul 2011 20:43:07 +0300 From: Felipe Balbi To: Kevin Hilman Subject: Re: [PATCH] OMAP: omap_device: replace pr_* prints with dev_* Message-ID: <20110721174306.GD32058@legolas.emea.dhcp.ti.com> References: <1311269976-4035-1-git-send-email-khilman@ti.com> MIME-Version: 1.0 In-Reply-To: <1311269976-4035-1-git-send-email-khilman@ti.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20090807-BlameThorstenAndJenny ( TRE 0.7.6 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20110721_134314_231236_B1DBA493 X-CRM114-Status: GOOD ( 17.57 ) X-Spam-Score: -2.3 (--) X-Spam-Report: SpamAssassin version 3.3.1 on canuck.infradead.org summary: Content analysis details: (-2.3 points) pts rule name description ---- ---------------------- -------------------------------------------------- -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/, medium trust [74.125.149.240 listed in list.dnswl.org] Cc: Russell King , Paul Walmsley , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Felipe Balbi X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: balbi@ti.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Thu, 21 Jul 2011 17:43:51 +0000 (UTC) On Thu, Jul 21, 2011 at 10:39:36AM -0700, Kevin Hilman wrote: > For consistency in kernel printk output for devices, use dev_dbg(), > dev_warn(), dev_err() instead of pr_debug(), pr_warning() and > pr_err(), some of which currently use direct access of name from > platform_device and others of which use dev_name(). Using the dev_* > versions uses the standard device naming from the driver core. > > Some pr_* prints were not converted with this patch since they are > used before the platform_device and struct device are created so > neither the dev_* prints or dev_name() is valid. > > Reported-by: Russell King > Cc: Felipe Balbi > Signed-off-by: Kevin Hilman > --- FWIW: Reviewed-by: Felipe Balbi BTW, I tried applying this to today's linus/master, but it fails (rejects below), are you depending on some other changes ? --- arch/arm/plat-omap/omap_device.c +++ arch/arm/plat-omap/omap_device.c @@ -242,28 +239,27 @@ if (!clk_alias || !clk_name) return; - pr_debug("omap_device: %s: Creating %s -> %s\n", - dev_name(&od->pdev.dev), clk_alias, clk_name); + dev_dbg(&od->pdev.dev, "Creating %s -> %s\n", clk_alias, clk_name); r = clk_get_sys(dev_name(&od->pdev.dev), clk_alias); if (!IS_ERR(r)) { - pr_warning("omap_device: %s: alias %s already exists\n", - dev_name(&od->pdev.dev), clk_alias); + dev_warn(&od->pdev.dev, + "alias %s already exists\n", clk_alias); clk_put(r); return; } r = omap_clk_get_by_name(clk_name); if (IS_ERR(r)) { - pr_err("omap_device: %s: omap_clk_get_by_name for %s failed\n", - dev_name(&od->pdev.dev), clk_name); + dev_err(&od->pdev.dev, + "omap_clk_get_by_name for %s failed\n", clk_name); return; } l = clkdev_alloc(r, clk_alias, dev_name(&od->pdev.dev)); if (!l) { - pr_err("omap_device: %s: clkdev_alloc for %s failed\n", - dev_name(&od->pdev.dev), clk_alias); + dev_err(&od->pdev.dev, + "clkdev_alloc for %s failed\n", clk_alias); return; }