diff options
| author | Vadim Bendebury <vbendeb@google.com> | 2017-08-02 11:35:25 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-08-02 17:26:42 -0700 |
| commit | 5a9d0de240fd7e964f15e7f2cae33277fd57f046 (patch) | |
| tree | fe364780f1597fe677bd0f08f22b870828c6f843 /util/signer | |
| parent | e156e014dda43438761985cf67ca95ec48cb3208 (diff) | |
| download | chrome-ec-5a9d0de240fd7e964f15e7f2cae33277fd57f046.tar.gz | |
g: stop converting hex device id values to ints
The new signer (version: 1.2 00840c1b6) allows hex values in the
manifest, which means there is no need to explicitly convert the
values before adding them to the manifest.
A nice side effect of this is the fact that there is no need to care
about the sign of the values any more, the signer does the right
thing.
BRANCH=none
BUG=none
TEST=built an image using the following invocation:
$ make BOARD=cr50 H1_DEVIDS='0x12009015 0x90e95664' -j
and successfully ran it on a device. Note that the old signer was
chocking on hex values exceeding 0x7fffffff, the new one handles
them properly.
Change-Id: I08c0339f922d287c82d56fb51570bfbf7107531e
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/598728
Reviewed-by: Nick Sanders <nsanders@chromium.org>
Diffstat (limited to 'util/signer')
| -rwxr-xr-x | util/signer/bs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/signer/bs b/util/signer/bs index ecf3cb221a..efca4d5170 100755 --- a/util/signer/bs +++ b/util/signer/bs @@ -74,7 +74,7 @@ tweak_manifest () { # If defined, plug in dev ID nodes before the 'fuses' node. if [[ -z "${do_prod}" && -n "${H1_DEVIDS}" ]]; then echo "creating a customized DEV image for DEV IDS ${H1_DEVIDS}" - sub=$(printf "\\\n \"DEV_ID0\": %d,\\\n \"DEV_ID1\": %d," ${H1_DEVIDS}) + sub=$(printf "\\\n \"DEV_ID0\": %s,\\\n \"DEV_ID1\": %s," ${H1_DEVIDS}) sed -i "s/\"fuses\": {/\"fuses\": {${sub}/" "${tmpf}" fi |