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

app: remove explicit CONFIG_DW_ICTL_ACE_V1X #6400

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

nashif
Copy link
Contributor

@nashif nashif commented Oct 11, 2022

This config was renamed is automatically selected for this platform, so
no need to include it here.

Signed-off-by: Anas Nashif [email protected]

This config was renamed is automatically selected for this platform, so
no need to include it here.

Signed-off-by: Anas Nashif <[email protected]>
@marc-hb
Copy link
Collaborator

marc-hb commented Oct 12, 2022

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

D'oh, didn't notice this. Need to update #6396 again

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 12, 2022

Given I need to update the west.yml PR again (for the dmic issue), @softwarecki @abonislawski are you good to merge this and remove this Kconfig on SOF side. Seems ok as it's enabled on Zephyr side for the board. I did a test build to confirm this indeed is the case.

Copy link
Member

@abonislawski abonislawski left a comment

Choose a reason for hiding this comment

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

Yes, please remove

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