From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 14 Apr 2016 00:08:02 +0200 Subject: [U-Boot] [PATCH] arm: socfpga: Add samtec VIN|ING board In-Reply-To: <570EBFE8.1000307@opensource.altera.com> References: <1460550259-9468-1-git-send-email-marex@denx.de> <570EBFE8.1000307@opensource.altera.com> Message-ID: <570EC342.8080700@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 04/13/2016 11:53 PM, Dinh Nguyen wrote: > On 04/13/2016 07:24 AM, Marek Vasut wrote: >> Add support for board based on the popular Altera Cyclone V SoC. >> This board has the following properties: >> - 1 GiB of DRAM >> - 1 Gigabit ethernet >> - 1 USB gadget port >> - 1 USB host port with an on-board hub >> - 2 QSPI NORs connected to the Cadence QSPI core >> - Multiple I2C EEPROMs and one I2C temperature sensor >> >> Signed-off-by: Marek Vasut >> Cc: Dinh Nguyen >> Cc: Chin Liang See >> --- > > [snip] > >> --- /dev/null >> +++ b/include/configs/socfpga_vining_fpga.h >> @@ -0,0 +1,246 @@ >> +/* >> + * Copyright (C) 2015 Marek Vasut >> + * >> + * SPDX-License-Identifier: GPL-2.0+ >> + */ >> +#ifndef __CONFIG_SAMTEC_VINING_FPGA_H__ >> +#define __CONFIG_SAMTEC_VINING_FPGA_H__ >> + >> +#include >> + >> +/* U-Boot Commands */ >> +#define CONFIG_SYS_NO_FLASH >> +#define CONFIG_DOS_PARTITION >> +#define CONFIG_FAT_WRITE >> +#define CONFIG_HW_WATCHDOG >> + >> +#define CONFIG_CMD_ASKENV >> +#define CONFIG_CMD_BOOTZ >> +#define CONFIG_CMD_CACHE >> +#define CONFIG_CMD_DFU >> +#define CONFIG_CMD_DHCP >> +#define CONFIG_CMD_EEPROM >> +#define CONFIG_CMD_EXT4 >> +#define CONFIG_CMD_EXT4_WRITE >> +#define CONFIG_CMD_FAT >> +#define CONFIG_CMD_FS_GENERIC >> +#define CONFIG_CMD_GREPENV >> +#define CONFIG_CMD_LED >> +#define CONFIG_CMD_MII >> +#define CONFIG_CMD_MMC >> +#define CONFIG_CMD_PING >> +#define CONFIG_CMD_USB >> +#define CONFIG_CMD_USB_MASS_STORAGE >> + > > With my patch[1], you don't need all of those common configs, all you > need is: > > #define CONFIG_CMD_DFU > #define CONFIG_CMD_USB > #define CONFIG_CMD_USB_MASS_STORAGE You're right, I also commented on your patch :) Let's see how the discussion goes and I'll ev. send a V2 here. > Thanks, > Dinh > > [1] http://lists.denx.de/pipermail/u-boot/2016-April/251654.html > -- Best regards, Marek Vasut