Re: [PATCH] mtd: Convert to using %pOF instead of full_name
Le Tue, 18 Jul 2017 16:43:17 -0500, Rob Herring a écrit : > Now that we have a custom printf format specifier, convert users of > full_name to use %pOF instead. This is preparation to remove storing > of the full path string for each node. Applied to l2-mtd/master. Thanks, Boris > > Signed-off-by: Rob Herring > Cc: David Woodhouse > Cc: Brian Norris > Cc: Boris Brezillon > Cc: Marek Vasut > Cc: Richard Weinberger > Cc: Cyrille Pitchen > Cc: linux-...@lists.infradead.org > --- > drivers/mtd/maps/physmap_of_core.c | 4 ++-- > drivers/mtd/maps/sun_uflash.c | 4 ++-- > drivers/mtd/ofpart.c | 23 +++ > drivers/mtd/spi-nor/hisi-sfc.c | 8 > 4 files changed, 19 insertions(+), 20 deletions(-) > > diff --git a/drivers/mtd/maps/physmap_of_core.c > b/drivers/mtd/maps/physmap_of_core.c > index 62fa6836f218..1343c34092ca 100644 > --- a/drivers/mtd/maps/physmap_of_core.c > +++ b/drivers/mtd/maps/physmap_of_core.c > @@ -178,8 +178,8 @@ static int of_flash_probe(struct platform_device *dev) >*/ > p = of_get_property(dp, "reg", &count); > if (!p || count % reg_tuple_size != 0) { > - dev_err(&dev->dev, "Malformed reg property on %s\n", > - dev->dev.of_node->full_name); > + dev_err(&dev->dev, "Malformed reg property on %pOF\n", > + dev->dev.of_node); > err = -EINVAL; > goto err_flash_remove; > } > diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c > index 414956eca0c9..1e73bba6e286 100644 > --- a/drivers/mtd/maps/sun_uflash.c > +++ b/drivers/mtd/maps/sun_uflash.c > @@ -55,8 +55,8 @@ int uflash_devinit(struct platform_device *op, struct > device_node *dp) > /* Non-CFI userflash device-- once I find one we >* can work on supporting it. >*/ > - printk(KERN_ERR PFX "Unsupported device at %s, 0x%llx\n", > -dp->full_name, (unsigned long > long)op->resource[0].start); > + printk(KERN_ERR PFX "Unsupported device at %pOF, 0x%llx\n", > +dp, (unsigned long long)op->resource[0].start); > > return -ENODEV; > } > diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c > index 2861c7079d7b..6bdf4e525677 100644 > --- a/drivers/mtd/ofpart.c > +++ b/drivers/mtd/ofpart.c > @@ -50,8 +50,8 @@ static int parse_ofpart_partitions(struct mtd_info *master, >* when using another parser), so don't be louder than >* KERN_DEBUG >*/ > - pr_debug("%s: 'partitions' subnode not found on %s. Trying to > parse direct subnodes as partitions.\n", > - master->name, mtd_node->full_name); > + pr_debug("%s: 'partitions' subnode not found on %pOF. Trying to > parse direct subnodes as partitions.\n", > + master->name, mtd_node); > ofpart_node = mtd_node; > dedicated = false; > } else if (!of_device_is_compatible(ofpart_node, "fixed-partitions")) { > @@ -87,9 +87,9 @@ static int parse_ofpart_partitions(struct mtd_info *master, > reg = of_get_property(pp, "reg", &len); > if (!reg) { > if (dedicated) { > - pr_debug("%s: ofpart partition %s (%s) missing > reg property.\n", > - master->name, pp->full_name, > - mtd_node->full_name); > + pr_debug("%s: ofpart partition %pOF (%pOF) > missing reg property.\n", > + master->name, pp, > + mtd_node); > goto ofpart_fail; > } else { > nr_parts--; > @@ -100,9 +100,9 @@ static int parse_ofpart_partitions(struct mtd_info > *master, > a_cells = of_n_addr_cells(pp); > s_cells = of_n_size_cells(pp); > if (len / 4 != a_cells + s_cells) { > - pr_debug("%s: ofpart partition %s (%s) error parsing > reg property.\n", > - master->name, pp->full_name, > - mtd_node->full_name); > + pr_debug("%s: ofpart partition %pOF (%pOF) error > parsing reg property.\n", > + master->name, pp, > + mtd_node); > goto ofpart_fail; > } > > @@ -131,8 +131,8 @@ static int parse_ofpart_partitions(struct mtd_info > *master, > return nr_parts; > > ofpart_fail: > - pr_err("%s: error parsing ofpart partition %s (%s)\n", > -master->name, pp->full_name, mtd_node->full_name); > + pr_err("%s: error parsing ofpart partition %pOF (%pOF)\n", > +master->name, pp, mtd_node
Re: [PATCH] mtd: Convert to using %pOF instead of full_name
On Wed, Jul 19, 2017 at 2:44 PM, Boris Brezillon wrote: > On Tue, 18 Jul 2017 16:43:17 -0500 > Rob Herring wrote: > >> Now that we have a custom printf format specifier, convert users of >> full_name to use %pOF instead. This is preparation to remove storing >> of the full path string for each node. >> >> Signed-off-by: Rob Herring >> Cc: David Woodhouse >> Cc: Brian Norris >> Cc: Boris Brezillon > > Acked-by: Boris Brezillon Thanks. Please take this thru the mtd tree. Rob
Re: [PATCH] mtd: Convert to using %pOF instead of full_name
On Tue, 18 Jul 2017 16:43:17 -0500 Rob Herring wrote: > Now that we have a custom printf format specifier, convert users of > full_name to use %pOF instead. This is preparation to remove storing > of the full path string for each node. > > Signed-off-by: Rob Herring > Cc: David Woodhouse > Cc: Brian Norris > Cc: Boris Brezillon Acked-by: Boris Brezillon > Cc: Marek Vasut > Cc: Richard Weinberger > Cc: Cyrille Pitchen > Cc: linux-...@lists.infradead.org > --- > drivers/mtd/maps/physmap_of_core.c | 4 ++-- > drivers/mtd/maps/sun_uflash.c | 4 ++-- > drivers/mtd/ofpart.c | 23 +++ > drivers/mtd/spi-nor/hisi-sfc.c | 8 > 4 files changed, 19 insertions(+), 20 deletions(-) > > diff --git a/drivers/mtd/maps/physmap_of_core.c > b/drivers/mtd/maps/physmap_of_core.c > index 62fa6836f218..1343c34092ca 100644 > --- a/drivers/mtd/maps/physmap_of_core.c > +++ b/drivers/mtd/maps/physmap_of_core.c > @@ -178,8 +178,8 @@ static int of_flash_probe(struct platform_device *dev) >*/ > p = of_get_property(dp, "reg", &count); > if (!p || count % reg_tuple_size != 0) { > - dev_err(&dev->dev, "Malformed reg property on %s\n", > - dev->dev.of_node->full_name); > + dev_err(&dev->dev, "Malformed reg property on %pOF\n", > + dev->dev.of_node); > err = -EINVAL; > goto err_flash_remove; > } > diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c > index 414956eca0c9..1e73bba6e286 100644 > --- a/drivers/mtd/maps/sun_uflash.c > +++ b/drivers/mtd/maps/sun_uflash.c > @@ -55,8 +55,8 @@ int uflash_devinit(struct platform_device *op, struct > device_node *dp) > /* Non-CFI userflash device-- once I find one we >* can work on supporting it. >*/ > - printk(KERN_ERR PFX "Unsupported device at %s, 0x%llx\n", > -dp->full_name, (unsigned long > long)op->resource[0].start); > + printk(KERN_ERR PFX "Unsupported device at %pOF, 0x%llx\n", > +dp, (unsigned long long)op->resource[0].start); > > return -ENODEV; > } > diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c > index 2861c7079d7b..6bdf4e525677 100644 > --- a/drivers/mtd/ofpart.c > +++ b/drivers/mtd/ofpart.c > @@ -50,8 +50,8 @@ static int parse_ofpart_partitions(struct mtd_info *master, >* when using another parser), so don't be louder than >* KERN_DEBUG >*/ > - pr_debug("%s: 'partitions' subnode not found on %s. Trying to > parse direct subnodes as partitions.\n", > - master->name, mtd_node->full_name); > + pr_debug("%s: 'partitions' subnode not found on %pOF. Trying to > parse direct subnodes as partitions.\n", > + master->name, mtd_node); > ofpart_node = mtd_node; > dedicated = false; > } else if (!of_device_is_compatible(ofpart_node, "fixed-partitions")) { > @@ -87,9 +87,9 @@ static int parse_ofpart_partitions(struct mtd_info *master, > reg = of_get_property(pp, "reg", &len); > if (!reg) { > if (dedicated) { > - pr_debug("%s: ofpart partition %s (%s) missing > reg property.\n", > - master->name, pp->full_name, > - mtd_node->full_name); > + pr_debug("%s: ofpart partition %pOF (%pOF) > missing reg property.\n", > + master->name, pp, > + mtd_node); > goto ofpart_fail; > } else { > nr_parts--; > @@ -100,9 +100,9 @@ static int parse_ofpart_partitions(struct mtd_info > *master, > a_cells = of_n_addr_cells(pp); > s_cells = of_n_size_cells(pp); > if (len / 4 != a_cells + s_cells) { > - pr_debug("%s: ofpart partition %s (%s) error parsing > reg property.\n", > - master->name, pp->full_name, > - mtd_node->full_name); > + pr_debug("%s: ofpart partition %pOF (%pOF) error > parsing reg property.\n", > + master->name, pp, > + mtd_node); > goto ofpart_fail; > } > > @@ -131,8 +131,8 @@ static int parse_ofpart_partitions(struct mtd_info > *master, > return nr_parts; > > ofpart_fail: > - pr_err("%s: error parsing ofpart partition %s (%s)\n", > -master->name, pp->full_name, mtd_node->full_name); > + pr_err("%s: error parsing ofpart partition %pOF (%pOF)\n", > +master->name, pp, mtd_node); > ret = -E