Skip to content

Skip triggering BE pipelines in FE trigger - #3914

Closed
ranj063 wants to merge 1 commit into
thesofproject:topic/sof-devfrom
ranj063:fix/be_pipeline
Closed

Skip triggering BE pipelines in FE trigger#3914
ranj063 wants to merge 1 commit into
thesofproject:topic/sof-devfrom
ranj063:fix/be_pipeline

Conversation

@ranj063

@ranj063ranj063 commented Oct 10, 2022•
edited
Loading

Copy link
Copy Markdown
Collaborator

No description provided.

@ranj063ranj063 mentioned this pull request Oct 10, 2022
Closed
plbossart
plbossart requested changes Oct 10, 2022
View reviewed changes
Comment threadinclude/uapi/sound/sof/tokens.h
Comment threadinclude/uapi/sound/sof/tokens.h Outdated
Comment threadsound/soc/sof/ipc4-pcm.c
@ranj063
ASoC: SOF: ipc4-pcm: skip backend pipelines during FE trigger
2d2441a
Skip triggering backend pipelines during FE trigger. This patch is in
preparation to add support for triggering intermediate pipelines that
are between the FE and BE pipelines in topology.
Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
@ranj063

ranj063 commented Oct 10, 2022

Copy link
Copy Markdown
CollaboratorAuthor

@plbossart I've updated this PR to use the information during topology parsing to set a backend pipeline. No need for a new token anymore like you suggested

plbossart
plbossart approved these changes Oct 10, 2022
View reviewed changes

@plbossartplbossart 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.

Sounds good, thanks @ranj063

@libinyang

libinyang commented Oct 11, 2022

Copy link
Copy Markdown

LGTM

@ranj063ranj063 mentioned this pull request Oct 11, 2022
Merged
@ranj063

ranj063 commented Oct 12, 2022

Copy link
Copy Markdown
CollaboratorAuthor

closing this one #3917 covers it as well

@ranj063ranj063 closed this Oct 12, 2022
@ranj063
ranj063 deleted the fix/be_pipeline branch October 12, 2022 16:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@plbossartplbossartplbossart approved these changes
@bardliaobardliaobardliao approved these changes
@lyakhlyakhAwaiting requested review from lyakh
@juimonenjuimonenAwaiting requested review from juimonen
@lgirdwoodlgirdwoodAwaiting requested review from lgirdwoodlgirdwood is a code owner
@kv2019ikv2019iAwaiting requested review from kv2019ikv2019i is a code owner
@dbalutadbalutaAwaiting requested review from dbalutadbaluta is a code owner
@ujfalusiujfalusiAwaiting requested review from ujfalusiujfalusi is a code owner
@ajitkupandeyajitkupandeyAwaiting requested review from ajitkupandey
@kuanhsunchengkuanhsunchengAwaiting requested review from kuanhsuncheng
@yaochunhungyaochunhungAwaiting requested review from yaochunhung
@libinyanglibinyangAwaiting requested review from libinyang

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ranj063@libinyang@plbossart@bardliao