[U-Boot] [PATCH 3/7] sunxi: Enable UBI and NAND support

Hans de Goede
2016-11-14T11:18:06Z

Hi,

On 08-11-16 17:21, Maxime Ripard wrote:
> From: Hans de Goede <hdegoede@redhat.com>
>
> Enable the NAND and UBI support in the configuration header so that we can
> (finally) use it.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
> include/configs/sunxi-common.h | 26 ++++++++++++++++++++++----
> 1 file changed, 22 insertions(+), 4 deletions(-)
>
> diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
> index 8363414828fa..1733767ba53b 100644
> --- a/include/configs/sunxi-common.h
> +++ b/include/configs/sunxi-common.h
> @@ -129,9 +129,23 @@
> #define CONFIG_SERIAL_TAG
>
> #ifdef CONFIG_NAND_SUNXI
> +#define CONFIG_SYS_NAND_U_BOOT_OFFS (8 << 20) /* 8 MiB */

As Boris already said, please put this in Kconfig.

> #define CONFIG_SYS_NAND_MAX_ECCPOS 1664
> #define CONFIG_SYS_NAND_ONFI_DETECTION
> #define CONFIG_SYS_MAX_NAND_DEVICE 8
> +
> +/* Requirements for UBI */
> +#define CONFIG_RBTREE
> +#define CONFIG_LZO
> +#define CONFIG_CMD_MTDPARTS
> +#define CONFIG_CMD_UBI
> +#define CONFIG_CMD_UBIFS
> +#define CONFIG_MTD_DEVICE
> +
> +#define CONFIG_MTD_PARTITIONS
> +
> +#define CONFIG_CMD_NAND
> +#define CONFIG_CMD_NAND_TRIMFFS
> #endif
>
> #ifdef CONFIG_SPL_SPI_SUNXI
> @@ -143,7 +157,14 @@
> #define CONFIG_GENERIC_MMC
> #define CONFIG_MMC_SUNXI
> #define CONFIG_MMC_SUNXI_SLOT 0
> -#define CONFIG_ENV_IS_IN_MMC
> +#endif
> +
> +#if defined(CONFIG_ENV_IS_IN_NAND)
> +#define CONFIG_ENV_OFFSET 0xc00000
> +#define CONFIG_ENV_SIZE 0x400000
> +#elif defined(CONFIG_ENV_IS_IN_MMC)
> +#define CONFIG_ENV_OFFSET (544 << 10) /* (8 + 24 + 512) KiB */
> +#define CONFIG_ENV_SIZE (128 << 10) /* 128 KiB */
> #define CONFIG_SYS_MMC_ENV_DEV 0 /* first detected MMC controller */
> #endif
>

I would greatly prefer putting the env in an UBI partition,
I thought that we had agreed on doing that ?

Regards,

Hans

> @@ -175,9 +196,6 @@
>
> #define CONFIG_SYS_MONITOR_LEN (768 << 10) /* 768 KiB */
>
> -#define CONFIG_ENV_OFFSET (544 << 10) /* (8 + 24 + 512) KiB */
> -#define CONFIG_ENV_SIZE (128 << 10) /* 128 KiB */
> -
> #define CONFIG_FAT_WRITE /* enable write access */
>
> #define CONFIG_SPL_FRAMEWORK
>