Skip to content

Commit

Permalink
rimage: mtl/lnl: fix key slot setup based on imr type
Browse files Browse the repository at this point in the history
Clean cherry-picks of two commits from separate rimage repo
https://github.com/thesofproject/rimage/commits/stable-v2.7

This will properly setup partition_usage field
and remove fixed 0x23 value from MTL toml

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
(cherry picked from commit fbea59358d06ffa86645cdf4ce0996e352742eb5)

Second, squashed LNL commit:

config/lnl.toml: drop now invalid signed_pkg.partition_usage

Fixes commit fbea59358d06 ("rimage: mtl: fix key slot setup based on imr
type") which broke LNL compilation like this:

```
error: 1 unparsed keys left in 'signed_pkg'
error: key 'signed_pkg' parsing error
```

This commit does not change the final image (compared to before rimage
commit fbea59358d06) because the default value is now 0x20 + IMR type.

Fixes are normally submitted to the main branch first and then cherry
picked to stable branches. However MTL fbea59358d06 has been made in
stable-v2.7 first so the LNL change was also in stable-v2.7 first for
consistency.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit 469102a8f6052ce6a374b870ea945a84d04c3b1a)
  • Loading branch information
abonislawski authored and marc-hb committed Oct 30, 2023
1 parent 1c729f4 commit 2fd28c6
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 7 deletions.
1 change: 0 additions & 1 deletion tools/rimage/config/lnl.toml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ length = "0x0" # calculated by rimage

[signed_pkg]
name = "ADSP"
partition_usage = "0x23"
[[signed_pkg.module]]
name = "ADSP.met"

Expand Down
1 change: 0 additions & 1 deletion tools/rimage/config/mtl.toml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ length = "0x0" # calculated by rimage

[signed_pkg]
name = "ADSP"
partition_usage = "0x23"
[[signed_pkg.module]]
name = "ADSP.met"

Expand Down
4 changes: 1 addition & 3 deletions tools/rimage/src/adsp_config.c
Original file line number Diff line number Diff line change
Expand Up @@ -1087,9 +1087,7 @@ static int parse_signed_pkg_ace_v1_5(const toml_table_t *toml, struct parse_ctx
if (ret < 0)
return ret;

out->partition_usage = parse_uint32_hex_key(signed_pkg, &ctx, "partition_usage", 0, &ret);
if (ret < 0)
return ret;
out->partition_usage = 0x20 + image->imr_type;

/* check everything parsed, expect 1 more array */
ctx.array_cnt += 1;
Expand Down
2 changes: 0 additions & 2 deletions tools/rimage/src/rimage.c
Original file line number Diff line number Diff line change
Expand Up @@ -191,8 +191,6 @@ int main(int argc, char *argv[])
}

if (image.adsp->man_ace_v1_5) {
if (imr_type_override)
image.adsp->man_ace_v1_5->adsp_file_ext.imr_type = image.imr_type;
image.adsp->man_ace_v1_5->css.reserved0 = pv_bit;
}

Expand Down

0 comments on commit 2fd28c6

Please sign in to comment.