-
Notifications
You must be signed in to change notification settings - Fork 17.6k
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
hwdef: Add support for Sierra-F1 #24695
Conversation
Hey @tridge @bugobliterator |
# as an alternative config setup the RX6 pin as a uart. This allows | ||
# for bi-directional UART based receiver protocols such as FPort | ||
# without any extra hardware | ||
PC7 USART6_RX USART6 ALT(1) |
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.
might need to set its default protocol to RC input in order to assure a DMA is assigned... @tridge would know for sure
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.
if you intend this to also communicatee with an IOMCU for more PWM, I think you may need to add something to the hwdef for the coprocessor use.... @tridge would be able to comment
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.
if you intend this to also communicatee with an IOMCU for more PWM, I think you may need to add something to the hwdef for the coprocessor use.... @tridge would be able to comment
It's planned for future use. I'm planning to add STM32L431xx for more expansion. I want to get this base setup running to plan the future ones.
</table> | ||
|
||
|
||
### GPS + RCIN(UART6) |
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.
this is not correct, right? no I2C for compass, UART6 RX is not on the connector
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.
Hey Henry, just checked. We have I2C only on power ports. This connector has UART6 or RCIN.
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.
that's unfortunate....so external GPS/compass modules (the typical kind) cant connect to the GPS connector....so only CAN GPS/Compass can be used (using an internal compass is usually too prone to motor interference)....
in any case the GPS/RC connector show TX1/TX2 and RX1/RX2 not UART6 and UART7(GPS)
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.
Yes. Most of our customers use CAN RM3100 for magnetometer. And I2C is there on Power ports for smart battery SMBus few our customers use.
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.
again, GPS is TX7/RX7 and RC is RX6....that is not what is listed below
db3f54e
to
a1c436a
Compare
f0d62b4
to
3523e7b
Compare
PC15 EXT_LDO_EN OUTPUT HIGH | ||
|
||
# PWM channels | ||
PE9 TIM1_CH1 TIM1 PWM(1) GPIO(50) |
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.
should try to make these bi-directional if possible
- SERIAL0 -> USB | ||
- SERIAL1 -> UART2 (Telem1, DMA-enabled) | ||
- SERIAL2 -> UART8 (Telem2, DMA-enabled) | ||
- SERIAL3 -> UART7 (GPS, NODMA) |
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.
this is not the order you defined them in in the hwdef!
</table> | ||
|
||
|
||
### GPS + RCIN(UART6) |
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.
again, GPS is TX7/RX7 and RC is RX6....that is not what is listed below
</tr> | ||
<tr> | ||
<td>3 (blk)</td> | ||
<td>RX (IN)</td> |
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.
RX2
</tr> | ||
<tr> | ||
<td>2 (blk)</td> | ||
<td>TX (OUT)</td> |
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.
TX2
</tr> | ||
<tr> | ||
<td>3 (blk)</td> | ||
<td>SCL I2C</td> |
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.
should note which I2C bus....3 or 4 on the SCL and SDA pins
# LED drivers | ||
NTF_LED_TYPES 289 | ||
|
||
# EKF setup |
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.
these are not required.....already defaults
@@ -0,0 +1,13 @@ | |||
# Serial3 default to GPS |
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.
my mistake....SERIAL3 already defaults to GPS....so not needed.....but need to fix UART order in hwdef so that UART7 is third and UART6 is forth, not the other way around...
@MallikarjunSE when I enable ethernet it watchdogs, we'll need to debug that |
PA7 ETH_RMII_CRS_DV ETH1 | ||
PA1 ETH_RMII_REF_CLK ETH1 | ||
|
||
define BOARD_PHY_ID MII_LAN8742A_ID |
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.
is it a LAN8742A ?
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.
Yes.
@MallikarjunSE can you send me a photo of exactly what the device you sent me looks like - I'm embarrassed to say I can't find it! @peterbarker found this PR in his archaeology efforts |
@MallikarjunSE I suspect the PHY is not properly connected to the H743. Please send me the schematic and also double check the layout and connections |
@MallikarjunSE status....if this is not active, we should close the PR |
Hey @tridge @bugobliterator
Possible to check this? There are few changes suggest by Sid for ADC3 support.
Thank you!