Hello,

Am 24.11.19 um 16:13 schrieb Tom Rini:
> On Thu, Oct 03, 2019 at 07:50:19PM +0200, Miquel Raynal wrote:
> 
>> There is no flash on this board, there is no reason to define MTD
>> environment variables nor UBI. Drop them from the configuration file.
>>
>> Signed-off-by: Miquel Raynal <[email protected]>
>> ---
>>  configs/socfpga_mcvevk_defconfig | 3 ---
>>  1 file changed, 3 deletions(-)
>>
>> diff --git a/configs/socfpga_mcvevk_defconfig 
>> b/configs/socfpga_mcvevk_defconfig
>> index 39cc753888..15c9f7fdb0 100644
>> --- a/configs/socfpga_mcvevk_defconfig
>> +++ b/configs/socfpga_mcvevk_defconfig
>> @@ -28,9 +28,6 @@ CONFIG_CMD_USB=y
>>  CONFIG_CMD_USB_MASS_STORAGE=y
>>  CONFIG_CMD_CACHE=y
>>  CONFIG_CMD_EXT4_WRITE=y
>> -CONFIG_MTDIDS_DEFAULT="nor0=ff705000.spi.0"
>> -CONFIG_MTDPARTS_DEFAULT="mtdparts=ff705000.spi.0:1m(u-boot),256k(env1),256k(env2),14848k(boot),16m(rootfs),-@1536k(UBI)0"
>> -CONFIG_CMD_UBI=y
>>  # CONFIG_ISO_PARTITION is not set
>>  # CONFIG_EFI_PARTITION is not set
>>  CONFIG_DEFAULT_DEVICE_TREE="socfpga_cyclone5_mcvevk"
> 
> This board has SPI flash and so the MTDIDS/PARTS make sense there.  I'm
> adding in the board maintainer for further comment about how UBI might
> be used, thanks!

There is *no* SPI-FLash connected to the HPS on that module/board.
Therefore these options can safely be removed. (There is a SPI-NOR
connected to the FPGA for configuration purposes).

Miquel, feel free to add my:

  Reviewed-by: Wolfgang Grandegger <[email protected]>

Thanks,

Wolfgang

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to