On 12/18/2017 01:30 PM, Andrey Zhizhikin wrote: > Hello Marek, Hi,
> On Mon, Dec 18, 2017 at 12:39 PM, Marek Vasut <ma...@denx.de> wrote: >> On 12/18/2017 12:10 PM, Andrey Zhizhikin wrote: >>> Enable CONFIG_CMD_PART item, as default environment requires it >>> and complains this command in unknown. >> >> Fine by me, but what about the other boards, don't they need this as well ? > > I'd say this is applicable to other boards in socfpga family as well. > The only reason I didn't include them as well into this commit is I > don't have those boards on hands to perform verification. But I can > assume since a lot of people uses ENV files to overwrite defaults - > this should not affect anyone. Please let me know what you think, and > I can re-submit this patch to include part command in all socfpga > family configs. Let's keep it consistent, include all the others. Also make sure to run make savedefconfig to place that defconfig entry at the right location if you don't already . >> btw diffstat is missing and the tags are wrong (should be ARM: >> socfpga:). I can fix those up, but see my question above... >> > > Thanks a lot for pointing this out! I quickly checked the log on those > files, and missed those points out... If you would agree on my > proposal below, then I get those resolved as well. Fine. -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot