* [PATCH 00/14] Use printf extension %pR for struct resource @ 2010-11-12 21:37 Joe Perches 2010-11-12 21:37 ` [PATCH 03/14] arch/powerpc: " Joe Perches 2010-11-12 21:37 ` [PATCH 08/14] drivers/mtd/nand: " Joe Perches 0 siblings, 2 replies; 5+ messages in thread From: Joe Perches @ 2010-11-12 21:37 UTC (permalink / raw) To: Jiri Kosina Cc: linux-mips, alsa-devel, linux-parisc, linux-rdma, netdev, linux-usb, linux-kernel, socketcan-core, linux-mtd, linuxppc-dev Yet more trivia... Joe Perches (14): arch/frv: Use printf extension %pR for struct resource arch/mips: Use printf extension %pR for struct resource arch/powerpc: Use printf extension %pR for struct resource drivers/dma/ppc4xx: Use printf extension %pR for struct resource drivers/infiniband: Use printf extension %pR for struct resource drivers/mfd: Use printf extension %pR for struct resource drivers/mtd/maps: Use printf extension %pR for struct resource drivers/mtd/nand: Use printf extension %pR for struct resource drivers/net/can/sja1000: Use printf extension %pR for struct resource drivers/parisc: Use printf extension %pR for struct resource drivers/rapidio: Use printf extension %pR for struct resource drivers/uwb: Use printf extension %pR for struct resource drivers/video: Use printf extension %pR for struct resource sound/ppc: Use printf extension %pR for struct resource arch/frv/mb93090-mb00/pci-vdk.c | 8 ++------ arch/mips/txx9/generic/pci.c | 7 ++----- arch/powerpc/kernel/pci_64.c | 3 +-- arch/powerpc/sysdev/tsi108_dev.c | 8 ++++---- drivers/dma/ppc4xx/adma.c | 5 ++--- drivers/infiniband/hw/ipath/ipath_driver.c | 5 ++--- drivers/mfd/sm501.c | 7 ++----- drivers/mtd/maps/amd76xrom.c | 7 ++----- drivers/mtd/maps/ck804xrom.c | 7 ++----- drivers/mtd/maps/esb2rom.c | 9 +++------ drivers/mtd/maps/ichxrom.c | 9 +++------ drivers/mtd/maps/physmap_of.c | 4 +--- drivers/mtd/maps/scx200_docflash.c | 5 ++--- drivers/mtd/nand/pasemi_nand.c | 2 +- drivers/net/can/sja1000/sja1000_of_platform.c | 8 ++------ drivers/parisc/dino.c | 13 +++++-------- drivers/parisc/hppb.c | 6 ++---- drivers/rapidio/rio.c | 4 ++-- drivers/uwb/umc-dev.c | 7 ++----- drivers/video/platinumfb.c | 8 ++------ sound/ppc/pmac.c | 12 ++++-------- 21 files changed, 48 insertions(+), 96 deletions(-) -- 1.7.3.1.g432b3.dirty ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 03/14] arch/powerpc: Use printf extension %pR for struct resource 2010-11-12 21:37 [PATCH 00/14] Use printf extension %pR for struct resource Joe Perches @ 2010-11-12 21:37 ` Joe Perches 2010-11-13 0:42 ` Stephen Rothwell 2010-11-12 21:37 ` [PATCH 08/14] drivers/mtd/nand: " Joe Perches 1 sibling, 1 reply; 5+ messages in thread From: Joe Perches @ 2010-11-12 21:37 UTC (permalink / raw) To: Jiri Kosina; +Cc: Paul Mackerras, linuxppc-dev, linux-kernel Using %pR standardizes the struct resource output. Signed-off-by: Joe Perches <joe@perches.com> --- arch/powerpc/kernel/pci_64.c | 3 +-- arch/powerpc/sysdev/tsi108_dev.c | 8 ++++---- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/arch/powerpc/kernel/pci_64.c b/arch/powerpc/kernel/pci_64.c index d43fc65..2ecb1de 100644 --- a/arch/powerpc/kernel/pci_64.c +++ b/arch/powerpc/kernel/pci_64.c @@ -193,8 +193,7 @@ int __devinit pcibios_map_io_space(struct pci_bus *bus) hose->io_resource.start += io_virt_offset; hose->io_resource.end += io_virt_offset; - pr_debug(" hose->io_resource=0x%016llx...0x%016llx\n", - hose->io_resource.start, hose->io_resource.end); + pr_debug(" hose->io_resource=%pR\n", &hose->io_resource.start); return 0; } diff --git a/arch/powerpc/sysdev/tsi108_dev.c b/arch/powerpc/sysdev/tsi108_dev.c index d4d15aa..5e249a8 100644 --- a/arch/powerpc/sysdev/tsi108_dev.c +++ b/arch/powerpc/sysdev/tsi108_dev.c @@ -83,8 +83,8 @@ static int __init tsi108_eth_of_init(void) memset(&tsi_eth_data, 0, sizeof(tsi_eth_data)); ret = of_address_to_resource(np, 0, &r[0]); - DBG("%s: name:start->end = %s:0x%lx-> 0x%lx\n", - __func__,r[0].name, r[0].start, r[0].end); + DBG("%s: name:start->end = %s:%pR\n", + __func__, r[0].name, &r[0]); if (ret) goto err; @@ -92,8 +92,8 @@ static int __init tsi108_eth_of_init(void) r[1].start = irq_of_parse_and_map(np, 0); r[1].end = irq_of_parse_and_map(np, 0); r[1].flags = IORESOURCE_IRQ; - DBG("%s: name:start->end = %s:0x%lx-> 0x%lx\n", - __func__,r[1].name, r[1].start, r[1].end); + DBG("%s: name:start->end = %s:%pR\n", + __func__, r[1].name, &r[1]); tsi_eth_dev = platform_device_register_simple("tsi-ethernet", i++, &r[0], -- 1.7.3.1.g432b3.dirty ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 03/14] arch/powerpc: Use printf extension %pR for struct resource 2010-11-12 21:37 ` [PATCH 03/14] arch/powerpc: " Joe Perches @ 2010-11-13 0:42 ` Stephen Rothwell 2010-11-13 0:49 ` [PATCH 03/14 V2] " Joe Perches 0 siblings, 1 reply; 5+ messages in thread From: Stephen Rothwell @ 2010-11-13 0:42 UTC (permalink / raw) To: Joe Perches; +Cc: Paul Mackerras, Jiri Kosina, linuxppc-dev, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1133 bytes --] Hi Joe, On Fri, 12 Nov 2010 13:37:53 -0800 Joe Perches <joe@perches.com> wrote: > > Using %pR standardizes the struct resource output. > > Signed-off-by: Joe Perches <joe@perches.com> > --- > arch/powerpc/kernel/pci_64.c | 3 +-- > arch/powerpc/sysdev/tsi108_dev.c | 8 ++++---- > 2 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/arch/powerpc/kernel/pci_64.c b/arch/powerpc/kernel/pci_64.c > index d43fc65..2ecb1de 100644 > --- a/arch/powerpc/kernel/pci_64.c > +++ b/arch/powerpc/kernel/pci_64.c > @@ -193,8 +193,7 @@ int __devinit pcibios_map_io_space(struct pci_bus *bus) > hose->io_resource.start += io_virt_offset; > hose->io_resource.end += io_virt_offset; > > - pr_debug(" hose->io_resource=0x%016llx...0x%016llx\n", > - hose->io_resource.start, hose->io_resource.end); > + pr_debug(" hose->io_resource=%pR\n", &hose->io_resource.start); ^^^^^^^^^^^^^^^^^^^^^^^^ Shouldn't this be just &hose->io_resource? -- Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/ [-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 03/14 V2] arch/powerpc: Use printf extension %pR for struct resource 2010-11-13 0:42 ` Stephen Rothwell @ 2010-11-13 0:49 ` Joe Perches 0 siblings, 0 replies; 5+ messages in thread From: Joe Perches @ 2010-11-13 0:49 UTC (permalink / raw) To: Stephen Rothwell; +Cc: Paul Mackerras, Jiri Kosina, linuxppc-dev, linux-kernel Using %pR standardizes the struct resource output. Signed-off-by: Joe Perches <joe@perches.com> --- On Sat, 2010-11-13 at 11:42 +1100, Stephen Rothwell wrote: > Shouldn't this be just &hose->io_resource? V2: Fix stupid editing mistake arch/powerpc/kernel/pci_64.c | 3 +-- arch/powerpc/sysdev/tsi108_dev.c | 8 ++++---- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/arch/powerpc/kernel/pci_64.c b/arch/powerpc/kernel/pci_64.c index d43fc65..2ecb1de 100644 --- a/arch/powerpc/kernel/pci_64.c +++ b/arch/powerpc/kernel/pci_64.c @@ -193,8 +193,7 @@ int __devinit pcibios_map_io_space(struct pci_bus *bus) hose->io_resource.start += io_virt_offset; hose->io_resource.end += io_virt_offset; - pr_debug(" hose->io_resource=0x%016llx...0x%016llx\n", - hose->io_resource.start, hose->io_resource.end); + pr_debug(" hose->io_resource=%pR\n", &hose->io_resource); return 0; } diff --git a/arch/powerpc/sysdev/tsi108_dev.c b/arch/powerpc/sysdev/tsi108_dev.c index d4d15aa..5e249a8 100644 --- a/arch/powerpc/sysdev/tsi108_dev.c +++ b/arch/powerpc/sysdev/tsi108_dev.c @@ -83,8 +83,8 @@ static int __init tsi108_eth_of_init(void) memset(&tsi_eth_data, 0, sizeof(tsi_eth_data)); ret = of_address_to_resource(np, 0, &r[0]); - DBG("%s: name:start->end = %s:0x%lx-> 0x%lx\n", - __func__,r[0].name, r[0].start, r[0].end); + DBG("%s: name:start->end = %s:%pR\n", + __func__, r[0].name, &r[0]); if (ret) goto err; @@ -92,8 +92,8 @@ static int __init tsi108_eth_of_init(void) r[1].start = irq_of_parse_and_map(np, 0); r[1].end = irq_of_parse_and_map(np, 0); r[1].flags = IORESOURCE_IRQ; - DBG("%s: name:start->end = %s:0x%lx-> 0x%lx\n", - __func__,r[1].name, r[1].start, r[1].end); + DBG("%s: name:start->end = %s:%pR\n", + __func__, r[1].name, &r[1]); tsi_eth_dev = platform_device_register_simple("tsi-ethernet", i++, &r[0], -- 1.7.3.1.g432b3.dirty ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 08/14] drivers/mtd/nand: Use printf extension %pR for struct resource 2010-11-12 21:37 [PATCH 00/14] Use printf extension %pR for struct resource Joe Perches 2010-11-12 21:37 ` [PATCH 03/14] arch/powerpc: " Joe Perches @ 2010-11-12 21:37 ` Joe Perches 1 sibling, 0 replies; 5+ messages in thread From: Joe Perches @ 2010-11-12 21:37 UTC (permalink / raw) To: Jiri Kosina Cc: Olof Johansson, linuxppc-dev, David Woodhouse, linux-kernel, linux-mtd Using %pR standardizes the struct resource output. Signed-off-by: Joe Perches <joe@perches.com> --- drivers/mtd/nand/pasemi_nand.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/mtd/nand/pasemi_nand.c b/drivers/mtd/nand/pasemi_nand.c index 6ddb246..bb277a5 100644 --- a/drivers/mtd/nand/pasemi_nand.c +++ b/drivers/mtd/nand/pasemi_nand.c @@ -107,7 +107,7 @@ static int __devinit pasemi_nand_probe(struct platform_device *ofdev, if (pasemi_nand_mtd) return -ENODEV; - pr_debug("pasemi_nand at %llx-%llx\n", res.start, res.end); + pr_debug("pasemi_nand at %pR\n", &res); /* Allocate memory for MTD device structure and private data */ pasemi_nand_mtd = kzalloc(sizeof(struct mtd_info) + -- 1.7.3.1.g432b3.dirty ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-11-13 0:49 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-11-12 21:37 [PATCH 00/14] Use printf extension %pR for struct resource Joe Perches 2010-11-12 21:37 ` [PATCH 03/14] arch/powerpc: " Joe Perches 2010-11-13 0:42 ` Stephen Rothwell 2010-11-13 0:49 ` [PATCH 03/14 V2] " Joe Perches 2010-11-12 21:37 ` [PATCH 08/14] drivers/mtd/nand: " Joe Perches
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).