From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D9D98C433F5 for ; Mon, 18 Oct 2021 12:07:53 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 550FB60EFF for ; Mon, 18 Oct 2021 12:07:53 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 550FB60EFF Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 9F3FB834B5; Mon, 18 Oct 2021 14:07:51 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1634558872; bh=8INl9HfyoviRi/m7M1sqw1q2auVysTvRW8U06jOeBtQ=; h=Subject:To:Cc:References:From:Date:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=RKRHwhP0K+mljz+K6J64Dm/ciG/ZQaGke5C0ZGVICfwiUqnZZ7S7djzNN4akacEaA 3YouRWEajjfCNrMrSz0c+mH/Hiyv0ZsQMzSVMgI3F+vMtIj0s6GSZkhlj/kHMP99Gy BSfhdQ4ZyC1nCm6XwT9QUbIzNNjZuoQZHkEd0MP9dTFAh21JeVCZ4hufdB9yyZgjNv eqb7rGcBOQ3nip3hNIVTXq7BZY3CgFxOXcr9OfjL+cbJVGJeH2xaRBJiMPqySHHiqN eU2rv+lfwOgHtuV0SZBQ0U+e9C2BKWefMSuONE+4QbUeqY2fYjy8QBng8WJYIcQew1 AAc5u+WB39kIg== Received: from [127.0.0.1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: marex@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 8CD18833F2; Mon, 18 Oct 2021 14:07:49 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1634558869; bh=8INl9HfyoviRi/m7M1sqw1q2auVysTvRW8U06jOeBtQ=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c1noM0pH/CDKyIefXPGffodI2JmQFssH8qzG1GBSLsUMetb95832+ossy6PzsdORL n4JpsErCdyQl4IKY9/SyLZIPAURdiqLY7403ZyqAehpHPvfMdNfXQMVmDssdNUtTde R2yZnlEK41mTkHBVrlZZ/9ydbOVx2nC5I3zAbiRysIqplZMAZG1XnOnhlynvG63NlE wqOuq7z6epPAV71aEy0aGTmATOc7+peN+evx/tymQDx3L7YdzP2LB8AmoGKX3nlf7g 6z+sDq+7YKscpUoiqZFB60f/RJ2YtTTER3vbIjSsOY/lx8fx/FdZHK4VBYFmzSXZXv gZsxWtTSAxGHg== Subject: Re: [PATCH v2] env: mmc: Add support for redundant env in both eMMC boot partitions To: Jaehoon Chung , u-boot@lists.denx.de Cc: Fabio Estevam , Peng Fan , Stefano Babic References: <20211017172336.47710-1-marex@denx.de> <359049b1-b526-3907-d28e-1e3877338ae2@samsung.com> From: Marek Vasut Message-ID: <89eb4efe-f184-9798-a242-63a56cd3cfda@denx.de> Date: Mon, 18 Oct 2021 13:47:12 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <359049b1-b526-3907-d28e-1e3877338ae2@samsung.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean On 10/18/21 4:02 AM, Jaehoon Chung wrote: > Hi Marek, Hi, > On 10/18/21 2:23 AM, Marek Vasut wrote: >> Currently the MMC environment driver supports storing redundant environment >> only in one eMMC partition at different offsets. This is sub-optimal, since >> if this one boot partition is erased, both copies of environment are lost. >> Since the eMMC has two boot partitions, add support for storing one copy of >> environment in each of the two boot partitions. >> >> To enable this functionality, select CONFIG_SYS_REDUNDAND_ENVIRONMENT to >> indicate redundant environment should be used. Set CONFIG_SYS_MMC_ENV_PART >> to 1 to indicate environment should be stored in eMMC boot partition. Set >> CONFIG_ENV_OFFSET equal to CONFIG_ENV_OFFSET_REDUND, and both to the offset >> from start of eMMC boot partition where the environment should be located. > > Is there any patch before apply this? When I had been trying to test this, I can't apply this from patchwork. Uh, pick this one fix, I missed it in this v2 resend: https://patchwork.ozlabs.org/project/uboot/patch/20211006162954.63596-1-marex@denx.de/