{"id":790626,"url":"http://patchwork.ozlabs.org/api/patches/790626/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20170718214339.7774-32-robh@kernel.org/","project":{"id":2,"url":"http://patchwork.ozlabs.org/api/projects/2/?format=json","name":"Linux PPC development","link_name":"linuxppc-dev","list_id":"linuxppc-dev.lists.ozlabs.org","list_email":"linuxppc-dev@lists.ozlabs.org","web_url":"https://github.com/linuxppc/wiki/wiki","scm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git","webscm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/","list_archive_url":"https://lore.kernel.org/linuxppc-dev/","list_archive_url_format":"https://lore.kernel.org/linuxppc-dev/{}/","commit_url_format":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id={}"},"msgid":"<20170718214339.7774-32-robh@kernel.org>","list_archive_url":"https://lore.kernel.org/linuxppc-dev/20170718214339.7774-32-robh@kernel.org/","date":"2017-07-18T21:43:12","name":"macintosh: Convert to using %pOF instead of full_name","commit_ref":"b6a945ae03fd3962b51b27ecedf4f1dd7e034229","pull_url":null,"state":"accepted","archived":false,"hash":"1e62505bb0145df6daefd9e56217bc27b37924fc","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/?format=json","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20170718214339.7774-32-robh@kernel.org/mbox/","series":[],"comments":"http://patchwork.ozlabs.org/api/patches/790626/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/790626/checks/","tags":{},"related":[],"headers":{"Return-Path":"<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xBvD16968z9sRg\n\tfor <patchwork-incoming@ozlabs.org>;\n\tWed, 19 Jul 2017 07:57:53 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xBvD15KgyzDrLk\n\tfor <patchwork-incoming@ozlabs.org>;\n\tWed, 19 Jul 2017 07:57:53 +1000 (AEST)","from mail-pf0-f193.google.com (mail-pf0-f193.google.com\n\t[209.85.192.193])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xBtwj62K3zDrJx\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tWed, 19 Jul 2017 07:44:37 +1000 (AEST)","by mail-pf0-f193.google.com with SMTP id o88so4065056pfk.1\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tTue, 18 Jul 2017 14:44:37 -0700 (PDT)","from localhost.localdomain (24-223-123-72.static.usa-companies.net.\n\t[24.223.123.72]) by smtp.googlemail.com with ESMTPSA id\n\tr207sm7186560pfr.106.2017.07.18.14.44.35\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tTue, 18 Jul 2017 14:44:35 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=H2KfYNxL/n/STjRAZCXNnh4AOhyrZnHd/+k5TbnwZB0=;\n\tb=cQLVbcSQHtQsfwbF2a5u0GY+uwepwZ9Ov84gOm0Kk5vBm6xkDDCdEp0Oziz1u1sWen\n\tQI3VONWW4N5bh7J8MZEJF5MflPfQKwK+LMBggacuo5yiQ7+T1L/DBuRTkT84N3kVJy5a\n\t726np/7fFMNJr1gOs8Yjw25q9+jr4hKTZws/vKnPPkrHsQneHEw0e4PQs0UacI9KQgfP\n\tbqMS75xGZLyk4i8eeeM3R6wOI7UsyZ62Jn80oGGggQeGxutDZ77QR1Fvk22drjRF4rI3\n\tX6uX6b7sJgbrXGbt2JBW5wSBLurhXpgLp9+6pu3VrBD/9xOM3/qpoU9MHBHk/y+TBEtA\n\t/Y4Q==","X-Gm-Message-State":"AIVw113wCnw3JJz+PnnQ4oR3xWixehqbS5/deMZmhgecFO7GWz3iPfOm\n\tNeMBoljnejuFrA==","X-Received":"by 10.98.110.65 with SMTP id j62mr3741000pfc.115.1500414276197; \n\tTue, 18 Jul 2017 14:44:36 -0700 (PDT)","From":"Rob Herring <robh@kernel.org>","To":"Benjamin Herrenschmidt <benh@kernel.crashing.org>","Subject":"[PATCH] macintosh: Convert to using %pOF instead of full_name","Date":"Tue, 18 Jul 2017 16:43:12 -0500","Message-Id":"<20170718214339.7774-32-robh@kernel.org>","X-Mailer":"git-send-email 2.11.0","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t<linuxppc-dev.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/linuxppc-dev/>","List-Post":"<mailto:linuxppc-dev@lists.ozlabs.org>","List-Help":"<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,\n\tlinux-kernel@vger.kernel.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>"},"content":"Now that we have a custom printf format specifier, convert users of\nfull_name to use %pOF instead. This is preparation to remove storing\nof the full path string for each node.\n\nSigned-off-by: Rob Herring <robh@kernel.org>\nCc: Benjamin Herrenschmidt <benh@kernel.crashing.org>\nCc: linuxppc-dev@lists.ozlabs.org\n---\n drivers/macintosh/macio_sysfs.c           | 2 +-\n drivers/macintosh/rack-meter.c            | 4 ++--\n drivers/macintosh/smu.c                   | 8 ++++----\n drivers/macintosh/via-cuda.c              | 4 ++--\n drivers/macintosh/windfarm_fcu_controls.c | 4 ++--\n drivers/macintosh/windfarm_lm87_sensor.c  | 4 ++--\n drivers/macintosh/windfarm_smu_sat.c      | 2 +-\n 7 files changed, 14 insertions(+), 14 deletions(-)\n\n--\n2.11.0","diff":"diff --git a/drivers/macintosh/macio_sysfs.c b/drivers/macintosh/macio_sysfs.c\nindex 2445274f7e4b..281f5345661e 100644\n--- a/drivers/macintosh/macio_sysfs.c\n+++ b/drivers/macintosh/macio_sysfs.c\n@@ -52,7 +52,7 @@ static ssize_t devspec_show(struct device *dev,\n \tstruct platform_device *ofdev;\n\n \tofdev = to_platform_device(dev);\n-\treturn sprintf(buf, \"%s\\n\", ofdev->dev.of_node->full_name);\n+\treturn sprintf(buf, \"%pOF\\n\", ofdev->dev.of_node);\n }\n static DEVICE_ATTR_RO(modalias);\n static DEVICE_ATTR_RO(devspec);\ndiff --git a/drivers/macintosh/rack-meter.c b/drivers/macintosh/rack-meter.c\nindex e199fd6c71ce..e9da80ceb7c8 100644\n--- a/drivers/macintosh/rack-meter.c\n+++ b/drivers/macintosh/rack-meter.c\n@@ -431,8 +431,8 @@ static int rackmeter_probe(struct macio_dev* mdev,\n \t    of_address_to_resource(i2s, 0, &ri2s) ||\n \t    of_address_to_resource(i2s, 1, &rdma)) {\n \t\tprintk(KERN_ERR\n-\t\t       \"rackmeter: found match but lacks resources: %s\",\n-\t\t       mdev->ofdev.dev.of_node->full_name);\n+\t\t       \"rackmeter: found match but lacks resources: %pOF\",\n+\t\t       mdev->ofdev.dev.of_node);\n \t\trc = -ENXIO;\n \t\tgoto bail_free;\n \t}\ndiff --git a/drivers/macintosh/smu.c b/drivers/macintosh/smu.c\nindex 1ac66421877a..ea9bdc85a21d 100644\n--- a/drivers/macintosh/smu.c\n+++ b/drivers/macintosh/smu.c\n@@ -589,14 +589,14 @@ static int smu_late_init(void)\n \tif (smu->db_node) {\n \t\tsmu->db_irq = irq_of_parse_and_map(smu->db_node, 0);\n \t\tif (!smu->db_irq)\n-\t\t\tprintk(KERN_ERR \"smu: failed to map irq for node %s\\n\",\n-\t\t\t       smu->db_node->full_name);\n+\t\t\tprintk(KERN_ERR \"smu: failed to map irq for node %pOF\\n\",\n+\t\t\t       smu->db_node);\n \t}\n \tif (smu->msg_node) {\n \t\tsmu->msg_irq = irq_of_parse_and_map(smu->msg_node, 0);\n \t\tif (!smu->msg_irq)\n-\t\t\tprintk(KERN_ERR \"smu: failed to map irq for node %s\\n\",\n-\t\t\t       smu->msg_node->full_name);\n+\t\t\tprintk(KERN_ERR \"smu: failed to map irq for node %pOF\\n\",\n+\t\t\t       smu->msg_node);\n \t}\n\n \t/*\ndiff --git a/drivers/macintosh/via-cuda.c b/drivers/macintosh/via-cuda.c\nindex c60415958dfe..147da4edd021 100644\n--- a/drivers/macintosh/via-cuda.c\n+++ b/drivers/macintosh/via-cuda.c\n@@ -297,8 +297,8 @@ static int __init via_cuda_start(void)\n #else\n     cuda_irq = irq_of_parse_and_map(vias, 0);\n     if (!cuda_irq) {\n-\tprintk(KERN_ERR \"via-cuda: can't map interrupts for %s\\n\",\n-\t       vias->full_name);\n+\tprintk(KERN_ERR \"via-cuda: can't map interrupts for %pOF\\n\",\n+\t       vias);\n \treturn -ENODEV;\n     }\n #endif\ndiff --git a/drivers/macintosh/windfarm_fcu_controls.c b/drivers/macintosh/windfarm_fcu_controls.c\nindex 0226b796a21c..fab7a21e9577 100644\n--- a/drivers/macintosh/windfarm_fcu_controls.c\n+++ b/drivers/macintosh/windfarm_fcu_controls.c\n@@ -470,8 +470,8 @@ static void wf_fcu_lookup_fans(struct wf_fcu_priv *pv)\n \t\t\t\tid = ((*reg) - 0x30) / 2;\n \t\t\tif (id > 7) {\n \t\t\t\tpr_warning(\"wf_fcu: Can't parse \"\n-\t\t\t\t       \"fan ID in device-tree for %s\\n\",\n-\t\t\t\t\t   np->full_name);\n+\t\t\t\t       \"fan ID in device-tree for %pOF\\n\",\n+\t\t\t\t\t   np);\n \t\t\t\tbreak;\n \t\t\t}\n \t\t\twf_fcu_add_fan(pv, name, type, id);\ndiff --git a/drivers/macintosh/windfarm_lm87_sensor.c b/drivers/macintosh/windfarm_lm87_sensor.c\nindex c071aab79dd1..913c4bfeef94 100644\n--- a/drivers/macintosh/windfarm_lm87_sensor.c\n+++ b/drivers/macintosh/windfarm_lm87_sensor.c\n@@ -126,8 +126,8 @@ static int wf_lm87_probe(struct i2c_client *client,\n \t\t}\n \t}\n \tif (!name) {\n-\t\tpr_warning(\"wf_lm87: Unsupported sensor %s\\n\",\n-\t\t\t   client->dev.of_node->full_name);\n+\t\tpr_warning(\"wf_lm87: Unsupported sensor %pOF\\n\",\n+\t\t\t   client->dev.of_node);\n \t\treturn -ENODEV;\n \t}\n\ndiff --git a/drivers/macintosh/windfarm_smu_sat.c b/drivers/macintosh/windfarm_smu_sat.c\nindex ad6223e88340..e9c828bf171b 100644\n--- a/drivers/macintosh/windfarm_smu_sat.c\n+++ b/drivers/macintosh/windfarm_smu_sat.c\n@@ -248,7 +248,7 @@ static int wf_sat_probe(struct i2c_client *client,\n \t\tcore = loc[5] - '0';\n \t\tif (chip > 1 || core > 1) {\n \t\t\tprintk(KERN_ERR \"wf_sat_create: don't understand \"\n-\t\t\t       \"location %s for %s\\n\", loc, child->full_name);\n+\t\t\t       \"location %s for %pOF\\n\", loc, child);\n \t\t\tcontinue;\n \t\t}\n \t\tcpu = 2 * chip + core;\n","prefixes":[]}