Skip to content

pipeline: fix logs in pipeline_comp_params_neg() - #3464

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
bkokoszx:fix-pipeline_comp_params_neg_logs
Sep 25, 2020
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
bkokoszx:fix-pipeline_comp_params_neg_logs

Conversation

@bkokoszx

@bkokoszx bkokoszx commented Sep 24, 2020 •

Copy link
Copy Markdown
Collaborator

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 bartoszx.kokoszko@linux.intel.com

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 <bartoszx.kokoszko@linux.intel.com>
@bkokoszx
bkokoszx force-pushed the fix-pipeline_comp_params_neg_logs branch from 9673cf2 to 7a36abb Compare September 24, 2020 13:31

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bkokoszx good catch, I did notice some confusing IDs on the logs....

@keyonjie keyonjie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Bartosz for spotting this.

@keyonjie

Copy link
Copy Markdown
Contributor

@lgirdwood worth to get this to 1.6 also.

@lgirdwood

Copy link
Copy Markdown
Member

@keyonjie too late for rc1, but yet for v1.6 final.

@lgirdwood
lgirdwood merged commit 5b8b543 into thesofproject:master Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants