diff options
author | Simon Glass <sjg@chromium.org> | 2016-07-31 17:35:02 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2016-08-05 20:55:19 -0400 |
commit | bcbd0c8fe1bdd95f07d773d076b5101a01e48287 (patch) | |
tree | 5d0088f4268a05b41d3d59a7683a206041a4869f /test/py | |
parent | 633cc7ae96d04c4929404f439242ff68177e85ba (diff) |
test: Fix typos and tidy up
Fix review comments that were missed at the time. Also explain why we need
to regenerate the device tree for each test.
Reported-by: Teddy Reed <teddy.reed@gmail.com>
Suggested-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Simon Glass <sjg@chromium.org>
Fixes: f6349c3c (test: Add a README)
Diffstat (limited to 'test/py')
-rw-r--r-- | test/py/tests/test_vboot.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/test/py/tests/test_vboot.py b/test/py/tests/test_vboot.py index 14ec85b853..46552fc803 100644 --- a/test/py/tests/test_vboot.py +++ b/test/py/tests/test_vboot.py @@ -106,7 +106,9 @@ def test_vboot(u_boot_console): algo = sha - # Compile our device tree files for kernel and U-Boot + # Compile our device tree files for kernel and U-Boot. These are + # regenerated here since mkimage will modify them (by adding a + # public key) below. dtc('sandbox-kernel.dts') dtc('sandbox-u-boot.dts') @@ -139,7 +141,7 @@ def test_vboot(u_boot_console): sig = util.cmd(cons, 'fdtget -t bx %s %s value' % (fit, sig_node)) byte_list = sig.split() byte = int(byte_list[0], 16) - byte_list = ['%x' % (byte + 1)] + byte_list[1:] + byte_list[0] = '%x' % (byte + 1) sig = ' '.join(byte_list) util.cmd(cons, 'fdtput -t bx %s %s value %s' % (fit, sig_node, sig)) |