[{"id":1764593,"web_url":"http://patchwork.ozlabs.org/comment/1764593/","msgid":"<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>","list_archive_url":null,"date":"2017-09-07T08:51:45","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"Kever,\n\nThanks for the fix.\n\nCould we move all RK3399 boards to use OF_CONTROL (i.e. not OF_PLATDATA)\nin SPL? We have been using this for the RK3399-Q7 and there is plenty of SRAM\nleft for future code-growth, so this should be a safe choice.\n\nRegards,\nPhilipp.\n\n> On 7 Sep 2017, at 05:20, Kever Yang wrote:\n> \n> After Simon's patch, the dtoc can work with 64bit address,\n> so we need to fix reg number for it.\n> Depend on Simon's patch set:\n> https://patchwork.ozlabs.org/cover/807266/\n> \n> Signed-off-by: Kever Yang \n\nReviewed-by: Philipp Tomsich \n\n> ---\n> \n> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c\n> index 63342ab..5ed4b03 100644\n> --- a/drivers/ram/rockchip/sdram_rk3399.c\n> +++ b/drivers/ram/rockchip/sdram_rk3399.c\n> @@ -1115,7 +1115,7 @@ static int conv_of_platdata(struct udevice *dev)\n> \tint ret;\n> \n> \tret = regmap_init_mem_platdata(dev, dtplat->reg,\n> -\t\t\tARRAY_SIZE(dtplat->reg) / 4,\n> +\t\t\tARRAY_SIZE(dtplat->reg) / 2,\n> \t\t\t&plat->map);\n> \tif (ret)\n> \t\treturn ret;\n> -- \n> 1.9.1\n>","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xnvMc6zZHz9s8J\n\tfor ;\n\tThu, 7 Sep 2017 18:51:56 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 9AF6BC21E2C; Thu, 7 Sep 2017 08:51:53 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 42481C21D70;\n\tThu, 7 Sep 2017 08:51:51 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 12911C21D70; Thu, 7 Sep 2017 08:51:49 +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 42FF3C21C97\n\tfor ; Thu, 7 Sep 2017 08:51:48 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:51811\n\thelo=[192.168.2.129]) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1dpsXK-00022b-Q5; Thu, 07 Sep 2017 10:51:46 +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 10.3 \\(3273\\))","From":"\"Dr. Philipp Tomsich\" ","In-Reply-To":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","Date":"Thu, 7 Sep 2017 10:51:45 +0200","Message-Id":"<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","To":"Kever Yang ","X-Mailer":"Apple Mail (2.3273)","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\" "}},{"id":1765046,"web_url":"http://patchwork.ozlabs.org/comment/1765046/","msgid":"","list_archive_url":null,"date":"2017-09-08T02:46:00","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":64532,"url":"http://patchwork.ozlabs.org/api/people/64532/","name":"Kever Yang","email":"kever.yang@rock-chips.com"},"content":"Philipp,\n\n We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n\n\n For OF_PLATDATA, I would prefer there always have a option to use for\nspeed up the boot time, not only for ram size.\n\nThanks,\n- Kever\nOn 09/07/2017 04:51 PM, Dr. Philipp Tomsich wrote:\n> Kever,\n>\n> Thanks for the fix.\n>\n> Could we move all RK3399 boards to use OF_CONTROL (i.e. not OF_PLATDATA)\n> in SPL? We have been using this for the RK3399-Q7 and there is plenty of SRAM\n> left for future code-growth, so this should be a safe choice.\n>\n> Regards,\n> Philipp.\n>\n>> On 7 Sep 2017, at 05:20, Kever Yang wrote:\n>>\n>> After Simon's patch, the dtoc can work with 64bit address,\n>> so we need to fix reg number for it.\n>> Depend on Simon's patch set:\n>> https://patchwork.ozlabs.org/cover/807266/\n>>\n>> Signed-off-by: Kever Yang \n> Reviewed-by: Philipp Tomsich \n>\n>> ---\n>>\n>> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c\n>> index 63342ab..5ed4b03 100644\n>> --- a/drivers/ram/rockchip/sdram_rk3399.c\n>> +++ b/drivers/ram/rockchip/sdram_rk3399.c\n>> @@ -1115,7 +1115,7 @@ static int conv_of_platdata(struct udevice *dev)\n>> \tint ret;\n>>\n>> \tret = regmap_init_mem_platdata(dev, dtplat->reg,\n>> -\t\t\tARRAY_SIZE(dtplat->reg) / 4,\n>> +\t\t\tARRAY_SIZE(dtplat->reg) / 2,\n>> \t\t\t&plat->map);\n>> \tif (ret)\n>> \t\treturn ret;\n>> -- \n>> 1.9.1\n>>\n>","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xpMCK3NKXz9sMN\n\tfor ;\n\tFri, 8 Sep 2017 12:46:20 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 9BB8EC21E21; Fri, 8 Sep 2017 02:46:15 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A864DC21D70;\n\tFri, 8 Sep 2017 02:46:13 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 12023C21D70; Fri, 8 Sep 2017 02:46:11 +0000 (UTC)","from regular1.263xmail.com (regular1.263xmail.com [211.150.99.138])\n\tby lists.denx.de (Postfix) with ESMTPS id 48E60C21D19\n\tfor ; Fri, 8 Sep 2017 02:46:10 +0000 (UTC)","from kever.yang?rock-chips.com (unknown [192.168.165.103])\n\tby regular1.263xmail.com (Postfix) with ESMTP id 4D14179D1;\n\tFri, 8 Sep 2017 10:46:06 +0800 (CST)","from [192.168.60.65] (localhost [127.0.0.1])\n\tby smtp.263.net (Postfix) with ESMTPA id D2A00416;\n\tFri, 8 Sep 2017 10:46:03 +0800 (CST)","from [192.168.60.65] (unknown [103.29.142.67])\n\tby smtp.263.net (Postfix) whith ESMTP id 300761MH9Q8;\n\tFri, 08 Sep 2017 10:46:06 +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=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H2 autolearn=unavailable autolearn_force=no\n\tversion=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":"sjg@chromium.org","X-SENDER-IP":"103.29.142.67","X-LOGIN-NAME":"kever.yang@rock-chips.com","X-UNIQUE-TAG":"<58faf8f05dca8bfaace4504c4e05df1d>","X-ATTACHMENT-NUM":"0","X-SENDER":"yk@rock-chips.com","X-DNS-TYPE":"0","To":"\"Dr. Philipp Tomsich\" ","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>","From":"Kever Yang ","Message-ID":"","Date":"Fri, 8 Sep 2017 10:46:00 +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":"<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1765128,"web_url":"http://patchwork.ozlabs.org/comment/1765128/","msgid":"<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>","list_archive_url":null,"date":"2017-09-08T07:34:11","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> On 8 Sep 2017, at 04:46, Kever Yang wrote:\n> \n> Philipp,\n> \n> We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n\nI meant “full OF_CONTROL” as opposed to “OF_PLATDATA” (which is dependent\nof OF_CONTROL, but is intended as a stopgap measure).\n\n> For OF_PLATDATA, I would prefer there always have a option to use for\n> speed up the boot time, not only for ram size.\n\nI had discussed that with Simon recently and he views OF_PLATDATA as a last\nresort to be used, when there is not enough memory for full OF_CONTROL.\n\n\n> Thanks,\n> - Kever\n> On 09/07/2017 04:51 PM, Dr. Philipp Tomsich wrote:\n>> Kever,\n>> \n>> Thanks for the fix.\n>> \n>> Could we move all RK3399 boards to use OF_CONTROL (i.e. not OF_PLATDATA)\n>> in SPL? We have been using this for the RK3399-Q7 and there is plenty of SRAM\n>> left for future code-growth, so this should be a safe choice.\n>> \n>> Regards,\n>> Philipp.\n>> \n>>> On 7 Sep 2017, at 05:20, Kever Yang wrote:\n>>> \n>>> After Simon's patch, the dtoc can work with 64bit address,\n>>> so we need to fix reg number for it.\n>>> Depend on Simon's patch set:\n>>> https://patchwork.ozlabs.org/cover/807266/\n>>> \n>>> Signed-off-by: Kever Yang \n>> Reviewed-by: Philipp Tomsich \n>> \n>>> ---\n>>> \n>>> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n>>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>> \n>>> diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c\n>>> index 63342ab..5ed4b03 100644\n>>> --- a/drivers/ram/rockchip/sdram_rk3399.c\n>>> +++ b/drivers/ram/rockchip/sdram_rk3399.c\n>>> @@ -1115,7 +1115,7 @@ static int conv_of_platdata(struct udevice *dev)\n>>> \tint ret;\n>>> \n>>> \tret = regmap_init_mem_platdata(dev, dtplat->reg,\n>>> -\t\t\tARRAY_SIZE(dtplat->reg) / 4,\n>>> +\t\t\tARRAY_SIZE(dtplat->reg) / 2,\n>>> \t\t\t&plat->map);\n>>> \tif (ret)\n>>> \t\treturn ret;\n>>> -- \n>>> 1.9.1\n>>> \n>> \n> \n>","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xpTbn3pvDz9s83\n\tfor ;\n\tFri, 8 Sep 2017 17:34:29 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 37FB4C21E42; Fri, 8 Sep 2017 07:34:21 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 427D4C21D19;\n\tFri, 8 Sep 2017 07:34:19 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 7A5D5C21D19; Fri, 8 Sep 2017 07:34:18 +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 DCE90C21C5C\n\tfor ; Fri, 8 Sep 2017 07:34:15 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:52251\n\thelo=[192.168.2.129]) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1dqDno-0006xZ-9e; Fri, 08 Sep 2017 09:34:12 +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 10.3 \\(3273\\))","From":"\"Dr. Philipp Tomsich\" ","In-Reply-To":"","Date":"Fri, 8 Sep 2017 09:34:11 +0200","Message-Id":"<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>\n\t","To":"Kever Yang ","X-Mailer":"Apple Mail (2.3273)","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\" "}},{"id":1765289,"web_url":"http://patchwork.ozlabs.org/comment/1765289/","msgid":"","list_archive_url":null,"date":"2017-09-08T12:17:38","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Kever,\n\nOn 8 September 2017 at 01:34, Dr. Philipp Tomsich\n wrote:\n>\n>\n> > On 8 Sep 2017, at 04:46, Kever Yang wrote:\n> >\n> > Philipp,\n> >\n> > We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n>\n> I meant “full OF_CONTROL” as opposed to “OF_PLATDATA” (which is dependent\n> of OF_CONTROL, but is intended as a stopgap measure).\n>\n> > For OF_PLATDATA, I would prefer there always have a option to use for\n> > speed up the boot time, not only for ram size.\n>\n> I had discussed that with Simon recently and he views OF_PLATDATA as a last\n> resort to be used, when there is not enough memory for full OF_CONTROL.\n\n\nCan you use bootstage to measure the boot time impact?\n\nRegards,\nSimon","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=\"FFcJO8OG\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"e1obpD15\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xpcBF3zMTz9t63\n\tfor ;\n\tFri, 8 Sep 2017 22:31:06 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 33FC4C21C5D; Fri, 8 Sep 2017 12:18:04 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 9A9C6C21C34;\n\tFri, 8 Sep 2017 12:18:02 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E4FC2C21C34; Fri, 8 Sep 2017 12:18:00 +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 532B2C21C33\n\tfor ; Fri, 8 Sep 2017 12:18:00 +0000 (UTC)","by mail-qk0-f180.google.com with SMTP id z143so5762317qkb.3\n\tfor ; Fri, 08 Sep 2017 05:18:00 -0700 (PDT)","by 10.200.37.200 with HTTP; Fri, 8 Sep 2017 05:17:38 -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=BK16pXRoVNo2EA/mbkwxYaaPvrQL9QdjVVrc2wHw+/c=;\n\tb=FFcJO8OGIYVuBgYPEyWovAN7ITaDr0XmioxQk//RqsI2kGMoFK/8QhlzGKdZYaMv22\n\tYLEsEmkXhNYgxQP+yDLPLSOeQKinnUv8s3tNcxQHcUlQxg95Lm9BhapZF5kOrp9LCHaS\n\tybZec+ByNsMOWx7J/OrGck8uQUxt5pWCUIVdPQISgkAqBao8FURdVdZU9QBkEmuvCtQg\n\t1hWmWlkYp9ApVxYgaf9EkFLHNXzj2Z2KSZOgb27evzDeGn3SWNA0ftACV7r/O1uSNJZb\n\tKF0ZV2awx7Wmq0yRTBRZ2JuK5c+dhJ7EMdQLkWqzTL5Kkq6ZfvB2xEDadUQExp+LyStp\n\t8HsQ==","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=BK16pXRoVNo2EA/mbkwxYaaPvrQL9QdjVVrc2wHw+/c=;\n\tb=e1obpD15a5O/CoVzTH6chWU8fL/O2zvanHWSwKipI51CUSjWEoR92AUC/dIN37smoL\n\t5T5nh5lMuH4IixAA0c4YWb6wh5DXcvJcZV6iRJ+Myy3R2rWArM0Vl+mHC1P8Nn/Q75NK\n\tK68cHg9ZCDT1Qu1CMHLxNL893YkMBIbTxQMH8="],"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=BK16pXRoVNo2EA/mbkwxYaaPvrQL9QdjVVrc2wHw+/c=;\n\tb=s7NC5idBKdbDaHp1PSCMbnttIsFhChgzJZddKTu8xkocNtdjVdcIYM1/ntevc944Ku\n\t+GnDVhzdxEMaSL3TmDnMrzjzXm1KC6f1/S1D39REZp/yZzyayPQsWy79WQjKUjbw6M6Q\n\tOrmOoOSQ1YTOs+03xTMABq2kYjlW3oBPEouzgjixeK47zXKGEydYJuH0TIufmV0sUwu1\n\tbecOtG+RDqQLaU7ZprHeyaJNBM2G9cGl184ncmpDNsEBV3gtatouRqXoEGv6sZ3P9Vtn\n\tmghrK7URjagtEnZ2PjUzPlH9pEZgUFIVosCh9jL4wv6ZYSfzv8bDJah5JPr6Z+zEeqiI\n\tKJDw==","X-Gm-Message-State":"AHPjjUhUiwxdaIoI19Osb6CkCeeUcXAIrF8HgNJEMX3go9LCJX0TriM+\n\tEK+2+D8EvkzGKASeL1b9oU0UR7mkONLyagPuv82jjw==","X-Google-Smtp-Source":"AOwi7QCVNivHWFpNucX/zSFJ26O7lnOyt12rPRJsRxYCgRVbfl2/61vwHNfL+M2G9IpHWq9cKqxsiSrFIKOWqhHsdH0=","X-Received":"by 10.55.81.215 with SMTP id f206mr3223314qkb.188.1504873078708; \n\tFri, 08 Sep 2017 05:17:58 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>\n\t\n\t<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>","From":"Simon Glass ","Date":"Fri, 8 Sep 2017 06:17:38 -0600","X-Google-Sender-Auth":"WK2O0DrzZ9_O6MP8W228gzT2olE","Message-ID":"","To":"\"Dr. Philipp Tomsich\" ","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\" "}},{"id":1766122,"web_url":"http://patchwork.ozlabs.org/comment/1766122/","msgid":"","list_archive_url":null,"date":"2017-09-11T08:17:23","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\nOn 09/08/2017 08:17 PM, Simon Glass wrote:\n> Hi Kever,\n>\n> On 8 September 2017 at 01:34, Dr. Philipp Tomsich\n> wrote:\n>>\n>>> On 8 Sep 2017, at 04:46, Kever Yang wrote:\n>>>\n>>> Philipp,\n>>>\n>>> We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n>> I meant “full OF_CONTROL” as opposed to “OF_PLATDATA” (which is dependent\n>> of OF_CONTROL, but is intended as a stopgap measure).\n>>\n>>> For OF_PLATDATA, I would prefer there always have a option to use for\n>>> speed up the boot time, not only for ram size.\n>> I had discussed that with Simon recently and he views OF_PLATDATA as a last\n>> resort to be used, when there is not enough memory for full OF_CONTROL.\n>\n> Can you use bootstage to measure the boot time impact?\n\nI think I have do the statistics before with the timer instead of bootstage.\nAnd here is the result I got with bootstage(with little change in spl to \nmove\nsecure_timer_init() before spl_early_init()):\n\nThe mark of end_spl:\nWith of-pladata:\n205952 205864 205857\nwithout of-platdata:\n279520 279495 279508\n\nThe difference is about 75ms.\n\nBTW:\nbug1: there is no timer_get_boot_us() in armv8,\nbug2: there is something wrong with \"dm_spl\", it use bootstage_start() + \nbootstage_accum()\n instead of bootstage_mark_name(), the \"board_init_f\" will \nreplace it, maybe some tag is\n not correct.\n\nThanks,\n- Kever\n>\n> Regards,\n> Simon\n>","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrLQR5PQXz9s81\n\tfor ;\n\tMon, 11 Sep 2017 18:17:50 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CD87AC21DD7; Mon, 11 Sep 2017 08:17:43 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 25FFEC21CB6;\n\tMon, 11 Sep 2017 08:17:38 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid BB66EC21D57; Mon, 11 Sep 2017 08:17:36 +0000 (UTC)","from regular1.263xmail.com (regular1.263xmail.com [211.150.99.140])\n\tby lists.denx.de (Postfix) with ESMTPS id 1E460C21C4C\n\tfor ; Mon, 11 Sep 2017 08:17:35 +0000 (UTC)","from kever.yang?rock-chips.com (unknown [192.168.165.141])\n\tby regular1.263xmail.com (Postfix) with ESMTP id 4C36C4B96;\n\tMon, 11 Sep 2017 16:17:29 +0800 (CST)","from [192.168.60.65] (localhost [127.0.0.1])\n\tby smtp.263.net (Postfix) with ESMTPA id 2E68D40F;\n\tMon, 11 Sep 2017 16:17:26 +0800 (CST)","from [192.168.60.65] (unknown [103.29.142.67])\n\tby smtp.263.net (Postfix) whith ESMTP id 19645U9GFJZ;\n\tMon, 11 Sep 2017 16:17:29 +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=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=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":"u-boot@lists.denx.de","X-SENDER-IP":"103.29.142.67","X-LOGIN-NAME":"kever.yang@rock-chips.com","X-UNIQUE-TAG":"<845b4915379174c0186a821e13f287be>","X-ATTACHMENT-NUM":"0","X-SENDER":"yk@rock-chips.com","X-DNS-TYPE":"0","To":"Simon Glass ,\n\t\"Dr. Philipp Tomsich\" ","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>\n\t\n\t<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>\n\t","From":"Kever Yang ","Message-ID":"","Date":"Mon, 11 Sep 2017 16:17:23 +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":"","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1766968,"web_url":"http://patchwork.ozlabs.org/comment/1766968/","msgid":"","list_archive_url":null,"date":"2017-09-12T12:30:39","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Kever,\n\nOn 11 September 2017 at 02:17, Kever Yang wrote:\n> Hi Simon,\n>\n>\n> On 09/08/2017 08:17 PM, Simon Glass wrote:\n>>\n>> Hi Kever,\n>>\n>> On 8 September 2017 at 01:34, Dr. Philipp Tomsich\n>> wrote:\n>>>\n>>>\n>>>> On 8 Sep 2017, at 04:46, Kever Yang wrote:\n>>>>\n>>>> Philipp,\n>>>>\n>>>> We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n>>>\n>>> I meant “full OF_CONTROL” as opposed to “OF_PLATDATA” (which is dependent\n>>> of OF_CONTROL, but is intended as a stopgap measure).\n>>>\n>>>> For OF_PLATDATA, I would prefer there always have a option to use\n>>>> for\n>>>> speed up the boot time, not only for ram size.\n>>>\n>>> I had discussed that with Simon recently and he views OF_PLATDATA as a\n>>> last\n>>> resort to be used, when there is not enough memory for full OF_CONTROL.\n>>\n>>\n>> Can you use bootstage to measure the boot time impact?\n>\n>\n> I think I have do the statistics before with the timer instead of bootstage.\n> And here is the result I got with bootstage(with little change in spl to\n> move\n> secure_timer_init() before spl_early_init()):\n>\n> The mark of end_spl:\n> With of-pladata:\n> 205952 205864 205857\n> without of-platdata:\n> 279520 279495 279508\n\n> The difference is about 75ms.\n\nThat is enough to justify using of-platdata here. I am not sure why\nthe difference is so large though.\n\n>\n> BTW:\n> bug1: there is no timer_get_boot_us() in armv8,\n> bug2: there is something wrong with \"dm_spl\", it use bootstage_start() +\n> bootstage_accum()\n> instead of bootstage_mark_name(), the \"board_init_f\" will replace\n> it, maybe some tag is\n> not correct.\n\nOK. Do you think you could send patches to fix these?\n\nRegards,\nSimon","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=\"h12Uwzep\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"KuoqYxQh\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs4Xh0hPZz9t6m\n\tfor ;\n\tTue, 12 Sep 2017 22:54:39 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 91724C22155; Tue, 12 Sep 2017 12:36:49 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BCF78C2203B;\n\tTue, 12 Sep 2017 12:36:47 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 720E9C21FFE; Tue, 12 Sep 2017 12:31:05 +0000 (UTC)","from mail-qt0-f174.google.com (mail-qt0-f174.google.com\n\t[209.85.216.174])\n\tby lists.denx.de (Postfix) with ESMTPS id 2495EC21FF1\n\tfor ; Tue, 12 Sep 2017 12:31:01 +0000 (UTC)","by mail-qt0-f174.google.com with SMTP id b1so15404042qtc.4\n\tfor ; Tue, 12 Sep 2017 05:31:01 -0700 (PDT)","by 10.200.37.200 with HTTP; Tue, 12 Sep 2017 05:30: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=vYFHtzMenojps+rpWDqQfF0hF5/nq03GpLfc5NqSyT0=;\n\tb=h12UwzepSL06p31YBATNEd5cfX/7AFSBri6TsItg7wWmp83rPaSvIcwAP8yU+plG9H\n\tqQTCtyF68ZROn5TAx9sUidM2pOIpFFLYjVCF2FZYALWm5JxTYF0DYqPLvtWqz6vphIYo\n\tmoh2UHms3LJlvyLnQzxmfoUE77Z1+QwjfhzM+32skAXaQNYPr5YNFfiOrALbm6JTJTKQ\n\tw6GvNnQSPSzo5Qr6w/OhZ29YrRD78gED9Np+0YTu6K+s9ox1Xd9lM1MNPlm1tyodsiRa\n\tQAG5w/DDwI0C7kAlCXKYMQfD/ifLz9ntwFWTt6US9HmpHBB+39cVQBBcy3OEOuttwvXb\n\tn35w==","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=vYFHtzMenojps+rpWDqQfF0hF5/nq03GpLfc5NqSyT0=;\n\tb=KuoqYxQh62EjL3S79gwSiawbRwIkLHxgqHK8IUKtm4wa/i+yGj814Ccm9+BahPbzP5\n\ttVx7NZyV9mYZiO3cRJTF0jlaMsBDETp1cA9Oo32I1zxlYrsPPA1JC2zSaiI3or+kB/0g\n\tXICfb4JBhgyhLW7TyhKkO6Ae9VmyJvut7ZmME="],"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=vYFHtzMenojps+rpWDqQfF0hF5/nq03GpLfc5NqSyT0=;\n\tb=szZbOXMqbBbfljA75SVGMDkOZNGjHYrQs+ikPLbk7XM+FiIzfyNiA4jkbGo3ZofsMj\n\t6vxBEjwQ/xKsb2xNgd22gTNg9TBwhd43SyRfp2Fs1886k7ZsWncM/B9OXXGqQTaJ0Qf6\n\teJHPb4mM97oUo87pxUL28eQzFlkOe5RoCz3ZhgoST0ZE/DokrX5F+Ki1MWwyt1Ek5BEn\n\t6zCZ2nD9xCkXZ2ypRjy5+tB+OiixB0dB5S2TITTIMC250aMsKakJFXVMnsFJPQsZYc2B\n\thgLjsCjQdGQbrnn7IfM+c1QABrpTzige/xTWKvYAvm0FHQtD7ltOEs6JFNF7mdz0ex3X\n\tHDSA==","X-Gm-Message-State":"AHPjjUjoqvkff++QdYHuwvB1yukH+6P1TsEZEwtFOlhJSAyGJMMNhxzi\n\t00tPB/HaMaBnjWJUT1SlJkkW4WmcscpIm9BjpWZhAQ==","X-Google-Smtp-Source":"AOwi7QDr81A9zpwPrcfJKrscSV9pDjfT1wChMSHW/EFdzfE1CYWYjvTIPzGYGUsgmU6MI9cqE3LXPj462V7w410AO1s=","X-Received":"by 10.237.35.238 with SMTP id k43mr21525988qtc.214.1505219459791;\n\tTue, 12 Sep 2017 05:30:59 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>\n\t\n\t<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>\n\t\n\t","From":"Simon Glass ","Date":"Tue, 12 Sep 2017 06:30:39 -0600","X-Google-Sender-Auth":"aqixvM-h-vKeYjMYOTUp2thwbwg","Message-ID":"","To":"Kever Yang ","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\" "}},{"id":1766969,"web_url":"http://patchwork.ozlabs.org/comment/1766969/","msgid":"","list_archive_url":null,"date":"2017-09-12T12:36:28","subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"Kever & Simon,\n\n> On 12 Sep 2017, at 14:30, Simon Glass wrote:\n> \n> Hi Kever,\n> \n> On 11 September 2017 at 02:17, Kever Yang > wrote:\n>> Hi Simon,\n>> \n>> \n>> On 09/08/2017 08:17 PM, Simon Glass wrote:\n>>> \n>>> Hi Kever,\n>>> \n>>> On 8 September 2017 at 01:34, Dr. Philipp Tomsich\n>>> wrote:\n>>>> \n>>>> \n>>>>> On 8 Sep 2017, at 04:46, Kever Yang wrote:\n>>>>> \n>>>>> Philipp,\n>>>>> \n>>>>> We already enable CONFIG_SPL_OF_CONTROL for all rk3399 board, right?\n>>>> \n>>>> I meant “full OF_CONTROL” as opposed to “OF_PLATDATA” (which is dependent\n>>>> of OF_CONTROL, but is intended as a stopgap measure).\n>>>> \n>>>>> For OF_PLATDATA, I would prefer there always have a option to use\n>>>>> for\n>>>>> speed up the boot time, not only for ram size.\n>>>> \n>>>> I had discussed that with Simon recently and he views OF_PLATDATA as a\n>>>> last\n>>>> resort to be used, when there is not enough memory for full OF_CONTROL.\n>>> \n>>> \n>>> Can you use bootstage to measure the boot time impact?\n>> \n>> \n>> I think I have do the statistics before with the timer instead of bootstage.\n>> And here is the result I got with bootstage(with little change in spl to\n>> move\n>> secure_timer_init() before spl_early_init()):\n>> \n>> The mark of end_spl:\n>> With of-pladata:\n>> 205952 205864 205857\n>> without of-platdata:\n>> 279520 279495 279508\n> \n>> The difference is about 75ms.\n> \n> That is enough to justify using of-platdata here. I am not sure why\n> the difference is so large though.\n\nI have noticed that the RK3399 U-Boot code does not clock up the boot\nCPU (i.e. rk3399_configure_cpu() is never called).\n\nCould this be the reason for the large difference?\n\n> \n>> \n>> BTW:\n>> bug1: there is no timer_get_boot_us() in armv8,\n>> bug2: there is something wrong with \"dm_spl\", it use bootstage_start() +\n>> bootstage_accum()\n>> instead of bootstage_mark_name(), the \"board_init_f\" will replace\n>> it, maybe some tag is\n>> not correct.\n> \n> OK. Do you think you could send patches to fix these?\n\nI woul prefer if we could move the timer-code away from the generic timer\nand start using Rockchip-specific DM-timer drivers. This will make the\nU-Boot code independent of the initialisation of the secure-timer block…\n\n> \n> Regards,\n> Simon","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs4Xh1WRXz9t6s\n\tfor ;\n\tTue, 12 Sep 2017 22:54:41 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid C933BC2214C; Tue, 12 Sep 2017 12:49:42 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5360CC22119;\n\tTue, 12 Sep 2017 12:49:40 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E95DFC22110; Tue, 12 Sep 2017 12:36:34 +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 B5908C2202C\n\tfor ; Tue, 12 Sep 2017 12:36:30 +0000 (UTC)","from [86.59.122.178] (port=55246 helo=[10.4.9.36])\n\tby mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1drkQW-0001j9-Um; Tue, 12 Sep 2017 14:36:29 +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","From":"\"Dr. Philipp Tomsich\" ","Message-Id":"","Mime-Version":"1.0 (Mac OS X Mail 10.3 \\(3273\\))","Date":"Tue, 12 Sep 2017 14:36:28 +0200","In-Reply-To":"","To":"Simon Glass ","References":"<1504754451-30534-1-git-send-email-kever.yang@rock-chips.com>\n\t<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>\n\t<22CB996A-C7D9-4578-8A68-84728FF10A06@theobroma-systems.com>\n\t\n\t<6AEC9D8B-F845-4F03-9520-A4C8E79C4BFB@theobroma-systems.com>\n\t\n\t\n\t","X-Mailer":"Apple Mail (2.3273)","X-Content-Filtered-By":"Mailman/MimeDel 2.1.18","Cc":"U-Boot Mailing List ","Subject":"Re: [U-Boot] [PATCH 2/2] rockchip: ram: rk3399: update reg map for\n\tof-platdata","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\" "}},{"id":1767200,"web_url":"http://patchwork.ozlabs.org/comment/1767200/","msgid":"","list_archive_url":null,"date":"2017-09-12T15:47:46","subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> After Simon's patch, the dtoc can work with 64bit address,\n> so we need to fix reg number for it.\n> Depend on Simon's patch set:\n> https://patchwork.ozlabs.org/cover/807266/\n> \n> Signed-off-by: Kever Yang \n> Reviewed-by: Philipp Tomsich \n> ---\n> \n> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n\nAcked-by: Philipp Tomsich ","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs8p72vKlz9s7f\n\tfor ;\n\tWed, 13 Sep 2017 02:07:43 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 2B712C222B0; Tue, 12 Sep 2017 16:04:58 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 23B96C2211C;\n\tTue, 12 Sep 2017 16:04:39 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid DD566C222B3; Tue, 12 Sep 2017 16:03:45 +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 BD69AC22042\n\tfor ; Tue, 12 Sep 2017 16:03:44 +0000 (UTC)","from [86.59.122.178] (port=60150 helo=vpn-10-11-0-14.lan)\n\tby mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1drnPe-00083Q-LH; Tue, 12 Sep 2017 17:47:46 +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=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich ","To":"Kever Yang ","In-Reply-To":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","References":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","Message-Id":"","Date":"Tue, 12 Sep 2017 17:47:46 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","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\" "}},{"id":1770397,"web_url":"http://patchwork.ozlabs.org/comment/1770397/","msgid":"","list_archive_url":null,"date":"2017-09-18T18:20:24","subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> After Simon's patch, the dtoc can work with 64bit address,\n> so we need to fix reg number for it.\n> Depend on Simon's patch set:\n> https://patchwork.ozlabs.org/cover/807266/\n> \n> Signed-off-by: Kever Yang \n> Reviewed-by: Philipp Tomsich \n> Acked-by: Philipp Tomsich \n> ---\n> \n> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n\nReviewed-by: Philipp Tomsich ","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwvYL2Rp2z9s7B\n\tfor ;\n\tTue, 19 Sep 2017 04:24:38 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid E14B1C21DB5; Mon, 18 Sep 2017 18:23:03 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 14F50C21D92;\n\tMon, 18 Sep 2017 18:22:23 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 08467C21D8D; Mon, 18 Sep 2017 18:20:29 +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 82229C21C41\n\tfor ; Mon, 18 Sep 2017 18:20:25 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:49928\n\thelo=vpn-10-11-0-14.lan) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1du0ee-0005Bu-Bt; Mon, 18 Sep 2017 20:20:24 +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=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich ","To":"Kever Yang ","In-Reply-To":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","References":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","Message-Id":"","Date":"Mon, 18 Sep 2017 20:20:24 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","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\" "}},{"id":1770414,"web_url":"http://patchwork.ozlabs.org/comment/1770414/","msgid":"","list_archive_url":null,"date":"2017-09-18T18:38:00","subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","submitter":{"id":53488,"url":"http://patchwork.ozlabs.org/api/people/53488/","name":"Philipp Tomsich","email":"philipp.tomsich@theobroma-systems.com"},"content":"> After Simon's patch, the dtoc can work with 64bit address,\n> so we need to fix reg number for it.\n> Depend on Simon's patch set:\n> https://patchwork.ozlabs.org/cover/807266/\n> \n> Signed-off-by: Kever Yang \n> Reviewed-by: Philipp Tomsich \n> Acked-by: Philipp Tomsich \n> Reviewed-by: Philipp Tomsich \n> ---\n> \n> drivers/ram/rockchip/sdram_rk3399.c | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n\nApplied to u-boot-rockchip, thanks!","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwvth2SPtz9s7v\n\tfor ;\n\tTue, 19 Sep 2017 04:39:40 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid BFD90C21CB6; Mon, 18 Sep 2017 18:39:22 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A02A4C21DAD;\n\tMon, 18 Sep 2017 18:38:06 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid BC761C21C4C; Mon, 18 Sep 2017 18:38:01 +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 7324CC21C4C\n\tfor ; Mon, 18 Sep 2017 18:38:01 +0000 (UTC)","from 89-104-28-141.customer.bnet.at ([89.104.28.141]:50258\n\thelo=vpn-10-11-0-14.lan) by mail.theobroma-systems.com with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_256_CBC_SHA256:256) (Exim 4.80)\n\t(envelope-from )\n\tid 1du0vg-0005fn-1Q; Mon, 18 Sep 2017 20:38:00 +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=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","MIME-Version":"1.0","From":"Philipp Tomsich ","To":"Kever Yang ","In-Reply-To":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","References":"<1504754451-30534-2-git-send-email-kever.yang@rock-chips.com>","Message-Id":"","Date":"Mon, 18 Sep 2017 20:38:00 +0200","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [U-Boot,\n\t2/2] rockchip: ram: rk3399: update reg map for of-platdata","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\" "}}]