Hi Masahiro,

On 07.09.2014 10:43, Masahiro Yamada wrote:
> This commit moves:
>   CONFIG_OF_CONTROL
>   CONFIG_OF_SEPARATE
>   CONFIG_OF_EMBED
>   CONFIG_OF_HOSTFILE
> 
> Because these options are currently not supported for SPL,
> the "Device Tree Control" menu does not appear in the SPL
> configuration.
> 
> Note:
> zynq-common.h should be adjusted so as not to change the
> default value of CONFIG_SPL_FAT_LOAD_PAYLOAD_NAME.
> 
> Signed-off-by: Masahiro Yamada <yamad...@jp.panasonic.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Stephen Warren <swar...@nvidia.com>
> Cc: Minkyu Kang <mk7.k...@samsung.com>
> Cc: Michal Simek <michal.si...@xilinx.com>
> ---

[...]

> diff --git a/dts/Kconfig b/dts/Kconfig
> new file mode 100644
> index 0000000..98b2357
> --- /dev/null
> +++ b/dts/Kconfig
> @@ -0,0 +1,46 @@
> +#
> +# Device Tree Control
> +#
> +# TODO:
> +#   This feature is not currently supported for SPL,
> +#    but this restriction should be removed in the future.
> +
> +menu "Device Tree Control"
> +     depends on !SPL_BUILD
> +     depends on ARM || MICROBLAZE || SANDBOX || X86

I suggest to use boolean symbols HAS_* or SUPPORTS_* like the kernel
does. For example:

config SUPPORTS_OF_CONTROL
        bool

menu "Device Tree Control"
        depends on !SPL_BUILD
        depends on SUPPORTS_OF_CONTROL


arch/Kconfig:
config ARM
        bool "ARM architecture"
        select SUPPORTS_OF_CONTROL


> +
> +config OF_CONTROL
> +     bool "Run-time configuration via Device Tree"
> +     help
> +       This feature provides for run-time configuration of U-Boot
> +       via a flattened device tree.
> +
> +choice
> +     prompt "Provider of DTB for DT control"
> +     depends on OF_CONTROL
> +     default OF_HOSTFILE if SANDBOX
> +     default OF_SEPARATE
> +
> +config OF_SEPARATE
> +     bool "Separate DTB for DT control"
> +     help
> +       If this option is enabled, the device tree will be built and
> +       placed as a separate u-boot.dtb file alongside the U-Boot image.
> +
> +config OF_EMBED
> +     bool "Embedded DTB for DT control"
> +     help
> +       If this option is enabled, the device tree will be picked up and
> +       built into the U-Boot image.
> +
> +config OF_HOSTFILE
> +     bool "Host filed DTB for DT control"
> +     depends on SANDBOX
> +     help
> +       If this option is enabled, DTB will be read from a file on startup.
> +       This is only useful for Sandbox.  Use the -d flag to U-Boot to
> +       specify the file to read.
> +
> +endchoice
> +
> +endmenu
> 

-- 
- Daniel
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to