[PATCH v3 1/3] doc: Tidy up the bindings for the config/ node

Heinrich Schuchardt xypron.glpk at gmx.de
Sat Sep 11 11:00:40 CEST 2021


On 9/9/21 10:10 PM, Simon Glass wrote:
> Sort these and add a type so it is clear how to set the value. Add a note
> about usage to the top. Correct the 'no-keyboard' binding which is missing
> a prefix.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> ---
> Note that some uses a u-boot prefix and some don't. Once [1] is applied
> we may want to update this to always use the prefix, or never. Another
> option would be to call the node u-boot,config. and drop the prefix. This
> has the advantage of complying with the devicetree spec, at little cost.
>
> [1] https://patchwork.ozlabs.org/project/uboot/patch/20210807132413.3513724-2-sjg@chromium.org/
>
> Changes in v3:
> - Add missing period
>
> Changes in v2:
> - Add a new patch to tidy up the /config bindings
>
>   doc/device-tree-bindings/config.txt | 46 ++++++++++++++++-------------

This text file is not integrated into the HTML documentation.

Either rename it to something like u-boot-bindings.rst and add it in an
appropriate place. Or use :download:`title <filepath>` to link to the
text file.

This all can be done in a future patch.

I will take this patch as is into my next pull request as has been
assigned to me.

Best regards

Heinrich

>   1 file changed, 26 insertions(+), 20 deletions(-)
>
> diff --git a/doc/device-tree-bindings/config.txt b/doc/device-tree-bindings/config.txt
> index 6cdc16da5b5..f704eb695f5 100644
> --- a/doc/device-tree-bindings/config.txt
> +++ b/doc/device-tree-bindings/config.txt
> @@ -5,15 +5,15 @@ A number of run-time configuration options are provided in the /config node
>   of the control device tree. You can access these using fdtdec_get_config_int(),
>   fdtdec_get_config_bool() and fdtdec_get_config_string().
>
> -Available options are:
> -
> -silent-console
> -	If present and non-zero, the console is silenced by default on boot.
> +These options are designed to affect the operation of U-Boot at runtime.
> +Runtime-configuration items can help avoid proliferation of different builds
> +with only minor changes, e.g. enabling and disabling console output. Items
> +here should be those that can usefully be set by the build system after U-Boot
> +is built.
>
> -no-keyboard
> -	Tells U-Boot not to expect an attached keyboard with a VGA console
> +Available options are:
>
> -u-boot,efi-partition-entries-offset
> +u-boot,efi-partition-entries-offset (int)
>   	If present, this provides an offset (in bytes, from the start of a
>   	device) that should be skipped over before the partition entries.
>   	This is used by the EFI/GPT partition implementation when a device
> @@ -21,17 +21,8 @@ u-boot,efi-partition-entries-offset
>
>   	This setting will override any values configured via Kconfig.
>
> -u-boot,mmc-env-partition
> -	if present, the environment shall be placed at the last
> -	CONFIG_ENV_SIZE blocks of the partition on the
> -	CONFIG_SYS_MMC_ENV_DEV.
> -
> -	if u-boot,mmc-env-offset* is present, this setting will take
> -	precedence. In that case, only if the partition is not found,
> -	mmc-env-offset* will be tried.
> -
> -u-boot,mmc-env-offset
> -u-boot,mmc-env-offset-redundant
> +u-boot,mmc-env-offset (int)
> +u-boot,mmc-env-offset-redundant (int)
>   	If present, the values of the 'u-boot,mmc-env-offset' and/or
>   	of the u-boot,mmc-env-offset-redundant' properties overrides
>   	CONFIG_ENV_OFFSET and CONFIG_ENV_OFFSET_REDUND, respectively,
> @@ -42,12 +33,27 @@ u-boot,mmc-env-offset-redundant
>   	will point at the beginning of a LBA and values that are not
>   	LBA-aligned will be rounded up to the next LBA address.
>
> -u-boot,spl-payload-offset
> +u-boot,mmc-env-partition (int)
> +	if present, the environment shall be placed at the last
> +	CONFIG_ENV_SIZE blocks of the partition on the
> +	CONFIG_SYS_MMC_ENV_DEV.
> +
> +	if u-boot,mmc-env-offset* is present, this setting will take
> +	precedence. In that case, only if the partition is not found,
> +	mmc-env-offset* will be tried.
> +
> +u-boot,no-keyboard (bool)
> +	Tells U-Boot not to expect an attached keyboard with a VGA console.
> +
> +silent-console (int)
> +	If present and non-zero, the console is silenced by default on boot.
> +
> +u-boot,spl-payload-offset (int)
>   	If present (and SPL is controlled by the device-tree), this allows
>   	to override the CONFIG_SYS_SPI_U_BOOT_OFFS setting using a value
>   	from the device-tree.
>
> -sysreset-gpio
> +sysreset-gpio (string)
>   	If present (and supported by the specific board), indicates a
>   	GPIO that can be set to trigger a system reset.  It is assumed
>   	that such a system reset will effect a complete platform reset,
>



More information about the U-Boot mailing list