Skip to content

Conversation

@nashif
Copy link
Contributor

@nashif nashif commented Oct 22, 2025

Adapt code to work with SOF not being a module in Zephyr.

Signed-off-by: Anas Nashif anas.nashif@intel.com

Adapt code to work with SOF not being a module in Zephyr.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

config CORE_COUNT
int
default 1
Copy link
Collaborator

@kv2019i kv2019i Oct 22, 2025

Choose a reason for hiding this comment

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

This is bit of a works-by-accident type of thing. This does work (I had to check the build logs), but normal Zephyr builds should not even use this file as this is supposed to be used by host (=x86) builds like testbench and ALSA SOF plugin only. This is ok as transitory solution, but I'll look for a more sane place to set the core count....

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is already another place where this is set src/platform/Kconfig

config CORE_COUNT
        int "Number of cores"
        default MP_MAX_NUM_CPUS if KERNEL_BIN_NAME = "zephyr"
        default MAX_CORE_COUNT
        range 1 MAX_CORE_COUNT
        help
          Number of used cores
          Lowering available core count could result in lower power consumption

Copy link
Contributor Author

Choose a reason for hiding this comment

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

but normal Zephyr builds should not even use this file as this is supposed to be used by host (=x86) builds like testbench and ALSA SOF plugin only.

aha, now I remember the issue, that was a weird one, without ARCH being defined after the migration to hwmv2, this starts failing and I guess those depend on ARCH

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can we actually just get rid of src/arch/host/Kconfig and just rely on what we have "src/platform/Kconfig"?

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.

2 participants