Skip to content
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.

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

Add Blues Cygnet board #822

Merged
merged 4 commits into from
Dec 18, 2024
Merged

Add Blues Cygnet board #822

merged 4 commits into from
Dec 18, 2024

Conversation

maxgerhardt
Copy link
Contributor

@maxgerhardt maxgerhardt commented Dec 10, 2024

Board see here, STM32Duino variant here. Closes #821

Tested by Blues employees.

  • we already have bw_swan_r5.json, "BluesWireless" seems to be the old name though? Now it's just "Blues"? (I named this file blues_cygnet.json)
    • added copy of Blues Swan as blues_swan_r5.json
  • I activate the USB CDC by default because this board has no onboard USB-to-UART converter and can be flashed via USB by default, so users will expect Serial to also go via USB
  • no on-board debugger probe: set DFU as default upload method for the same reason: Works by default with just USB cable
  • No HWIDs, the code in main.py correctly selects 0x0483:0xDF11 (STM32 DFU) (Swan R5 also didn't set it)

@zfields
Copy link

zfields commented Dec 10, 2024

In stm32duino, we allowed for both bw_swan_r5 as blues_swan_r5; see here. Does this mean, we also need a blues_swan_r5.json?

If so, would you mind to create a copy of bw_swan_r5.json that changes "Blues Wireless" references to "Blues" and add it to this PR?

I think it should look like this:

{
  "build": {
    "core": "stm32",
    "cpu": "cortex-m4",
    "extra_flags": "-DSTM32L4 -DSTM32L4xx -DSTM32L4R5xx",
    "f_cpu": "120000000L",
    "framework_extra_flags": {
      "arduino": "-DCUSTOM_PERIPHERAL_PINS"
    },
    "mcu": "stm32l4r5ziy6",
    "product_line": "STM32L4R5xx",
    "variant": "STM32L4xx/L4R5Z(G-I)Y_L4R9Z(G-I)Y_L4S5ZIY_L4S9ZIY"
  },
  "connectivity": [
    "can"
  ],
  "debug": {
    "jlink_device": "STM32L4R5ZI",
    "openocd_target": "stm32l4x",
    "svd_path": "STM32L4R5.svd"
  },
  "frameworks": [
    "arduino",
    "stm32cube"
  ],
  "name": "Swan R5",
  "upload": {
    "maximum_ram_size": 655360,
    "maximum_size": 2097152,
    "protocol": "stlink",
    "protocols": [
      "dfu",
      "blackmagic",
      "cmsis-dap",
      "jlink",
      "serial",
      "stlink",
      "mbed"
    ]
  },
  "url": "https://blues.io/products/swan",
  "vendor": "Blues"
}

@zfields
Copy link

zfields commented Dec 10, 2024

feat: Blues Swan R5 (alias)
@zfields
Copy link

zfields commented Dec 12, 2024

Thank you for helping us with this work!

When do you expect it will be merged and released? We have a lot of customers who love PlatformIO, and we would like to let them know when it will support our new board.

@maxgerhardt
Copy link
Contributor Author

I dunno. But for sure only after an employee reviewed it. CC @valeros

- Updated board names with vendor
- Updated Swan R5 Arduino settings
@valeros valeros merged commit 8651b16 into platformio:develop Dec 18, 2024
110 of 111 checks passed
@valeros
Copy link
Member

valeros commented Dec 18, 2024

Thanks, added!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Blues Cygnet (STM32L433)
3 participants