-
Notifications
You must be signed in to change notification settings - Fork 18k
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
Conversation
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.
Looks pretty good, thanks! This needs a bootloader. You can compile it by running:
./Tools/scripts/build_bootloaders.py LongBowF405WING
The commits should also be squashed |
@HiRCBro I've fixed up the commits in your PR |
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 |
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.... |
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 |
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 |
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 |
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 |
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 |
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 |
HI |
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.... |
Please help me complete it, thank you |
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.
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 |
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? |
4.6 will be automatic...its backports that are not...but IF another 4.5 release occurs, it should be added now |
Ok. thank you
…---- Replied Message ----
| From | Henry ***@***.***> |
| Date | 10/15/2024 20:43 |
| To | ArduPilot/ardupilot ***@***.***> |
| Cc | HiRCBro ***@***.***>,
Mention ***@***.***> |
| Subject | Re: [ArduPilot/ardupilot] AP_HAL_ChibiOS: add hwdef for LongBowF405WING (PR #27009) |
4.6 will be automatic...its backports that are not...but IF another 4.5 release occurs, it should be added now
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
hi
lefeirc PR new board with F405