-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add sof-v2.9 for Intel hardware #164
Conversation
V2 uploaded, now tests pass. |
bf00d93
to
559970b
Compare
V3: fix tarball_multi_releases
V4: fix BATS tests again |
559970b
to
99ca1e7
Compare
V5: one more fixup to the sof-ace-tplg symlink handling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kv2019i, nice and clean, thank you!
99ca1e7
to
2296aba
Compare
The community binaries have "v2.9-rc1" tag (it's the same commit as v2.9 tag), causing difference in binaries. We really should get rid of this in SOF main. |
V6:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated the communiy-signed binaries (now binary uses "v2.9" tag instead of "v2.9-rc1" so there's more obvious match of binaries...
Why keep the v2.9-rc1 binaries ?
# the compatibility note for Intel MTL in | ||
# https://thesofproject.github.io/latest/getting_started/intel_debug/introduction.html#user-space-and-filesystem-requirements | ||
( cd "${archive_name}" | ||
if test -e "sof-ipc4-tplg-WIP" -a ! -e "sof-ace-tplg-WIP" ; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellcheck would likely say:
if test -e "sof-ipc4-tplg-WIP" -a ! -e "sof-ace-tplg-WIP" ; then | |
if test -e "sof-ipc4-tplg-WIP" && ! test -e "sof-ace-tplg-WIP" ; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marc-hb Hmm, shellcheck seems happy with original.
tests/tests.bats
Outdated
diff -qr "$EXTR_REFS"/sof-bin-"$ver" "$(pwd)/sof-bin-$ver"/ | ||
# install.sh was updated in 2024.03, so it won't match | ||
# anymore | ||
diff -qr --exclude="install.sh" "$EXTR_REFS"/sof-bin-"$ver" "$(pwd)/sof-bin-$ver"/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please try this instead?
git show v2023.12.1:install.sh > "$(pwd)/sof-bin-$ver"/install.sh
v2.9.x/tools-v2.9/mtrace-reader.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there any tools change between v2.8 and v2.9? If not, can you just re-use tools-v2.8
and reduce noice?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marc-hb There was a minor change in sof-ctl, so we need to update.
Add binaries for supported Intel systems. Community binaries build identification "Build #3513". Also contains topology2/IPC4 topology files for all configurations supported in the release. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Add support for new sof-ipc4-tplg directory as documented in updated SOF user-space and filesystem requirements. Link: https://thesofproject.github.io/latest/getting_started/intel_debug/introduction.html#user-space-and-filesystem-requirements Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
The install.sh has been updated to include more directories, so recreating 2023.09 will cause a mismatch in installed install.sh. Exclude the known difference in the test case. Suggested-by: Marc Herbert <marc.herbert@intel.com> Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
…link Not ideal, but it seems adding a special case rule is easier to understand and maintain than trying to modify the existing FW/tplg copy rules to handle this special case. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Add signed Intel binaries for v2.9 SOF release. Intel build identification 13778510. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
d2ea2ef
to
1edb6ca
Compare
V7:
@marc-hb please check |
No description provided.