Skip to content

[bsp] add n32 pwm driver #11236

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

Open
chejia12 wants to merge 1 commit into
base: master
Choose a base branch
from
+2,398 −2
Open

[bsp] add n32 pwm driver #11236

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32G43x_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ if GetDepend(['RT_USING_DAC']):

if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g43x_std_periph_driver/src/n32g43x_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32g43x_std_periph_driver/src/n32g43x_tim.c']

Comment on lines 36 to 41
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32g43x_tim.c

English: n32g43x_tim.c is appended in both the RT_USING_CLOCK_TIME and RT_USING_PWM conditions. When both are enabled, the same source may be compiled twice and break the build with duplicate symbols. Ensure it is included only once.
中文：n32g43x_tim.c 同时在 RT_USING_CLOCK_TIME 与 RT_USING_PWM 条件下加入；两者同时开启时可能导致重复编译并引发重复符号的构建失败。请确保只加入一次。

Suggested change
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g43x_std_periph_driver/src/n32g43x_tim.c']
if GetDepend(['RT_USING_PWM']):
src += ['n32g43x_std_periph_driver/src/n32g43x_tim.c']
if GetDepend(['RT_USING_CLOCK_TIME']) or GetDepend(['RT_USING_PWM']):
src += ['n32g43x_std_periph_driver/src/n32g43x_tim.c']

Copilot uses AI. Check for mistakes.
if GetDepend(['RT_USING_RTC']):
src += ['n32g43x_std_periph_driver/src/n32g43x_rtc.c']
Expand Down
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32G45x_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,9 @@ if GetDepend(['RT_USING_DAC']):
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g45x_std_periph_driver/src/n32g45x_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32g45x_std_periph_driver/src/n32g45x_tim.c']

Comment on lines 41 to +46
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32g45x_tim.c

English: n32g45x_tim.c is added in both RT_USING_CLOCK_TIME and RT_USING_PWM blocks. If both options are enabled, this can compile the same file twice and cause duplicate-symbol build failures. Add it only once.
中文：n32g45x_tim.c 同时在 RT_USING_CLOCK_TIME 与 RT_USING_PWM 条件块中被加入；两者同时开启可能导致同一文件重复编译并产生重复符号错误。应确保只加入一次。

Suggested change
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g45x_std_periph_driver/src/n32g45x_tim.c']
if GetDepend(['RT_USING_PWM']):
src += ['n32g45x_std_periph_driver/src/n32g45x_tim.c']
if GetDepend(['RT_USING_CLOCK_TIME']) or GetDepend(['RT_USING_PWM']):
src += ['n32g45x_std_periph_driver/src/n32g45x_tim.c']

Copilot uses AI. Check for mistakes.
if GetDepend(['RT_USING_RTC']):
src += ['n32g45x_std_periph_driver/src/n32g45x_rtc.c']
src += ['n32g45x_std_periph_driver/src/n32g45x_pwr.c']
Expand Down
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32G4FR_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ if GetDepend(['RT_USING_DAC']):
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_tim.c']

Comment on lines 35 to +40
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32g4fr_tim.c

English: n32g4fr_tim.c is appended under both RT_USING_CLOCK_TIME and RT_USING_PWM. With both enabled, the same C file may be built twice causing duplicate symbols. Ensure it is only added once.
中文：n32g4fr_tim.c 同时在 RT_USING_CLOCK_TIME 和 RT_USING_PWM 下加入；两者同时开启时可能重复编译导致重复符号。请保证只加入一次。

Suggested change
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_tim.c']
if GetDepend(['RT_USING_PWM']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_tim.c']
if GetDepend(['RT_USING_CLOCK_TIME']) or GetDepend(['RT_USING_PWM']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_tim.c']

Copilot uses AI. Check for mistakes.
if GetDepend(['RT_USING_RTC']):
src += ['n32g4fr_std_periph_driver/src/n32g4fr_rtc.c']

Expand Down
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32L40x_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ if GetDepend(['RT_USING_DAC']):
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32l40x_std_periph_driver/src/n32l40x_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32l40x_std_periph_driver/src/n32l40x_tim.c']

Comment on lines 36 to +41
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32l40x_tim.c

English: n32l40x_tim.c is added for both RT_USING_CLOCK_TIME and RT_USING_PWM. If both are enabled, this can compile the same file twice and trigger duplicate-symbol errors. Add it only once.
中文：n32l40x_tim.c 同时在 RT_USING_CLOCK_TIME 与 RT_USING_PWM 条件下被加入；两者同时开启可能导致源文件重复编译并触发重复符号错误。应确保只加入一次。

Suggested change
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32l40x_std_periph_driver/src/n32l40x_tim.c']
if GetDepend(['RT_USING_PWM']):
src += ['n32l40x_std_periph_driver/src/n32l40x_tim.c']
if GetDepend(['RT_USING_CLOCK_TIME']) or GetDepend(['RT_USING_PWM']):
src += ['n32l40x_std_periph_driver/src/n32l40x_tim.c']

Copilot uses AI. Check for mistakes.
if GetDepend(['RT_USING_RTC']):
src += ['n32l40x_std_periph_driver/src/n32l40x_rtc.c']
src += ['n32l40x_std_periph_driver/src/n32l40x_pwr.c']
Expand Down
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32L43x_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ if GetDepend(['RT_USING_DAC']):
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32l43x_std_periph_driver/src/n32l43x_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32l43x_std_periph_driver/src/n32l43x_tim.c']
Comment on lines 36 to +40
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32l43x_tim.c

English: n32l43x_tim.c is appended under both RT_USING_CLOCK_TIME and RT_USING_PWM. Enabling both options will compile the same source twice and may break the build with duplicate symbols. Ensure it is only added once.
中文：n32l43x_tim.c 同时在 RT_USING_CLOCK_TIME 与 RT_USING_PWM 条件下被加入。两者同时开启时会导致同一源文件重复编译，可能出现重复符号从而构建失败。请保证只加入一次。

Suggested change
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32l43x_std_periph_driver/src/n32l43x_tim.c']
if GetDepend(['RT_USING_PWM']):
src += ['n32l43x_std_periph_driver/src/n32l43x_tim.c']
if GetDepend(['RT_USING_CLOCK_TIME']) or GetDepend(['RT_USING_PWM']):
src += ['n32l43x_std_periph_driver/src/n32l43x_tim.c']

Copilot uses AI. Check for mistakes.

if GetDepend(['RT_USING_RTC']):
src += ['n32l43x_std_periph_driver/src/n32l43x_rtc.c']
src += ['n32l43x_std_periph_driver/src/n32l43x_pwr.c']
Expand Down
3 changes: 3 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/N32WB452_Firmware_Library/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ if GetDepend(['RT_USING_DAC']):
if GetDepend(['RT_USING_CLOCK_TIME']):
src += ['n32wb452_std_periph_driver/src/n32wb452_tim.c']

if GetDepend(['RT_USING_PWM']):
src += ['n32wb452_std_periph_driver/src/n32wb452_tim.c']

Comment on lines 36 to +41
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

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

[bug/缺陷]: Duplicate source inclusion of n32wb452_tim.c

English: n32wb452_tim.c is added under both RT_USING_CLOCK_TIME and RT_USING_PWM. If both are enabled, the same C file will be compiled twice, typically causing duplicate object/duplicate symbol build failures. Add the file only once (e.g., gate it behind a combined condition or de-duplicate the src list).
中文：n32wb452_tim.c 同时在 RT_USING_CLOCK_TIME 和 RT_USING_PWM 条件下被加入 src。如果两者同时使能，同一源文件会被编译两次，通常会导致重复目标文件/重复符号的构建失败。应保证该文件只被加入一次（例如使用合并条件或对 src 去重）。

Copilot uses AI. Check for mistakes.
if GetDepend(['RT_USING_RTC']):
src += ['n32wb452_std_periph_driver/src/n32wb452_rtc.c']
src += ['n32wb452_std_periph_driver/src/n32wb452_pwr.c']
Expand Down
2 changes: 2 additions & 0 deletions bsp/n32/n32gxx_lxx/libraries/n32_drivers/SConscript
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ if GetDepend(['RT_USING_RTC']):
if GetDepend(['RT_USING_WDT']):
src += ['drv_wdt.c']

if GetDepend(['BSP_USING_PWM']):
src += ['drv_pwm.c']
path = [cwd]
path += [cwd + '/config']

Expand Down
Loading