-
Notifications
You must be signed in to change notification settings - Fork 311
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
audio: base_fw_intel: Fix problem with send structure ext_system_time_data #9277
Conversation
95b6c74
to
52e38e6
Compare
src/audio/base_fw_intel.c
Outdated
memcpy_s(ext_system_time, sizeof(ext_system_time), &ext_system_time_data, | ||
sizeof(ext_system_time)); | ||
memcpy_s(ext_system_time, sizeof(struct ipc4_ext_system_time), &ext_system_time_data, | ||
sizeof(struct ipc4_ext_system_time)); |
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.
I guess "sizeof(ext_system_time_data)" was the original intent ("sizeof(*ext_system_time)" would have done as well). A good catch nevertheless. This is not used by the Linux host so was not caught with Linux driver tests.
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.
yes, a very good catch, but indeed a
memcpy_s(ext_system_time, sizeof(*ext_system_time), &ext_system_time_data,
sizeof(ext_system_time_data));
would be even better IMHO.
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.
@lyakh Good point, I changed this line. Thank you
…_data There was a problem with test test_102_08_extended_time_check. The structure ext_system_time_data was sending with wrong size. Because of this it was not possible to correctly read the value from the art clock to determine the HH counter. Signed-off-by: Grzegorz Bernat <grzegorzx.bernat@intel.com>
52e38e6
to
222a33c
Compare
SOFCI TEST |
There was a problem with test test_102_08_extended_time_check. The structure ext_system_time_data was sending with wrong size. Because of this it was not possible to correctly read the value from the art clock to determine the HH counter.