-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[BSP]适配珠海泰为tae32g5800的部分外设(GPIO/UART/CAN/FDCAN/IWDG/FAL) #10951
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
base: master
Are you sure you want to change the base?
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
|
根据bsp瘦身计划 您的提交可能不会被合并 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request adds Board Support Package (BSP) support for the Zhuhai Taiwei TAE32G5800 microcontroller, implementing peripheral drivers for GPIO, UART, CAN, FDCAN, IWDG, and FAL (Flash Abstraction Layer).
Key Changes
- New BSP directory structure for TAE32G5800 evaluation board
- Firmware library including TAE32G58xx device drivers and CMSIS core files
- Peripheral driver implementations for GPIO, UART, CAN/FDCAN, and IWDG
- Build configuration files (SConscript, Kconfig, linker scripts for GCC and ARM)
- OpenOCD configuration files and startup code for the TAE32G58xx series
Reviewed Changes
Copilot reviewed 39 out of 194 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/TAE32G58xx_Driver/Inc/*.h | ADC and general low-level driver header files |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/TAE32G58xx_Device/Src/*.c | System initialization and startup code |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/TAE32G58xx_Device/Src/GCC/* | GCC linker scripts and OpenOCD configuration |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/TAE32G58xx_Device/Src/ARM/* | ARM compiler scatter files |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/TAE32G58xx_Device/Inc/*.h | Device and system header files |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/SConscript | SCons build script for library compilation |
| bsp/tae32/libraries/TAE32G58xx_Firmware_Library/CMSIS/Include/*.h | CMSIS core support headers |
| bsp/tae32/libraries/Kconfig | Kconfig configuration for TAE32 SOC family |
| bsp/tae32/libraries/.ignore_format.yml | Format check exclusion configuration |
| #define __LL_ADC_CalMode_Get(__ADC__) READ_BIT_SHIFT((__ADC__)->CR, ADC0_CR_ADCALDIF_Msk, ADC0_CR_ADCALDIF_Pos) | ||
|
|
||
| /** | ||
| * @brief Calibration Start |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: Trailing whitespace after "Start" on line 85. Remove the extra space at the end of the comment.
中文:第 85 行 "Start" 后面有多余的空格。请删除注释末尾的多余空格。
| * @brief Calibration Start | |
| * @brief Calibration Start |
| #define __LL_ADC_Cal_Start(__ADC__) SET_BIT((__ADC__)->CR, ADC0_CR_ADCAL_Msk) | ||
|
|
||
| /** | ||
| * @brief Judge is Calibration Starting or not |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: Trailing whitespace after "not" on line 92. Remove the extra space at the end of the comment.
中文:第 92 行 "not" 后面有多余的空格。请删除注释末尾的多余空格。
| * @brief Judge is Calibration Starting or not | |
| * @brief Judge is Calibration Starting or not |
| /* | ||
| * SG veneers: | ||
| * All SG veneers are placed in the special output section .gnu.sgstubs. Its start address | ||
| * must be set, either with the command line option �--section-start� or in a linker script, |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: Line 137 contains an invalid character '�' (likely a corrupted em-dash). Replace with proper ASCII characters like "--section-start".
中文:第 137 行包含无效字符 '�'(可能是损坏的破折号)。请替换为正确的 ASCII 字符,如 "--section-start"。
| * must be set, either with the command line option �--section-start� or in a linker script, | |
| * must be set, either with the command line option --section-start or in a linker script, |
| /* | ||
| * SG veneers: | ||
| * All SG veneers are placed in the special output section .gnu.sgstubs. Its start address | ||
| * must be set, either with the command line option �--section-start� or in a linker script, |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: Line 137 contains an invalid character '�' (likely a corrupted em-dash). Replace with proper ASCII characters like "--section-start".
中文:第 137 行包含无效字符 '�'(可能是损坏的破折号)。请替换为正确的 ASCII 字符,如 "--section-start"。
| * must be set, either with the command line option �--section-start� or in a linker script, | |
| * must be set, either with the command line option --section-start or in a linker script, |
| * Secondary data section, optional | ||
| * | ||
| * Remember to add each additional data section | ||
| * to the .copy.table above to asure proper |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: "asure" should be spelled "assure" on line 243.
中文:第 243 行 "asure" 应拼写为 "assure"。
| * Secondary bss section, optional | ||
| * | ||
| * Remember to add each additional bss section | ||
| * to the .zero.table above to asure proper |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: "asure" should be spelled "assure" on line 278.
中文:第 278 行 "asure" 应拼写为 "assure"。
| * to the .zero.table above to asure proper | |
| * to the .zero.table above to assure proper |
| * Secondary data section, optional | ||
| * | ||
| * Remember to add each additional data section | ||
| * to the .copy.table above to asure proper |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: "asure" should be spelled "assure" on line 243.
中文:第 243 行 "asure" 应拼写为 "assure"。
| * Secondary bss section, optional | ||
| * | ||
| * Remember to add each additional bss section | ||
| * to the .zero.table above to asure proper |
Copilot
AI
Nov 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
English: "asure" should be spelled "assure" on line 278.
中文:第 278 行 "asure" 应拼写为 "assure"。
| * to the .zero.table above to asure proper | |
| * to the .zero.table above to assure proper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
关于芯片的BSP驱动库已经不可以加入到master了,还请使用软件包形式提交,谢谢
请参考:https://club.rt-thread.org/ask/article/4839095b91c2f849.html
|
@yeminlong 作者有联系方式吗?我们可以加个好友看看怎么推动下 |
|
@Rbb666 已通过B站的站内消息发送了联系方式,感谢支持! |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
新增BSP--适配珠海泰为tae32g5800的部分外设(GPIO/UART/CAN/FDCAN/IWDG/FAL)
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up