Skip to content

IPC4: Remove from handler-kernel.c function ipc4_get_pipeline_data()#10705

Open
singalsu wants to merge 1 commit intothesofproject:mainfrom
singalsu:xt_testbench_unused_build_fix
Open

IPC4: Remove from handler-kernel.c function ipc4_get_pipeline_data()#10705
singalsu wants to merge 1 commit intothesofproject:mainfrom
singalsu:xt_testbench_unused_build_fix

Conversation

@singalsu
Copy link
Copy Markdown
Collaborator

The function is unused and with xt-clang toolchain and warnings treated as errors this fails the xt-testbench4 build.

The function is unused and with xt-clang toolchain and warnings
treated as errors this fails the xt-testbench4 build.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu singalsu marked this pull request as ready for review April 15, 2026 14:19
Copilot AI review requested due to automatic review settings April 15, 2026 14:19
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes an unused helper from the IPC4 kernel handler to avoid -Wunused-function build failures when using the xt-clang toolchain with warnings treated as errors (notably in xt-testbench4 builds).

Changes:

  • Delete the unused static inline function ipc4_get_pipeline_data() from handler-kernel.c.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Collaborator

@lyakh lyakh left a comment

Choose a reason for hiding this comment

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

oh, that's a handler.c splitting artifact - we now have 2 copies of this function - one in handler-user.c and one in handler-kernel.c, and only the former is used...

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