Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753269AbcCGQIi (ORCPT ); Mon, 7 Mar 2016 11:08:38 -0500 Received: from mga11.intel.com ([192.55.52.93]:18136 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752803AbcCGQIe (ORCPT ); Mon, 7 Mar 2016 11:08:34 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,552,1449561600"; d="scan'208";a="904003772" Date: Mon, 7 Mar 2016 09:08:00 -0700 From: Ross Zwisler To: Joe Perches Cc: Nicolas Ferre , Vinod Koul , Dan Williams , Benoit Parrot , Ross Zwisler , Jiri Kosina , Mauro Carvalho Chehab , Matt Porter , Alexandre Bounine , linux-arm-kernel@lists.infradead.org, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-nvdimm@ml01.01.org Subject: Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses Message-ID: <20160307160800.GA9185@linux.intel.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1228 Lines: 31 On Fri, Mar 04, 2016 at 11:46:32PM -0800, Joe Perches wrote: > Since commit 3cab1e711297 ("lib/vsprintf: refactor duplicate code > to special_hex_number()") %pa uses have been ouput with a 0x prefix. > > These 0x prefixes in the formats are unnecessary. > > Signed-off-by: Joe Perches > --- > drivers/dma/at_hdmac_regs.h | 2 +- > drivers/media/platform/ti-vpe/cal.c | 2 +- > drivers/nvdimm/pmem.c | 2 +- > drivers/rapidio/devices/rio_mport_cdev.c | 4 ++-- > drivers/rapidio/devices/tsi721.c | 8 ++++---- > 5 files changed, 9 insertions(+), 9 deletions(-) <> > diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c > index 8d0b546..eb619d1 100644 > --- a/drivers/nvdimm/pmem.c > +++ b/drivers/nvdimm/pmem.c > @@ -172,7 +172,7 @@ static struct pmem_device *pmem_alloc(struct device *dev, > > if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size, > dev_name(dev))) { > - dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n", > + dev_warn(dev, "could not reserve region [%pa:0x%zx]\n", > &pmem->phys_addr, pmem->size); > return ERR_PTR(-EBUSY); > } For the pmem part: Acked-by: Ross Zwisler