Skip to content

Conversation

@jsbatch
Copy link
Contributor

@jsbatch jsbatch commented Nov 8, 2025

  • Adds SOC and DTS files to support CYW20829 B1 revision parts.
  • Updates the CYW920829M2EVK_02 evaluation board to support multiple SOC configurations.
  • Updates tests and samples to support multiple soc configurations for CYW920829M2EVK_02 board.
  • Updates bluetooth to load the correct bluetooth firmware blob for B0 or B1 part revision.

Renames mpn device tree files to lowercase for consistency with other
devices.
Removes incorrect mpn and package devicetree files.

Signed-off-by: John Batch <[email protected]>
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_infineon zephyrproject-rtos/hal_infineon@f3c571f (master) zephyrproject-rtos/hal_infineon#35 zephyrproject-rtos/hal_infineon#35/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.


# SOC die
config SOC_DIE_CYW20829
config SOC_DIE_20829
Copy link
Contributor

Choose a reason for hiding this comment

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

these symbols have no place in sysbuild, use the correct SOC_X or SOC_SERIES_X Kconfigs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is currently being done consistently across Infineon devices. I have reworked this file to remove the SOC_DIE_* for this device, but changes will need to be made to other Infineon devices to remove the SOC_DIE_* references. This is outside the scope of this PR.

Comment on lines 21 to 25
config SOC_REV_B0
bool

config SOC_REV_B1
bool
Copy link
Contributor

Choose a reason for hiding this comment

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

not acceptable at all

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've removed this.

Comment on lines 37 to 44
config SOC_PACKAGE_CYW20829_48_QFN
bool

config SOC_PACKAGE_CYW20829_64_BGA
bool

config SOC_PACKAGE_CYW20829_56_QFN_MT
bool
Copy link
Contributor

Choose a reason for hiding this comment

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

no relevance for sysbuild

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is currently being done consistently across Infineon devices. I have reworked this file to remove the SOC_DIE_* for this device, but changes will need to be made to other Infineon devices to remove the SOC_DIE_* references. This is outside the scope of this PR.

default CYW20829B0_BT_FW_PAWR_TX0 if (BT_PER_ADV_RSP || BT_PER_ADV_SYNC_RSP) && !CYW20829_BT_FW_TX10DBM_POWER && SOC_REV_B0
default CYW20829B0_BT_FW_TX10 if CYW20829_BT_FW_TX10DBM_POWER && SOC_REV_B0
default CYW20829B0_BT_FW_TX0 if SOC_REV_B0
default CYW20829B1_BT_FW_TX10 if CYW20829_BT_FW_TX10DBM_POWER && SOC_REV_B1
Copy link
Contributor

Choose a reason for hiding this comment

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

no extra spaces before the if, just 1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've corrected the formatting.

if(CONFIG_CYW20829_BT_FW_TX0)
# CYW20829B0 device (FW patch for 0dBm TX Power)
if(CONFIG_CYW20829B0_BT_FW_TX0)
set(blob_hcd_file ${hal_blobs_dir}/COMPONENT_CYW20829B0/COMPONENT_BTFW-TX0/bt_firmware.hcd)
Copy link
Contributor

Choose a reason for hiding this comment

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

these should be fixed to have one space after blob_hcd_file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've corrected the formatting in this file.

Adding new MPN files for B1 part revision.
Updating existing parts to be consistent with changes made for other new
devices (PSOC Edge, PSOC Control C3).

Signed-off-by: John Batch <[email protected]>
Adding B1 revision devices to CYW20829 family.
Clean up unsupported devices from SOC files.

Signed-off-by: John Batch <[email protected]>
Updates Bluetooth driver to include bluetooth firmware for B1 device
revision.

Signed-off-by: John Batch <[email protected]>
Reorganizing board hardware definition in the device tree to better
reflect hardware design of separate processor board and base board.

Signed-off-by: John Batch <[email protected]>
Adds the following processor variants and associated cards to the
cyw920829m2evk development kit:
- cyw20829b1010
- cyw20829b1340

Signed-off-by: John Batch <[email protected]>
Updates all supported samples for the cyw920829m2evk_02 board to support
multiple SOC options.

Signed-off-by: John Batch <[email protected]>
Updating supported tests for the cyw920829m2evk_02 board to support
multiple socs.

Signed-off-by: John Batch <[email protected]>
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_infineon platform: Infineon Infineon Technologies AG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants