Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760566AbcCEKCs (ORCPT ); Sat, 5 Mar 2016 05:02:48 -0500 Received: from lists.s-osg.org ([54.187.51.154]:36444 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760102AbcCEKCo (ORCPT ); Sat, 5 Mar 2016 05:02:44 -0500 Date: Sat, 5 Mar 2016 07:02:37 -0300 From: Mauro Carvalho Chehab To: Joe Perches Cc: Nicolas Ferre , Vinod Koul , Dan Williams , Benoit Parrot , Ross Zwisler , Jiri Kosina , 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: <20160305070237.235b32c5@recife.lan> In-Reply-To: References: Organization: Samsung X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.29; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5014 Lines: 127 Em Fri, 4 Mar 2016 23:46:32 -0800 Joe Perches escreveu: > 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 +- Acked-by: Mauro Carvalho Chehab > 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/dma/at_hdmac_regs.h b/drivers/dma/at_hdmac_regs.h > index 7f58f06..0474e4a 100644 > --- a/drivers/dma/at_hdmac_regs.h > +++ b/drivers/dma/at_hdmac_regs.h > @@ -385,7 +385,7 @@ static void vdbg_dump_regs(struct at_dma_chan *atchan) {} > static void atc_dump_lli(struct at_dma_chan *atchan, struct at_lli *lli) > { > dev_crit(chan2dev(&atchan->chan_common), > - " desc: s%pad d%pad ctrl0x%x:0x%x l0x%pad\n", > + " desc: s%pad d%pad ctrl0x%x:0x%x l%pad\n", > &lli->saddr, &lli->daddr, > lli->ctrla, lli->ctrlb, &lli->dscr); > } > diff --git a/drivers/media/platform/ti-vpe/cal.c b/drivers/media/platform/ti-vpe/cal.c > index 82001e6..7dce489 100644 > --- a/drivers/media/platform/ti-vpe/cal.c > +++ b/drivers/media/platform/ti-vpe/cal.c > @@ -498,7 +498,7 @@ static inline void cal_runtime_put(struct cal_dev *dev) > > static void cal_quickdump_regs(struct cal_dev *dev) > { > - cal_info(dev, "CAL Registers @ 0x%pa:\n", &dev->res->start); > + cal_info(dev, "CAL Registers @ %pa:\n", &dev->res->start); > print_hex_dump(KERN_INFO, "", DUMP_PREFIX_OFFSET, 16, 4, > (__force const void *)dev->base, > resource_size(dev->res), false); > 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); > } > diff --git a/drivers/rapidio/devices/rio_mport_cdev.c b/drivers/rapidio/devices/rio_mport_cdev.c > index a3369d1..211a67d 100644 > --- a/drivers/rapidio/devices/rio_mport_cdev.c > +++ b/drivers/rapidio/devices/rio_mport_cdev.c > @@ -2223,7 +2223,7 @@ static void mport_mm_open(struct vm_area_struct *vma) > { > struct rio_mport_mapping *map = vma->vm_private_data; > > -rmcd_debug(MMAP, "0x%pad", &map->phys_addr); > + rmcd_debug(MMAP, "%pad", &map->phys_addr); > kref_get(&map->ref); > } > > @@ -2231,7 +2231,7 @@ static void mport_mm_close(struct vm_area_struct *vma) > { > struct rio_mport_mapping *map = vma->vm_private_data; > > -rmcd_debug(MMAP, "0x%pad", &map->phys_addr); > + rmcd_debug(MMAP, "%pad", &map->phys_addr); > mutex_lock(&map->md->buf_mutex); > kref_put(&map->ref, mport_release_mapping); > mutex_unlock(&map->md->buf_mutex); > diff --git a/drivers/rapidio/devices/tsi721.c b/drivers/rapidio/devices/tsi721.c > index b5b4556..4c20e99 100644 > --- a/drivers/rapidio/devices/tsi721.c > +++ b/drivers/rapidio/devices/tsi721.c > @@ -1101,7 +1101,7 @@ static int tsi721_rio_map_inb_mem(struct rio_mport *mport, dma_addr_t lstart, > ibw_start = lstart & ~(ibw_size - 1); > > tsi_debug(IBW, &priv->pdev->dev, > - "Direct (RIO_0x%llx -> PCIe_0x%pad), size=0x%x, ibw_start = 0x%llx", > + "Direct (RIO_0x%llx -> PCIe_%pad), size=0x%x, ibw_start = 0x%llx", > rstart, &lstart, size, ibw_start); > > while ((lstart + size) > (ibw_start + ibw_size)) { > @@ -1120,7 +1120,7 @@ static int tsi721_rio_map_inb_mem(struct rio_mport *mport, dma_addr_t lstart, > > } else { > tsi_debug(IBW, &priv->pdev->dev, > - "Translated (RIO_0x%llx -> PCIe_0x%pad), size=0x%x", > + "Translated (RIO_0x%llx -> PCIe_%pad), size=0x%x", > rstart, &lstart, size); > > if (!is_power_of_2(size) || size < 0x1000 || > @@ -1215,7 +1215,7 @@ static int tsi721_rio_map_inb_mem(struct rio_mport *mport, dma_addr_t lstart, > priv->ibwin_cnt--; > > tsi_debug(IBW, &priv->pdev->dev, > - "Configured IBWIN%d (RIO_0x%llx -> PCIe_0x%pad), size=0x%llx", > + "Configured IBWIN%d (RIO_0x%llx -> PCIe_%pad), size=0x%llx", > i, ibw_start, &loc_start, ibw_size); > > return 0; > @@ -1237,7 +1237,7 @@ static void tsi721_rio_unmap_inb_mem(struct rio_mport *mport, > int i; > > tsi_debug(IBW, &priv->pdev->dev, > - "Unmap IBW mapped to PCIe_0x%pad", &lstart); > + "Unmap IBW mapped to PCIe_%pad", &lstart); > > /* Search for matching active inbound translation window */ > for (i = 0; i < TSI721_IBWIN_NUM; i++) { -- Thanks, Mauro