[{"id":1757993,"web_url":"http://patchwork.ozlabs.org/comment/1757993/","msgid":"","list_archive_url":null,"date":"2017-08-26T13:36:48","subject":"Re: [U-Boot] [PATCH v2 02/10] fdt: Allow stacked overlays phandle\n\treferences","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"On 11 August 2017 at 02:52, Pantelis Antoniou\n wrote:\n> This patch enables an overlay to refer to a previous overlay's\n> labels by performing a merge of symbol information at application\n> time.\n>\n> In a nutshell it allows an overlay to refer to a symbol that a previous\n> overlay has defined. It requires both the base and all the overlays\n> to be compiled with the -@ command line switch so that symbol\n> information is included.\n>\n> base.dts\n> --------\n>\n> /dts-v1/;\n> / {\n> foo: foonode {\n> foo-property;\n> };\n> };\n>\n> $ dtc -@ -I dts -O dtb -o base.dtb base.dts\n>\n> bar.dts\n> -------\n>\n> /dts-v1/;\n> /plugin/;\n> / {\n> fragment@1 {\n> target = <&foo>;\n> __overlay__ {\n> overlay-1-property;\n> bar: barnode {\n> bar-property;\n> };\n> };\n> };\n> };\n>\n> $ dtc -@ -I dts -O dtb -o bar.dtb bar.dts\n>\n> baz.dts\n> -------\n>\n> /dts-v1/;\n> /plugin/;\n> / {\n> fragment@1 {\n> target = <&bar>;\n> __overlay__ {\n> overlay-2-property;\n> baz: baznode {\n> baz-property;\n> };\n> };\n> };\n> };\n>\n> $ dtc -@ -I dts -O dtb -o baz.dtb baz.dts\n>\n> Applying the overlays:\n>\n> $ fdtoverlay -i base.dtb -o target.dtb bar.dtb baz.dtb\n>\n> Dumping:\n>\n> $ fdtdump target.dtb\n> / {\n> foonode {\n> overlay-1-property;\n> foo-property;\n> linux,phandle = <0x00000001>;\n> phandle = <0x00000001>;\n> barnode {\n> overlay-2-property;\n> phandle = <0x00000002>;\n> linux,phandle = <0x00000002>;\n> bar-property;\n> baznode {\n> phandle = <0x00000003>;\n> linux,phandle = <0x00000003>;\n> baz-property;\n> };\n> };\n> };\n> __symbols__ {\n> baz = \"/foonode/barnode/baznode\";\n> bar = \"/foonode/barnode\";\n> foo = \"/foonode\";\n> };\n> };\n>\n> Signed-off-by: Pantelis Antoniou \n> Signed-off-by: David Gibson \n> ---\n> lib/libfdt/fdt_overlay.c | 228 ++++++++++++++++++++++++++++++++++++++++++-----\n> 1 file changed, 206 insertions(+), 22 deletions(-)\n\nReviewed-by: Simon Glass ","headers":{"Return-Path":"","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=)","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=\"Bw1gv2t9\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"eyBDuSxD\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xffHz6cLkz9s8V\n\tfor ;\n\tSat, 26 Aug 2017 23:38:39 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 67F02C22074; Sat, 26 Aug 2017 13:37:28 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 6DCDAC2208A;\n\tSat, 26 Aug 2017 13:37:18 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8355FC22094; Sat, 26 Aug 2017 13:37:14 +0000 (UTC)","from mail-qt0-f181.google.com (mail-qt0-f181.google.com\n\t[209.85.216.181])\n\tby lists.denx.de (Postfix) with ESMTPS id C6C0AC2207D\n\tfor ; Sat, 26 Aug 2017 13:37:10 +0000 (UTC)","by mail-qt0-f181.google.com with SMTP id u11so9825419qtu.1\n\tfor ; Sat, 26 Aug 2017 06:37:10 -0700 (PDT)","by 10.200.52.117 with HTTP; Sat, 26 Aug 2017 06:36:48 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\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=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=3d7atPfVFF+bXWNa76QJY7NNM9LU2ZavNUEzdCMMFQI=;\n\tb=Bw1gv2t9WSCNY5Cme67ujuIn9ffiRCsb5y1fxvRmHFp09we5GfZRiVwEFiPSbz4BN9\n\t9w+YBe1za5CTV8txY+dvUBHAItbrF2h3dZLy8gK+LT6bhpAcMy+xrn6iZXPTwfjdYDHd\n\tm1xb7HztYqLH7wt5Z3GOTc+Kxz66+u6e7eoY9W2i/GuVHdUNcbXnYcsrZlblggUwXdKX\n\tBcyxRbO38WxJ1WBPYz+5tCefsSMigI9yavZxgaycCzEqqkxYVl7viYsaA8y4oTL2WV3l\n\t+rNRE2dGYHPvaks7U/jBh8iu/UsGsO7sQb5iKZZH6lTNghlBsxcrf4rDNjaIqlt1e7Av\n\tvNIQ==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=3d7atPfVFF+bXWNa76QJY7NNM9LU2ZavNUEzdCMMFQI=;\n\tb=eyBDuSxDidvSXXbX4WOe5nwAhklu12DdqetvZiWGOs31J1GiGuCd/1tVLTyzao5KW8\n\t7rv2c5eQ07BVE+ZF4RJ6WQoWH1idMr8yy11qAM53biWCneyVLSWm3Dh0AgMcVXX/epdN\n\tJQnrAEIWQKzhUvya1rnUJX/4ZSYfQ605tCUG8="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=3d7atPfVFF+bXWNa76QJY7NNM9LU2ZavNUEzdCMMFQI=;\n\tb=ZqaEeC25PNN/vSqmCs3R92MGNVN2aCI3dAii5RSU5iNUexJC6Cej+luRayfYzSHGHV\n\tGSUDpHQDWdUbaoVxjp41Gnupio5sWS1I4uXpMwcUWBmjieOVVKTCOoj7BJvX9phpxRqX\n\t1xozlnRXYwnikIAZHJyBW521pPFr7C0Q/LXbRq09skn4B3QOArVc4xQs1uHEp2zriGj6\n\tG/HbRIUzBRPzoiUNpWEtxGrTUhMrDzFE65YXMNJzeWma/1AfKwa9r52+7sLnsVL12PTP\n\taOlBUZvuZ1VZdI2QGbl47Smh9BaTVzS6o1GP+bZMUvtwS8g3multG0XXNf6vzHbOe00k\n\tIUiQ==","X-Gm-Message-State":"AHYfb5jxnZP4LJ7mlkhYhFO+O0hlQOnuq4XX5LU2FflpGX3b4RKogJsZ\n\tc9uYrecrhzbDb2+D6cWV3nn5HWeZrHhR","X-Google-Smtp-Source":"ADKCNb4+cKCnVFC7HS5fCaeg/t/2vHsuxfsgI0cVDUWhSYW/pnW/PCDZhfo1LrNdvGGpTLpcubp+puop7mRADb4h974=","X-Received":"by 10.200.5.133 with SMTP id a5mr2141127qth.38.1503754629529;\n\tSat, 26 Aug 2017 06:37:09 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1502441568-22896-3-git-send-email-pantelis.antoniou@konsulko.com>","References":"<1502441568-22896-1-git-send-email-pantelis.antoniou@konsulko.com>\n\t<1502441568-22896-3-git-send-email-pantelis.antoniou@konsulko.com>","From":"Simon Glass ","Date":"Sat, 26 Aug 2017 07:36:48 -0600","X-Google-Sender-Auth":"4cJDom0BM4FxopIRj_05TkfI9_c","Message-ID":"","To":"Pantelis Antoniou ","Cc":"Marek Vasut , Tom Rini ,\n\tTero Kristo ,\n\tU-Boot Mailing List , \n\tMaxime Ripard ,\n\tStefan Roese , Alan Ott ,\n\tDavid Gibson ","Subject":"Re: [U-Boot] [PATCH v2 02/10] fdt: Allow stacked overlays phandle\n\treferences","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]