i think this should the opt-in not opt-out. target maintainers should
set it only if they have boards with the env stored inside ubi.

        John

On 12/12/2014 23:59, Daniel Golle wrote:
> We've been carrying around that patch for a while now and didn't
> ever encounter any problem on non-UBI flashtypes. So I guess the
> 8kB extra space won't hurt on modern platforms which might even make
> use of their U-Boot environment being stored in UBI.
> 
> Let me know if there are any targets missing in the blacklist
> (i.e. platforms/SoCs without the option to deploy NAND flash or
> commonly found with only 4MB flash where those 8kB might actually
> matter, or platforms where we haven't got U-Boot, ...)
> 
> Signed-off-by: Daniel Golle <dan...@makrotopia.org>
> ---
>  package/boot/uboot-envtools/Config.in | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/package/boot/uboot-envtools/Config.in 
> b/package/boot/uboot-envtools/Config.in
> index 9fd8103..10c03f9 100644
> --- a/package/boot/uboot-envtools/Config.in
> +++ b/package/boot/uboot-envtools/Config.in
> @@ -1,7 +1,13 @@
>  config UBOOT_ENVTOOLS_UBI
>       bool "Support environment in UBI volume"
>       depends on PACKAGE_uboot-envtools
> -     default n
> +     depends on !TARGET_adm5120 && !TARGET_adm8668 && !TARGET_ar7 &&        \
> +                !TARGET_ar71xx_generic && !TARGET_atheros &&                \
> +                !TARGET_au1000 && !TARGET_avr32 && !TARGET_ramips_rt288x && \
> +                !TARGET_ramips_rt305x && !TARGET_uml && !TARGET_x86 &&      \
> +                !TARGET_x86_64
> +
> +     default y
>       help
>               Add support for reading and writing U-Boot environment
>               stored in UBI volume(s).
> 
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to