[{"id":1774224,"web_url":"http://patchwork.ozlabs.org/comment/1774224/","msgid":"<20170924144037.GR30205@bill-the-cat>","list_archive_url":null,"date":"2017-09-24T14:40:37","subject":"Re: [U-Boot] [PATCH 2/4] arm: dra7xx: 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:57PM +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 <semen.protsenko@linaro.org>\n\nReviewed-by: Tom Rini <trini@konsulko.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\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"Jc0dw/xa\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0VQP1lvPz9sDB\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 25 Sep 2017 00:46:05 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 0727AC22105; Sun, 24 Sep 2017 14:43:36 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id CF75BC2210A;\n\tSun, 24 Sep 2017 14:41:06 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 00CC5C220F3; Sun, 24 Sep 2017 14:40:47 +0000 (UTC)","from mail-yw0-f176.google.com (mail-yw0-f176.google.com\n\t[209.85.161.176])\n\tby lists.denx.de (Postfix) with ESMTPS id 3F062C22007\n\tfor <u-boot@lists.denx.de>; Sun, 24 Sep 2017 14:40:37 +0000 (UTC)","by mail-yw0-f176.google.com with SMTP id w22so3274765ywa.13\n\tfor <u-boot@lists.denx.de>; Sun, 24 Sep 2017 07:40:37 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:9cbd:ca90:bff9:8e97])\n\tby smtp.gmail.com with ESMTPSA id\n\tj6sm1756748ywg.44.2017.09.24.07.40.35\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tSun, 24 Sep 2017 07:40:35 -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=oVED6AkJA2MCZL0SUI+xEaxN+LqEq3khv/LiKgFcVo8=;\n\tb=Jc0dw/xaHJg5U2k9iymW8jGNAVy0jx9O2/Y0xc5aY7i5G/JKhKR5O6tapnGw3awpjy\n\t+DZDpq0tqm6g14XexUXRhfNfrj4UZFwTtnZDw4otcMz6g087YQYc1z9qZJpf0uscm++V\n\t+NGm7jDOaYwg9fEO6t6EhWGYlK6porHY5mFuw=","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=oVED6AkJA2MCZL0SUI+xEaxN+LqEq3khv/LiKgFcVo8=;\n\tb=bex7+HiZHQwzhMqyug5VYPwfaTttNQ0GLqKnSAtpJjcGzlZiyChtt3al82MIUIGFvK\n\t5PRTc7BGkgG0YkQKVWszhMkUEwcuX6Z/BRC4Ahh6KRUYkbjB75LirzsjC410tGa3di+L\n\tlpZFr0ipl8jYomCYTR+5bZX4gfqxErQXh4MEke01H4Hz/Hmk8lZvvT5FpP55LTEcyBdQ\n\tENpecUzcuw0hDVEGGGp9zrUVOjEVZmGxrX2URMFHVeVbJ0IxRRq+WJ/8lYMdnPhvOjN9\n\tvVaKhz0Bw3Lm45v5yMnkLYE6kEaKM2YqD5zidMIl7N6GIV5d5oDvOJqpboAo8NtfLg+Z\n\tytww==","X-Gm-Message-State":"AHPjjUh6znVqxVexOjehjD+748Wk4PZxFm2QIzcX1kxf2LftITMP02/J\n\tIvRf28PrlSjf5ero3f4tzyGNaw==","X-Google-Smtp-Source":"AOwi7QD6G+s4ohqwHOY4InQOniQKNiEBxysEh++P9kvlMslGx57wCxW3xg50MDWisvGNd1RKxinDgg==","X-Received":"by 10.129.50.88 with SMTP id y85mr3059418ywy.36.1506264036058;\n\tSun, 24 Sep 2017 07:40:36 -0700 (PDT)","Date":"Sun, 24 Sep 2017 10:40:37 -0400","From":"Tom Rini <trini@konsulko.com>","To":"Sam Protsenko <semen.protsenko@linaro.org>","Message-ID":"<20170924144037.GR30205@bill-the-cat>","References":"<20170921193759.3914-1-semen.protsenko@linaro.org>\n\t<20170921193759.3914-3-semen.protsenko@linaro.org>","MIME-Version":"1.0","In-Reply-To":"<20170921193759.3914-3-semen.protsenko@linaro.org>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Praneeth Bajjuri <praneeth@ti.com>, u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH 2/4] arm: dra7xx: 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 <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":"multipart/mixed;\n\tboundary=\"===============1765210954061874324==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1777791,"web_url":"http://patchwork.ozlabs.org/comment/1777791/","msgid":"<20170929202647.GA3112@bill-the-cat>","list_archive_url":null,"date":"2017-09-29T20:26:47","subject":"Re: [U-Boot] [U-Boot,2/4] arm: dra7xx: 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:57PM +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 <trini@konsulko.com>\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 <semen.protsenko@linaro.org>\n\nApplied to u-boot/master, thanks!","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\" (1024-bit key;\n\tunprotected) header.d=konsulko.com header.i=@konsulko.com\n\theader.b=\"AXknq+cl\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y3k4X4VgNz9t2Q\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat, 30 Sep 2017 06:41:48 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 5ED86C21EC3; Fri, 29 Sep 2017 20:37:10 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id E072AC21EFB;\n\tFri, 29 Sep 2017 20:34:54 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 4BB91C21EA5; Fri, 29 Sep 2017 20:26:49 +0000 (UTC)","from mail-yw0-f181.google.com (mail-yw0-f181.google.com\n\t[209.85.161.181])\n\tby lists.denx.de (Postfix) with ESMTPS id 8FAE3C21E7C\n\tfor <u-boot@lists.denx.de>; Fri, 29 Sep 2017 20:26:45 +0000 (UTC)","by mail-yw0-f181.google.com with SMTP id s62so549756ywg.0\n\tfor <u-boot@lists.denx.de>; Fri, 29 Sep 2017 13:26:45 -0700 (PDT)","from bill-the-cat ([2606:a000:1401:811b:59d1:1e23:9925:e8d6])\n\tby smtp.gmail.com with ESMTPSA id\n\tc13sm2194910ywm.101.2017.09.29.13.26.43\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tFri, 29 Sep 2017 13:26:43 -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=sJ5YlXh/xBztrjlrB0D+S5iuviMrOtpjbNHYdZx3lXQ=;\n\tb=AXknq+clHtzqmId0BfjBCD3+aonezGt9vfn/DnO4B4XW+LnhFS+xozdguB46Fb71Xt\n\t99r4f/KRQEEiVviMCGaAEi8uxw43m+iB2m6hZ9SoXeAXrKS86Mc7JsiJC3BoCa10tFL/\n\t0UzQDjvHulkChOPY+AxDGlUEfKSWpyTxBL6y4=","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=sJ5YlXh/xBztrjlrB0D+S5iuviMrOtpjbNHYdZx3lXQ=;\n\tb=Zg6jMsaHZqR80ZVu7OtEtZ9ZgpN+8cPLdUV12Ojtncv/SotzWYi552p0A31k5xHBlM\n\tyNLx38vSQTF1NX0X47LFTuk+NYeQ0pg2xPYjSChw+thA5+Q46A8JDxNsKoSLsUa7qiQk\n\tFkpjHmmGJtpqj6dfk+EXEwSYWHHLuuZIMhdMbIuDDWAqrI4sTyQpP2hA3Fl5MgMEzfuK\n\ttE/wEBuEIWb1v0LO2lkwKHF+zoYcFASlr1wBAHztqPm3OpTo5z51PiQlh6cBl2VUXzfc\n\t1mYNPwBUYieb90b7nIgH9Gle5ZdK3iGu/EEkztikJk6+u1hAsHZk40Rcx26dxS3O7XoJ\n\ttAZg==","X-Gm-Message-State":"AHPjjUi1MtjBsWjPde66vy6NSaSY6f1wVpIawZvzLNbDZxJnyF0TdGoe\n\tyNBL0I43ylImOvfmTVHlOaweVA==","X-Google-Smtp-Source":"AOwi7QBYdIBaJDcDyeeCuOs9hwYRu0WqU8yEujgzw0ThYiXQWYc5q2Ux0icqDn951M+OOWE0qBR1dw==","X-Received":"by 10.129.121.4 with SMTP id u4mr6779241ywc.71.1506716804367;\n\tFri, 29 Sep 2017 13:26:44 -0700 (PDT)","Date":"Fri, 29 Sep 2017 16:26:47 -0400","From":"Tom Rini <trini@konsulko.com>","To":"Sam Protsenko <semen.protsenko@linaro.org>","Message-ID":"<20170929202647.GA3112@bill-the-cat>","References":"<20170921193759.3914-3-semen.protsenko@linaro.org>","MIME-Version":"1.0","In-Reply-To":"<20170921193759.3914-3-semen.protsenko@linaro.org>","User-Agent":"Mutt/1.5.21 (2010-09-15)","Cc":"Praneeth Bajjuri <praneeth@ti.com>, u-boot@lists.denx.de","Subject":"Re: [U-Boot] [U-Boot,2/4] arm: dra7xx: 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 <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":"multipart/mixed;\n\tboundary=\"===============4217063924193138842==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]