[{"id":1759672,"web_url":"http://patchwork.ozlabs.org/comment/1759672/","msgid":"<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>","list_archive_url":null,"date":"2017-08-29T20:31:38","subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"Simon,\n\nI should be able to give this a test drive on the RK3368 by the end of the week.\nFor the RK3399 our board uses full OF_CONTROL, so I won’t really exercise\nthis there.\n\nGiven that the RK3399 can always fall back to OF_CONTROL for SPL (as is the\ndefault on Puma) and this only blocks the RK3368 TPL (which is not enabled\nfor any other boards than Lion): should we try to get this merged for this release\ncycle or let it slip into the new merge window? \n\nRegards,\nPhilipp.\n\n> On 29 Aug 2017, at 22:15, Simon Glass <sjg@chromium.org> wrote:\n> \n> This series updates dtoc to support 64-bit addresses automatically. These\n> appear in C code as fdt64_t arrays:\n> \n> struct dtd_test1 {\n> \tfdt64_t\t\treg[2];\n> \n> };\n> \n> static struct dtd_test1 dtv_test1 = {\n> \t.reg\t\t= {0x123400000000, 0x5678},\n> };\n> \n> C code can then process these address and size parents easily. This\n> feature is controlled by the #address-cells and #size-cells values of the\n> parent.\n> \n> The v2 series also enhances phandle support so that it is now possible to\n> support phandles with different number of arguments. Several clean-up\n> patches are included also.\n> \n> Changes in v2:\n> - Support 'reg' properties with a single cell (e.g. #size-cells = 0)\n> - Introduce an fdt_val_t type which is either 32- or 64-bits long\n> - Update rk3368 and rk3399 uses\n> - Drop review tags since there are significant changes in this patch\n> \n> Simon Glass (16):\n>  fdt: Sync libfdt up to upstream\n>  dtoc: Adjust Node to record its parent\n>  dtoc: Add a 64-bit type and a way to convert cells into 64 bits\n>  dtoc: Avoid very long lines in output\n>  dtoc: Add support for 32 or 64-bit addresses\n>  dtoc: Handle 'reg' properties with unusual sizes\n>  dtoc: Update the Fdt class to record phandles\n>  dtoc: Use the Fdt's class's phandle map\n>  dtoc: Make is_phandle() a member function\n>  dtoc: Rename is_phandle() and adjust it to return more detail\n>  dtoc: Rename the phandle struct\n>  dtoc: Put each phandle on a separate line\n>  dtoc: Put phandle args in an array\n>  dtoc: Support properties containing multiple phandle values\n>  dtoc: Rename the auto-generated dt-structs.h file\n>  dtoc: Add a header to the generated files\n> \n> doc/driver-model/of-plat.txt       |   2 +-\n> drivers/clk/clk-uclass.c           |   4 +-\n> drivers/clk/rockchip/clk_rk3368.c  |   2 +-\n> drivers/clk/rockchip/clk_rk3399.c  |   4 +-\n> drivers/core/regmap.c              |   2 +-\n> include/clk.h                      |   4 +-\n> include/dt-structs.h               |  16 ++-\n> include/fdtdec.h                   |   2 +\n> include/regmap.h                   |   2 +-\n> include/syscon.h                   |   6 +-\n> lib/libfdt/fdt_rw.c                |  20 ++-\n> lib/libfdt/libfdt.h                |  31 +++++\n> lib/libfdt/pylibfdt/libfdt.i       |  58 ++++++++\n> scripts/Makefile.spl               |   5 +-\n> tools/dtoc/dtb_platdata.py         | 204 ++++++++++++++++++++++------\n> tools/dtoc/dtoc_test_addr32.dts    |  27 ++++\n> tools/dtoc/dtoc_test_addr32_64.dts |  33 +++++\n> tools/dtoc/dtoc_test_addr64.dts    |  33 +++++\n> tools/dtoc/dtoc_test_addr64_32.dts |  33 +++++\n> tools/dtoc/dtoc_test_phandle.dts   |  16 ++-\n> tools/dtoc/dtoc_test_simple.dts    |  14 ++\n> tools/dtoc/fdt.py                  |  19 ++-\n> tools/dtoc/fdt_util.py             |  16 +++\n> tools/dtoc/test_dtoc.py            | 270 ++++++++++++++++++++++++++++++++++++-\n> 24 files changed, 748 insertions(+), 75 deletions(-)\n> create mode 100644 tools/dtoc/dtoc_test_addr32.dts\n> create mode 100644 tools/dtoc/dtoc_test_addr32_64.dts\n> create mode 100644 tools/dtoc/dtoc_test_addr64.dts\n> create mode 100644 tools/dtoc/dtoc_test_addr64_32.dts\n> \n> -- \n> 2.14.1.342.g6490525c54-goog\n>","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>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xhgV60R2Dz9s9Y\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 06:39:25 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 9B974C226B4; Tue, 29 Aug 2017 20:37:41 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 3F715C2262E;\n\tTue, 29 Aug 2017 20:37:39 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D4C50C21E92; Tue, 29 Aug 2017 20:31:42 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id 268C1C22385\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 20:31:42 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:60613\n\thelo=[192.168.2.129]) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1dmnAh-0004AJ-Pt; Tue, 29 Aug 2017 22:31:39 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"*","X-Spam-Status":"No, score=1.0 required=5.0 tests=HK_NAME_DR autolearn=no\n\tautolearn_force=no version=3.4.0","Mime-Version":"1.0 (Mac OS X Mail 8.2 \\(2104\\))","From":"\"Dr. Philipp Tomsich\" <philipp.tomsich@theobroma-systems.com>","In-Reply-To":"<20170829201601.64312-1-sjg@chromium.org>","Date":"Tue, 29 Aug 2017 22:31:38 +0200","Message-Id":"<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>","References":"<20170829201601.64312-1-sjg@chromium.org>","To":"Simon Glass <sjg@chromium.org>","X-Mailer":"Apple Mail (2.2104)","Cc":"U-Boot Mailing List <u-boot@lists.denx.de>,\n\tJerry Van Baren <vanbaren@cideas.com>, Tom Rini <trini@konsulko.com>","Subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","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":1759997,"web_url":"http://patchwork.ozlabs.org/comment/1759997/","msgid":"<CAPnjgZ3gFp2kcwADtmL=k5TU2OaWnGzJyLTMEe46wQAGd+jn0g@mail.gmail.com>","list_archive_url":null,"date":"2017-08-30T09:37:06","subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Philipp,\n\nOn 30 August 2017 at 04:31, Dr. Philipp Tomsich\n<philipp.tomsich@theobroma-systems.com> wrote:\n> Simon,\n>\n> I should be able to give this a test drive on the RK3368 by the end of the week.\n> For the RK3399 our board uses full OF_CONTROL, so I won’t really exercise\n> this there.\n>\n> Given that the RK3399 can always fall back to OF_CONTROL for SPL (as is the\n> default on Puma) and this only blocks the RK3368 TPL (which is not enabled\n> for any other boards than Lion): should we try to get this merged for this release\n> cycle or let it slip into the new merge window?\n\nMy preference would be to put it in a next branch, but if nothing else\nuses it then I suppose it is possible (up to you and Tom)? What is\nbroken that this fixes?\n\n>\n> Regards,\n> Philipp.\n>\nRegards,\nSimon","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=\"bmawCnwu\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"HxgDfgGy\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xj0mv39bbz9sN7\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 19:38:23 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 25EFEC22816; Wed, 30 Aug 2017 09:37:44 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 57602C22814;\n\tWed, 30 Aug 2017 09:37:42 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 3EC01C22817; Wed, 30 Aug 2017 09:37:33 +0000 (UTC)","from mail-qk0-f180.google.com (mail-qk0-f180.google.com\n\t[209.85.220.180])\n\tby lists.denx.de (Postfix) with ESMTPS id DA286C22814\n\tfor <u-boot@lists.denx.de>; Wed, 30 Aug 2017 09:37:27 +0000 (UTC)","by mail-qk0-f180.google.com with SMTP id p67so25619780qkd.2\n\tfor <u-boot@lists.denx.de>; Wed, 30 Aug 2017 02:37:27 -0700 (PDT)","by 10.200.28.106 with HTTP; Wed, 30 Aug 2017 02:37:06 -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:content-transfer-encoding;\n\tbh=rKORtiwSd5AqXd9jQhrZyepiY3goSV6uFU9pgYrKuYs=;\n\tb=bmawCnwuQqUc5qMCWpSNPrN4OW4R+8ku07B/PULTF2eEkmRw0JpjjEy6dm4QYDZj97\n\tXyzfmiIMs9ZhHvCbZpRWM6gyezKuqlMhJY+SHW6VFCtu0JsZHxxu//XqyZwGLEoslcrh\n\tAm+RDNTUPL80SsmOW99p8r6o81FSDtWP1EC4MxiowteKWYh7EC7PT+MCd7I+0CsVx8By\n\tnahsSCems4c8/4z53lCJ61bnZxciRB2VtB3deEzotsrqTZv/ELC4qpXouduAF9yTOxmI\n\tPwidsCstiIiWduZmeTzKdqMMgF8GK1yn6/OyZtPGWeDP3ShClC0gKS7eMWL9/ywEyQPL\n\tS11w==","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:content-transfer-encoding;\n\tbh=rKORtiwSd5AqXd9jQhrZyepiY3goSV6uFU9pgYrKuYs=;\n\tb=HxgDfgGymSK6h5EIH+dR5G6n70lXmGs8sufwNewHibcpWDuI0Xh7lKlF7EHp4Q9YZj\n\ttyh7a1huVYzBnCK/zo/Qb3Nt69rFDYmCH8zIvRwDOL35p6+hL6bulRQ7jj3ebzG5zG4d\n\tESwPIWL/OVNEOWa3/1CQUnNEdE8l9VcCCAIDw="],"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:content-transfer-encoding;\n\tbh=rKORtiwSd5AqXd9jQhrZyepiY3goSV6uFU9pgYrKuYs=;\n\tb=kiIi2dTJMdzbmr0hcCoNXWS/32M8c6tPAXWfQrp3d+HIy3runwKVNOnLUroKIOqGOG\n\t5wOMzF3OhZsvC3aEIRmxFwAEQVZVq8FdD3oopE6wg3yaelcVUx3N0w2IUDs/iZPB2xO8\n\tFQTzFwYarypBD2izZOcNU9Lu+1J/tfGTdSHS3P4VRusln+J9RwI9d6qT5e/PAB/u35Uq\n\tfh6KuEEO7eYzSOtNqKnrKfk/poPEoF8yQJu2Divs4kc6pjoTL+bH3lA8jEZZYb2lMUOj\n\tBRXtD0ZTdqLtr85e7vGGk06gSO9SD4gEzyid4LIYznAulEgztY70ppplB8TYNPPBNQmB\n\thHuw==","X-Gm-Message-State":"AHYfb5giEhbPgyHldUbSc3mNTu1b3KiSe7eeClq/BkfdhTeydcgTs0EO\n\tsYM8RyH/D7t1f537I4gPmDK62C5RQ3czPrU=","X-Google-Smtp-Source":"ADKCNb7jLvtT5CuvWM5VzaXb6b/JoALeflipe7bwFHDo9gHtTX11Wnub22SI/SHRY+YpOLhexc+xGdPZt0wfLrzU8MY=","X-Received":"by 10.55.74.88 with SMTP id x85mr9356910qka.294.1504085846709;\n\tWed, 30 Aug 2017 02:37:26 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>","References":"<20170829201601.64312-1-sjg@chromium.org>\n\t<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Wed, 30 Aug 2017 17:37:06 +0800","X-Google-Sender-Auth":"U8E9GUHDZqeSG_pKgv_47EZhfmk","Message-ID":"<CAPnjgZ3gFp2kcwADtmL=k5TU2OaWnGzJyLTMEe46wQAGd+jn0g@mail.gmail.com>","To":"\"Dr. Philipp Tomsich\" <philipp.tomsich@theobroma-systems.com>","Cc":"U-Boot Mailing List <u-boot@lists.denx.de>,\n\tJerry Van Baren <vanbaren@cideas.com>, Tom Rini <trini@konsulko.com>","Subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","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":1760018,"web_url":"http://patchwork.ozlabs.org/comment/1760018/","msgid":"<DEFAB8A2-D826-4F1F-B184-C551E58B785F@theobroma-systems.com>","list_archive_url":null,"date":"2017-08-30T10:12:33","subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> On 30 Aug 2017, at 11:37, Simon Glass <sjg@chromium.org> wrote:\n> \n> Hi Philipp,\n> \n> On 30 August 2017 at 04:31, Dr. Philipp Tomsich\n> <philipp.tomsich@theobroma-systems.com> wrote:\n>> Simon,\n>> \n>> I should be able to give this a test drive on the RK3368 by the end of the week.\n>> For the RK3399 our board uses full OF_CONTROL, so I won’t really exercise\n>> this there.\n>> \n>> Given that the RK3399 can always fall back to OF_CONTROL for SPL (as is the\n>> default on Puma) and this only blocks the RK3368 TPL (which is not enabled\n>> for any other boards than Lion): should we try to get this merged for this release\n>> cycle or let it slip into the new merge window?\n> \n> My preference would be to put it in a next branch, but if nothing else\n> uses it then I suppose it is possible (up to you and Tom)? What is\n> broken that this fixes?\n\nRK3399: SPL for all platforms using OF_PLATDATA\nRK3368: TPL (uses OF_PLATDATA)\n\nGiven that TPL for the RK3368 is not used much (and we have a private branch\nfor people that need it), I don’t mind going with the ‘next’ branch.\n\nI know that Kever has a few board configurations (EVB-RK3399, Firefly-RK3399)\nthat are affected, but I’d rather see these moved to OF_CONTROL instead of\nOF_PLATDATA.\n\nRegards,\nPhilipp.","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>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xj1XR6Sthz9t0F\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 20:12:38 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 0F4A9C227D1; Wed, 30 Aug 2017 10:12:31 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 3DEF2C22729;\n\tWed, 30 Aug 2017 10:12:29 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid AE799C227D1; Wed, 30 Aug 2017 10:12:27 +0000 (UTC)","from mail.theobroma-systems.com (vegas.theobroma-systems.com\n\t[144.76.126.164])\n\tby lists.denx.de (Postfix) with ESMTPS id 4CCE6C22729\n\tfor <u-boot@lists.denx.de>; Wed, 30 Aug 2017 10:12:27 +0000 (UTC)","from [86.59.122.178] (port=63690 helo=[10.4.9.36])\n\tby mail.theobroma-systems.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80)\n\t(envelope-from <philipp.tomsich@theobroma-systems.com>)\n\tid 1dmzyz-0002Jq-Gz; Wed, 30 Aug 2017 12:12:25 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"*","X-Spam-Status":"No, score=1.0 required=5.0 tests=HK_NAME_DR autolearn=no\n\tautolearn_force=no version=3.4.0","Mime-Version":"1.0 (Mac OS X Mail 8.2 \\(2104\\))","From":"\"Dr. Philipp Tomsich\" <philipp.tomsich@theobroma-systems.com>","In-Reply-To":"<CAPnjgZ3gFp2kcwADtmL=k5TU2OaWnGzJyLTMEe46wQAGd+jn0g@mail.gmail.com>","Date":"Wed, 30 Aug 2017 12:12:33 +0200","Message-Id":"<DEFAB8A2-D826-4F1F-B184-C551E58B785F@theobroma-systems.com>","References":"<20170829201601.64312-1-sjg@chromium.org>\n\t<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>\n\t<CAPnjgZ3gFp2kcwADtmL=k5TU2OaWnGzJyLTMEe46wQAGd+jn0g@mail.gmail.com>","To":"Simon Glass <sjg@chromium.org>","X-Mailer":"Apple Mail (2.2104)","Cc":"U-Boot Mailing List <u-boot@lists.denx.de>,\n\tJerry Van Baren <vanbaren@cideas.com>, Tom Rini <trini@konsulko.com>","Subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","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":1760961,"web_url":"http://patchwork.ozlabs.org/comment/1760961/","msgid":"<CAPnjgZ1Xc+gKdTx2KWEVE=PiDFJJvRO+Rzpp_egNrx7JQeMWgw@mail.gmail.com>","list_archive_url":null,"date":"2017-08-31T12:52:39","subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Philipp,\n\nOn 30 August 2017 at 18:12, Dr. Philipp Tomsich\n<philipp.tomsich@theobroma-systems.com> wrote:\n>\n>> On 30 Aug 2017, at 11:37, Simon Glass <sjg@chromium.org> wrote:\n>>\n>> Hi Philipp,\n>>\n>> On 30 August 2017 at 04:31, Dr. Philipp Tomsich\n>> <philipp.tomsich@theobroma-systems.com> wrote:\n>>> Simon,\n>>>\n>>> I should be able to give this a test drive on the RK3368 by the end of the week.\n>>> For the RK3399 our board uses full OF_CONTROL, so I won’t really exercise\n>>> this there.\n>>>\n>>> Given that the RK3399 can always fall back to OF_CONTROL for SPL (as is the\n>>> default on Puma) and this only blocks the RK3368 TPL (which is not enabled\n>>> for any other boards than Lion): should we try to get this merged for this release\n>>> cycle or let it slip into the new merge window?\n>>\n>> My preference would be to put it in a next branch, but if nothing else\n>> uses it then I suppose it is possible (up to you and Tom)? What is\n>> broken that this fixes?\n>\n> RK3399: SPL for all platforms using OF_PLATDATA\n> RK3368: TPL (uses OF_PLATDATA)\n>\n> Given that TPL for the RK3368 is not used much (and we have a private branch\n> for people that need it), I don’t mind going with the ‘next’ branch.\n>\n> I know that Kever has a few board configurations (EVB-RK3399, Firefly-RK3399)\n> that are affected, but I’d rather see these moved to OF_CONTROL instead of\n> OF_PLATDATA.\n\nYes, I see OF_PLATDATA as an extreme measure to be used when we really have to.\n\nRegards\nSimon","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=\"DaRGMzkO\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"NFW9Ri+U\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xjjJF2tVrz9s81\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 31 Aug 2017 23:04:29 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 21266C21E76; Thu, 31 Aug 2017 12:57:19 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 7549DC21E27;\n\tThu, 31 Aug 2017 12:57:17 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid BC850C21DF7; Thu, 31 Aug 2017 12:53:05 +0000 (UTC)","from mail-qk0-f173.google.com (mail-qk0-f173.google.com\n\t[209.85.220.173])\n\tby lists.denx.de (Postfix) with ESMTPS id BA697C21E01\n\tfor <u-boot@lists.denx.de>; Thu, 31 Aug 2017 12:53:01 +0000 (UTC)","by mail-qk0-f173.google.com with SMTP id o63so2288125qkb.3\n\tfor <u-boot@lists.denx.de>; Thu, 31 Aug 2017 05:53:01 -0700 (PDT)","by 10.200.28.106 with HTTP; Thu, 31 Aug 2017 05:52:39 -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:content-transfer-encoding;\n\tbh=+cEaRee2z+QrSLY95j3bgnaJAr/2zS3h4OSVjdqIAuE=;\n\tb=DaRGMzkOSlQ+N0RwtzhjGhYhqzyLYotvwASgVritSmfLORPGiNO5dIgkaqs5/6zoNI\n\tHSnl7ctekGgxXwsNWHJFxSHbCZbkU7GAQJOfIGSpeb9X0sQryOO2o3lFX7F/JLwkXzcw\n\tWVMKUxzKPPW4zthnagKF93mrMCHJESvcn79dQ+XmbfMErQ0iOErDi09B74RXy7aJnzNT\n\tkgGwKNNoBq+TDhIDyhvjLMDLfZeOMwcQgsEsPd0rKl3Mmb+LSrdLxYtkOnvJmurHzikO\n\tZUkR5BF7No4zNGRMy9KHLQcPllaDqcoSdhEwW1oiXDKb12F4pj5rt6jeRyiDYKbHWEZN\n\tiVuA==","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:content-transfer-encoding;\n\tbh=+cEaRee2z+QrSLY95j3bgnaJAr/2zS3h4OSVjdqIAuE=;\n\tb=NFW9Ri+UXKK2BKEhScF9cyGDCKIsQdpASpjvVpATozou6gj3MviVC087nzVeR/Fpyg\n\tAClyFcbQM1EroJMKyw1RwmVCzxA/L685CNTqJ2B2/o65dViqEBtB5cF6fbNvRyycPNMx\n\tbx0zFg3y+N0txpQGlP3A1RbvE+vNBiRikPq64="],"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:content-transfer-encoding;\n\tbh=+cEaRee2z+QrSLY95j3bgnaJAr/2zS3h4OSVjdqIAuE=;\n\tb=Qt8vRZQnGkd7uY7cfLTXkwmj10lUtzY6Nxt7oy6l0/xynlNU4MZBbrq80+RCnX+oPG\n\t5BHjR4CcdjNlIbDixArYaZZdTmi2Xws9B43mMnkX+p8k1iCCl3kaP8Yd0mHnq7SpVwOK\n\t8LT+dMmOeM2B/do+kDbR+EpsDy6cFzz5yGF7wX5G68YEAPBVOqOLlPYV3P99uYVBr9tx\n\tIuYXmOMIU6pqAmpX5FI5BKrmq9ZlmHjCWDIicAx80m43zTdD9sEmPgvpdK8DGqhpR6WC\n\tb1EffRuRid8vfJ72JNGJ9kA8S+hjaKCaxbuUhO+B9pxAi42WoF5vxgbVYLnhDDe0xShi\n\t7W+w==","X-Gm-Message-State":"AHYfb5jukJCpa8iUBjjUYEFncmAkoGLjmEKpWet7u7ALDraV4QxNTIfC\n\teAgG5+2dnOXUMGG+n/wL62OekJ0vNO+EGk4=","X-Google-Smtp-Source":"ADKCNb7BZuQlCqKc2ai7rXI6Y7OfOrvQICPxLB/TDtC1bkDOVFhxG2UOnreALaD+fs4dZ/TFMr4RhnQXgd1otV3DJwY=","X-Received":"by 10.55.39.84 with SMTP id n81mr3378319qkn.236.1504183980460;\n\tThu, 31 Aug 2017 05:53:00 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<DEFAB8A2-D826-4F1F-B184-C551E58B785F@theobroma-systems.com>","References":"<20170829201601.64312-1-sjg@chromium.org>\n\t<B8DD20CB-DF95-4394-B5C3-8C2FBB8ED8AB@theobroma-systems.com>\n\t<CAPnjgZ3gFp2kcwADtmL=k5TU2OaWnGzJyLTMEe46wQAGd+jn0g@mail.gmail.com>\n\t<DEFAB8A2-D826-4F1F-B184-C551E58B785F@theobroma-systems.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Thu, 31 Aug 2017 20:52:39 +0800","X-Google-Sender-Auth":"4Bi5NIaWbST4f9sXGnV5iK07LDY","Message-ID":"<CAPnjgZ1Xc+gKdTx2KWEVE=PiDFJJvRO+Rzpp_egNrx7JQeMWgw@mail.gmail.com>","To":"\"Dr. Philipp Tomsich\" <philipp.tomsich@theobroma-systems.com>","Cc":"U-Boot Mailing List <u-boot@lists.denx.de>,\n\tJerry Van Baren <vanbaren@cideas.com>, Tom Rini <trini@konsulko.com>","Subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","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":1764483,"web_url":"http://patchwork.ozlabs.org/comment/1764483/","msgid":"<127f5814-33a2-86ed-2940-6b20038f21e4@rock-chips.com>","list_archive_url":null,"date":"2017-09-07T03:11:06","subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","submitter":{"id":64532,"url":"http://patchwork.ozlabs.org/api/people/64532/","name":"Kever Yang","email":"kever.yang@rock-chips.com"},"content":"Hi Simon,\n\n     Thanks for your patches, It works very good. I have test the patch \nset on firefly-rk3399,\n\nand the SPL works with OF_PLATDATA after I fix issue in clk(Philipp has \npoint out) driver,\n\nsdram driver and sdhci driver, I will send out my fix later.\n\n\nPls add my test tag for this patch set.\nTested-by: Kever Yang <kever.yang@rock-chips.com>\n\nThanks,\n- Kever\nOn 08/30/2017 04:15 AM, Simon Glass wrote:\n> This series updates dtoc to support 64-bit addresses automatically. These\n> appear in C code as fdt64_t arrays:\n>\n> struct dtd_test1 {\n> \tfdt64_t\t\treg[2];\n>\n> };\n>\n> static struct dtd_test1 dtv_test1 = {\n> \t.reg\t\t= {0x123400000000, 0x5678},\n> };\n>\n> C code can then process these address and size parents easily. This\n> feature is controlled by the #address-cells and #size-cells values of the\n> parent.\n>\n> The v2 series also enhances phandle support so that it is now possible to\n> support phandles with different number of arguments. Several clean-up\n> patches are included also.\n>\n> Changes in v2:\n> - Support 'reg' properties with a single cell (e.g. #size-cells = 0)\n> - Introduce an fdt_val_t type which is either 32- or 64-bits long\n> - Update rk3368 and rk3399 uses\n> - Drop review tags since there are significant changes in this patch\n>\n> Simon Glass (16):\n>    fdt: Sync libfdt up to upstream\n>    dtoc: Adjust Node to record its parent\n>    dtoc: Add a 64-bit type and a way to convert cells into 64 bits\n>    dtoc: Avoid very long lines in output\n>    dtoc: Add support for 32 or 64-bit addresses\n>    dtoc: Handle 'reg' properties with unusual sizes\n>    dtoc: Update the Fdt class to record phandles\n>    dtoc: Use the Fdt's class's phandle map\n>    dtoc: Make is_phandle() a member function\n>    dtoc: Rename is_phandle() and adjust it to return more detail\n>    dtoc: Rename the phandle struct\n>    dtoc: Put each phandle on a separate line\n>    dtoc: Put phandle args in an array\n>    dtoc: Support properties containing multiple phandle values\n>    dtoc: Rename the auto-generated dt-structs.h file\n>    dtoc: Add a header to the generated files\n>\n>   doc/driver-model/of-plat.txt       |   2 +-\n>   drivers/clk/clk-uclass.c           |   4 +-\n>   drivers/clk/rockchip/clk_rk3368.c  |   2 +-\n>   drivers/clk/rockchip/clk_rk3399.c  |   4 +-\n>   drivers/core/regmap.c              |   2 +-\n>   include/clk.h                      |   4 +-\n>   include/dt-structs.h               |  16 ++-\n>   include/fdtdec.h                   |   2 +\n>   include/regmap.h                   |   2 +-\n>   include/syscon.h                   |   6 +-\n>   lib/libfdt/fdt_rw.c                |  20 ++-\n>   lib/libfdt/libfdt.h                |  31 +++++\n>   lib/libfdt/pylibfdt/libfdt.i       |  58 ++++++++\n>   scripts/Makefile.spl               |   5 +-\n>   tools/dtoc/dtb_platdata.py         | 204 ++++++++++++++++++++++------\n>   tools/dtoc/dtoc_test_addr32.dts    |  27 ++++\n>   tools/dtoc/dtoc_test_addr32_64.dts |  33 +++++\n>   tools/dtoc/dtoc_test_addr64.dts    |  33 +++++\n>   tools/dtoc/dtoc_test_addr64_32.dts |  33 +++++\n>   tools/dtoc/dtoc_test_phandle.dts   |  16 ++-\n>   tools/dtoc/dtoc_test_simple.dts    |  14 ++\n>   tools/dtoc/fdt.py                  |  19 ++-\n>   tools/dtoc/fdt_util.py             |  16 +++\n>   tools/dtoc/test_dtoc.py            | 270 ++++++++++++++++++++++++++++++++++++-\n>   24 files changed, 748 insertions(+), 75 deletions(-)\n>   create mode 100644 tools/dtoc/dtoc_test_addr32.dts\n>   create mode 100644 tools/dtoc/dtoc_test_addr32_64.dts\n>   create mode 100644 tools/dtoc/dtoc_test_addr64.dts\n>   create mode 100644 tools/dtoc/dtoc_test_addr64_32.dts\n>","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>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xnlpr5Rm0z9s7C\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu,  7 Sep 2017 13:11:31 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 1C937C21DAD; Thu,  7 Sep 2017 03:11:22 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id B7461C21C40;\n\tThu,  7 Sep 2017 03:11:19 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D9108C21C40; Thu,  7 Sep 2017 03:11:18 +0000 (UTC)","from regular1.263xmail.com (regular1.263xmail.com [211.150.99.132])\n\tby lists.denx.de (Postfix) with ESMTPS id 17EC7C21C3C\n\tfor <u-boot@lists.denx.de>; Thu,  7 Sep 2017 03:11:17 +0000 (UTC)","from kever.yang?rock-chips.com (unknown [192.168.167.152])\n\tby regular1.263xmail.com (Postfix) with ESMTP id B53BA9431;\n\tThu,  7 Sep 2017 11:11:11 +0800 (CST)","from [172.16.12.149] (localhost [127.0.0.1])\n\tby smtp.263.net (Postfix) with ESMTPA id 6FA473F5;\n\tThu,  7 Sep 2017 11:11:07 +0800 (CST)","from [172.16.12.149] (unknown [58.22.7.114])\n\tby smtp.263.net (Postfix) whith ESMTP id 2909745BTY5;\n\tThu, 07 Sep 2017 11:11:10 +0800 (CST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"*","X-Spam-Status":"No, score=1.9 required=5.0 tests=RCVD_IN_BL_SPAMCOP_NET,\n\tRCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,RCVD_IN_SORBS_WEB autolearn=no\n\tautolearn_force=no version=3.4.0","X-263anti-spam":"KSV:0;BIG:0;","X-MAIL-GRAY":"0","X-MAIL-DELIVERY":"1","X-KSVirus-check":"0","X-ADDR-CHECKED4":"1","X-ABS-CHECKED":"1","X-SKE-CHECKED":"1","X-ANTISPAM-LEVEL":"2","X-RL-SENDER":"kever.yang@rock-chips.com","X-FST-TO":"vanbaren@cideas.com","X-SENDER-IP":"58.22.7.114","X-LOGIN-NAME":"kever.yang@rock-chips.com","X-UNIQUE-TAG":"<2e773b151bbf5e2259a5a3a04bb85803>","X-ATTACHMENT-NUM":"0","X-SENDER":"yk@rock-chips.com","X-DNS-TYPE":"0","To":"Simon Glass <sjg@chromium.org>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>","References":"<20170829201601.64312-1-sjg@chromium.org>","From":"Kever Yang <kever.yang@rock-chips.com>","Message-ID":"<127f5814-33a2-86ed-2940-6b20038f21e4@rock-chips.com>","Date":"Thu, 7 Sep 2017 11:11:06 +0800","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"<20170829201601.64312-1-sjg@chromium.org>","Cc":"Tom Rini <trini@konsulko.com>, Jerry Van Baren <vanbaren@cideas.com>","Subject":"Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]