[{"id":1758789,"web_url":"http://patchwork.ozlabs.org/comment/1758789/","msgid":"<be33a789-4a68-8cfb-490b-d6ccac59934a@denx.de>","list_archive_url":null,"date":"2017-08-28T19:00:20","subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","submitter":{"id":70701,"url":"http://patchwork.ozlabs.org/api/people/70701/","name":"Lukasz Majewski","email":"lukma@denx.de"},"content":"Hi Anatolij,\n\n> bss section is cleared in crt0.S. board_init_r() is also\n> entered from crt0 code.\n\nReviewed-by: Lukasz Majewski <lukma@denx.de>\n\n>\n> Signed-off-by: Anatolij Gustschin <agust@denx.de>\n> Cc: Christian Gmeiner <christian.gmeiner@gmail.com>\n> Cc: Nikita Kiryanov <nikita@compulab.co.il>\n> Cc: Jagan Teki <jagan@amarulasolutions.com>\n> Cc: Tim Harvey <tharvey@gateworks.com>\n> Cc: Marek Vasut <marex@denx.de>\n> Cc: Lukasz Majewski <lukma@denx.de>\n> Cc: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n>  board/bachmann/ot1200/ot1200_spl.c          | 6 ------\n>  board/compulab/cm_fx6/spl.c                 | 3 ---\n>  board/engicam/common/spl.c                  | 6 ------\n>  board/gateworks/gw_ventana/gw_ventana_spl.c | 3 ---\n>  board/kosagi/novena/novena_spl.c            | 6 ------\n>  board/liebherr/mccmon6/spl.c                | 6 ------\n>  board/udoo/udoo_spl.c                       | 6 ------\n>  board/wandboard/spl.c                       | 6 ------\n>  8 files changed, 42 deletions(-)\n>\n> diff --git a/board/bachmann/ot1200/ot1200_spl.c b/board/bachmann/ot1200/ot1200_spl.c\n> index 9d28da4..f3dff95 100644\n> --- a/board/bachmann/ot1200/ot1200_spl.c\n> +++ b/board/bachmann/ot1200/ot1200_spl.c\n> @@ -151,10 +151,4 @@ void board_init_f(ulong dummy)\n>\n>  \t/* configure MMDC for SDRAM width/size and per-model calibration */\n>  \tot1200_spl_dram_init();\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n> diff --git a/board/compulab/cm_fx6/spl.c b/board/compulab/cm_fx6/spl.c\n> index bba977f..56aac60 100644\n> --- a/board/compulab/cm_fx6/spl.c\n> +++ b/board/compulab/cm_fx6/spl.c\n> @@ -336,9 +336,6 @@ void board_init_f(ulong dummy)\n>  \t\tputs(\"!!!ERROR!!! DRAM detection failed!!!\\n\");\n>  \t\thang();\n>  \t}\n> -\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\tboard_init_r(NULL, 0);\n>  }\n>\n>  void board_boot_order(u32 *spl_boot_list)\n> diff --git a/board/engicam/common/spl.c b/board/engicam/common/spl.c\n> index a8a7cf3..1f2826a 100644\n> --- a/board/engicam/common/spl.c\n> +++ b/board/engicam/common/spl.c\n> @@ -384,10 +384,4 @@ void board_init_f(ulong dummy)\n>\n>  \t/* DDR initialization */\n>  \tspl_dram_init();\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n> diff --git a/board/gateworks/gw_ventana/gw_ventana_spl.c b/board/gateworks/gw_ventana/gw_ventana_spl.c\n> index 9524da7..e107f15 100644\n> --- a/board/gateworks/gw_ventana/gw_ventana_spl.c\n> +++ b/board/gateworks/gw_ventana/gw_ventana_spl.c\n> @@ -637,9 +637,6 @@ void board_init_f(ulong dummy)\n>  \tspl_dram_init(8 << ventana_info.sdram_width,\n>  \t\t      16 << ventana_info.sdram_size,\n>  \t\t      board_model);\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n>  }\n>\n>  void board_boot_order(u32 *spl_boot_list)\n> diff --git a/board/kosagi/novena/novena_spl.c b/board/kosagi/novena/novena_spl.c\n> index 3645b75..ffc99b8 100644\n> --- a/board/kosagi/novena/novena_spl.c\n> +++ b/board/kosagi/novena/novena_spl.c\n> @@ -607,10 +607,4 @@ void board_init_f(ulong dummy)\n>  \tudelay(100);\n>  \tmmdc_do_write_level_calibration(&novena_ddr_info);\n>  \tmmdc_do_dqs_calibration(&novena_ddr_info);\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n> diff --git a/board/liebherr/mccmon6/spl.c b/board/liebherr/mccmon6/spl.c\n> index 15844ef..56e7d3b 100644\n> --- a/board/liebherr/mccmon6/spl.c\n> +++ b/board/liebherr/mccmon6/spl.c\n> @@ -307,11 +307,5 @@ void board_init_f(ulong dummy)\n>\n>  \t/* DDR initialization */\n>  \tspl_dram_init();\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n>  #endif\n> diff --git a/board/udoo/udoo_spl.c b/board/udoo/udoo_spl.c\n> index e83e7c3..34ef413 100644\n> --- a/board/udoo/udoo_spl.c\n> +++ b/board/udoo/udoo_spl.c\n> @@ -263,11 +263,5 @@ void board_init_f(ulong dummy)\n>\n>  \t/* DDR initialization */\n>  \tspl_dram_init();\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n>  #endif\n> diff --git a/board/wandboard/spl.c b/board/wandboard/spl.c\n> index 47082a8..c3f5fff 100644\n> --- a/board/wandboard/spl.c\n> +++ b/board/wandboard/spl.c\n> @@ -311,11 +311,5 @@ void board_init_f(ulong dummy)\n>\n>  \t/* DDR initialization */\n>  \tspl_dram_init();\n> -\n> -\t/* Clear the BSS. */\n> -\tmemset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -\t/* load/boot image from boot device */\n> -\tboard_init_r(NULL, 0);\n>  }\n>  #endif\n>","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>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xh1Lh5cXFz9sNq\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 29 Aug 2017 05:00:44 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 17BB7C22600; Mon, 28 Aug 2017 19:00:29 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 8A264C225F8;\n\tMon, 28 Aug 2017 19:00:26 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid CA8B5C225B3; Mon, 28 Aug 2017 19:00:24 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id 5BDD2C225F7\n\tfor <u-boot@lists.denx.de>; Mon, 28 Aug 2017 19:00:24 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xh1LJ0wmdz1rCgQ;\n\tMon, 28 Aug 2017 21:00:24 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xh1LH6826z3jgYb;\n\tMon, 28 Aug 2017 21:00:23 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id 2e_ythur9y84; Mon, 28 Aug 2017 21:00:21 +0200 (CEST)","from [192.168.2.222] (89-77-92-62.dynamic.chello.pl [89.77.92.62])\n\t(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 28 Aug 2017 21:00:21 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"5ZQFkw99LkZghBU/OsDFq0m6hAdglp6jnHa6tVqPGuk=","To":"Anatolij Gustschin <agust@denx.de>, u-boot@lists.denx.de","References":"<1503946718-23973-1-git-send-email-agust@denx.de>","From":"=?utf-8?q?=C5=81ukasz_Majewski?= <lukma@denx.de>","Organization":"DENX","Message-ID":"<be33a789-4a68-8cfb-490b-d6ccac59934a@denx.de>","Date":"Mon, 28 Aug 2017 21:00:20 +0200","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":"<1503946718-23973-1-git-send-email-agust@denx.de>","Cc":"Marek Vasut <marex@denx.de>, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1759296,"web_url":"http://patchwork.ozlabs.org/comment/1759296/","msgid":"<CAH9NwWeu=rSWxAHfTGdhd4xGnw+T8UmiXnjwyvqbMCt7hyZT+A@mail.gmail.com>","list_archive_url":null,"date":"2017-08-29T11:02:27","subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","submitter":{"id":17280,"url":"http://patchwork.ozlabs.org/api/people/17280/","name":"Christian Gmeiner","email":"christian.gmeiner@gmail.com"},"content":"2017-08-28 20:58 GMT+02:00 Anatolij Gustschin <agust@denx.de>:\n> bss section is cleared in crt0.S. board_init_r() is also\n> entered from crt0 code.\n>\n> Signed-off-by: Anatolij Gustschin <agust@denx.de>\n\nReviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>\n\n> Cc: Christian Gmeiner <christian.gmeiner@gmail.com>\n> Cc: Nikita Kiryanov <nikita@compulab.co.il>\n> Cc: Jagan Teki <jagan@amarulasolutions.com>\n> Cc: Tim Harvey <tharvey@gateworks.com>\n> Cc: Marek Vasut <marex@denx.de>\n> Cc: Lukasz Majewski <lukma@denx.de>\n> Cc: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n>  board/bachmann/ot1200/ot1200_spl.c          | 6 ------\n>  board/compulab/cm_fx6/spl.c                 | 3 ---\n>  board/engicam/common/spl.c                  | 6 ------\n>  board/gateworks/gw_ventana/gw_ventana_spl.c | 3 ---\n>  board/kosagi/novena/novena_spl.c            | 6 ------\n>  board/liebherr/mccmon6/spl.c                | 6 ------\n>  board/udoo/udoo_spl.c                       | 6 ------\n>  board/wandboard/spl.c                       | 6 ------\n>  8 files changed, 42 deletions(-)\n>\n> diff --git a/board/bachmann/ot1200/ot1200_spl.c b/board/bachmann/ot1200/ot1200_spl.c\n> index 9d28da4..f3dff95 100644\n> --- a/board/bachmann/ot1200/ot1200_spl.c\n> +++ b/board/bachmann/ot1200/ot1200_spl.c\n> @@ -151,10 +151,4 @@ void board_init_f(ulong dummy)\n>\n>         /* configure MMDC for SDRAM width/size and per-model calibration */\n>         ot1200_spl_dram_init();\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n> diff --git a/board/compulab/cm_fx6/spl.c b/board/compulab/cm_fx6/spl.c\n> index bba977f..56aac60 100644\n> --- a/board/compulab/cm_fx6/spl.c\n> +++ b/board/compulab/cm_fx6/spl.c\n> @@ -336,9 +336,6 @@ void board_init_f(ulong dummy)\n>                 puts(\"!!!ERROR!!! DRAM detection failed!!!\\n\");\n>                 hang();\n>         }\n> -\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -       board_init_r(NULL, 0);\n>  }\n>\n>  void board_boot_order(u32 *spl_boot_list)\n> diff --git a/board/engicam/common/spl.c b/board/engicam/common/spl.c\n> index a8a7cf3..1f2826a 100644\n> --- a/board/engicam/common/spl.c\n> +++ b/board/engicam/common/spl.c\n> @@ -384,10 +384,4 @@ void board_init_f(ulong dummy)\n>\n>         /* DDR initialization */\n>         spl_dram_init();\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n> diff --git a/board/gateworks/gw_ventana/gw_ventana_spl.c b/board/gateworks/gw_ventana/gw_ventana_spl.c\n> index 9524da7..e107f15 100644\n> --- a/board/gateworks/gw_ventana/gw_ventana_spl.c\n> +++ b/board/gateworks/gw_ventana/gw_ventana_spl.c\n> @@ -637,9 +637,6 @@ void board_init_f(ulong dummy)\n>         spl_dram_init(8 << ventana_info.sdram_width,\n>                       16 << ventana_info.sdram_size,\n>                       board_model);\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n>  }\n>\n>  void board_boot_order(u32 *spl_boot_list)\n> diff --git a/board/kosagi/novena/novena_spl.c b/board/kosagi/novena/novena_spl.c\n> index 3645b75..ffc99b8 100644\n> --- a/board/kosagi/novena/novena_spl.c\n> +++ b/board/kosagi/novena/novena_spl.c\n> @@ -607,10 +607,4 @@ void board_init_f(ulong dummy)\n>         udelay(100);\n>         mmdc_do_write_level_calibration(&novena_ddr_info);\n>         mmdc_do_dqs_calibration(&novena_ddr_info);\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n> diff --git a/board/liebherr/mccmon6/spl.c b/board/liebherr/mccmon6/spl.c\n> index 15844ef..56e7d3b 100644\n> --- a/board/liebherr/mccmon6/spl.c\n> +++ b/board/liebherr/mccmon6/spl.c\n> @@ -307,11 +307,5 @@ void board_init_f(ulong dummy)\n>\n>         /* DDR initialization */\n>         spl_dram_init();\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n>  #endif\n> diff --git a/board/udoo/udoo_spl.c b/board/udoo/udoo_spl.c\n> index e83e7c3..34ef413 100644\n> --- a/board/udoo/udoo_spl.c\n> +++ b/board/udoo/udoo_spl.c\n> @@ -263,11 +263,5 @@ void board_init_f(ulong dummy)\n>\n>         /* DDR initialization */\n>         spl_dram_init();\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n>  #endif\n> diff --git a/board/wandboard/spl.c b/board/wandboard/spl.c\n> index 47082a8..c3f5fff 100644\n> --- a/board/wandboard/spl.c\n> +++ b/board/wandboard/spl.c\n> @@ -311,11 +311,5 @@ void board_init_f(ulong dummy)\n>\n>         /* DDR initialization */\n>         spl_dram_init();\n> -\n> -       /* Clear the BSS. */\n> -       memset(__bss_start, 0, __bss_end - __bss_start);\n> -\n> -       /* load/boot image from boot device */\n> -       board_init_r(NULL, 0);\n>  }\n>  #endif\n> --\n> 2.7.4\n>\n\ngreets\n--\nChristian Gmeiner, MSc\n\nhttps://christian-gmeiner.info","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=gmail.com header.i=@gmail.com\n\theader.b=\"DxZrWU4D\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xhT1D0tBhz9t1t\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 29 Aug 2017 22:47:04 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 5D2BFC22042; Tue, 29 Aug 2017 11:30:45 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 1B73EC21C39;\n\tTue, 29 Aug 2017 11:30:32 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 018F3C21FCB; Tue, 29 Aug 2017 11:02:48 +0000 (UTC)","from mail-wm0-f67.google.com (mail-wm0-f67.google.com\n\t[74.125.82.67])\n\tby lists.denx.de (Postfix) with ESMTPS id 34567C2203F\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 11:02:48 +0000 (UTC)","by mail-wm0-f67.google.com with SMTP id x189so3323122wmg.4\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 04:02:48 -0700 (PDT)","by 10.28.109.23 with HTTP; Tue, 29 Aug 2017 04:02:27 -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=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=6WajgU0jx1k/LaWqXNKrzUTcoHowHMP54dPlaniwl3c=;\n\tb=DxZrWU4DXAmINXSebw3U6dhtTGeePjTKoZ7KK4C6kTvZNNwtSa/xrIgpWkmazNGKWk\n\tbPHnvuv+bDXLgBHDxGvsmjfBiJDXFXi5cFZWnrLnUITzcegTLMDNVfNZoTSAqEV2xAR2\n\tM3+yX9VVOfXtr63rB9w6IL3nEowQsQCoK8/AalzD6Rd12g8b5/0N0AqLiaKaeBafl5zO\n\tTcJLqWFHnA31jVoQpTwseG7rm7qNdDjLQbyqd75xoPBPKwCtqt+1XnvgQQc9uJDIDOWf\n\t2SRYm3rzTyoiKKYnWq4qt7GNjRSiduY58LJyF3FFq62CgcAqaaanJ9FjMc89R9hEs94M\n\t1ZOg==","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:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=6WajgU0jx1k/LaWqXNKrzUTcoHowHMP54dPlaniwl3c=;\n\tb=t1ucT5ST0hS1002Qr4SInjetbcTjzz9ybRyEJyXZtlI7JqwzYBy+htwkSeR/utodLY\n\t6gENf4MVdc3NhFivfVeOm/Ki+YZguQRq+y0cyrqbRfd2BqrQ2OSOA8WM4j29DZUkOoXW\n\tGtkISxMJzL1Nl5lBqbQ8UhFRaX4cKLbWsoWBWrOfy+QuxqNqc/pcegNG7DD+O5QhgauM\n\twsBMul3Wa5FsWkUyDxMI3vH3RzuzwPBGKPPA/bsYw7cchqizzKLeDiudtpl8BJx5z1Na\n\t9dOW32S3RjcruQEJCqcb9bGT7v40sICkFTz0rBaWth0xSMhobpuveXOMFoyMIXRd9KWg\n\t8IcQ==","X-Gm-Message-State":"AHYfb5i8I87AvqTntkqntqbXdEdi2oC2MRpOiNdsqOul1x6EvPiNljaz\n\trQqGRWLJCq+1kqEqRbPSpNzcIwYx5w==","X-Received":"by 10.28.84.84 with SMTP id p20mr1977946wmi.5.1504004567782; Tue,\n\t29 Aug 2017 04:02:47 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1503946718-23973-1-git-send-email-agust@denx.de>","References":"<1503946718-23973-1-git-send-email-agust@denx.de>","From":"Christian Gmeiner <christian.gmeiner@gmail.com>","Date":"Tue, 29 Aug 2017 13:02:27 +0200","Message-ID":"<CAH9NwWeu=rSWxAHfTGdhd4xGnw+T8UmiXnjwyvqbMCt7hyZT+A@mail.gmail.com>","To":"Anatolij Gustschin <agust@denx.de>","Cc":"Marek Vasut <marex@denx.de>,\n\t\"u-boot@lists.denx.de\" <u-boot@lists.denx.de>, \n\tFabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","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>"}},{"id":1759403,"web_url":"http://patchwork.ozlabs.org/comment/1759403/","msgid":"<CAOMZO5AvyeonMs525a4KLCNjBePCw42qkTZNNudE2hQBLqR_rQ@mail.gmail.com>","list_archive_url":null,"date":"2017-08-29T13:30:00","subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","submitter":{"id":6978,"url":"http://patchwork.ozlabs.org/api/people/6978/","name":"Fabio Estevam","email":"festevam@gmail.com"},"content":"On Mon, Aug 28, 2017 at 3:58 PM, Anatolij Gustschin <agust@denx.de> wrote:\n> bss section is cleared in crt0.S. board_init_r() is also\n> entered from crt0 code.\n>\n> Signed-off-by: Anatolij Gustschin <agust@denx.de>\n\nReviewed-by: Fabio Estevam <fabio.estevam@nxp.com>","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=gmail.com header.i=@gmail.com\n\theader.b=\"t2v5/8io\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xhWbJ6rVJz9s7f\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 00:43:24 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 8CA7DC221BE; Tue, 29 Aug 2017 14:19:51 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id C1C2AC221B5;\n\tTue, 29 Aug 2017 14:19:43 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 51DE9C221AA; Tue, 29 Aug 2017 13:30:03 +0000 (UTC)","from mail-oi0-f66.google.com (mail-oi0-f66.google.com\n\t[209.85.218.66])\n\tby lists.denx.de (Postfix) with ESMTPS id 5EB58C2208A\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 13:30:02 +0000 (UTC)","by mail-oi0-f66.google.com with SMTP id t124so3007188oih.0\n\tfor <u-boot@lists.denx.de>; Tue, 29 Aug 2017 06:30:02 -0700 (PDT)","by 10.157.42.41 with HTTP; Tue, 29 Aug 2017 06:30:00 -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=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=S/5CPS1jBroUESxy2F9NyiCmWxpUl/jGqYBy2gEnvgQ=;\n\tb=t2v5/8io01O9fb4isJ8dehi95FHnECiqOTAqRUO2rCTZxZB2ECjGjyk58R+49F1tOC\n\tcUu+/nRBlEkZEoAS510CYm1Hm96woP7d63CcfZtrThCuMhTVpBt8/AQzQPbL1Vxh0rXR\n\ttD+XoVvMMtWzDanzNvNGZzqy8qIqz/5PSpZImj+jYQshwIVoGABQkNUKVL7JagdMH8RX\n\tQ+kL3xWt6xe07XknnFW9QG2wSBcVc3PuAnVRzbHWf/d0loq5gi83zM93IhPxSiaq/bYE\n\timls7qgmRP+qUrZUhMPDJGx2cILu8WEHwtzXAkmuebFC1YAHL6Zeseum796LtpSTuxbK\n\tQeFQ==","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:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=S/5CPS1jBroUESxy2F9NyiCmWxpUl/jGqYBy2gEnvgQ=;\n\tb=mwDA+ujX69wNu1NPm4Doy88aJ8wpHVN0i/CG6DsXlDVop7P0QklRkaBQ2TXhvDe5lX\n\tH79u4mQVXEm63D4qDHTKSFE+/SPDdX9Kc4c1JGyAyxT+A24XPYnV6pFDMRF7Ktp4LSj2\n\t0Xvm2OTOkepTCXPobkHHC/5SZ185CBXXs4qG6fkSbK7OMNHlfUW4z0QIxT9Dda4gvX2v\n\thT/BGlg3Sy6xFf01FT4NRLFTpgV2jbaccwGYUAZ5VIJx6voz/HZDEU4Y09clrr+J7I3J\n\t29/J7XnR+v6GXaMlgyQIOwFzJ1MQ9haWiV0bbgzjBCAmEhW/iIksnHR017eFbzzoEzC7\n\tBfvw==","X-Gm-Message-State":"AHYfb5hBXctwuBHEAOCdK98+O53MU4CCRqpfB2Ji91odeeJpYrRMSHAa\n\tNLs9NqHAPbv98UW4Y89BEUbUVwxSFQ==","X-Received":"by 10.202.206.208 with SMTP id e199mr290637oig.253.1504013400739;\n\tTue, 29 Aug 2017 06:30:00 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1503946718-23973-1-git-send-email-agust@denx.de>","References":"<1503946718-23973-1-git-send-email-agust@denx.de>","From":"Fabio Estevam <festevam@gmail.com>","Date":"Tue, 29 Aug 2017 10:30:00 -0300","Message-ID":"<CAOMZO5AvyeonMs525a4KLCNjBePCw42qkTZNNudE2hQBLqR_rQ@mail.gmail.com>","To":"Anatolij Gustschin <agust@denx.de>","Cc":"Marek Vasut <marex@denx.de>, U-Boot-Denx <u-boot@lists.denx.de>,\n\tFabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","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>"}},{"id":1771856,"web_url":"http://patchwork.ozlabs.org/comment/1771856/","msgid":"<b2766ec2-0345-1be2-6b57-9b3d00e126c4@denx.de>","list_archive_url":null,"date":"2017-09-20T13:38:34","subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","submitter":{"id":5771,"url":"http://patchwork.ozlabs.org/api/people/5771/","name":"Stefano Babic","email":"sbabic@denx.de"},"content":"On 28/08/2017 20:58, Anatolij Gustschin wrote:\n> bss section is cleared in crt0.S. board_init_r() is also\n> entered from crt0 code.\n> \n> Signed-off-by: Anatolij Gustschin <agust@denx.de>\n> Cc: Christian Gmeiner <christian.gmeiner@gmail.com>\n> Cc: Nikita Kiryanov <nikita@compulab.co.il>\n> Cc: Jagan Teki <jagan@amarulasolutions.com>\n> Cc: Tim Harvey <tharvey@gateworks.com>\n> Cc: Marek Vasut <marex@denx.de>\n> Cc: Lukasz Majewski <lukma@denx.de>\n> Cc: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n\n\nApplied to u-boot-imx, thanks !\n\nBest regards,\nStefano Babic","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>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xy17X0yH8z9s7c\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 20 Sep 2017 23:39:36 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid DF723C21F20; Wed, 20 Sep 2017 13:39:01 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 2FB08C21EDC;\n\tWed, 20 Sep 2017 13:38:59 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 6B0FAC21F19; Wed, 20 Sep 2017 13:38:44 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id 023DDC21F05\n\tfor <u-boot@lists.denx.de>; Wed, 20 Sep 2017 13:38:41 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xy16S4mxxz1r8WB;\n\tWed, 20 Sep 2017 15:38:40 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xy16S3vkfz1r0wp;\n\tWed, 20 Sep 2017 15:38:40 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id yzh51YeSSCsJ; Wed, 20 Sep 2017 15:38:38 +0200 (CEST)","from babic.homelinux.org (host-88-217-136-221.customer.m-online.net\n\t[88.217.136.221])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPS;\n\tWed, 20 Sep 2017 15:38:38 +0200 (CEST)","from localhost (mail.babic.homelinux.org [127.0.0.1])\n\tby babic.homelinux.org (Postfix) with ESMTP id D171345406A6;\n\tWed, 20 Sep 2017 15:38:37 +0200 (CEST)","from babic.homelinux.org ([127.0.0.1])\n\tby localhost (mail.babic.homelinux.org [127.0.0.1]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id eHNz_7hw5y0f; Wed, 20 Sep 2017 15:38:35 +0200 (CEST)","from [192.168.178.132] (papero.fritz.box [192.168.178.132])\n\tby babic.homelinux.org (Postfix) with ESMTP id F255E454055D;\n\tWed, 20 Sep 2017 15:38:34 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":["amavisd-new at mnet-online.de","Debian amavisd-new at babic.homelinux.org"],"To":"Anatolij Gustschin <agust@denx.de>, u-boot@lists.denx.de","References":"<1503946718-23973-1-git-send-email-agust@denx.de>","From":"Stefano Babic <sbabic@denx.de>","Message-ID":"<b2766ec2-0345-1be2-6b57-9b3d00e126c4@denx.de>","Date":"Wed, 20 Sep 2017 15:38:34 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<1503946718-23973-1-git-send-email-agust@denx.de>","Content-Language":"de-DE","Cc":"Marek Vasut <marex@denx.de>, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] imx6: drop duplicated bss memset and\n\tboard_init_r() call","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>"}}]