xtensa-build-zephyr.py: re-enable sof_ri_info.py on MTL and LNL

Now we can avoid bitrot again thanks to recent rimage fix thesofproject/rimage@4fb9fe00575bc Also update outdated comments. Signed-off-by: Marc Herbert <[email protected]>
marc-hb · Aug 28, 2023 · e4a75ce · e4a75ce
1 parent d0cb478
commit e4a75ce
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/scripts/xtensa-build-zephyr.py b/scripts/xtensa-build-zephyr.py
@@ -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")
 
@@ -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):