Skip to content

Commit

Permalink
xtensa-build-zephyr.py: re-enable sof_ri_info.py on MTL and LNL
Browse files Browse the repository at this point in the history
Now we can avoid bitrot again thanks to recent rimage fix
thesofproject/rimage@4fb9fe00575bc

Also update outdated comments.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb authored and lgirdwood committed Aug 28, 2023
1 parent d0cb478 commit c0fd911
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions scripts/xtensa-build-zephyr.py
Original file line number Diff line number Diff line change
Expand Up @@ -779,6 +779,11 @@ def build_platforms():
for p_alias in platform_configs[platform].aliases:
symlink_or_copy(sof_platform_output_dir, f"sof-{platform}.ldc", f"sof-{p_alias}.ldc")

# reproducible-zephyr.ri is less useful now that show_installed_files() shows
# checksums too. However: - it's still useful when only the .ri file is
# available (no build logs for the other image), - it makes sure sof_ri_info.py
# itself does not bitrot, - it can catch rimage issues, see for instance rimage
# fix 4fb9fe00575b
if platform not in RI_INFO_UNSUPPORTED:
reproducible_checksum(platform, west_top / platform_build_dir_name / "zephyr" / "zephyr.ri")

Expand Down Expand Up @@ -968,8 +973,8 @@ def gzip_compress(fname, gzdst=None):
RI_INFO_UNSUPPORTED += ['rn']
RI_INFO_UNSUPPORTED += ['mt8186', 'mt8195']

# sof_ri_info.py has not caught up with the latest rimage yet: these will print a warning.
RI_INFO_FIXME = ['mtl', 'lnl']
# For temporary workarounds. Unlike _UNSUPPORTED above, the platforms below will print a warning.
RI_INFO_FIXME = [ ]

def reproducible_checksum(platform, ri_file):

Expand Down

0 comments on commit c0fd911

Please sign in to comment.