Skip to content

Commit

Permalink
modules: Kconfig.simplelink: Remove POSIX_API, use fine-grained opts
Browse files Browse the repository at this point in the history
PR zephyrproject-rtos#18780 introduces a way to decouple pthread support from the general
CONFIG_POSIX_API global switch. This commit modifies the build of
SimpleLink components to take advantage of it, since SimpleLink
libraries only require pthread, sem, clock, and sleep support, not
entire POSIX API.

This fixes the build errors in the http_get sample introduced
by the merge of zephyrproject-rtos#18736. As such, this patch also removes
cc3220sf_launchxl exclude from sample.yaml of that sample.

Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
Signed-off-by: Paul Sokolovsky <paul.sokolovsky@linaro.org>
  • Loading branch information
vanti authored and pfalcon committed Sep 3, 2019
1 parent f882b02 commit 73049db
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion modules/Kconfig.simplelink
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ config SIMPLELINK_HOST_DRIVER
depends on MULTITHREADING
select NEWLIB_LIBC
select ERRNO
select POSIX_API
select PTHREAD_IPC
select POSIX_CLOCK
help
Build the SimpleLink host driver

Expand Down
1 change: 0 additions & 1 deletion samples/net/sockets/http_get/sample.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ tests:
- CONFIG_NET_SOCKETS_POSIX_NAMES=y
# Forcibly defines CONFIG_POSIX_API, which is incompatible with
# CONFIG_NET_SOCKETS_POSIX_NAMES.
platform_exclude: cc3220sf_launchxl
sample.net.sockets.http_get.posix:
filter: and not CONFIG_NET_SOCKETS_OFFLOAD
extra_configs:
Expand Down

0 comments on commit 73049db

Please sign in to comment.