From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Wed, 16 Sep 2015 08:59:50 +0200 Subject: [U-Boot] [PATCH] ARM: socfpga: Enable saveenv for SD/MMC In-Reply-To: <20150916065716.GB11562@amd> References: <1442346443-22613-1-git-send-email-dinguyen@opensource.altera.com> <201509152222.33768.marex@denx.de> <20150916065716.GB11562@amd> Message-ID: <55F91366.7020401@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 16.09.2015 08:57, Pavel Machek wrote: > On Tue 2015-09-15 22:22:33, Marek Vasut wrote: >> On Tuesday, September 15, 2015 at 09:47:23 PM, dinguyen at opensource.altera.com >> wrote: >>> From: Dinh Nguyen >>> >>> Enable the able to save the environment variables when SD/MMC is used. >>> >>> Signed-off-by: Dinh Nguyen >> >> Hi, >> >> this should be done per-board, so I'd suggest the following: > > Why? MMC layout is same on all of them... It doesn't have to be identical. And other boards might want to use e.g. SPI NOR as the environment storage device (I have such a custom board here). So this is definitely a board specific thing. >>> #define CONFIG_ENV_SIZE 4096 > > While you are changing that, bigger environment size would be > welcome. 32K? Yes. Thanks, Stefan