Skip to content

Commit

Permalink
xtensa: do not imply atomic ops kconfig
Browse files Browse the repository at this point in the history
Xtensa's arch level atomic implementation requries S32C1I
support which may not exist as Xtensa cores are highly
configurable. Implying CONFIG_ATOMIC_OPERATIONS_ARCH at
arch level has a side effect that it is enabled
regardless if the other options are enabled. Given how
the header file's #ifdef is structured, only C atomic
ops can override this, which means enabling
CONFIG_ATOMIC_OPERATIONS_BUILTIN has no effect at all.
So let the SoC or board decide which atomic ops are
being used instead.

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
  • Loading branch information
dcpleung authored and nashif committed Jan 18, 2024
1 parent 8bcec18 commit 3b99fb1
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion arch/Kconfig
Expand Up @@ -126,7 +126,6 @@ config XTENSA
select ARCH_HAS_CODE_DATA_RELOCATION
select ARCH_HAS_TIMING_FUNCTIONS
select ARCH_MEM_DOMAIN_DATA if USERSPACE
imply ATOMIC_OPERATIONS_ARCH
help
Xtensa architecture

Expand Down

0 comments on commit 3b99fb1

Please sign in to comment.