[BSP] 添加raspberry-rp2350适配

[hydevcode]

拉取/合并请求描述：(PR description)

[

为什么提交这份PR (why to submit this PR)

添加raspberry-rp2350适配

你的解决方案是什么 (what is your solution)

软件包索引仓库pr：
RT-Thread/packages#1917
软件包仓库pr:
RT-Thread-packages/raspberrypi-pico-rp2350-sdk#1

请提供验证的bsp和config (provide the config and bsp)

• BSP:raspberry-rp2350

• .config:

• action:

]

当前拉取/合并请求的状态 Intent for your PR

必须选择一项 Choose one (Mandatory):

• 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
• 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo

代码质量 Code Quality：

我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:

• 已经仔细查看过代码改动的对比 Already check the difference between PR and old code
• 代码风格正确，包括缩进空格，命名及其他风格 Style guide is adhered to, including spacing, naming and other styles
• 没有垃圾代码，代码尽量精简，不包含#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up
• 所有变更均有原因及合理的，并且不会影响到其他软件组件代码或BSP All modifications are justified and not affect other components or BSP
• 对难懂代码均提供对应的注释 I've commented appropriately where code is tricky
• 代码是高质量的 Code in this PR is of high quality
• 已经使用formatting 等源码格式化工具确保格式符合RT-Thread代码规范 This PR complies with RT-Thread code specification
• 如果是新增bsp, 已经添加ci检查到.github/workflows/bsp_buildings.yml 详细请参考链接BSP自查

[github-actions]

📌 Code Review Assignment

🏷️ Tag: workflow

Reviewers: Rbb666 kurisaW supperthomas

Changed Files (Click to expand)

• .github/ALL_BSP_COMPILE.json

---

📊 Current Review Status (Last Updated: 2025-08-04 21:25 CST)

• ⌛ Rbb666 Pending Review
• ⌛ kurisaW Pending Review
• ⌛ supperthomas Pending Review

---

📝 Review Instructions

1. 维护者可以通过单击此处来刷新审查状态: 🔄 刷新状态
   Maintainers can refresh the review status by clicking here: 🔄 Refresh Status

2. 确认审核通过后评论 LGTM/lgtm
   Comment LGTM/lgtm after confirming approval

3. PR合并前需至少一位维护者确认
   PR must be confirmed by at least one maintainer before merging

ℹ️ 刷新CI状态操作需要具备仓库写入权限。
ℹ️ Refresh CI status operation requires repository Write permission.

[Rbb666]

这些外设都有验证过吗?如果没有建议先删掉，保留已验证过的即可

[Rbb666]

LVGL这部分如果没验证过建议删除

[Rbb666]

ardunio也是

[Rbb666]

多余注释请删除

[Rbb666]

-WL部分参数可以删除的，没必要添加这么多，是自定义的函数实现了吗？

[Copilot]

Pull Request Overview

This PR adds adaptation support for the Raspberry Pi RP2350 microcontroller by creating a new BSP (Board Support Package). The RP2350 is a newer dual-core ARM Cortex-M33 microcontroller from Raspberry Pi with enhanced security features.

• New BSP implementation for raspberry-rp2350 based on existing raspberry-pico BSP
• Driver support for common peripherals including UART, GPIO, SPI, I2C, PWM, ADC, WDT, and RTC
• LVGL support for LCD display functionality
• Arduino compatibility layer for easier migration

Reviewed Changes

Copilot reviewed 56 out of 69 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
bsp/raspberry-rp2350/rtconfig.py	Build configuration script for ARM Cortex-M33 target
bsp/raspberry-rp2350/rtconfig.h	RT-Thread kernel and component configuration
bsp/raspberry-rp2350/link.ld	Linker script defining memory layout for RP2350
bsp/raspberry-rp2350/drivers/*	Hardware abstraction layer drivers for peripherals
bsp/raspberry-rp2350/board/*	Board initialization and configuration
bsp/raspberry-rp2350/applications/*	Sample applications and Arduino pinout definitions

Comments suppressed due to low confidence (1)

bsp/raspberry-rp2350/drivers/drv_soft_spi.h:46

• The macro name 'BSP_S_SPI_MOSI_PIN' should be 'BSP_S_SPI1_MOSI_PIN' to match the pattern used in other SPI1 pin definitions.

        .mosi = BSP_S_SPI_MOSI_PIN,                        \

[Copilot]

This extremely long line (over 5000 characters) should be refactored or removed. It appears to be commented CMake command output that doesn't belong in a configuration file and makes the code difficult to read and maintain.

[Copilot]

The hardcoded absolute path '/home/qemu/picotool/build/picotool' should be made configurable or use relative paths to avoid environment-specific dependencies.

Suggested change

	# elf2uf2_path = os.path.join(os.getcwd(), "tools", "elf2uf2")
	elf2uf2_path="/home/qemu/picotool/build/picotool"
	elf2uf2_path = os.getenv("ELF2UF2_PATH", os.path.join(os.getcwd(), "tools", "elf2uf2"))

[Copilot]

The function always returns -RT_ERROR regardless of successful I2C transfers. It should return the number of messages processed on success or a negative error code on failure.

Suggested change

	rt_err_t ret = -RT_ERROR;
	struct pico_i2c_bus *pico_i2c = (struct pico_i2c_bus *)bus;
	for (i = 0; i < num; i++)
	{
	msg = &msgs[i];
	if (msg->flags & RT_I2C_RD)
	{
	i2c_read_blocking(pico_i2c->handle, msg->addr, msg->buf, msg->len, false);
	}
	else
	{
	i2c_write_blocking(pico_i2c->handle, msg->addr, msg->buf, msg->len, false);
	}
	}
	return ret;
	struct pico_i2c_bus *pico_i2c = (struct pico_i2c_bus *)bus;
	for (i = 0; i < num; i++)
	{
	msg = &msgs[i];
	int res;
	if (msg->flags & RT_I2C_RD)
	{
	res = i2c_read_blocking(pico_i2c->handle, msg->addr, msg->buf, msg->len, false);
	if (res != msg->len)
	{
	return -RT_ERROR;
	}
	}
	else
	{
	res = i2c_write_blocking(pico_i2c->handle, msg->addr, msg->buf, msg->len, false);
	if (res != msg->len)
	{
	return -RT_ERROR;
	}
	}
	}
	return num;

[Copilot]

The erase function checks for size alignment but doesn't return an error value when the condition fails, continuing with the potentially invalid erase operation.

Suggested change

	LOG_E("erase size must be %d-byte alignment", FLASH_SECTOR_SIZE);
	LOG_E("erase size must be %d-byte alignment", FLASH_SECTOR_SIZE);
	return -RT_EINVAL;