logger: Refactor fread() error check in logger_read()

fread() returns number of readed blocks, 0 when nothing read.
Comparison fread return value with "!ret" is quite misleading -
may suggests that negative value is returrned after fail.

Swapping if content makes flow easier, then first is error check,
and eventyally return statement, next try to reopen file.
It allows to check error condition only in one place, so there
won't be possibility to use different error checks in subsequent
stages (like ferror() and errno).

in_file alignment with trace entry size check has been added,
to warn about corrupted file.

Signed-off-by: Karol Trzcinski <[email protected]>

trace: null-check for comp context

Check if returned component is NULL.
Reported by static analysis tool.

Signed-off-by: Janusz Jankowski <[email protected]>

Audio: Continue volume ramp until all channels are complete

The volume component works correct when all channels receive the same
gain value in volume set command.

This patch fixes a bug that is triggered by applying different
gain values for the channels. The logic with setting cd->ramp_finished
to true caused the check in volume copy() to no more call volume_ramp()
when one of the channel reached their target volume. When the ramp
updating was stopped all other channels remain in intermediate gain
value.

In the fix the logic is set to opposite. Whenever a channel needs a ramp
update it sets a temporary flag. The ramp finish is set only when
no channels needed gain update.

Fixes thesofproject#3455

Signed-off-by: Seppo Ingalsuo <[email protected]>

Post processing release

The initial version of post processing component

component: switch to use UUID for component creation

Switch to use UUID for component creation, if it is provided from the
host, otherwise, use component type for the component driver matching.

Signed-off-by: Keyon Jie <[email protected]>
Signed-off-by: Marc Herbert <[email protected]>

ll: set initial value for num_clients

Value for 'num_clients' has to be set, otherwise it may
have undefined value in later ifs.

Signed-off-by: Janusz Jankowski <[email protected]>

mux: simplifies mux processing functions

This commit simplifies mux processing functions and
does not allow mux to mix channels.

Signed-off-by: Bartosz Kokoszko <[email protected]>

chrome: jsl: disable system agent

To match TGL release config.

Signed-off-by: Janusz Jankowski <[email protected]>

topology: fix playback & capture pipelines to be timer driven

This patch fixes two SSP pipelines to be scheduled in timer
domain.

Signed-off-by: Marcin Rajwa <[email protected]>