8122f396 | 17-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Enhance the map and fdt-update output
At present the .map file produced for each image does not include the overall image size. This is useful information.
Update the code to generate it in
binman: Enhance the map and fdt-update output
At present the .map file produced for each image does not include the overall image size. This is useful information.
Update the code to generate it in the .map file as well as the updated FDT. Also fix a few comments while we are here.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
ea6922e3 | 17-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Tidy up some comments in the tests
A few lines are commented out and can be removed. Also fix return-value docs for _DoReadFile() and _DoReadFileDtb().
Signed-off-by: Simon Glass <sjg@chrom
binman: Tidy up some comments in the tests
A few lines are commented out and can be removed. Also fix return-value docs for _DoReadFile() and _DoReadFileDtb().
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
b2b0df8f | 17-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Add comments to elf_test
The purpose of some of the tests is not obvious from the function names. Add a few comments to help with understanding.
Signed-off-by: Simon Glass <sjg@chromium.org> |
3ab9598d | 01-Aug-2018 |
Simon Glass <sjg@chromium.org> |
binman: Rename 'position' to 'offset'
After some thought, I believe there is an unfortunate naming flaw in binman. Entries have a position and size, but now that we support hierarchical sections it
binman: Rename 'position' to 'offset'
After some thought, I believe there is an unfortunate naming flaw in binman. Entries have a position and size, but now that we support hierarchical sections it is unclear whether a position should be an absolute position within the image, or a relative position within its parent section.
At present 'position' actually means the relative position. This indicates a need for an 'image position' for code that wants to find the location of an entry without having to do calculations back through parents to discover this image position.
A better name for the current 'position' or 'pos' is 'offset'. It is not always an absolute position, but it is always an offset from its parent offset.
It is unfortunate to rename this concept now, 18 months after binman was introduced. However I believe it is the right thing to do. The impact is mostly limited to binman itself and a few changes to in-tree users to binman:
tegra sunxi x86
The change makes old binman definitions (e.g. downstream or out-of-tree) incompatible if they use the 'pos = <...>' property. Later work will adjust binman to generate an error when it is used.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
46d61a2f | 17-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Don't depend on dict order in ELF testOutsideFile()
At present this test assumes that the symbols are returned in address order. However, objdump can list symbols in any order and dictionari
binman: Don't depend on dict order in ELF testOutsideFile()
At present this test assumes that the symbols are returned in address order. However, objdump can list symbols in any order and dictionaries do not guarantee any particular order when iterating through item.
Update elf.GetSymbols() to return an OrderedDict, sorted by address, to avoid any problems.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
60801702 | 24-Jul-2018 |
Peng Fan <peng.fan@nxp.com> |
tools: genboardscfg: move buildman path to first
To system which has kconfiglib installed, genboardscfg will use system kconfiglib, we need it use U-Boot owned version, so move the buildman path to
tools: genboardscfg: move buildman path to first
To system which has kconfiglib installed, genboardscfg will use system kconfiglib, we need it use U-Boot owned version, so move the buildman path to first.
Signed-off-by: Peng Fan <peng.fan@nxp.com> Cc: Masahiro Yamada <yamada.m@jp.panasonic.com> Cc: Simon Glass <sjg@chromium.org> Cc: Tom Rini <trini@konsulko.com> Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com> Reviewed-by: Simon Glass <sjg@chromium.org>
show more ...
|
2ae23a28 | 26-Jul-2018 |
Tom Rini <trini@konsulko.com> |
Merge git://git.denx.de/u-boot-dm |
b8a48fbc | 27-Jun-2018 |
Adam Sampson <ats@offog.org> |
patman: Split alias entries on arbitrary whitespace
Previously the first three words in a git-mailrc alias entry could only be separated by spaces. git-send-email and Mutt both allow arbitrary white
patman: Split alias entries on arbitrary whitespace
Previously the first three words in a git-mailrc alias entry could only be separated by spaces. git-send-email and Mutt both allow arbitrary whitespace here.
Signed-off-by: Adam Sampson <ats@offog.org> Reviewed-by: Simon Glass <sjg@chromium.org>
show more ...
|
4a8b6e01 | 20-Jul-2018 |
Michal Simek <michal.simek@xilinx.com> |
mkimage: fit_image: Use macros from image.h
There is no reason not to use macros which are already defined. It is also much easier for grepping.
Signed-off-by: Michal Simek <michal.simek@xilinx.com
mkimage: fit_image: Use macros from image.h
There is no reason not to use macros which are already defined. It is also much easier for grepping.
Signed-off-by: Michal Simek <michal.simek@xilinx.com> Reviewed-by: Simon Glass <sjg@chromium.org>
show more ...
|
7e1a0483 | 06-Jul-2018 |
Holger Dengler <dengler@linutronix.de> |
tools/imximage: get HAB information from header
Signing parts of a u-boot imximage for image verification in High Assurance Boot (HAB) in a post-build process, requires some information from the imx
tools/imximage: get HAB information from header
Signing parts of a u-boot imximage for image verification in High Assurance Boot (HAB) in a post-build process, requires some information from the imximage header. Currently, this information is only provided during the image build, which makes the transfer of this information to the post-build process harder than necessary.
The i.MX HAB information (start and length) can be calculated either by using information from the image-configuration file, or from the information in the flash header of the imximage. The advantage of using information from flash header is, that they are not only available during image creation, but also available if existing images are processed.
Example: $ tools/mkimage -l u-boot.imx Image Type: Freescale IMX Boot Image Image Ver: 2 (i.MX53/6/7 compatible) Mode: DCD Data Size: 483328 Bytes = 472.00 KiB = 0.46 MiB Load Address: 877ff420 Entry Point: 87800000 HAB Blocks: 0x877ff400 0x00000000 0x00071c00 DCD Blocks: 0x00910000 0x0000002c 0x00000208
Signed-off-by: Holger Dengler <dengler@linutronix.de> Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
show more ...
|
12308b12 | 16-Jul-2018 |
Siva Durga Prasad Paladugu <siva.durga.paladugu@xilinx.com> |
lib: fdtdec: Rename routine fdtdec_setup_memory_size()
This patch renames the routine fdtdec_setup_memory_size() to fdtdec_setup_mem_size_base() as it now fills the mem base as well along with size.
lib: fdtdec: Rename routine fdtdec_setup_memory_size()
This patch renames the routine fdtdec_setup_memory_size() to fdtdec_setup_mem_size_base() as it now fills the mem base as well along with size.
Signed-off-by: Siva Durga Prasad Paladugu <siva.durga.paladugu@xilinx.com> Signed-off-by: Michal Simek <michal.simek@xilinx.com> Reviewed-by: Tom Rini <trini@konsulko.com>
show more ...
|
795f452e | 20-Jun-2018 |
Alex Kiernan <alex.kiernan@gmail.com> |
mkimage: fit_image: Add support for SOURCE_DATE_EPOCH in signatures
When generating timestamps in signatures, use imagetool_get_source_date() so we can be overridden by SOURCE_DATE_EPOCH to generate
mkimage: fit_image: Add support for SOURCE_DATE_EPOCH in signatures
When generating timestamps in signatures, use imagetool_get_source_date() so we can be overridden by SOURCE_DATE_EPOCH to generate reproducible images.
Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com> Reviewed-by: Simon Glass <sjg@chromum.org>
show more ...
|
87925df2 | 20-Jun-2018 |
Alex Kiernan <alex.kiernan@gmail.com> |
mkimage: Refactor imagetool_get_source_date to take command name
So we can use imagetool_get_source_date() from callers who do not have the image tool params struct, just pass in the command name fo
mkimage: Refactor imagetool_get_source_date to take command name
So we can use imagetool_get_source_date() from callers who do not have the image tool params struct, just pass in the command name for the error message.
Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com> Reviewed-by: Simon Glass <sjg@chromum.org>
show more ...
|
7346c1e1 | 09-Jun-2018 |
Teddy Reed <teddy.reed@gmail.com> |
vboot: Do not use hashed-strings offset
The hashed-strings signature property includes two uint32_t values. The first is unneeded as there should never be a start offset into the strings region. The
vboot: Do not use hashed-strings offset
The hashed-strings signature property includes two uint32_t values. The first is unneeded as there should never be a start offset into the strings region. The second, the size, is needed because the added signature node appends to this region.
See tools/image-host.c, where a static 0 value is used for the offset.
Signed-off-by: Teddy Reed <teddy.reed@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
show more ...
|
72239fc8 | 09-Jun-2018 |
Teddy Reed <teddy.reed@gmail.com> |
vboot: Add FIT_SIGNATURE_MAX_SIZE protection
This adds a new config value FIT_SIGNATURE_MAX_SIZE, which controls the max size of a FIT header's totalsize field. The field is checked before signature
vboot: Add FIT_SIGNATURE_MAX_SIZE protection
This adds a new config value FIT_SIGNATURE_MAX_SIZE, which controls the max size of a FIT header's totalsize field. The field is checked before signature checks are applied to protect from reading past the intended FIT regions.
This field is not part of the vboot signature so it should be sanity checked. If the field is corrupted then the structure or string region reads may have unintended behavior, such as reading from device memory. A default value of 256MB is set and intended to support most max storage sizes.
Suggested-by: Simon Glass <sjg@chromium.org> Signed-off-by: Teddy Reed <teddy.reed@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
show more ...
|
16b8d6b7 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Support updating the device tree with calc'd info
It is useful to write the position and size of each entry back to the device tree so that U-Boot can access this at runtime. Add a feature t
binman: Support updating the device tree with calc'd info
It is useful to write the position and size of each entry back to the device tree so that U-Boot can access this at runtime. Add a feature to support this, along with associated tests.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
078ab1a2 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Add a SetCalculatedProperties() method
Once binman has packed the image, the position and size of each entry is known. It is then possible for binman to update the device tree with these pos
binman: Add a SetCalculatedProperties() method
Once binman has packed the image, the position and size of each entry is known. It is then possible for binman to update the device tree with these positions. Since placeholder values have been added, this does not affect the size of the device tree and therefore the packing does not need to be performed again.
Add a new SetCalculatedProperties method to handle this.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
ecab8973 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Add a ProcessFdt() method
Some entry types modify the device tree, e.g. to remove microcode or add a property. So far this just modifies their local copy and does not affect a 'shared' devic
binman: Add a ProcessFdt() method
Some entry types modify the device tree, e.g. to remove microcode or add a property. So far this just modifies their local copy and does not affect a 'shared' device tree.
Rather than doing this modification in the ObtainContents() method, and a new ProcessFdt() method which is specifically designed to modify this shared device tree.
Move the existing device-tree code over to use this method, reducing ObtainContents() to the goal of just obtaining the contents without any processing, even for device tree.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
0a4357c4 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Complete documentation of stages
At present one of the stages is badly numbered and not described. Fix this.
Signed-off-by: Simon Glass <sjg@chromium.org> |
116adecb | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
dtoc: Add functions to add integer properties
Add a few simple functions to add a placeholder integer property, and set its value.
Signed-off-by: Simon Glass <sjg@chromium.org> |
fe57c784 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
dtoc: Avoid unwanted output during tests
At present some warnings are printed to indicate failures which are a known part of running the tests. Suppress these.
Signed-off-by: Simon Glass <sjg@chrom
dtoc: Avoid unwanted output during tests
At present some warnings are printed to indicate failures which are a known part of running the tests. Suppress these.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
8512ea2e | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
dtoc: Increase code coverage to 100%
Add more tests to increase dtoc code coverage to 100%.
Correct a whitespace error in some test .dts files at the same time.
Signed-off-by: Simon Glass <sjg@chr
dtoc: Increase code coverage to 100%
Add more tests to increase dtoc code coverage to 100%.
Correct a whitespace error in some test .dts files at the same time.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
c3f94541 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
binman: Move capture_sys_output() to test_util
This function is useful in various tests. Move it into the common test utility module.
Signed-off-by: Simon Glass <sjg@chromium.org> |
ba765217 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
dtoc: Add a test for code coverage
Add a -T option to run a code-coverage test on dtoc. At present this is about 96%. Future work will increase it to 100%.
Signed-off-by: Simon Glass <sjg@chromium.
dtoc: Add a test for code coverage
Add a -T option to run a code-coverage test on dtoc. At present this is about 96%. Future work will increase it to 100%.
Signed-off-by: Simon Glass <sjg@chromium.org>
show more ...
|
dfe5f5b9 | 06-Jul-2018 |
Simon Glass <sjg@chromium.org> |
dtoc: Fix some minor errors
Fix some comments and a printf string which is incorrect.
Signed-off-by: Simon Glass <sjg@chromium.org> |