clk: fix unused variable

This patch removes unused variables form platform_clock_on_waiti()

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

kpb: check pm_is_active and dont enable it at the end of draining

This patch will prevent from enabling pm_runtime dsp at the end of draining
to avoid enabling it when already received disable ipc during draining

Signed-off-by: Adrian Bonislawski <[email protected]>

cavs: clk: return to HPRO on wakeup

This will allow to recover hpro instead of wovcro on wakeup

Signed-off-by: Adrian Bonislawski <[email protected]>

cavs: clk: use WOVCRO for D0I3 only.

Change to use WOVCRO for D0I3 only, for WFI, use LPRO instead.

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

zephyr: remove infinite recursion on Broadwell

Broadwell support with Zephyr had only been dry-coded, it has never
been tested, and it contains an infinite recursion bug: functions
platform_timer_get() and arch_timer_get_system() call each other
infinitely. Break the recursion formally, until a proper fix is
available.

Signed-off-by: Guennadi Liakhovetski <[email protected]>

demux: add demux_prepare_active_look_up() function

This commit adds demux_prepare_active_look_up()
function. It contains channel parameter checking.
Look up table should not include elements referring
to channels higher than those actually set in
source/sink buffers.

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

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