[PATCH 0/5] trivia: Use vsprintf extension %pR

[PATCH 0/5] trivia: Use vsprintf extension %pR

[Joe Perches]

Joe Perches (5):
  ste_dma40.c: Use vsprintf extension %pR for struct resource
  lba_pci.c: Use vsprintf extension %pR for struct resource
  samsung.c: Use vsprintf extension %pR for struct resource
  mx3fb.c: Use vsprintf extension %pR for struct resource
  ml403-ac97cr.c: Use vsprintf extension %pR for struct resource

 drivers/dma/ste_dma40.c      |    5 ++---
 drivers/parisc/lba_pci.c     |   13 +++++--------
 drivers/serial/samsung.c     |    2 +-
 drivers/video/mx3fb.c        |    3 +--
 sound/drivers/ml403-ac97cr.c |    4 ++--
 5 files changed, 11 insertions(+), 16 deletions(-)

-- 
1.7.3.3.398.g0b0cd.dirty

[PATCH 1/5] dma: ste_dma40: Use vsprintf extension %pR for struct resource

[Joe Perches]

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/dma/ste_dma40.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index fab68a5..f1a8e1d 100644
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
@@ -2855,9 +2855,8 @@ static int __init d40_probe(struct platform_device *pdev)
 	if (request_mem_region(res->start, resource_size(res),
 			       D40_NAME " I/O lcpa") == NULL) {
 		ret = -EBUSY;
-		dev_err(&pdev->dev,
-			"[%s] Failed to request LCPA region 0x%x-0x%x\n",
-			__func__, res->start, res->end);
+		dev_err(&pdev->dev, "[%s] Failed to request LCPA region %pR\n",
+			__func__, res);
 		goto failure;
 	}
 
-- 
1.7.3.3.398.g0b0cd.dirty

[PATCH 2/5] parisc: lba_pci: Use vsprintf extension %pR for struct resource

[Joe Perches]

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/parisc/lba_pci.c |   13 +++++--------
 1 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/drivers/parisc/lba_pci.c b/drivers/parisc/lba_pci.c
index 3aeb327..b1b6644 100644
--- a/drivers/parisc/lba_pci.c
+++ b/drivers/parisc/lba_pci.c
@@ -749,24 +749,21 @@ lba_fixup_bus(struct pci_bus *bus)
 				continue;
 
 			if (res->flags & IORESOURCE_IO) {
-				DBG("lba_fixup_bus() I/O Ports [%lx/%lx] -> ",
-					res->start, res->end);
+				DBG("lba_fixup_bus() I/O Ports [%pR] -> ", res);
 				res->start |= lba_portbase;
 				res->end   |= lba_portbase;
-				DBG("[%lx/%lx]\n", res->start, res->end);
+				DBG("[%pR]\n", res);
 			} else if (res->flags & IORESOURCE_MEM) {
 				/*
 				** Convert PCI (IO_VIEW) addresses to
 				** processor (PA_VIEW) addresses
 				 */
-				DBG("lba_fixup_bus() MMIO [%lx/%lx] -> ",
-					res->start, res->end);
+				DBG("lba_fixup_bus() MMIO [%pR] -> ", res);
 				res->start = PCI_HOST_ADDR(HBA_DATA(ldev), res->start);
 				res->end   = PCI_HOST_ADDR(HBA_DATA(ldev), res->end);
-				DBG("[%lx/%lx]\n", res->start, res->end);
+				DBG("[%pR]\n", res);
 			} else {
-				DBG("lba_fixup_bus() WTF? 0x%lx [%lx/%lx] XXX",
-					res->flags, res->start, res->end);
+				DBG("lba_fixup_bus() WTF? [%pR] XXX", res);
 			}
 
 			/*
-- 
1.7.3.3.398.g0b0cd.dirty

[PATCH 3/5] serial: samsung: Use vsprintf extension %pR for struct resource

[Joe Perches]

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/serial/samsung.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/samsung.c b/drivers/serial/samsung.c
index 7ac2bf5..f71f511 100644
--- a/drivers/serial/samsung.c
+++ b/drivers/serial/samsung.c
@@ -1098,7 +1098,7 @@ static int s3c24xx_serial_init_port(struct s3c24xx_uart_port *ourport,
 		return -EINVAL;
 	}
 
-	dbg("resource %p (%lx..%lx)\n", res, res->start, res->end);
+	dbg("resource %pR\n", res);
 
 	port->mapbase = res->start;
 	port->membase = S3C_VA_UART + (res->start & 0xfffff);
-- 
1.7.3.3.398.g0b0cd.dirty

[PATCH 4/5] video: mx3fb: Use vsprintf extension %pR for struct resource

[Joe Perches]

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/video/mx3fb.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/video/mx3fb.c b/drivers/video/mx3fb.c
index ca0f6be..cb01391 100644
--- a/drivers/video/mx3fb.c
+++ b/drivers/video/mx3fb.c
@@ -1474,8 +1474,7 @@ static int mx3fb_probe(struct platform_device *pdev)
 		goto eremap;
 	}
 
-	pr_debug("Remapped %x to %x at %p\n", sdc_reg->start, sdc_reg->end,
-		 mx3fb->reg_base);
+	pr_debug("Remapped %pR at %p\n", sdc_reg, mx3fb->reg_base);
 
 	/* IDMAC interface */
 	dmaengine_get();
-- 
1.7.3.3.398.g0b0cd.dirty

Re: [PATCH 4/5] video: mx3fb: Use vsprintf extension %pR for struct resource

[Paul Mundt]

On Mon, Dec 13, 2010 at 01:42:21PM -0800, Joe Perches wrote:
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
>  drivers/video/mx3fb.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
Applied, thanks.

[PATCH 5/5] sound: ml403-ac97cr: Use vsprintf extension %pR for struct resource

[Joe Perches]

Signed-off-by: Joe Perches <joe@perches.com>
---
 sound/drivers/ml403-ac97cr.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/drivers/ml403-ac97cr.c b/sound/drivers/ml403-ac97cr.c
index a1282c1..5cfcb90 100644
--- a/sound/drivers/ml403-ac97cr.c
+++ b/sound/drivers/ml403-ac97cr.c
@@ -1143,8 +1143,8 @@ snd_ml403_ac97cr_create(struct snd_card *card, struct platform_device *pfdev,
 					     (resource->start) + 1);
 	if (ml403_ac97cr->port == NULL) {
 		snd_printk(KERN_ERR SND_ML403_AC97CR_DRIVER ": "
-			   "unable to remap memory region (%x to %x)\n",
-			   resource->start, resource->end);
+			   "unable to remap memory region (%pR)\n",
+			   resource);
 		snd_ml403_ac97cr_free(ml403_ac97cr);
 		return -EBUSY;
 	}
-- 
1.7.3.3.398.g0b0cd.dirty

Re: [PATCH 5/5] sound: ml403-ac97cr: Use vsprintf extension %pR for struct resource

[Takashi Iwai]

At Mon, 13 Dec 2010 13:42:22 -0800,
Joe Perches wrote:
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Applied now.  Thanks.


Takashi

> ---
>  sound/drivers/ml403-ac97cr.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/drivers/ml403-ac97cr.c b/sound/drivers/ml403-ac97cr.c
> index a1282c1..5cfcb90 100644
> --- a/sound/drivers/ml403-ac97cr.c
> +++ b/sound/drivers/ml403-ac97cr.c
> @@ -1143,8 +1143,8 @@ snd_ml403_ac97cr_create(struct snd_card *card, struct platform_device *pfdev,
>  					     (resource->start) + 1);
>  	if (ml403_ac97cr->port == NULL) {
>  		snd_printk(KERN_ERR SND_ML403_AC97CR_DRIVER ": "
> -			   "unable to remap memory region (%x to %x)\n",
> -			   resource->start, resource->end);
> +			   "unable to remap memory region (%pR)\n",
> +			   resource);
>  		snd_ml403_ac97cr_free(ml403_ac97cr);
>  		return -EBUSY;
>  	}
> -- 
> 1.7.3.3.398.g0b0cd.dirty
>