Skip to content

Commit

Permalink
tools/plugin: drop bogus -Wl,-EL binutils endianness option
Browse files Browse the repository at this point in the history
This mysterious option has been causing portability issues and has never
made any difference, see previous commits for details.

I compiled the plugin with and without it and there was absolutely zero
binary difference.

Let's remove it from all tools/plugin/ CMakeLists.txt files before the
mass copy/paste/diverge there spreads it even more.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb authored and kv2019i committed Aug 9, 2024
1 parent dc90583 commit 21e2028
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions tools/plugin/alsaplug/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ target_include_directories(asound_module_pcm_sof PRIVATE
${CMAKE_CURRENT_SOURCE_DIR}/..
${sof_source_directory}/src/audio)

target_compile_options(asound_module_pcm_sof PRIVATE -DPIC -g -O3 -Wl,-EL -Wmissing-prototypes
target_compile_options(asound_module_pcm_sof PRIVATE -DPIC -g -O3 -Wmissing-prototypes
-Wimplicit-fallthrough -DCONFIG_LIBRARY -imacros${config_h})

install(TARGETS asound_module_pcm_sof DESTINATION /usr/lib/x86_64-linux-gnu/alsa-lib)
Expand Down Expand Up @@ -47,7 +47,7 @@ target_include_directories(asound_module_ctl_sof PRIVATE
${CMAKE_CURRENT_SOURCE_DIR}/..
${sof_source_directory}/src/audio)

target_compile_options(asound_module_ctl_sof PRIVATE -DPIC -g -O3 -Wl,-EL -Wmissing-prototypes
target_compile_options(asound_module_ctl_sof PRIVATE -DPIC -g -O3 -Wmissing-prototypes
-Wimplicit-fallthrough -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})

install(TARGETS asound_module_ctl_sof DESTINATION /usr/lib/x86_64-linux-gnu/alsa-lib)
Expand Down
4 changes: 2 additions & 2 deletions tools/plugin/modules/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ target_include_directories(sof_mod_shm PRIVATE
${CMAKE_CURRENT_SOURCE_DIR}/../pipe
${sof_source_directory}/src/audio)

target_compile_options(sof_mod_shm PRIVATE -DPIC -g -O3 -Wl,-EL -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})
target_compile_options(sof_mod_shm PRIVATE -DPIC -g -O3 -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})

install(TARGETS sof_mod_shm
DESTINATION /usr/lib/x86_64-linux-gnu/alsa-lib)
Expand Down Expand Up @@ -39,7 +39,7 @@ target_include_directories(sof_mod_alsa PRIVATE
${CMAKE_CURRENT_SOURCE_DIR}/../pipe
${sof_source_directory}/src/audio)

target_compile_options(sof_mod_alsa PRIVATE -DPIC -g -O3 -Wl,-EL -Wmissing-prototypes
target_compile_options(sof_mod_alsa PRIVATE -DPIC -g -O3 -Wmissing-prototypes
-Wimplicit-fallthrough -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})

install(TARGETS sof_mod_alsa
Expand Down
2 changes: 1 addition & 1 deletion tools/plugin/modules/ov_noise_suppression/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ add_library(sof_ns MODULE noise_suppression.c)
target_link_libraries(sof_ns PRIVATE sof_ns_interface)
target_link_libraries(sof_ns PRIVATE -Wl,--export-dynamic)
sof_append_relative_path_definitions(sof_ns)
target_compile_options(sof_ns PRIVATE -DPIC -g -O3 -Wl,-EL -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})
target_compile_options(sof_ns PRIVATE -DPIC -g -O3 -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})
install(TARGETS sof_ns DESTINATION /usr/lib/x86_64-linux-gnu/alsa-lib)

target_include_directories(sof_ns PRIVATE
Expand Down
2 changes: 1 addition & 1 deletion tools/plugin/pipe/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ target_include_directories(sof-pipe PRIVATE
${CMAKE_CURRENT_SOURCE_DIR}/..
${sof_source_directory}/src/audio)

target_compile_options(sof-pipe PRIVATE -DPIC -g -O3 -Wl,-EL -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})
target_compile_options(sof-pipe PRIVATE -DPIC -g -O3 -Wall -Werror -DCONFIG_LIBRARY -imacros${config_h})

target_include_directories(sof-pipe PRIVATE ${sof_install_directory}/include)
target_include_directories(sof-pipe PRIVATE ${parser_install_dir}/include)
Expand Down

0 comments on commit 21e2028

Please sign in to comment.