u-boot.git/tools/env, branch v2016.09-rc1

tools/env: reuse fw_getenv in fw_printenv function

2016-07-22T18:46:22+00:00

Try to avoid adhoc iteration of the environment. Reuse fw_getenv
to find the variables that should be printed. Only use open-coded
iteration when printing all variables.
For backwards compatibility, keep emitting a newline when
printing with value_only.

Signed-off-by: Andreas Fenkart

tools/env: move envmatch further up in file to avoid forward declarations

2016-07-22T18:46:22+00:00

forward declaration not needed when re-ordered

Reviewed-by: Simon Glass 
Signed-off-by: Andreas Fenkart

tools/env: kernel-doc for fw_printenv, fw_getenv and fw_parse_script

2016-07-22T18:46:21+00:00

there are two groups of functions:
- application ready tools: fw_setenv/fw_getenv/fw_parse_script
these are used, when creating a single binary containing multiple
tools (busybox like)
- file access like: open/read/write/close
above functions are implemented on top of these. applications
can use those to modify several variables without creating a
temporary batch script file
tested with "./scripts/kernel-doc -html -v tools/env/fw_env.h"

Reviewed-by: Simon Glass 
Signed-off-by: Andreas Fenkart

tools/env: allow negative offsets

2016-07-22T18:46:20+00:00

A negative value for the offset is treated as a backwards offset for
from the end of the device/partition for block devices. This aligns
the behavior of the config file with the syntax of CONFIG_ENV_OFFSET
where the functionality has been introduced with
commit 5c088ee841f9 ("env_mmc: allow negative CONFIG_ENV_OFFSET").

Signed-off-by: Stefan Agner

tools/env: complete environment device config early

2016-07-22T18:46:20+00:00

Currently flash_read completes a crucial part of the environment
device configuration, the device type (mtd_type). This is rather
confusing as flash_io calls flash_read conditionally, and one might
think flash_write, which also makes use of mtd_type, gets called
before flash_read. But since flash_io is always called with O_RDONLY
first, this is not actually the case in reality.

However, it is much cleaner to complete and verify the config early
in parse_config. This also prepares the code for further extension.

Signed-off-by: Stefan Agner 
Reviewed-by: Andreas Fenkart

Various, unrelated tree-wide typo fixes.

2016-07-16T13:43:12+00:00

    Fix a number of typos, including:

     * "compatble" -> "compatible"
     * "eanbeld" -> "enabled"
     * "envrionment" -> "environment"
     * "FTD" -> "FDT" (for "flattened device tree")
     * "ommitted" -> "omitted"
     * "overriden" -> "overridden"
     * "partiton" -> "partition"
     * "propogate" -> "propagate"
     * "resourse" -> "resource"
     * "rest in piece" -> "rest in peace"
     * "suport" -> "support"
     * "varible" -> "variable"

Signed-off-by: Robert P. J. Day

tools/env: allow to pass NULL for environment options

2016-05-31T14:23:10+00:00

If users of the library are happy with the default, e.g. config file
name. They can pass NULL as the opts pointer. This simplifies the
transition of existing library users.
FIXES a compile error. since common_args has been removed by
a previous patch

Signed-off-by: Andreas Fenkart

tools/env: no global variable sharing between application and library

2016-05-27T19:39:56+00:00

Signed-off-by: Andreas Fenkart

tools/env: compute size of usable area only once

2016-05-27T13:56:19+00:00

for double buffering to work, redundant buffers must have equal size

Signed-off-by: Andreas Fenkart

tools/env: fw_printenv pass value_only as argument

2016-05-27T13:56:19+00:00

Signed-off-by: Andreas Fenkart