Skip to content

Changes for pico2_w #1816

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.

Sign up for GitHub

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

Merged
kilograham merged 10 commits into from
Sep 18, 2024
+351 −85
Merged

Changes for pico2_w #1816

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
76d10e8
Add board file for pico2_w
peterharperuk Aug 12, 2024
1c107e1
Simplify CYW43 PIO config
peterharperuk Aug 14, 2024
d86c8a2
Make CYW43 gpio pins configurable at build time
peterharperuk Aug 14, 2024
32cbccf
Fix pio initialisation
peterharperuk Aug 15, 2024
f5d842c
Add missing doxygen for cyw43_set_pio_clock_divisor
peterharperuk Aug 15, 2024
76cb985
Support dynamic configuration of cyw43 gpio pins
peterharperuk Aug 15, 2024
d1a779c
Some fixes for cyw32 gpio > 32
peterharperuk Aug 16, 2024
fa774d1
Allow CYW43 to be configured with cmake
peterharperuk Aug 23, 2024
66bcde5
Add default config of CYW43_WL_GPIO_COUNT to cyw43_configport.h
peterharperuk Aug 23, 2024
9326733
Fix some review comments
peterharperuk Sep 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions src/rp2_common/pico_cyw43_driver/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,34 @@ if (EXISTS ${PICO_CYW43_DRIVER_PATH}/${CYW43_DRIVER_TEST_FILE})
hardware_exception
)

if (CYW43_DEFAULT_PIN_WL_REG_ON)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_REG_ON=${CYW43_DEFAULT_PIN_WL_REG_ON})
endif()
if (CYW43_DEFAULT_PIN_WL_DATA_OUT)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_DATA_OUT=${CYW43_DEFAULT_PIN_WL_DATA_OUT})
endif()
if (CYW43_DEFAULT_PIN_WL_DATA_IN)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_DATA_IN=${CYW43_DEFAULT_PIN_WL_DATA_IN})
endif()
if (CYW43_DEFAULT_PIN_WL_HOST_WAKE)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_HOST_WAKE=${CYW43_DEFAULT_PIN_WL_HOST_WAKE})
endif()
if (CYW43_DEFAULT_PIN_WL_CLOCK)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_CLOCK=${CYW43_DEFAULT_PIN_WL_CLOCK})
endif()
if (CYW43_DEFAULT_PIN_WL_CS)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_DEFAULT_PIN_WL_CS=${CYW43_DEFAULT_PIN_WL_CS})
endif()
if (CYW43_PIO_CLOCK_DIV_INT)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_PIO_CLOCK_DIV_INT=${CYW43_PIO_CLOCK_DIV_INT})
endif()
if (CYW43_PIO_CLOCK_DIV_FRAC)
Copy link
Contributor

@kilograham kilograham Sep 16, 2024

Choose a reason for hiding this comment

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

this should probably be _FRAC8 now, or probably we should add _FRAC8 too

Copy link
Contributor

@kilograham kilograham Sep 16, 2024

Choose a reason for hiding this comment

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

actually let's leave this for now; i'll fix with #1926

target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_PIO_CLOCK_DIV_FRAC=${CYW43_PIO_CLOCK_DIV_FRAC})
endif()
if (CYW43_PIO_CLOCK_DIV_DYNAMIC)
target_compile_definitions(cyw43_driver_picow INTERFACE CYW43_PIO_CLOCK_DIV_DYNAMIC=${CYW43_PIO_CLOCK_DIV_DYNAMIC})
endif()

# Note: This is used by MP, so check for issues when making changes
# e.g. Don't add new depenedences
pico_add_library(pico_btstack_hci_transport_cyw43 NOFLAG)
Expand Down