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_HAL_ChibiOS: add hwdef for LongBowF405WING #27009

Merged
merged 2 commits into from
Jul 8, 2024
Merged

Conversation

HiRCBro
Copy link
Contributor

@HiRCBro HiRCBro commented May 7, 2024

hi
lefeirc PR new board with F405

Copy link
Contributor

@magicrub magicrub left a comment

Choose a reason for hiding this comment

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

Looks pretty good, thanks! This needs a bootloader. You can compile it by running:
./Tools/scripts/build_bootloaders.py LongBowF405WING

libraries/AP_HAL_ChibiOS/hwdef/LongBowF405WING/hwdef.dat Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/LongBowF405WING/hwdef.dat Outdated Show resolved Hide resolved
@magicrub
Copy link
Contributor

magicrub commented May 7, 2024

The commits should also be squashed

@tridge
Copy link
Contributor

tridge commented May 7, 2024

@HiRCBro I've fixed up the commits in your PR

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label May 7, 2024
@Hwurzburg Hwurzburg self-requested a review May 7, 2024 22:27
@Hwurzburg
Copy link
Collaborator

In order to review I will need pinout/board image.....also, any UART intended for RC operation needs DMA so UART2 in alt configuration will not support CRSF/ELRS and perhaps even SRXL2 ptotocols....so those need to be on another UART...but no others have full DMA either...so RC protocols using bidir serial will have an issue with current definition I beleive

@Hwurzburg
Copy link
Collaborator

Hwurzburg commented May 7, 2024

the pinnout of each back side jst-gh connector is missing, labeled solder pads are clear, but what signals are in each JST-GH are not indicated on the back....

@HiRCBro
Copy link
Contributor Author

HiRCBro commented May 8, 2024

In order to review I will need pinout/board image.....also, any UART intended for RC operation needs DMA so UART2 in alt configuration will not support CRSF/ELRS and perhaps even SRXL2 ptotocols....so those need to be on another UART...but no others have full DMA either...so RC protocols using bidir serial will have an issue with current definition I beleive

SBUS connect to UART2 with an inverter, if you use SBUS then UART2 cant work. if you use CRSF you cant connect to SBUS you need connect to UART2 or other UART

@Hwurzburg
Copy link
Collaborator

UART2 and all other UARTs do not have full DMA required for CRSF/ELRS for sure...so you cant make those RC systems work reliably with this hwdef

@HiRCBro
Copy link
Contributor Author

HiRCBro commented May 8, 2024

UART2 and all other UARTs do not have full DMA required for CRSF/ELRS for sure...so you cant make those RC systems work reliably with this hwdef


So i need change
PA2 USART2_TX USART2 NODMA
PA3 USART2_RX USART2 NODMA ALT(1)
to:
PA2 USART2_TX USART2
PA3 USART2_RX USART2ALT(1)

@Hwurzburg
Copy link
Collaborator

unfortunately dma does not get assigned to alternate function pins....so that wont do it....I dont know of a way to do it on a uart whose primary pin function is RCINT for RC .....best to get one of the other UARTs DMA enabled....UART6 unfortunately was not assigned a DMA channel in the resolver....maybe @tridge can make a suggestion, or you cna play with NODMA on some UARTS and see what the resolver produces for the others....you have to look at the hwdef.h file in the build directory after to waf configure for you board

@HiRCBro
Copy link
Contributor Author

HiRCBro commented May 8, 2024

se primary pin function is RCINT

i am confused, i test uart2 with CRSF is OK, do you mean, CRSF must work with a full DMA uart, i check hwdef.h only uart1 is full DMA with RX and TX.

@HiRCBro
Copy link
Contributor Author

HiRCBro commented May 8, 2024

se primary pin function is RCINT

i am confused, i test uart2 with CRSF is OK, do you mean, CRSF must work with a full DMA uart, i check hwdef.h only uart1 is full DMA with RX and TX.

if uart2 not suitbale for CRSF, we can use UART1, it is full DMA with RX and TX

@Hwurzburg
Copy link
Collaborator

yes UART1 is the only one that can support CRSF/ELRS...so SBUS->RX2 s RCINT and all single wire RC, and UART1 can handle the two wire protocols

@Hwurzburg
Copy link
Collaborator

se primary pin function is RCINT

i am confused, i test uart2 with CRSF is OK, do you mean, CRSF must work with a full DMA uart, i check hwdef.h only uart1 is full DMA with RX and TX.

its a matter of RELIABILITY.....CRSF and to a lesser extent ELRS will work without DMA, but not reliably...

@HiRCBro
Copy link
Contributor Author

HiRCBro commented May 10, 2024

se primary pin function is RCINT

i am confused, i test uart2 with CRSF is OK, do you mean, CRSF must work with a full DMA uart, i check hwdef.h only uart1 is full DMA with RX and TX.

its a matter of RELIABILITY.....CRSF and to a lesser extent ELRS will work without DMA, but not reliably...

Understood, DMA can reduce CPU load and improve data processing capabilities

@HiRCBro
Copy link
Contributor Author

HiRCBro commented Jul 3, 2024

se primary pin function is RCINT

i am confused, i test uart2 with CRSF is OK, do you mean, CRSF must work with a full DMA uart, i check hwdef.h only uart1 is full DMA with RX and TX.

its a matter of RELIABILITY.....CRSF and to a lesser extent ELRS will work without DMA, but not reliably...

HI
When can this be merged into the master,thankyou.

@Hwurzburg
Copy link
Collaborator

once the open review comments are addressed, the commits squashed and split into two commits one for Tools and one for hwdef it can be posted for devcall review and probable merge after that call....
if you wish, I can fix the readme recommended changes and squash and split commits for you....let me know

@HiRCBro
Copy link
Contributor Author

HiRCBro commented Jul 3, 2024

once the open review comments are addressed, the commits squashed and split into two commits one for Tools and one for hwdef it can be posted for devcall review and probable merge after that call....
if you wish, I can fix the readme recommended changes and squash and split commits for you....let me know

Please help me complete it, thank you

Copy link
Collaborator

@Hwurzburg Hwurzburg left a comment

Choose a reason for hiding this comment

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

I have applied the changes, rebased, squashed, and library split....since I never got the board (delivered to wrong address I suppose) please retest the firmware on your board...once confirmed I will mark it for devcall (next one is Monday night GMT)

@tridge tridge merged commit 8e28e6a into ArduPilot:master Jul 8, 2024
45 checks passed
@HiRCBro
Copy link
Contributor Author

HiRCBro commented Oct 15, 2024

I have applied the changes, rebased, squashed, and library split....since I never got the board (delivered to wrong address I suppose) please retest the firmware on your board...once confirmed I will mark it for devcall (next one is Monday night GMT)

I have applied the changes, rebased, squashed, and library split....since I never got the board (delivered to wrong address I suppose) please retest the firmware on your board...once confirmed I will mark it for devcall (next one is Monday night GMT)

HI
why LongBowF405WING still cant get from https://firmware.ardupilot.org/Plane/stable-4.5.7/

@Hwurzburg
Copy link
Collaborator

because backports must be requested...I have marked for backport...but I am not sure it will be since 4.6 is about to beta release soon..

@HiRCBro
Copy link
Contributor Author

HiRCBro commented Oct 15, 2024

because backports must be requested...I have marked for backport...but I am not sure it will be since 4.6 is about to beta release soon..

It's been 3 months since I submitted, please help to add it to 4.6?

@Hwurzburg
Copy link
Collaborator

4.6 will be automatic...its backports that are not...but IF another 4.5 release occurs, it should be added now

@HiRCBro
Copy link
Contributor Author

HiRCBro commented Oct 15, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WikiNeeded needs wiki update
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

5 participants