[{"id":1765720,"web_url":"http://patchwork.ozlabs.org/comment/1765720/","msgid":"<CAPnjgZ13L7+D2KS7=pHVnpt8Fw4q6HhWxNKxkWmAOoM03CaP8g@mail.gmail.com>","list_archive_url":null,"date":"2017-09-09T04:53:40","subject":"Re: [U-Boot] [PATCH v3 04/14] 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 4 September 2017 at 14:12, Pantelis Antoniou\n<pantelis.antoniou@konsulko.com> 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 <pantelis.antoniou@konsulko.com>\n> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>\n> ---\n>  lib/libfdt/fdt_overlay.c | 228 ++++++++++++++++++++++++++++++++++++++++++-----\n>  1 file changed, 206 insertions(+), 22 deletions(-)\n>\n\nAcked-by: Simon Glass <sjg@chromium.org>","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=\"KhI5ohkR\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"N6dFBaX/\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xq2DX4rn7z9sBZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat,  9 Sep 2017 15:04:44 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 5EFD9C21F6A; Sat,  9 Sep 2017 04:58:16 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id AA621C21F0F;\n\tSat,  9 Sep 2017 04:56:33 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 3DFC4C21E4A; Sat,  9 Sep 2017 04:56:29 +0000 (UTC)","from mail-qt0-f177.google.com (mail-qt0-f177.google.com\n\t[209.85.216.177])\n\tby lists.denx.de (Postfix) with ESMTPS id 690BCC21EC1\n\tfor <u-boot@lists.denx.de>; Sat,  9 Sep 2017 04:54:02 +0000 (UTC)","by mail-qt0-f177.google.com with SMTP id s18so8802488qta.3\n\tfor <u-boot@lists.denx.de>; Fri, 08 Sep 2017 21:54:02 -0700 (PDT)","by 10.200.37.200 with HTTP; Fri, 8 Sep 2017 21:53:40 -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.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=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=ZzT8Fag9fcq31lx4xblYNiSLqqJV9oGLBxuVVwl5JP4=;\n\tb=KhI5ohkR0Wl57hejfuwoz8qRLoXO6Ri1XyFp0kwD9DKZDaqbrTGRVoSeolLHCfunec\n\tQFsWiD31USxZqcFtyu6pKjhOh6WDIxwVQ1ZfihdUkYprdFegXJ46n7GMUs9fOIR3cTcC\n\t3BoJp1AfjTs2I57/3XBBdQi7+JHI4W54K1WllwbEgpCqFI90wHjwD5YM/fv0vfIPffM7\n\tTXrrHv27AQqFnodyJvXfQcsExDUA4TibarCv/9CqXqrQyNpte+4ulEhA7VxM9EvwkVG6\n\tI65NCjKRt2eeM470plLTHoaHX07qiJKaLx8HIAC84c5tiv0rc7d+YCaPovs2D/VOnOVR\n\tRXYg==","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=ZzT8Fag9fcq31lx4xblYNiSLqqJV9oGLBxuVVwl5JP4=;\n\tb=N6dFBaX/6ILaizvxeoNFk+0vaNpWNW4oMs8amfFepqe43nRLPMRGy2LTRCrM6CqAmu\n\tgF681G7Qm1RJuMLWNPDl9a0lj6/PHOlBeIf7QiCZNWhLMb1El8WcXckFURk6CDdwLTWM\n\tqEycJBX8te9e2GMiM17WpP5boiN3FpvJUCjGU="],"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=ZzT8Fag9fcq31lx4xblYNiSLqqJV9oGLBxuVVwl5JP4=;\n\tb=Lak3njwlDOhj1IH4P4Q2IJxPBViB45/6FYlMiiL9YpXtjks1/Bh/1HBKLRnNOrzpK7\n\tMIsvvUgVtycMidHIAhr23A7ETiJmk81/Qu4RSWWchxngblloTXddEAU7olyeLqNGIf3u\n\tkAPJcIVyGHi+Eyn6IMSsg+oDD4bqh5cPj21g0G0S7I7xoh3abKNNBntXMvNQljq0XKLv\n\tv12sxBACL4LxfO4z8KmNXLLf2j4V2P5vf3+Q5v+DmwGIaG/EzOdWwDQRWvyxncyQFlIJ\n\tPfnMnPl2Dw+R/ktk/l1p+zthuMJUbC4wmKd3W6empdmZVj6RFAiWEAC9id7Qkn8VoJQr\n\tdYww==","X-Gm-Message-State":"AHPjjUhh4BpgS3t2N1NFMWVtoYdsH/6Be1Lu4FQtgYwiijWcTpTpJgQx\n\t+0yBY+k8cRyWs0mdjtljgFgSESbnhVqpm+CyYprgzg==","X-Google-Smtp-Source":"AOwi7QAakjwrfcB5ODWL7O+8760B+o7U1A6U+NmROKvJQCTKN0m4GKO6cyA/mXr/jaDC/UmgR2tELTOx15THq7Q3wrg=","X-Received":"by 10.200.5.132 with SMTP id a4mr7515382qth.318.1504932841148;\n\tFri, 08 Sep 2017 21:54:01 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1504555943-12893-5-git-send-email-pantelis.antoniou@konsulko.com>","References":"<1504555943-12893-1-git-send-email-pantelis.antoniou@konsulko.com>\n\t<1504555943-12893-5-git-send-email-pantelis.antoniou@konsulko.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Fri, 8 Sep 2017 22:53:40 -0600","X-Google-Sender-Auth":"kvXDsQcWBiTISBNiSohKQrT3Uqk","Message-ID":"<CAPnjgZ13L7+D2KS7=pHVnpt8Fw4q6HhWxNKxkWmAOoM03CaP8g@mail.gmail.com>","To":"Pantelis Antoniou <pantelis.antoniou@konsulko.com>","Cc":"Marek Vasut <marex@denx.de>, Tom Rini <trini@konsulko.com>,\n\tTero Kristo <t-kristo@ti.com>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>, \n\tMaxime Ripard <maxime.ripard@free-electrons.com>,\n\tStefan Roese <sr@denx.de>, Alan Ott <alan@signal11.us>,\n\tDavid Gibson <david@gibson.dropbear.id.au>","Subject":"Re: [U-Boot] [PATCH v3 04/14] 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 <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>","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>"}},{"id":1769385,"web_url":"http://patchwork.ozlabs.org/comment/1769385/","msgid":"<CAPnjgZ04coCv43sLEj6GghB5EqCt_GFyNsmt1XhsngbcRw6mbw@mail.gmail.com>","list_archive_url":null,"date":"2017-09-15T19:19:50","subject":"Re: [U-Boot] [PATCH v3 04/14] fdt: Allow stacked overlays phandle\n\treferences","submitter":{"id":12501,"url":"http://patchwork.ozlabs.org/api/people/12501/","name":"Simon Glass","email":"sjg@google.com"},"content":"On 4 September 2017 at 14:12, Pantelis Antoniou\n<pantelis.antoniou@konsulko.com> 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 <pantelis.antoniou@konsulko.com>\n> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>\n> ---\n>  lib/libfdt/fdt_overlay.c | 228 ++++++++++++++++++++++++++++++++++++++++++-----\n>  1 file changed, 206 insertions(+), 22 deletions(-)\n>\n\nAcked-by: Simon Glass <sjg@chromium.org>\n\nApplied to u-boot-fdt thanks!","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=\"mHK/zHgj\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xv58f1b5bz9s7g\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat, 16 Sep 2017 05:30:26 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid DD277C22103; Fri, 15 Sep 2017 19:22:06 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 1BBDAC21F32;\n\tFri, 15 Sep 2017 19:20:44 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 72506C21EF9; Fri, 15 Sep 2017 19:20:02 +0000 (UTC)","from mail-io0-f175.google.com (mail-io0-f175.google.com\n\t[209.85.223.175])\n\tby lists.denx.de (Postfix) with ESMTPS id E4264C21F5F\n\tfor <u-boot@lists.denx.de>; Fri, 15 Sep 2017 19:19:58 +0000 (UTC)","by mail-io0-f175.google.com with SMTP id k101so10892477iod.0\n\tfor <u-boot@lists.denx.de>; Fri, 15 Sep 2017 12:19:58 -0700 (PDT)","from 480794996271 named unknown by gmailapi.google.com with\n\tHTTPREST; Fri, 15 Sep 2017 21:19:51 +0200"],"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_H2, T_DKIM_INVALID,\n\tUNPARSEABLE_RELAY 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:from:in-reply-to:references:date:message-id\n\t:subject:to:cc;\n\tbh=ob+XunRRUMIqReUxa0yGxpkjvl+OsLFPHPWCvPuLHmg=;\n\tb=mHK/zHgjpKdv7uOxaso2/EqRIYzL3VelQxUj4q0+xX/2jYP9JoJ+dcfrhYS2tAlUBf\n\tzkgttdZK2TbZxgbE6F+BE7EWJ/Wgw4IGnsZhcmNWMoHkaFN272mNv+q6HGqDm7Dxzn8O\n\t3yAiMyGlHQKvJjRPuJBDCSTTLxS5f91Znf44b0IF7/yQIe233YmbB1n5VpWhhasux4Dp\n\t6x4fCfpl3ju+NwNthqbaeFumhFTrzOGP/eHRCWewxmcsblhKhxGJMXmpI3bEZnVHYRKZ\n\tztnmZjgn6F9EwPIhWkPYwu4HgGaFy7pb5xzTGdcHkaBlpr+8JgFyKE7OsNP2Uew3SJCH\n\tky0w==","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:from:in-reply-to:references\n\t:date:message-id:subject:to:cc;\n\tbh=ob+XunRRUMIqReUxa0yGxpkjvl+OsLFPHPWCvPuLHmg=;\n\tb=qR9K+N3ZbG5arsuNHjXicHK5CNeaA3+miKdTdFLQ0dfWzJtyqCWm5IEQBldsmpjzb9\n\t5ZanRREJfwSH2Agw/Sp9m9uctRk+Gq3etiXNnF3xUKvjJloDYkeiY6N3tRkalnmnu2x1\n\t1OLhKkhmCNQPOYWSQ3bzD53RtLiW6qsnte2auAO4gclGBbOdpX5iLxL2BaqDti8kU0IF\n\txJnfUCAZmCsT2DW7SAwTqNjpCbUX38nTf/zy+QlY0t/0mzfBKGrOVL2UVDfpzWxqrJX+\n\tofshH8AfnWKmM50zUclML8JjB7KqNTSwOOFB+KZbEooawmDNnVk8NpQqTk+DG42HsFEn\n\tXfFA==","X-Gm-Message-State":"AHPjjUgdd3gowM84VMZool618y5HLIYgeS+7srFVJbXXkF9v119HCi02\n\t2OogoiqyA6hK32/QUzXWjFyZhLimdY9uvxolH24CzQ==","X-Google-Smtp-Source":"AOwi7QDuGYrnScetEP25qufKRnSRSdz259XAb7tdHqNwmqTkO1mF0EujZaywdyTiQ+esM7YJ5rZZBnVeceUldAL8lBw=","X-Received":"by 10.202.170.204 with SMTP id\n\tt195mr13961044oie.277.1505503196932; \n\tFri, 15 Sep 2017 12:19:56 -0700 (PDT)","MIME-Version":"1.0","From":"sjg@google.com","In-Reply-To":"<CAPnjgZ13L7+D2KS7=pHVnpt8Fw4q6HhWxNKxkWmAOoM03CaP8g@mail.gmail.com>","References":"<CAPnjgZ13L7+D2KS7=pHVnpt8Fw4q6HhWxNKxkWmAOoM03CaP8g@mail.gmail.com>\n\t<1504555943-12893-1-git-send-email-pantelis.antoniou@konsulko.com>\n\t<1504555943-12893-5-git-send-email-pantelis.antoniou@konsulko.com>","Date":"Fri, 15 Sep 2017 21:19:50 +0200","X-Google-Sender-Auth":"L1EzIHXBxgK02w1g0hlJcU3k_QY","Message-ID":"<CAPnjgZ04coCv43sLEj6GghB5EqCt_GFyNsmt1XhsngbcRw6mbw@mail.gmail.com>","To":"Simon Glass <sjg@chromium.org>","Cc":"Marek Vasut <marex@denx.de>, Tom Rini <trini@konsulko.com>,\n\tPantelis Antoniou <pantelis.antoniou@konsulko.com>,\n\tTero Kristo <t-kristo@ti.com>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>, \n\tMaxime Ripard <maxime.ripard@free-electrons.com>,\n\tStefan Roese <sr@denx.de>, Alan Ott <alan@signal11.us>,\n\tDavid Gibson <david@gibson.dropbear.id.au>","Subject":"Re: [U-Boot] [PATCH v3 04/14] 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 <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>","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>"}}]