Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

system-agent: remove an unused variable #8188

Merged
merged 1 commit into from
Sep 18, 2023
Merged

Conversation

lyakh
Copy link
Collaborator

@lyakh lyakh commented Sep 12, 2023

sys_service_p in
src/audio/module_adapter/library/native_system_agent.c is unused, remove it.

sys_service_p in
src/audio/module_adapter/library/native_system_agent.c is unused,
remove it.

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

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

@marcinszkudlinski used in the future ?

@pjdobrowolski
Copy link
Contributor

That pointer will be used in future for passing system service which provides layer of communication between FW and loadable modules. We can remove that lines but they will come back.

@lyakh
Copy link
Collaborator Author

lyakh commented Sep 15, 2023

That pointer will be used in future for passing system service which provides layer of communication between FW and loadable modules. We can remove that lines but they will come back.

@pjdobrowolski I think it would be better to remove them now and see if and when we really need them.

@kv2019i kv2019i merged commit 0675b2a into thesofproject:main Sep 18, 2023
37 of 41 checks passed
@lyakh lyakh deleted the sys-agent branch September 19, 2023 09:31
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.

5 participants