Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/807278/?format=api
{ "id": 807278, "url": "http://patchwork.ozlabs.org/api/patches/807278/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/patch/20170829201601.64312-12-sjg@chromium.org/", "project": { "id": 18, "url": "http://patchwork.ozlabs.org/api/projects/18/?format=api", "name": "U-Boot", "link_name": "uboot", "list_id": "u-boot.lists.denx.de", "list_email": "u-boot@lists.denx.de", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170829201601.64312-12-sjg@chromium.org>", "list_archive_url": null, "date": "2017-08-29T20:15:56", "name": "[U-Boot,v2,11/16] dtoc: Rename the phandle struct", "commit_ref": "0d15463c0537806f70ea2359e32e4deb8c4766c2", "pull_url": null, "state": "accepted", "archived": false, "hash": "d62130b39f0379a98d4f40e18572d427cf744620", "submitter": { "id": 6170, "url": "http://patchwork.ozlabs.org/api/people/6170/?format=api", "name": "Simon Glass", "email": "sjg@chromium.org" }, "delegate": { "id": 3184, "url": "http://patchwork.ozlabs.org/api/users/3184/?format=api", "username": "sjg", "first_name": "Simon", "last_name": "Glass", "email": "sjg@chromium.org" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/patch/20170829201601.64312-12-sjg@chromium.org/mbox/", "series": [ { "id": 462, "url": "http://patchwork.ozlabs.org/api/series/462/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=462", "date": "2017-08-29T20:15:45", "name": "dtoc: Add support for 64-bit addresses", "version": 2, "mbox": "http://patchwork.ozlabs.org/series/462/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/807278/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/807278/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<u-boot-bounces@lists.denx.de>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"Qt+5v59X\"; dkim-atps=neutral" ], "Received": [ "from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xhg870766z9sQl\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 06:23:50 +1000 (AEST)", "by lists.denx.de (Postfix, from userid 105)\n\tid 96280C2219B; Tue, 29 Aug 2017 20:20:05 +0000 (UTC)", "from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 2E6AEC225C8;\n\tTue, 29 Aug 2017 20:17:04 +0000 (UTC)", "by lists.denx.de (Postfix, from userid 105)\n\tid 35B89C22643; Tue, 29 Aug 2017 20:16:26 +0000 (UTC)", "from mail-oi0-f51.google.com (mail-oi0-f51.google.com\n\t[209.85.218.51])\n\tby lists.denx.de (Postfix) with ESMTPS id 52C8FC22578\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 20:16:21 +0000 (UTC)", "by mail-oi0-f51.google.com with SMTP id k77so36402008oib.2\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 13:16:21 -0700 (PDT)", "from kiwi.bld.corp.google.com ([100.101.164.89])\n\tby smtp.gmail.com with ESMTPSA id\n\tv128sm3553158oie.1.2017.08.29.13.16.18\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tTue, 29 Aug 2017 13:16:19 -0700 (PDT)", "by kiwi.bld.corp.google.com (Postfix, from userid 121222)\n\tid 2304C13FAB7; Tue, 29 Aug 2017 14:16:18 -0600 (MDT)" ], "X-Spam-Checker-Version": "SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de", "X-Spam-Level": "", "X-Spam-Status": "No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=sender:from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=t79eqeF5lA1VVNjM9H4DnL76Yaj4HkESUcgdl4Y7bJ0=;\n\tb=Qt+5v59XI/iD/Ski/9FVRQcNZaEzs9bGDrQL5ijyeHqcQOYMHmCT8211IgZDXQmruJ\n\t4F+cOYshDBRPajo8KbrDz0/8JDR+ESvoGwxlIiH95T+prLa8lSVwlM+vA/3ibs7OiDxk\n\te6Xh9cISDYEMcpJ7UZT3jNsFHjFUGDlLzOADjIcLm/vlLbQ9NqgaxkjmNTkXii2scXtz\n\t/BCbPCyMjNHkOEeM7zPKRAyFabC3sfI4rfpv0/pOp5CP9kpjJRzx196473bORX8OrDeb\n\tJcxwBSawR88N7rN1v9AATdQ1Z4VD6iks9K2DRfqoJQxYCZ+C1hAcCpFDI4dWfFSYjB/0\n\tvP/Q==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:sender:from:to:cc:subject:date:message-id\n\t:in-reply-to:references;\n\tbh=t79eqeF5lA1VVNjM9H4DnL76Yaj4HkESUcgdl4Y7bJ0=;\n\tb=l/VtCVMhP3ZYMyw3iYWDtDNHIINEF/LVLI4uVHQhlM+2avvS9Xix83tk69MySG4QWB\n\tGmYLBC0j4+EodjoY90GYthaao+wcawl549MDgLT38LbHpsLkQPMpnn31FNs3pnYsaCog\n\tovb3dFBfQ0vbFVgcU7KqvvFgEn/ozXek34aoaU1isTMCfVyW5cOgu/YXnKttwCiA/xIi\n\tO4lhgiED+0ay5MYFwBrDtoE9koQFZ8UMveMQ3T6nkek5XliuEyTKxZuRAknEpILd4OM8\n\tp9MjlY8UcXJwDOeqsd3oAEcdiVxTaj6sVI5DFslspBeJcJPFFlRwlJ+Sp8r25itcFZ6Q\n\tZNrQ==", "X-Gm-Message-State": "AHYfb5iZx0gJdCsAw3AOAq4XX1CNf+G1Dazk2skVNo7Nw5shs0PIAYId\n\tFG8+RyKDaSlFQ7pc", "X-Google-Smtp-Source": "ADKCNb5YkeukDGl60BBFJJZxJ7BP3Z60D6ntEBpZSFAG5N3larS6IbYri9PPBXPKbR7CZOzqMaOs/w==", "X-Received": "by 10.202.199.86 with SMTP id x83mr1655372oif.116.1504037779786; \n\tTue, 29 Aug 2017 13:16:19 -0700 (PDT)", "From": "Simon Glass <sjg@chromium.org>", "To": "U-Boot Mailing List <u-boot@lists.denx.de>", "Date": "Tue, 29 Aug 2017 14:15:56 -0600", "Message-Id": "<20170829201601.64312-12-sjg@chromium.org>", "X-Mailer": "git-send-email 2.14.1.342.g6490525c54-goog", "In-Reply-To": "<20170829201601.64312-1-sjg@chromium.org>", "References": "<20170829201601.64312-1-sjg@chromium.org>", "Cc": "Tom Rini <trini@konsulko.com>", "Subject": "[U-Boot] [PATCH v2 11/16] dtoc: Rename the phandle struct", "X-BeenThere": "u-boot@lists.denx.de", "X-Mailman-Version": "2.1.18", "Precedence": "list", "List-Id": "U-Boot discussion <u-boot.lists.denx.de>", "List-Unsubscribe": "<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>", "List-Archive": "<http://lists.denx.de/pipermail/u-boot/>", "List-Post": "<mailto:u-boot@lists.denx.de>", "List-Help": "<mailto:u-boot-request@lists.denx.de?subject=help>", "List-Subscribe": "<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "base64", "Errors-To": "u-boot-bounces@lists.denx.de", "Sender": "\"U-Boot\" <u-boot-bounces@lists.denx.de>" }, "content": "Rather than naming the phandle struct according to the number of cells it\nuses (e.g. struct phandle_2_cell) name it according to the number of\narguments it has (e.g. struct phandle_1_arg). This is a more intuitive\nnaming.\n\nSigned-off-by: Simon Glass <sjg@chromium.org>\n---\n\nChanges in v2: None\n\n doc/driver-model/of-plat.txt | 2 +-\n drivers/clk/clk-uclass.c | 2 +-\n include/clk.h | 4 ++--\n include/dt-structs.h | 7 ++++++-\n tools/dtoc/dtb_platdata.py | 3 ++-\n tools/dtoc/test_dtoc.py | 2 +-\n 6 files changed, 13 insertions(+), 7 deletions(-)", "diff": "diff --git a/doc/driver-model/of-plat.txt b/doc/driver-model/of-plat.txt\nindex 3ed8c759d6..732bc34f06 100644\n--- a/doc/driver-model/of-plat.txt\n+++ b/doc/driver-model/of-plat.txt\n@@ -111,7 +111,7 @@ struct dtd_rockchip_rk3288_dw_mshc {\n bool cap_sd_highspeed;\n fdt32_t card_detect_delay;\n fdt32_t clock_freq_min_max[2];\n- struct phandle_2_cell clocks[4];\n+ struct phandle_1_arg clocks[4];\n bool disable_wp;\n fdt32_t fifo_depth;\n fdt32_t interrupts[3];\ndiff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c\nindex e68d9279b9..8b40326efa 100644\n--- a/drivers/clk/clk-uclass.c\n+++ b/drivers/clk/clk-uclass.c\n@@ -23,7 +23,7 @@ static inline struct clk_ops *clk_dev_ops(struct udevice *dev)\n #if CONFIG_IS_ENABLED(OF_CONTROL)\n # if CONFIG_IS_ENABLED(OF_PLATDATA)\n int clk_get_by_index_platdata(struct udevice *dev, int index,\n-\t\t\t struct phandle_2_cell *cells, struct clk *clk)\n+\t\t\t struct phandle_1_arg *cells, struct clk *clk)\n {\n \tint ret;\n \ndiff --git a/include/clk.h b/include/clk.h\nindex c5988f78a8..e7ce3e8576 100644\n--- a/include/clk.h\n+++ b/include/clk.h\n@@ -61,9 +61,9 @@ struct clk {\n };\n \n #if CONFIG_IS_ENABLED(OF_CONTROL) && CONFIG_IS_ENABLED(CLK)\n-struct phandle_2_cell;\n+struct phandle_1_arg;\n int clk_get_by_index_platdata(struct udevice *dev, int index,\n-\t\t\t struct phandle_2_cell *cells, struct clk *clk);\n+\t\t\t struct phandle_1_arg *cells, struct clk *clk);\n \n /**\n * clock_get_by_index - Get/request a clock by integer index.\ndiff --git a/include/dt-structs.h b/include/dt-structs.h\nindex 0732c442ff..2ed997115a 100644\n--- a/include/dt-structs.h\n+++ b/include/dt-structs.h\n@@ -9,7 +9,12 @@\n \n /* These structures may only be used in SPL */\n #if CONFIG_IS_ENABLED(OF_PLATDATA)\n-struct phandle_2_cell {\n+struct phandle_0_arg {\n+\tconst void *node;\n+\tint id[0];\n+};\n+\n+struct phandle_1_arg {\n \tconst void *node;\n \tint id;\n };\ndiff --git a/tools/dtoc/dtb_platdata.py b/tools/dtoc/dtb_platdata.py\nindex 001bc4ea66..0234f71b76 100644\n--- a/tools/dtoc/dtb_platdata.py\n+++ b/tools/dtoc/dtb_platdata.py\n@@ -419,7 +419,8 @@ class DtbPlatdata(object):\n info = self.get_phandle_argc(prop, structs[name])\n if info:\n # For phandles, include a reference to the target\n- self.out('\\t%s%s[%d]' % (tab_to(2, 'struct phandle_2_cell'),\n+ struct_name = 'struct phandle_%d_arg' % info.max_args\n+ self.out('\\t%s%s[%d]' % (tab_to(2, struct_name),\n conv_name_to_c(prop.name),\n len(prop.value) / 2))\n else:\ndiff --git a/tools/dtoc/test_dtoc.py b/tools/dtoc/test_dtoc.py\nindex 62460acb7c..23c4439ed9 100644\n--- a/tools/dtoc/test_dtoc.py\n+++ b/tools/dtoc/test_dtoc.py\n@@ -228,7 +228,7 @@ U_BOOT_DEVICE(pmic_at_9) = {\n self.assertEqual('''#include <stdbool.h>\n #include <libfdt.h>\n struct dtd_source {\n-\\tstruct phandle_2_cell clocks[1];\n+\\tstruct phandle_1_arg clocks[1];\n };\n struct dtd_target {\n \\tfdt32_t\\t\\tintval;\n", "prefixes": [ "U-Boot", "v2", "11/16" ] }