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

AP_BoardConfig: support flow control control on UARTs 6->8 #29130

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

andyp1per
Copy link
Collaborator

Currently four boards support SERIAL7 with flow control, but there is no support for controlling this via BRD_SER7_RTSCTS. This fixes the problem

@andyp1per
Copy link
Collaborator Author

Durandal

 Binary Name      Text [B]     Data [B]     BSS (B)      Total Flash Change [B] (%)      Flash Free After PR (B)
---------------  -----------  -----------  -----------  ----------------------------  -------------------------
blimp            0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      591888
ardurover        0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      291156
ardusub          0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      347128
arducopter-heli  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      135980
arduplane        0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      149660
arducopter       0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      134380
antennatracker   0 (0.0000%)  0 (0.0000%)  0 (0.0000%)  0 (0.0000%)                                      614092

@IamPete1
Copy link
Member

Could we add a #error somewhere to remind us to add these if we add support for more serial ports? We really should move them over to serial options....

@tridge tridge merged commit e20a00c into ArduPilot:master Jan 28, 2025
100 checks passed
@andyp1per andyp1per deleted the pr-all-ctsrts branch January 28, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Pending
Development

Successfully merging this pull request may close these issues.

4 participants