On 01/22/2017 12:04 AM, Westergreen, Dalon wrote: > On Sat, 2017-01-21 at 20:29 +0100, Marek Vasut wrote: >> On 01/21/2017 06:31 PM, Dalon Westergreen wrote: >>> >>> From: Dalon Westergreen <dalon.westergr...@intel.com> >>> >>> The default values for CONFIG_SYS_MMCSD_FS_BOOT_PARTITION >>> and CONFIG_SYS_MMCSD_FS_OS_PARTITION have changed and as >>> as result the default uboot environment for this board >>> needs updating. This sets the default envirnment to >>> use the CONFIG_SYS_MMCSD_FS_BOOT_PARTITION and >>> CONFIG_SYS_MMCSD_FS_OS_PARTITION configs for the boot >>> and os partitions. >>> >>> Also set the default fdtimage value to match the >>> devicetree name in the linux kernel for this board. >>> >>> Signed-off-by: Dalon Westergreen <dalon.westergr...@intel.com> >> >> Acked-by: Marek Vasut <ma...@denx.de> >> >> but please see my comment on 1/7 >> >> btw I think this is repeating too much, why don't we pull the env into >> socfpga_common.h first ? > > I'd prefer to keep the boards separate and look at > moving to disto boot.
If you create common env in socfpga_common.h first and then update it, the update will be in a single patch changing a single file. So will then be the switch to distro bootcmd. I would prefer such course of action over many patches doing the same thing in many files. Given how similar (and broken) the envs are for those boards, pulling the common env should be easy and very beneficial. -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
