{"id":810364,"url":"http://patchwork.ozlabs.org/api/1.2/patches/810364/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20170906014950.132960-5-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-5-sjg@chromium.org>","list_archive_url":null,"date":"2017-09-06T01:49:48","name":"[U-Boot,v3,4/6] bootstage: Drop unused options","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"a80dc193ed0271217e03281ea0c54ba9c7cbae68","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-5-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/810364/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/810364/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=\"cwhts53p\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xn6843yQKz9sNq\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed,  6 Sep 2017 11:54:12 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 4B3D1C21DBA; Wed,  6 Sep 2017 01:51:38 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5B7EFC21E27;\n\tWed,  6 Sep 2017 01:51:28 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 5A5CCC21DC0; Wed,  6 Sep 2017 01:50:19 +0000 (UTC)","from mail-oi0-f47.google.com (mail-oi0-f47.google.com\n\t[209.85.218.47])\n\tby lists.denx.de (Postfix) with ESMTPS id 742C5C21DB9\n\tfor <u-boot@lists.denx.de>; Wed,  6 Sep 2017 01:50:19 +0000 (UTC)","by mail-oi0-f47.google.com with SMTP id n18so34507521oig.2\n\tfor <u-boot@lists.denx.de>; Tue, 05 Sep 2017 18:50:19 -0700 (PDT)","from kiwi.bld.corp.google.com ([100.101.164.89])\n\tby smtp.gmail.com with ESMTPSA id\n\t188sm2012872oid.5.2017.09.05.18.50.14\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 A7460140719; Tue,  5 Sep 2017 19:50:13 -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_H2,\n\tT_DKIM_INVALID autolearn=unavailable autolearn_force=no\n\tversion=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=v28r37qJN+A0Iuk5BBDMTNnts8I+74OYTTCpF4Yu8Gw=;\n\tb=cwhts53psERBHo8Ofkl2CStY5wtS0cXK6DQrhLQQx+r8pOH646m88pFofJz/oXLG8F\n\t3J9gKZVHZlOojaDGWP+ICbZqX7CKOoNbjKm0VVnb1PvfzJhoDMurxzqlTo2mo3DP+8u5\n\toqz0LXg+naiLslhhDPofefKVZYbjaejigntpeJZSOJse72yrvlMoKFAjYuWVvmZBLF/j\n\tFijiJ6Y2G7+zgJ3FSy5L6yWTkwV9HKEVckmklVg4Y4c172822rDZ8k/ow/HaxRE2YUIM\n\tzQ2WkBrGac6aHt7yTJ+o7zVlSZBaT/FveXscZqNU5g+03BVxgY8Dm8JCLHYx2+DiNbdW\n\tjP6A==","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=v28r37qJN+A0Iuk5BBDMTNnts8I+74OYTTCpF4Yu8Gw=;\n\tb=BucudXNJYRaCWFMPsrocgpVGapr9YQVv7+fq1vZK25S4kMYXX/YR0XvwpwjmNG1Ndc\n\tweK8T0NLG4HYvahDKyMAgGboDTHO49eUeZpUxzqDAjE0tXT41rd3Lf5Lf7f58n05mA8a\n\tkgRwh4BiINkwqltaaQ2opd+BCq5LT0gBy8J58Rp7mtbGM6kwd3Ij07mp2ShRVzUfEyDC\n\t7cRVliPSoX83ynDKUYH5SWkW3blWb8tVOuwl0+Zh9HAKpHitgesQi2JLHVARQFXRIs1Y\n\tnKgDh+W2/U0Dl6RjoJvzZF4dsiM1k4CaZs9TCArMonhUuyhQXGBeDf80zFhted69b8zN\n\t8R5w==","X-Gm-Message-State":"AHPjjUiYOdT71Ihwaj4zPluilrkk8QYRwaXIL4TXdeJP8GaXaqZfdC+n\n\t7EBzKpnD/SjINZvb","X-Google-Smtp-Source":"ADKCNb6xoVaWCinyo4fwR2Orw6+0HjjZywEr/hI9pOwothsW1eJ1Cd+Uq5Pl7iGSw9CpIQo2CbZNFQ==","X-Received":"by 10.202.76.144 with SMTP id z138mr1107252oia.245.1504662618023;\n\tTue, 05 Sep 2017 18:50:18 -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:48 -0600","Message-Id":"<20170906014950.132960-5-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 4/6] bootstage: Drop unused options","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":"The CONFIG_BOOTSTAGE_USER_COUNT option is no-longer needed since we can now\nsupport any number of user IDs. Also BOOTSTAGE_ID_COUNT is not needed now.\n\nDrop these unused options.\n\nSigned-off-by: Simon Glass <sjg@chromium.org>\nReviewed-by: Bin Meng <bmeng.cn@gmail.com>\n---\n\nChanges in v3: None\nChanges in v2: None\n\n common/Kconfig                     | 9 ---------\n common/bootstage.c                 | 2 +-\n configs/sandbox_defconfig          | 1 -\n configs/sandbox_flattree_defconfig | 1 -\n configs/sandbox_noblk_defconfig    | 1 -\n configs/sandbox_spl_defconfig      | 1 -\n include/bootstage.h                | 6 ------\n 7 files changed, 1 insertion(+), 20 deletions(-)","diff":"diff --git a/common/Kconfig b/common/Kconfig\nindex 4d8cae9610..abd4146922 100644\n--- a/common/Kconfig\n+++ b/common/Kconfig\n@@ -46,15 +46,6 @@ config BOOTSTAGE_REPORT\n \t\t 29,916,167 26,005,792  bootm_start\n \t\t 30,361,327    445,160  start_kernel\n \n-config BOOTSTAGE_USER_COUNT\n-\tint \"Number of boot ID numbers available for user use\"\n-\tdefault 20\n-\thelp\n-\t  This is the number of available user bootstage records.\n-\t  Each time you call bootstage_mark(BOOTSTAGE_ID_ALLOC, ...)\n-\t  a new ID will be allocated from this stash. If you exceed\n-\t  the limit, recording will stop.\n-\n config BOOTSTAGE_RECORD_COUNT\n \tint \"Number of boot stage records to store\"\n \tdefault 30\ndiff --git a/common/bootstage.c b/common/bootstage.c\nindex 61479d7f07..efc99fc681 100644\n--- a/common/bootstage.c\n+++ b/common/bootstage.c\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_USER_COUNT\\n\",\n+\t\t\t\"- please increase CONFIG_BOOTSTAGE_RECORD_COUNT\\n\",\n \t\t      __func__, hdr->count, RECORD_COUNT - data->rec_count);\n \t\treturn -ENOSPC;\n \t}\ndiff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig\nindex 4c4e4809be..5d8440fbcc 100644\n--- a/configs/sandbox_defconfig\n+++ b/configs/sandbox_defconfig\n@@ -7,7 +7,6 @@ CONFIG_FIT_SIGNATURE=y\n CONFIG_FIT_VERBOSE=y\n CONFIG_BOOTSTAGE=y\n CONFIG_BOOTSTAGE_REPORT=y\n-CONFIG_BOOTSTAGE_USER_COUNT=32\n CONFIG_BOOTSTAGE_FDT=y\n CONFIG_BOOTSTAGE_STASH=y\n CONFIG_BOOTSTAGE_STASH_ADDR=0x0\ndiff --git a/configs/sandbox_flattree_defconfig b/configs/sandbox_flattree_defconfig\nindex 325f4ff57f..50ea35e64b 100644\n--- a/configs/sandbox_flattree_defconfig\n+++ b/configs/sandbox_flattree_defconfig\n@@ -7,7 +7,6 @@ CONFIG_FIT_SIGNATURE=y\n CONFIG_FIT_VERBOSE=y\n CONFIG_BOOTSTAGE=y\n CONFIG_BOOTSTAGE_REPORT=y\n-CONFIG_BOOTSTAGE_USER_COUNT=32\n CONFIG_BOOTSTAGE_FDT=y\n CONFIG_BOOTSTAGE_STASH=y\n CONFIG_BOOTSTAGE_STASH_ADDR=0x0\ndiff --git a/configs/sandbox_noblk_defconfig b/configs/sandbox_noblk_defconfig\nindex e152917fe4..693c14bc60 100644\n--- a/configs/sandbox_noblk_defconfig\n+++ b/configs/sandbox_noblk_defconfig\n@@ -6,7 +6,6 @@ CONFIG_FIT_SIGNATURE=y\n CONFIG_FIT_VERBOSE=y\n CONFIG_BOOTSTAGE=y\n CONFIG_BOOTSTAGE_REPORT=y\n-CONFIG_BOOTSTAGE_USER_COUNT=32\n CONFIG_BOOTSTAGE_FDT=y\n CONFIG_BOOTSTAGE_STASH=y\n CONFIG_BOOTSTAGE_STASH_ADDR=0x0\ndiff --git a/configs/sandbox_spl_defconfig b/configs/sandbox_spl_defconfig\nindex be4a85da33..db9129456b 100644\n--- a/configs/sandbox_spl_defconfig\n+++ b/configs/sandbox_spl_defconfig\n@@ -13,7 +13,6 @@ CONFIG_FIT_VERBOSE=y\n CONFIG_SPL_LOAD_FIT=y\n CONFIG_BOOTSTAGE=y\n CONFIG_BOOTSTAGE_REPORT=y\n-CONFIG_BOOTSTAGE_USER_COUNT=32\n CONFIG_BOOTSTAGE_FDT=y\n CONFIG_BOOTSTAGE_STASH=y\n CONFIG_BOOTSTAGE_STASH_ADDR=0x0\ndiff --git a/include/bootstage.h b/include/bootstage.h\nindex c5d93f57fd..7a524782ba 100644\n--- a/include/bootstage.h\n+++ b/include/bootstage.h\n@@ -12,11 +12,6 @@\n #ifndef _BOOTSTAGE_H\n #define _BOOTSTAGE_H\n \n-/* Define this for host tools */\n-#ifndef CONFIG_BOOTSTAGE_USER_COUNT\n-#define CONFIG_BOOTSTAGE_USER_COUNT\t20\n-#endif\n-\n /* Flags for each bootstage record */\n enum bootstage_flags {\n \tBOOTSTAGEF_ERROR\t= 1 << 0,\t/* Error record */\n@@ -208,7 +203,6 @@ enum bootstage_id {\n \n \t/* a few spare for the user, from here */\n \tBOOTSTAGE_ID_USER,\n-\tBOOTSTAGE_ID_COUNT = BOOTSTAGE_ID_USER + CONFIG_BOOTSTAGE_USER_COUNT,\n \tBOOTSTAGE_ID_ALLOC,\n };\n \n","prefixes":["U-Boot","v3","4/6"]}