From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 23 Sep 2015 21:37:19 +0200 Subject: [U-Boot] [PATCHv3] arm: socfpga: Enable saveenv for SD/MMC In-Reply-To: <1443036108-24630-1-git-send-email-dinguyen@opensource.altera.com> References: <1443036108-24630-1-git-send-email-dinguyen@opensource.altera.com> Message-ID: <201509232137.19568.marex@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 Wednesday, September 23, 2015 at 09:21:48 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 > --- > v3: Only define ENV_IS_NOWHERE if env is not in MMC, NAND, FAT, and > SPI_FLASH v2: Move this option to be on a per-board basis > Add config for socfpga_cyclone5 and socfpga_arria5 > --- [...] Hi! > --- a/include/configs/socfpga_common.h > +++ b/include/configs/socfpga_common.h > @@ -281,7 +281,12 @@ unsigned int cm_get_qspi_controller_clk_hz(void); > #define CONFIG_SYS_CONSOLE_IS_IN_ENV > #define CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE > #define CONFIG_SYS_CONSOLE_ENV_OVERWRITE > +#if !defined CONFIG_ENV_IS_IN_MMC && \ > + !defined CONFIG_ENV_IS_IN_NAND && \ > + !defined CONFIG_ENV_IS_IN_FAT && \ > + !defined CONFIG_ENV_IS_IN_SPI_FLASH This really doesn't scale, I can have env in UBI for example, or on EEPROM and this condition would trigger and cause a problem. Just move the CONFIG_ENV_IS_NOWHERE into the respective board files and be done with it. This bit of duplication is pretty OK I'd say. > #define CONFIG_ENV_IS_NOWHERE > +#endif > #define CONFIG_ENV_SIZE 4096 > > /* [...] Sorry to pester you this way, the rest is great of course :) Best regards, Marek Vasut