~canonical-hwe-team/hwe-next/+git/sof:lrg/topic/cmake

Last commit made on 2020-08-23
Get this branch:
git clone -b lrg/topic/cmake https://git.launchpad.net/~canonical-hwe-team/hwe-next/+git/sof

Branch merges

Branch information

Name:
lrg/topic/cmake
Repository:
lp:~canonical-hwe-team/hwe-next/+git/sof

Recent commits

5712792... by Liam Girdwood <email address hidden> on 2020-08-21

cmake: set git hash to 8 digit

GIT_LOG_HASH can be more than 8 digits. Limit hash value up to
first 8 digits.

fixes: #3322

Signed-off-by: Fred Oh <email address hidden>

2e07bb9... by Mohana Datta Yelugoti <email address hidden> on 2020-08-18

tools: testbench: send valid pointer to free system call

free() system call frees the memory space pointed to by
ptr, which must have been returned by previous call to
malloc(), calloc() or realloc().

Now, when incrementing the array pointer, we may have
exceeded the actual allocated size. If we do pointer arithmetic
and subtract size from the array pointer, we may not point to
the actual location returned by malloc().

This patch makes sure that when we do pointer arithmetic on array,
the array's value at the end would be same as the one returned
by malloc.

Signed-off-by: Mohana Datta Yelugoti <email address hidden>

87e4f5a... by Daniel Baluta <email address hidden> on 2020-08-17

host: Fix first period not being copied to/from host

After commit 048145fe6006f8 ("pipeline: remove code related to preload")
buffers are no longer preloaded so we don't need to manually
update first period transfer.

Taking playback as an example, without this patch first period bytes
weren't copied downstream to DAIs removing typically first 0.1 seconds
from the input file.

With PCM streams this is not really noticeable. Anyhow, this becomes
a problem with compressed streams where we need every byte for
decompression.

Signed-off-by: Daniel Baluta <email address hidden>

96a4748... by Mohana Datta Yelugoti <email address hidden> on 2020-08-15

tools: testbench: check for array size mismatch for components

This patch uses the functionality added by the c9e090ccf37 commit
to check for array size mismatch in various components.

Not doing so may result in cases, where we try to write into the
space not allocated leading to segmentation fault.

Signed-off-by: Mohana Datta Yelugoti <email address hidden>

782d4ce... by Mohana Datta Yelugoti <email address hidden> on 2020-08-14

testbench: check array size mismatch while loading topology buffer

This patch uses the functionality added by the c9e090c commit
to check for array size mismatch while loading topology
buffer.

Signed-off-by: Mohana Datta Yelugoti <email address hidden>

d9712ef... by Mohana Datta Yelugoti <email address hidden> on 2020-08-14

testbench: topology:check array size mismatch in filewrite component

This patch uses the functionality added by the c9e090ccf37 commit
to check for array size mismatch in components.

Since, the components are in places other than tplg_parser, the
function which checks for the mismatch has been made an external
function from a static one. The function declaration has been
added to the tplg_parser header files, so that the function can
be used by any code which needs array size checks in components.

Signed-off-by: Mohana Datta Yelugoti <email address hidden>

6c14e76... by Karol Trzcinski <email address hidden> on 2020-06-26

trace: Log FW ABI and hash numbers

This log allows to fully identify source code and check ldc
compatibility with FW by comparing source hash value saved in ldc
file with value received from FW in runtimee.
Without this mechanism it's impossible to
be sure about right connection without booting firmware
(to read fw_ready message content), because value of fw_hash
hasn't been provided in output logs file. This is especially
important during debugging process based on logs received by
client.

Signed-off-by: Karol Trzcinski <email address hidden>

2263a8b... by Karol Trzcinski <email address hidden> on 2020-07-29

cmake: Include GIT_TAG in generated version.h file

GIT_TAG is user readable form of used source code version with
commit identifier, what is important for bugs reproducibility,
so it will be convenient to have this information in output logs.

Signed-off-by: Karol Trzcinski <email address hidden>

901f991... by Karol Trzcinski <email address hidden> on 2020-07-08

logger: Validate by src_hash instead of abi version from fw_ready

ABI version saved in fw_ready doesn't match with DBG_ABI version
saved in ldc file even for proper pair of fw and ldc file.
Moreover ldc file content changes at any log modyfication, what
is not related with DBG_ABI change, so this way of solving this
problem is incorrect.
Introduced src_hash value change solves both problems.

Signed-off-by: Karol Trzcinski <email address hidden>

9f293b0... by Karol Trzcinski <email address hidden> on 2020-07-17

sof: Add `src_hash` to `sof_ipc_fw_version` structure

This field will be used to compare ldc file with loaded fw version,
to assert validity of trace logs. Value used in sof-logger.

Signed-off-by: Karol Trzcinski <email address hidden>