From: Roger Quadros <rogerq@ti.com>
To: Rob Herring <robh@kernel.org>, arm@kernel.org
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Alexandre Belloni <alexandre.belloni@free-electrons.com>,
Tony Lindgren <tony@atomide.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org
Subject: Re: [PATCH] memory: Convert to using %pOF instead of full_name
Date: Thu, 10 Aug 2017 10:10:52 +0300 [thread overview]
Message-ID: <bf4e23a9-ee21-71c6-af13-adf3fe12b5f4@ti.com> (raw)
In-Reply-To: <20170718214339.7774-34-robh@kernel.org>
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 19/07/17 00:43, 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 <robh@kernel.org>
> Cc: Nicolas Ferre <nicolas.ferre@microchip.com>
> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> Cc: Roger Quadros <rogerq@ti.com>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-omap@vger.kernel.org
> ---
> drivers/memory/atmel-ebi.c | 17 ++++++++---------
> drivers/memory/jz4780-nemc.c | 12 ++++++------
> drivers/memory/mvebu-devbus.c | 12 ++++++------
> drivers/memory/omap-gpmc.c | 18 +++++++++---------
> 4 files changed, 29 insertions(+), 30 deletions(-)
for omap-gpmc.c
Acked-by: Roger Quadros <rogerq@ti.com>
>
> diff --git a/drivers/memory/atmel-ebi.c b/drivers/memory/atmel-ebi.c
> index 99e644cda4d1..597282e35c69 100644
> --- a/drivers/memory/atmel-ebi.c
> +++ b/drivers/memory/atmel-ebi.c
> @@ -156,8 +156,8 @@ static int atmel_ebi_xslate_smc_timings(struct atmel_ebi_dev *ebid,
> out:
> if (ret) {
> dev_err(ebid->ebi->dev,
> - "missing or invalid timings definition in %s",
> - np->full_name);
> + "missing or invalid timings definition in %pOF",
> + np);
> return ret;
> }
>
> @@ -267,8 +267,8 @@ static int atmel_ebi_xslate_smc_config(struct atmel_ebi_dev *ebid,
> return -EINVAL;
>
> if ((ret > 0 && !required) || (!ret && required)) {
> - dev_err(ebid->ebi->dev, "missing atmel,smc- properties in %s",
> - np->full_name);
> + dev_err(ebid->ebi->dev, "missing atmel,smc- properties in %pOF",
> + np);
> return -EINVAL;
> }
>
> @@ -311,8 +311,7 @@ static int atmel_ebi_dev_setup(struct atmel_ebi *ebi, struct device_node *np,
>
> if (cs >= AT91_MATRIX_EBI_NUM_CS ||
> !(ebi->caps->available_cs & BIT(cs))) {
> - dev_err(dev, "invalid reg property in %s\n",
> - np->full_name);
> + dev_err(dev, "invalid reg property in %pOF\n", np);
> return -EINVAL;
> }
>
> @@ -321,7 +320,7 @@ static int atmel_ebi_dev_setup(struct atmel_ebi *ebi, struct device_node *np,
> }
>
> if (!numcs) {
> - dev_err(dev, "invalid reg property in %s\n", np->full_name);
> + dev_err(dev, "invalid reg property in %pOF\n", np);
> return -EINVAL;
> }
>
> @@ -569,8 +568,8 @@ static int atmel_ebi_probe(struct platform_device *pdev)
>
> ret = atmel_ebi_dev_setup(ebi, child, reg_cells);
> if (ret) {
> - dev_err(dev, "failed to configure EBI bus for %s, disabling the device",
> - child->full_name);
> + dev_err(dev, "failed to configure EBI bus for %pOF, disabling the device",
> + child);
>
> ret = atmel_ebi_dev_disable(ebi, child);
> if (ret)
> diff --git a/drivers/memory/jz4780-nemc.c b/drivers/memory/jz4780-nemc.c
> index 919d1925acb9..bcf06adefc96 100644
> --- a/drivers/memory/jz4780-nemc.c
> +++ b/drivers/memory/jz4780-nemc.c
> @@ -322,8 +322,8 @@ static int jz4780_nemc_probe(struct platform_device *pdev)
> bank = of_read_number(prop, 1);
> if (bank < 1 || bank >= JZ4780_NEMC_NUM_BANKS) {
> dev_err(nemc->dev,
> - "%s requests invalid bank %u\n",
> - child->full_name, bank);
> + "%pOF requests invalid bank %u\n",
> + child, bank);
>
> /* Will continue the outer loop below. */
> referenced = 0;
> @@ -334,12 +334,12 @@ static int jz4780_nemc_probe(struct platform_device *pdev)
> }
>
> if (!referenced) {
> - dev_err(nemc->dev, "%s has no addresses\n",
> - child->full_name);
> + dev_err(nemc->dev, "%pOF has no addresses\n",
> + child);
> continue;
> } else if (nemc->banks_present & referenced) {
> - dev_err(nemc->dev, "%s conflicts with another node\n",
> - child->full_name);
> + dev_err(nemc->dev, "%pOF conflicts with another node\n",
> + child);
> continue;
> }
>
> diff --git a/drivers/memory/mvebu-devbus.c b/drivers/memory/mvebu-devbus.c
> index 24852812fd44..981860879d02 100644
> --- a/drivers/memory/mvebu-devbus.c
> +++ b/drivers/memory/mvebu-devbus.c
> @@ -105,8 +105,8 @@ static int get_timing_param_ps(struct devbus *devbus,
>
> err = of_property_read_u32(node, name, &time_ps);
> if (err < 0) {
> - dev_err(devbus->dev, "%s has no '%s' property\n",
> - name, node->full_name);
> + dev_err(devbus->dev, "%pOF has no '%s' property\n",
> + node, name);
> return err;
> }
>
> @@ -127,8 +127,8 @@ static int devbus_get_timing_params(struct devbus *devbus,
> err = of_property_read_u32(node, "devbus,bus-width", &r->bus_width);
> if (err < 0) {
> dev_err(devbus->dev,
> - "%s has no 'devbus,bus-width' property\n",
> - node->full_name);
> + "%pOF has no 'devbus,bus-width' property\n",
> + node);
> return err;
> }
>
> @@ -180,8 +180,8 @@ static int devbus_get_timing_params(struct devbus *devbus,
> &w->sync_enable);
> if (err < 0) {
> dev_err(devbus->dev,
> - "%s has no 'devbus,sync-enable' property\n",
> - node->full_name);
> + "%pOF has no 'devbus,sync-enable' property\n",
> + node);
> return err;
> }
> }
> diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
> index a80e17de906d..f7f393541934 100644
> --- a/drivers/memory/omap-gpmc.c
> +++ b/drivers/memory/omap-gpmc.c
> @@ -960,7 +960,7 @@ static int gpmc_cs_remap(int cs, u32 base)
> * Make sure we ignore any device offsets from the GPMC partition
> * allocated for the chip select and that the new base confirms
> * to the GPMC 16MB minimum granularity.
> - */
> + */
> base &= ~(SZ_16M - 1);
>
> gpmc_cs_get_memconf(cs, &old_base, &size);
> @@ -1930,8 +1930,8 @@ static int gpmc_probe_onenand_child(struct platform_device *pdev,
> struct omap_onenand_platform_data *gpmc_onenand_data;
>
> if (of_property_read_u32(child, "reg", &val) < 0) {
> - dev_err(&pdev->dev, "%s has no 'reg' property\n",
> - child->full_name);
> + dev_err(&pdev->dev, "%pOF has no 'reg' property\n",
> + child);
> return -ENODEV;
> }
>
> @@ -1979,14 +1979,14 @@ static int gpmc_probe_generic_child(struct platform_device *pdev,
> struct gpmc_device *gpmc = platform_get_drvdata(pdev);
>
> if (of_property_read_u32(child, "reg", &cs) < 0) {
> - dev_err(&pdev->dev, "%s has no 'reg' property\n",
> - child->full_name);
> + dev_err(&pdev->dev, "%pOF has no 'reg' property\n",
> + child);
> return -ENODEV;
> }
>
> if (of_address_to_resource(child, 0, &res) < 0) {
> - dev_err(&pdev->dev, "%s has malformed 'reg' property\n",
> - child->full_name);
> + dev_err(&pdev->dev, "%pOF has malformed 'reg' property\n",
> + child);
> return -ENODEV;
> }
>
> @@ -2084,8 +2084,8 @@ static int gpmc_probe_generic_child(struct platform_device *pdev,
> ret = of_property_read_u32(child, "bank-width",
> &gpmc_s.device_width);
> if (ret < 0) {
> - dev_err(&pdev->dev, "%s has no 'bank-width' property\n",
> - child->full_name);
> + dev_err(&pdev->dev, "%pOF has no 'bank-width' property\n",
> + child);
> goto err;
> }
> }
> --
> 2.11.0
>
--
cheers,
-roger
prev parent reply other threads:[~2017-08-10 7:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-18 21:43 [PATCH] memory: Convert to using %pOF instead of full_name Rob Herring
[not found] ` <20170718214339.7774-34-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-07-19 5:47 ` Alexandre Belloni
[not found] ` <20170719054702.x6h2fkz6vplzyqg2-m++hUPXGwpdeoWH0uzbU5w@public.gmane.org>
2017-08-09 22:22 ` Rob Herring
2017-08-16 19:38 ` Arnd Bergmann
2017-08-10 7:10 ` Roger Quadros [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bf4e23a9-ee21-71c6-af13-adf3fe12b5f4@ti.com \
--to=rogerq@ti.com \
--cc=alexandre.belloni@free-electrons.com \
--cc=arm@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=robh@kernel.org \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).