[{"id":1774232,"web_url":"http://patchwork.ozlabs.org/comment/1774232/","msgid":"<20170924144032.GQ30205@bill-the-cat>","list_archive_url":null,"date":"2017-09-24T14:40:32","subject":"Re: [U-Boot] [PATCH 1/4] arm: am57xx: Fix Linux boot from eMMC","submitter":{"id":65875,"url":"http://patchwork.ozlabs.org/api/people/65875/","name":"Tom Rini","email":"trini@konsulko.com"},"content":"On Thu, Sep 21, 2017 at 10:37:56PM +0300, Sam Protsenko wrote:\n\n> Right now on OMAP5-based boards we have only one partition defined for\n> Linux boot, which is rootfs. That doesn't work with bootpart=1:2 (that\n> is defined in include/environment/ti/boot.h). To fix Linux boot we may\n> either:\n> \n> 1. Change bootpart to be 1:1\n> 2. Or add preceding partition, so that rootfs is actually 1:2\n> \n> Second choice seems more reasonable, as DFU is already using similar\n> partition table and can rely on bootpart to be 1:2.\n> \n> This patch adds \"bootloader\" partition. So now eMMC layout for Linux\n> boot looks like this:\n> \n> offset content size partition\n> (KiB) (KiB)\n> =======================================================================\n> \n> 0 +--------------------------------+\n> | MBR/GPT header | 128 -\n> 128 +--------------------------------+\n> | MLO | 256 -\n> 384 +--------------------------------+\n> | u-boot.img | 1792 bootloader\n> 2176 +--------------------------------+\n> | //////////// hole //////////// | 256 -\n> 2432 +--------------------------------+\n> | U-Boot environment | 128 -\n> 2560 +--------------------------------+\n> | U-Boot environment (redundant) | 128 -\n> 2688 +--------------------------------+\n> | rootfs | remaining rootfs\n> end +--------------------------------+\n> \n> =======================================================================\n> \n> Guard hole appears because U-Boot environment offset was calculated for\n> Android partition table, which has two additional partitions in place of\n> that hole (\"environment\" and \"misc\" partitions).\n> \n> This patch also changes rootfs offset from 2 MiB further to 2688 KiB,\n> so that there won't be any collisions with U-Boot environment when we\n> flash rootfs.\n> \n> Signed-off-by: Sam Protsenko \n\nReviewed-by: Tom Rini ","headers":{"Return-Path":"","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=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"g4OUzHqy\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0VVY0vDjz9s8J\n\tfor ;\n\tMon, 25 Sep 2017 00:49:41 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid DF5CFC22129; Sun, 24 Sep 2017 14:44:39 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id EE20DC22113;\n\tSun, 24 Sep 2017 14:41:21 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 67872C22111; Sun, 24 Sep 2017 14:40:43 +0000 (UTC)","from mail-yw0-f175.google.com (mail-yw0-f175.google.com\n\t[209.85.161.175])\n\tby lists.denx.de (Postfix) with ESMTPS id B6B3BC220CB\n\tfor ; Sun, 24 Sep 2017 14:40:32 +0000 (UTC)","by mail-yw0-f175.google.com with SMTP id o143so3281214ywd.12\n\tfor ; Sun, 24 Sep 2017 07:40:32 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:9cbd:ca90:bff9:8e97])\n\tby smtp.gmail.com with ESMTPSA id\n\ts187sm762266yws.55.2017.09.24.07.40.30\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tSun, 24 Sep 2017 07:40:30 -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=konsulko.com;\n\ts=google; \n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=iFY6QxRaFwyH+d3np+8PkoYQGSuq4LpXgg2cNigQdPs=;\n\tb=g4OUzHqyFnZ/4Sl6DjAo2pr83jc9/2ZJtrYHPr4OsabUtPDFunjt2VO6rlidrBWWvk\n\tCTRGtY8++ficjN78jTml4X4I9TmKA/W2gbKAAX4QdTTilxBDH3L/1mTiWli7HKcSDz2p\n\tVfUa88fznU1zAn6vWcU0xukPt9VeJwr4DUll4=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=iFY6QxRaFwyH+d3np+8PkoYQGSuq4LpXgg2cNigQdPs=;\n\tb=j1TpDALDbFZ4SM+hQs1/B5zsgiZhpJ4cjPQ44f5jfT3sWMkTAKiKHbAJ3Ifci0i760\n\t4VDA6PtZ5U4KiYpFxaCkuq+d8PfFJSluU54XSVkpTIr1itQa5zOX01h6XiLMA3OIk1r7\n\tFaJnetRSKVDU2XIqEh9R9/QIqrJYR4n/4KnbfzSlZs9tMV647WpNJNbbuj/MCGsi6rol\n\tncwkbNirtu5PMhkmSDlyv9qwlIlvmHZjqNplYDcYa5ZSyrBEFDZYyAKF2HjIccDFR5jZ\n\tsSrrgb79n8SEFj+LRXWvK+oiZqOlfIQYvtX49NF0L17zuHCrJ6yk8FrQfB8r1Vg+TATt\n\tT99g==","X-Gm-Message-State":"AHPjjUgGQ9gitoJHsGXmJQsS4SB0+vD+dHM0eD6YwKO7R1qjW71ciM2G\n\twn7mmc628j3VwDzasVAVZcd/Lw==","X-Google-Smtp-Source":"AOwi7QDxppaEpmX5n8Ltb9QgKIWzQptp/JRV+GmJVHDLyu03RD21r8n0xWDcRedQfKOgxbiIwyolig==","X-Received":"by 10.37.135.67 with SMTP id e3mr2911078ybn.10.1506264031541;\n\tSun, 24 Sep 2017 07:40:31 -0700 (PDT)","Date":"Sun, 24 Sep 2017 10:40:32 -0400","From":"Tom Rini ","To":"Sam Protsenko ","Message-ID":"<20170924144032.GQ30205@bill-the-cat>","References":"<20170921193759.3914-1-semen.protsenko@linaro.org>\n\t<20170921193759.3914-2-semen.protsenko@linaro.org>","MIME-Version":"1.0","In-Reply-To":"<20170921193759.3914-2-semen.protsenko@linaro.org>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Praneeth Bajjuri , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 1/4] arm: am57xx: Fix Linux boot from eMMC","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============2462444934048202789==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1777785,"web_url":"http://patchwork.ozlabs.org/comment/1777785/","msgid":"<20170929202643.GZ3112@bill-the-cat>","list_archive_url":null,"date":"2017-09-29T20:26:43","subject":"Re: [U-Boot] [U-Boot,1/4] arm: am57xx: Fix Linux boot from eMMC","submitter":{"id":65875,"url":"http://patchwork.ozlabs.org/api/people/65875/","name":"Tom Rini","email":"trini@konsulko.com"},"content":"On Thu, Sep 21, 2017 at 10:37:56PM +0300, Sam Protsenko wrote:\n\n> Right now on OMAP5-based boards we have only one partition defined for\n> Linux boot, which is rootfs. That doesn't work with bootpart=1:2 (that\n> is defined in include/environment/ti/boot.h). To fix Linux boot we may\n> either:\n> \n> 1. Change bootpart to be 1:1\n> 2. Or add preceding partition, so that rootfs is actually 1:2\n> \n> Second choice seems more reasonable, as DFU is already using similar\n> partition table and can rely on bootpart to be 1:2.\n> \n> This patch adds \"bootloader\" partition. So now eMMC layout for Linux\n> boot looks like this:\n> \n> offset content size partition\n> (KiB) (KiB)\n> Reviewed-by: Tom Rini \n> \n> =======================================================================\n> \n> 0 +--------------------------------+\n> | MBR/GPT header | 128 -\n> 128 +--------------------------------+\n> | MLO | 256 -\n> 384 +--------------------------------+\n> | u-boot.img | 1792 bootloader\n> 2176 +--------------------------------+\n> | //////////// hole //////////// | 256 -\n> 2432 +--------------------------------+\n> | U-Boot environment | 128 -\n> 2560 +--------------------------------+\n> | U-Boot environment (redundant) | 128 -\n> 2688 +--------------------------------+\n> | rootfs | remaining rootfs\n> end +--------------------------------+\n> \n> =======================================================================\n> \n> Guard hole appears because U-Boot environment offset was calculated for\n> Android partition table, which has two additional partitions in place of\n> that hole (\"environment\" and \"misc\" partitions).\n> \n> This patch also changes rootfs offset from 2 MiB further to 2688 KiB,\n> so that there won't be any collisions with U-Boot environment when we\n> flash rootfs.\n> \n> Signed-off-by: Sam Protsenko \n\nApplied to u-boot/master, thanks!","headers":{"Return-Path":"","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=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"oObIB2hA\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y3k1B2HK9z9t4k\n\tfor ;\n\tSat, 30 Sep 2017 06:38:54 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 0BCF1C21EA5; Fri, 29 Sep 2017 20:36:45 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id CB658C21DDA;\n\tFri, 29 Sep 2017 20:34:12 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E2065C21E9C; Fri, 29 Sep 2017 20:26:46 +0000 (UTC)","from mail-yw0-f178.google.com (mail-yw0-f178.google.com\n\t[209.85.161.178])\n\tby lists.denx.de (Postfix) with ESMTPS id A6F60C21DA5\n\tfor ; Fri, 29 Sep 2017 20:26:40 +0000 (UTC)","by mail-yw0-f178.google.com with SMTP id u11so536936ywh.7\n\tfor ; Fri, 29 Sep 2017 13:26:40 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:59d1:1e23:9925:e8d6])\n\tby smtp.gmail.com with ESMTPSA id\n\tx140sm2041795ywd.58.2017.09.29.13.26.38\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tFri, 29 Sep 2017 13:26:39 -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_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL,\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=konsulko.com;\n\ts=google; \n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=jsu0TIJAnjVOEtRyMM9lL8z8adY2Gc8M7qF67iT4Ll0=;\n\tb=oObIB2hAbuJLmbFwAkMDop4MAdgd/QJWGeOxuSfjsyOycSFAgJQwBHXDN+VUdauIIo\n\tXz9ziC5bnw5Wh7xxrBlc0m9eu+41mv/MorI/6uujHeV6fujFUuEn3JC8vnRHdEQKGhnj\n\teVDXy7Y/GcoKqZQMGMdcD9yQEhuIx/TrYGMZY=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=jsu0TIJAnjVOEtRyMM9lL8z8adY2Gc8M7qF67iT4Ll0=;\n\tb=WZx1g+RjPzeVvDbf9/Zc3LGUPbmtna7ksHp3GvcMWZO0HbkWUwwt23VF4XBd967JKv\n\tb92XGZSGXEVB6n01SGuy/TYhqlGM9UPRw9X2PNjtExtPaW/YRaaug5bGKKMT+vr2r1vA\n\txsl3XUQqYmZgcSZd20ou1+aIoCWWUgvgo8+MIyurj+9hgUFAWvMJCoeSaJd6wlFe3A/i\n\thYtVTVg912pM5Xxjg8k6EOU0kfM8aaiJQOwYzy0xABgJcvcaBjxobvp04Lh2kV2WsjfE\n\tbLABSMPQlUGVHyY3foCEnAdvqp8RAfKSgLIrIFRuognuF2wCUMpLwFVFvSd3Lw4OFloh\n\tB74g==","X-Gm-Message-State":"AHPjjUjGaIyhJvN5HIX10nrmdmtyYSLe4RGZCb8gOB8Ev/DZy/YPhLR0\n\tPw75+aTn5SJ/1Yf/qpXYFT7gWA==","X-Google-Smtp-Source":"AOwi7QCxw1GoFKO3I69c3ytz5i/8pPxpuWHH2Ecvaj4pZW/H++6PmB7vbl7j2n3152Imcl4HGDc0JQ==","X-Received":"by 10.129.79.147 with SMTP id d141mr7293417ywb.302.1506716799521;\n\tFri, 29 Sep 2017 13:26:39 -0700 (PDT)","Date":"Fri, 29 Sep 2017 16:26:43 -0400","From":"Tom Rini ","To":"Sam Protsenko ","Message-ID":"<20170929202643.GZ3112@bill-the-cat>","References":"<20170921193759.3914-2-semen.protsenko@linaro.org>","MIME-Version":"1.0","In-Reply-To":"<20170921193759.3914-2-semen.protsenko@linaro.org>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Praneeth Bajjuri , u-boot@lists.denx.de","Subject":"Re: [U-Boot] [U-Boot,1/4] arm: am57xx: Fix Linux boot from eMMC","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============8658237997415072443==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]