From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrey Zhizhikin Date: Mon, 18 Dec 2017 13:30:05 +0100 Subject: [U-Boot] [PATCH] config: Enable part command for socfpga platform In-Reply-To: References: <1513595456-37953-1-git-send-email-andrey.z@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Marek, On Mon, Dec 18, 2017 at 12:39 PM, Marek Vasut 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. > > 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. -- Regards, Andrey.