{"id":806261,"url":"http://patchwork.ozlabs.org/api/1.0/patches/806261/?format=json","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.0/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},"msgid":"<20170827152351.20609-6-sjg@chromium.org>","date":"2017-08-27T15:23:50","name":"[U-Boot,v2,5/6] bootstage: Provide a separate record count setting for SPL","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"a9f7a420365dfc33a651139714992605fe00dcb2","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/1.0/people/6170/?format=json","name":"Simon Glass","email":"sjg@chromium.org"},"delegate":{"id":56520,"url":"http://patchwork.ozlabs.org/api/1.0/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/20170827152351.20609-6-sjg@chromium.org/mbox/","series":[{"id":26,"url":"http://patchwork.ozlabs.org/api/1.0/series/26/?format=json","date":"2017-08-27T15:23:45","name":"x86: bootstage: Fix bootstage operation on link","version":2,"mbox":"http://patchwork.ozlabs.org/series/26/mbox/"}],"check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/806261/checks/","tags":{},"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=\"O8ufUiEB\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xgJgj1sB1z9s8V\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 28 Aug 2017 01:28:01 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 08B85C22229; Sun, 27 Aug 2017 15:24:37 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BB02DC2223D;\n\tSun, 27 Aug 2017 15:24:23 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D4573C22235; Sun, 27 Aug 2017 15:24:08 +0000 (UTC)","from mail-oi0-f50.google.com (mail-oi0-f50.google.com\n\t[209.85.218.50])\n\tby lists.denx.de (Postfix) with ESMTPS id C350EC22238\n\tfor <u-boot@lists.denx.de>; Sun, 27 Aug 2017 15:24:04 +0000 (UTC)","by mail-oi0-f50.google.com with SMTP id k77so29324578oib.2\n\tfor <u-boot@lists.denx.de>; Sun, 27 Aug 2017 08:24:04 -0700 (PDT)","from kaki.bld.corp.google.com ([100.100.197.131])\n\tby smtp.gmail.com with ESMTPSA id\n\te136sm16358353oih.36.2017.08.27.08.24.01\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSun, 27 Aug 2017 08:24:01 -0700 (PDT)","by kaki.bld.corp.google.com (Postfix, from userid 121222)\n\tid 94A2940199; Sun, 27 Aug 2017 09:24:00 -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=WeR0BaKXHmGApoL46/+NCU591OZfRzuUVZi7Rt8m6x4=;\n\tb=O8ufUiEBxHKpmyKFIoSrjJmiSyLBBzmE0jX0rIDIXV6thuvIC69jbmoA1XCXiuy+4L\n\tUiylsufcjj9/3LPwNOOyaMrsl4JJ6la9uTNQyDW6PDhW1vOLWq/RdzpyjogBi+PwGIKd\n\t1bNCyb3Sq4S2U57vZS2xHVYoMnrh/NMl8oE6zWOO6fMbtYMJ25j9GZEOKUaO12SDYuUX\n\tsEiGMsPZilUMqi1zzh7/m+nxehEZ2sABxxd2Wfb6OBlXG0aX0ISF45LnggOAkLlA0eDj\n\traGSVEwhb0AVFIEZTFvYSnl0oxs0lTVwNdY1sJ/bbA+NQfqtQCD2ZzIYkhVNKhCbGs9u\n\t4DyQ==","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=WeR0BaKXHmGApoL46/+NCU591OZfRzuUVZi7Rt8m6x4=;\n\tb=slE9XDebCxEBOC0ke32M9f+TtV1zxD7G93XKqn6qgtRWDgAC0i8VDCdhzXx0VK+9cW\n\t90piaPrvUzB4aECAPed5jjmtKmLVsLJsjb8QN6C/LroLihI/yIsgRrYvC36h82xEhmld\n\tpLeuBq4TQlxSmL3UciSPpI1J2sCu2AehjQrcA0ermCVHAEnn1tA/HnojZJtCTsISlxS3\n\tOYXXYrO+/kxsrMDXvyh6Q9RKk1RLLy46hFUhlB1eBOQCgKWMQXrFs0Ifur86wErtm9VB\n\tzNAG7GtK+d672Odgtwo8g7Cn6g7tRXx2ggPhncOGJ9iUR2+lnNNhfrvsCC0X/1yTgNof\n\tfR7A==","X-Gm-Message-State":"AHYfb5gTruteBVpR7+jBgQwyR89TBaWKUoF+Mdm13ILW0ZNvLKbbDBGm\n\tVnA0lLqv+aTTREA8","X-Google-Smtp-Source":"ADKCNb4gp71uYkeQriiRHJhix3qMlQ/w1Xwg//0m4D0l15fdcwq5YvfKcJwR2d9qDYLDJ0uxMp5SUw==","X-Received":"by 10.202.104.33 with SMTP id d33mr6225476oic.58.1503847443383; \n\tSun, 27 Aug 2017 08:24:03 -0700 (PDT)","From":"Simon Glass <sjg@chromium.org>","To":"U-Boot Mailing List <u-boot@lists.denx.de>","Date":"Sun, 27 Aug 2017 09:23:50 -0600","Message-Id":"<20170827152351.20609-6-sjg@chromium.org>","X-Mailer":"git-send-email 2.14.1.342.g6490525c54-goog","In-Reply-To":"<20170827152351.20609-1-sjg@chromium.org>","References":"<20170827152351.20609-1-sjg@chromium.org>","Cc":"Jagan Teki <jagan@openedev.com>,\n\tMaxime Ripard <maxime.ripard@free-electrons.com>","Subject":"[U-Boot] [PATCH v2 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 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 abd4146922e..540cc9999bc 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 efc99fc6810..b866e66979e 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","v2","5/6"]}