RE: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses

Path	csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	"Bounine, Alexandre" <Alexandre.Bounine@idt.com>
Newsgroups	linux.kernel
Subject	RE: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Date	Mon, 07 Mar 2016 13:50:03 +0100
Message-ID	<ra2LN-2Ud-15@gated-at.bofh.it> (permalink)
References	<r9f8m-2Tl-3@gated-at.bofh.it>
Thread-Topic	[TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Thread-Index	AQHRdrMqYsa44JSHvUe6lJUA3nP3qJ9N7l9w
Accept-Language	en-US
Content-Language	en-US
X-Originating-IP	[157.165.140.29]
Content-Type	text/plain; charset="us-ascii"
Content-Transfer-Encoding	8BIT
MIME-Version	1.0
X-Tm-As-Mml	disable
X-Scanned-By	MIMEDefang 2.43
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	99
Organization	linux.* mail to news gateway
X-Original-Cc	Mauro Carvalho Chehab <mchehab@osg.samsung.com>, Matt Porter <mporter@kernel.crashing.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "dmaengine@vger.kernel.org" <dmaengine@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>, "linux-nvdimm@ml01.01.org" <linux-nvdimm@ml01.01.org>
X-Original-Date	Mon, 7 Mar 2016 12:39:35 +0000
X-Original-Message-ID	<8D983423E7EDF846BB3056827B8CC5D15CFC6F84@corpmail1.na.ads.idt.com>
X-Original-References	<dcdebc08a17023fac93595312281cb8e91185668.1457162650.git.joe@perches.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1351583

Show key headers only | View raw

> From: Joe Perches [mailto:joe@perches.com]
> Sent: Saturday, March 05, 2016 2:47 AM
.....
> Subject: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes
> before %pa extension uses
> 
> 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 <joe@perches.com>
> ---
....

>  drivers/rapidio/devices/rio_mport_cdev.c | 4 ++--
>  drivers/rapidio/devices/tsi721.c         | 8 ++++----

For RapidIO part -
Acked-by: Alexandre Bounine <alexandre.bounine@idt.com>

.....

> 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++) {
> --
> 2.6.3.368.gf34be46

Thread

[TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses Joe Perches <joe@perches.com> - 2016-03-05 08:50 +0100
  Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before  %pa extension uses Mauro Carvalho Chehab <mchehab@osg.samsung.com> - 2016-03-05 11:10 +0100
  Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before  %pa extension uses Nicolas Ferre <nicolas.ferre@atmel.com> - 2016-03-07 11:00 +0100
  RE: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before  %pa extension uses "Bounine, Alexandre" <Alexandre.Bounine@idt.com> - 2016-03-07 13:50 +0100
  Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before  %pa extension uses Vinod Koul <vinod.koul@intel.com> - 2016-03-07 15:50 +0100
  Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before  %pa extension uses Ross Zwisler <ross.zwisler@linux.intel.com> - 2016-03-07 17:10 +0100

csiph-web