{"id":810363,"url":"http://patchwork.ozlabs.org/api/1.2/patches/810363/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20170906014950.132960-6-sjg@chromium.org/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.2/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170906014950.132960-6-sjg@chromium.org>","list_archive_url":null,"date":"2017-09-06T01:49:49","name":"[U-Boot,v3,5/6] bootstage: Provide a separate record count setting for SPL","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"a9f7a420365dfc33a651139714992605fe00dcb2","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/1.2/people/6170/?format=json","name":"Simon Glass","email":"sjg@chromium.org"},"delegate":{"id":56520,"url":"http://patchwork.ozlabs.org/api/1.2/users/56520/?format=json","username":"bmeng","first_name":"Bin","last_name":"Meng","email":"bmeng.cn@gmail.com"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20170906014950.132960-6-sjg@chromium.org/mbox/","series":[{"id":1680,"url":"http://patchwork.ozlabs.org/api/1.2/series/1680/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=1680","date":"2017-09-06T01:49:44","name":"x86: bootstage: Fix bootstage operation on link","version":3,"mbox":"http://patchwork.ozlabs.org/series/1680/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/810363/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/810363/checks/","tags":{},"related":[],"headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"ZQlQy3mQ\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xn67f2kV5z9sNq\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed,  6 Sep 2017 11:53:50 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 09196C21E6F; Wed,  6 Sep 2017 01:51:09 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 6A7C2C21EA2;\n\tWed,  6 Sep 2017 01:50:41 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 5570AC21E66; Wed,  6 Sep 2017 01:50:19 +0000 (UTC)","from mail-oi0-f41.google.com (mail-oi0-f41.google.com\n\t[209.85.218.41])\n\tby lists.denx.de (Postfix) with ESMTPS id DF2FEC21D94\n\tfor <u-boot@lists.denx.de>; Wed,  6 Sep 2017 01:50:18 +0000 (UTC)","by mail-oi0-f41.google.com with SMTP id x190so18837881oix.3\n\tfor <u-boot@lists.denx.de>; Tue, 05 Sep 2017 18:50:18 -0700 (PDT)","from kiwi.bld.corp.google.com ([100.101.164.89])\n\tby smtp.gmail.com with ESMTPSA id\n\tm63sm1947539oia.21.2017.09.05.18.50.15\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tTue, 05 Sep 2017 18:50:15 -0700 (PDT)","by kiwi.bld.corp.google.com (Postfix, from userid 121222)\n\tid 91BEB1407FF; Tue,  5 Sep 2017 19:50:14 -0600 (MDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=sender:from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=+NDtmpk1vuvhYHQNSJBJH6vZ2efTE8YIUyHOLaXTIOw=;\n\tb=ZQlQy3mQXN+GRh1oKdRVXQ37JVeukxQyiBey8I63cUd/6tvbcc2JEr+WM9e0dsXo0R\n\t6lrlv7mIQ8zXCtaPLSB5gdXWBLDabn443tA9d/c2Lj/aRn1I8zMM/gE1G//hdeDMCF/o\n\t1jvbrEIsJRtacTQeVEwLjjxOxAjHv14Gd6Fm/qwKMsEcOTyHBqrFgxW+LJcdJGStg8XQ\n\tb/j0mCYA+bgvZdPPa0i43cmJPaSGK27WO7Yi5X8S+Jy8eO6SFEjwqW4X1SoO69ERMKGq\n\tYqzfMGLF890ZMuEC9eBLUKpJ34tcZCai9T6AvUGehtkKl2GOHWXO5le9kZ1jBW1int0C\n\tzHbg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:sender:from:to:cc:subject:date:message-id\n\t:in-reply-to:references;\n\tbh=+NDtmpk1vuvhYHQNSJBJH6vZ2efTE8YIUyHOLaXTIOw=;\n\tb=ZDByX+iZ6CYgR8WekDibOCkqU0hMfUXM/GC+J7TtW9o561HO57W6OSCVrtWxCESJB8\n\tDg2XV4ezKDnPHSnvZ8cNchUG4pmovF+riYGjI9CBSafMZocjlXbhYKjn575LqabSxDSc\n\tXIjfOIvDDOAWKgd1OvBoCApe0C9TwmOMe9x4su4GdDI45SD3Y7ydt8OX9VN31+Alr9sh\n\tMA/Es6bVG1uoEOekIe1AXhTy99c74uvjtHh20Lkdv3uIZB2TfdyV7tSjnCTFlODEXCoF\n\tpVC+sLirK3UMWNU9r4wPcfvzTiSpdeRFSrOh42k5Sfl+MEL0MjdjRA+Af8n6VqLQd1Ak\n\tiBUA==","X-Gm-Message-State":"AHPjjUitd4wWv3pbqGTTL72FOTUcBf+qejxl1n/tiASNpsQDtXXNsTt3\n\tb3yA6tsTuvJyi7V3","X-Google-Smtp-Source":"ADKCNb58kippu0Jww91BUrcnCJr4x6JqMn6amWDcKqIes+nfqu6yy0OiElpV03T0FuSeTpL4Md0iXg==","X-Received":"by 10.202.237.23 with SMTP id l23mr1262201oih.63.1504662617443; \n\tTue, 05 Sep 2017 18:50:17 -0700 (PDT)","From":"Simon Glass <sjg@chromium.org>","To":"U-Boot Mailing List <u-boot@lists.denx.de>","Date":"Tue,  5 Sep 2017 19:49:49 -0600","Message-Id":"<20170906014950.132960-6-sjg@chromium.org>","X-Mailer":"git-send-email 2.14.1.581.gf28d330327-goog","In-Reply-To":"<20170906014950.132960-1-sjg@chromium.org>","References":"<20170906014950.132960-1-sjg@chromium.org>","Subject":"[U-Boot] [PATCH v3 5/6] bootstage: Provide a separate record count\n\tsetting for SPL","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"},"content":"With SPL we often have limited memory and do not need very many bootstage\nrecords. Add a separate Kconfig option for SPL.\n\nSigned-off-by: Simon Glass <sjg@chromium.org>\n---\n\nChanges in v3: None\nChanges in v2: None\n\n common/Kconfig     | 7 +++++++\n common/bootstage.c | 6 +++---\n 2 files changed, 10 insertions(+), 3 deletions(-)","diff":"diff --git a/common/Kconfig b/common/Kconfig\nindex abd4146922..540cc9999b 100644\n--- a/common/Kconfig\n+++ b/common/Kconfig\n@@ -53,6 +53,13 @@ config BOOTSTAGE_RECORD_COUNT\n \t  This is the size of the bootstage record list and is the maximum\n \t  number of bootstage records that can be recorded.\n \n+config SPL_BOOTSTAGE_RECORD_COUNT\n+\tint \"Number of boot stage records to store for SPL\"\n+\tdefault 5\n+\thelp\n+\t  This is the size of the bootstage record list and is the maximum\n+\t  number of bootstage records that can be recorded.\n+\n config BOOTSTAGE_FDT\n \tbool \"Store boot timing information in the OS device tree\"\n \tdepends on BOOTSTAGE\ndiff --git a/common/bootstage.c b/common/bootstage.c\nindex efc99fc681..b866e66979 100644\n--- a/common/bootstage.c\n+++ b/common/bootstage.c\n@@ -18,7 +18,7 @@\n DECLARE_GLOBAL_DATA_PTR;\n \n enum {\n-\tRECORD_COUNT = CONFIG_BOOTSTAGE_RECORD_COUNT,\n+\tRECORD_COUNT = CONFIG_VAL(BOOTSTAGE_RECORD_COUNT),\n };\n \n struct bootstage_record {\n@@ -327,7 +327,7 @@ void bootstage_report(void)\n \t}\n \tif (data->rec_count > RECORD_COUNT)\n \t\tprintf(\"Overflowed internal boot id table by %d entries\\n\"\n-\t\t       \"- please increase CONFIG_BOOTSTAGE_RECORD_COUNT\\n\",\n+\t\t       \"Please increase CONFIG_(SPL_)BOOTSTAGE_RECORD_COUNT\\n\",\n \t\t       data->rec_count - RECORD_COUNT);\n \n \tputs(\"\\nAccumulated time:\\n\");\n@@ -456,7 +456,7 @@ int bootstage_unstash(const void *base, int size)\n \n \tif (data->rec_count + hdr->count > RECORD_COUNT) {\n \t\tdebug(\"%s: Bootstage has %d records, we have space for %d\\n\"\n-\t\t\t\"- please increase CONFIG_BOOTSTAGE_RECORD_COUNT\\n\",\n+\t\t\t\"Please increase CONFIG_(SPL_)BOOTSTAGE_RECORD_COUNT\\n\",\n \t\t      __func__, hdr->count, RECORD_COUNT - data->rec_count);\n \t\treturn -ENOSPC;\n \t}\n","prefixes":["U-Boot","v3","5/6"]}