[{"id":1766952,"web_url":"http://patchwork.ozlabs.org/comment/1766952/","msgid":"<CAPnjgZ2Tz5jQfUvTcc0YY05+edhF+4L0kgCnR3pyryQVZD63+Q@mail.gmail.com>","list_archive_url":null,"date":"2017-09-12T12:28:31","subject":"Re: [U-Boot] [PATCH 09/13] cmd: fastboot: Rework fastboot dependency","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"On 8 September 2017 at 11:34, Maxime Ripard\n<maxime.ripard@free-electrons.com> wrote:\n> Fastboot need a bunch of options to be operating properly, such as the\n> g_dnl gadget, the fastboot command, and some options that make sense. Since\n> fastboot is now part of Kconfig, make sure we have them right.\n>\n> That will also reduce the boilerplate in the defconfigs.\n>\n> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>\n> ---\n>  arch/arm/Kconfig                               | 2 --\n>  cmd/fastboot/Kconfig                           | 4 ++++\n>  configs/A13-OLinuXino_defconfig                | 1 -\n>  configs/A20-OLinuXino-Lime2-eMMC_defconfig     | 1 -\n>  configs/A20-OLinuXino-Lime2_defconfig          | 1 -\n>  configs/CHIP_defconfig                         | 1 -\n>  configs/CHIP_pro_defconfig                     | 1 -\n>  configs/Cubietruck_defconfig                   | 1 -\n>  configs/Nintendo_NES_Classic_Edition_defconfig | 1 -\n>  configs/Sinlinx_SinA33_defconfig               | 1 -\n>  configs/am335x_boneblack_defconfig             | 2 --\n>  configs/am335x_boneblack_vboot_defconfig       | 2 --\n>  configs/am335x_evm_defconfig                   | 2 --\n>  configs/am335x_evm_nor_defconfig               | 2 --\n>  configs/am335x_evm_norboot_defconfig           | 2 --\n>  configs/am335x_evm_spiboot_defconfig           | 2 --\n>  configs/am335x_evm_usbspl_defconfig            | 2 --\n>  configs/am57xx_evm_defconfig                   | 2 --\n>  configs/am57xx_evm_nodt_defconfig              | 3 ---\n>  configs/am57xx_hs_evm_defconfig                | 2 --\n>  configs/bcm23550_w1d_defconfig                 | 3 ---\n>  configs/bcm28155_ap_defconfig                  | 3 ---\n>  configs/birdland_bav335a_defconfig             | 3 ---\n>  configs/birdland_bav335b_defconfig             | 3 ---\n>  configs/cgtqmx6eval_defconfig                  | 3 ---\n>  configs/chromebit_mickey_defconfig             | 1 -\n>  configs/chromebook_jerry_defconfig             | 1 -\n>  configs/chromebook_minnie_defconfig            | 1 -\n>  configs/dra7xx_evm_defconfig                   | 2 --\n>  configs/dra7xx_hs_evm_defconfig                | 2 --\n>  configs/evb-rk3036_defconfig                   | 1 -\n>  configs/evb-rk3229_defconfig                   | 1 -\n>  configs/evb-rk3288_defconfig                   | 1 -\n>  configs/fennec-rk3288_defconfig                | 1 -\n>  configs/firefly-rk3288_defconfig               | 1 -\n>  configs/kc1_defconfig                          | 3 ---\n>  configs/kylin-rk3036_defconfig                 | 1 -\n>  configs/miqi-rk3288_defconfig                  | 1 -\n>  configs/mx6qsabrelite_defconfig                | 3 ---\n>  configs/nitrogen6dl2g_defconfig                | 3 ---\n>  configs/nitrogen6dl_defconfig                  | 3 ---\n>  configs/nitrogen6q2g_defconfig                 | 3 ---\n>  configs/nitrogen6q_defconfig                   | 3 ---\n>  configs/nitrogen6s1g_defconfig                 | 3 ---\n>  configs/nitrogen6s_defconfig                   | 3 ---\n>  configs/omap3_beagle_defconfig                 | 3 ---\n>  configs/omap3_logic_defconfig                  | 2 --\n>  configs/parrot_r16_defconfig                   | 1 -\n>  configs/phycore-rk3288_defconfig               | 1 -\n>  configs/popmetal-rk3288_defconfig              | 1 -\n>  configs/rock2_defconfig                        | 1 -\n>  configs/sniper_defconfig                       | 3 ---\n>  configs/tinker-rk3288_defconfig                | 1 -\n>  53 files changed, 4 insertions(+), 97 deletions(-)\n\nReviewed-by: Simon Glass <sjg@chromium.org>","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=\"QxcH3LFy\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"NNWwt2Jf\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs45p46N1z9t4V\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 12 Sep 2017 22:35:40 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid A6820C21FED; Tue, 12 Sep 2017 12:30:44 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BBF83C22001;\n\tTue, 12 Sep 2017 12:30:08 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A9763C21FE9; Tue, 12 Sep 2017 12:28:56 +0000 (UTC)","from mail-qt0-f169.google.com (mail-qt0-f169.google.com\n\t[209.85.216.169])\n\tby lists.denx.de (Postfix) with ESMTPS id DBDFFC21FAB\n\tfor <u-boot@lists.denx.de>; Tue, 12 Sep 2017 12:28:52 +0000 (UTC)","by mail-qt0-f169.google.com with SMTP id m35so25125321qte.1\n\tfor <u-boot@lists.denx.de>; Tue, 12 Sep 2017 05:28:52 -0700 (PDT)","by 10.200.37.200 with HTTP; Tue, 12 Sep 2017 05:28:31 -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;\n\tbh=aoTIqtaKw1rTzXZgJCxvzuNTSmWbzJkpNsLFfqgFVvk=;\n\tb=QxcH3LFyGKZvemCANJDCCJ6hwALQ5a1vbzMv7/pnKYNBdRyhtbLd1v6Eox8+L2G4oH\n\t786IPNc7Ml0q8h6YIK5laV7zpM09z32eAITLz7koneUKBAjs6gtidEhjkmH8ML9rRuM3\n\tfx0lrt7TKlAIo114eflJ7MUojXR/soTqnz4NyPTNyuRrY/SR/+pIJY+48B61Xt71pldI\n\tfiw8tiN0DOlOh5MCQBDK0EJsJRxZca/YTwOheW8vpqIViz0hlLWaKljIT5rgAS3EvpWl\n\tncclRILmQ4beCsBznMGFP9KfPwD2qfqH563Oo4+tCmOkHe6saiP4eIOGkBp7p6taq93k\n\t3KTA==","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;\n\tbh=aoTIqtaKw1rTzXZgJCxvzuNTSmWbzJkpNsLFfqgFVvk=;\n\tb=NNWwt2Jf7W2epKImSO8uHlrbE7c5PO3V+xcPZWzqreLoLWR2QaB/uTWYWrpLT3ZdAO\n\tTabB9EZl8DS2NbnItJYGkJ+SH5AUUluNros1Npwp0bjh+2qsUIOHTR5lUcY7Yx8UUa1W\n\tlC/a0EMOG5lphgu9xVlJhxGfXRgBLWRK9TTm8="],"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;\n\tbh=aoTIqtaKw1rTzXZgJCxvzuNTSmWbzJkpNsLFfqgFVvk=;\n\tb=GyChxFMo8UkeZxzMZdC9WsZaUtCy8sqsAWbdRVFGq5vQnNXEnaYifsdjJ51jVykLoT\n\t0KyaKoKNiDtlg2v1vENdaPmzxdouMUKaJiIpPsz3rECSMQ1QIMqWkn1Uz+0Gh1WcheNy\n\tTnk3HlNOirpacdcENW2kCsHh3eXVUMSI/grQaqbnaSgDmgmNHWp+N1slQUHNPh+m6Zty\n\tf/vxF2xQBlNuQsy0/nW6x8WzTeU0q8422N6d/Vfj/VPIBKSB+y8KJX6ErNvMMmL723uE\n\tosdNu99c1Qe50+0AC2cRZOFhXMpdLY66D8T9psX09K2ijOJDfh0HPuoOdJPxqrXfzNqT\n\tXMqg==","X-Gm-Message-State":"AHPjjUjSzZIND+xzbeSyLiqqztlS715sw9iQcPOmUGGZuA2dKYwmqtSQ\n\t/9fomjohfUgUEbkz7M0fktB0BeF4QpTCLawWoegfgQ==","X-Google-Smtp-Source":"AOwi7QDVDBAQyC7aEsKs0Esev1ptgHqc5Qu0qecTIddAYxIB2CnLFN4Cll/xgSHrdEnDAf9V9Kqpmij+qhTKQu7HHK4=","X-Received":"by 10.200.41.239 with SMTP id 44mr19796922qtt.138.1505219331657; \n\tTue, 12 Sep 2017 05:28:51 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<2baebcd54ad7fed6d22a56d3dae4b15b8f81443c.1504891631.git-series.maxime.ripard@free-electrons.com>","References":"<cover.3b9463250a55d25afdf3b221b80a90b4e7811c1e.1504891631.git-series.maxime.ripard@free-electrons.com>\n\t<2baebcd54ad7fed6d22a56d3dae4b15b8f81443c.1504891631.git-series.maxime.ripard@free-electrons.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Tue, 12 Sep 2017 06:28:31 -0600","X-Google-Sender-Auth":"x1vXGyNwXauRfbbzaJ8Y68a-FS0","Message-ID":"<CAPnjgZ2Tz5jQfUvTcc0YY05+edhF+4L0kgCnR3pyryQVZD63+Q@mail.gmail.com>","To":"Maxime Ripard <maxime.ripard@free-electrons.com>","Cc":"Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,\n\tTom Rini <trini@konsulko.com>, Marek Vasut <marex@denx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tJagan Teki <jagan@openedev.com>","Subject":"Re: [U-Boot] [PATCH 09/13] cmd: fastboot: Rework fastboot dependency","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>"}}]