[U-Boot,v5,17/26] configs: socfpga: mcvevk: Remove useless UBI infos

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 <miquel.raynal@bootlin.com>
---
 configs/socfpga_mcvevk_defconfig | 3 ---
 1 file changed, 3 deletions(-)

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 <miquel.raynal@bootlin.com>
> ---
>  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!

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 <miquel.raynal@bootlin.com>
>> ---
>>  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 <wg@aries-embedded.de>

Thanks,

Wolfgang

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 <miquel.raynal@bootlin.com>
> Reviewed-by: Wolfgang Grandegger <wg@aries-embedded.de>

Applied to u-boot/master, thanks!