From 7a36abba2b76f09cd7b2e2e417a49392c51443d7 Mon Sep 17 00:00:00 2001 From: Bartosz Kokoszko Date: Thu, 24 Sep 2020 14:29:42 +0200 Subject: [PATCH] pipeline: fix logs in pipeline_comp_params_neg() Use current->pipeline instead of ppl_data->p as a pipe_dbg() and pipe_err() arguments. ppl_data->p is not set at that point and can cause unexpected errors. Signed-off-by: Bartosz Kokoszko --- src/audio/pipeline.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/audio/pipeline.c b/src/audio/pipeline.c index da125ebde285..b3db7ae1531d 100644 --- a/src/audio/pipeline.c +++ b/src/audio/pipeline.c @@ -398,7 +398,7 @@ static int pipeline_comp_params_neg(struct comp_dev *current, uint32_t flags = 0; int err = 0; - pipe_dbg(ppl_data->p, "pipeline_comp_params_neg(), current->comp.id = %u, dir = %u", + pipe_dbg(current->pipeline, "pipeline_comp_params_neg(), current->comp.id = %u, dir = %u", dev_comp_id(current), dir); /* check if 'current' is already configured */ @@ -414,7 +414,7 @@ static int pipeline_comp_params_neg(struct comp_dev *current, * the param is conflict with an active pipeline, * drop an error and reject the .params() command. */ - pipe_err(ppl_data->p, "pipeline_comp_params_neg(): params conflict with existed active pipeline!"); + pipe_err(current->pipeline, "pipeline_comp_params_neg(): params conflict with existed active pipeline!"); return -EINVAL; }