TBS LUCID Pro

[andyp1per]

TBS LUCID PRO

[Hwurzburg]

@andyp1per missing image

[andyp1per]

@andyp1per missing image

Very draft right now - don't review until its out of draft

[Hwurzburg]

Need pinots for GH connectors

[Hwurzburg]

@andyp1per started looking at this...some questions before a I post a formal review since there are some big disconnects:

1. UART1, RCin, is not pinned out in the images...where is it?
2. Images show UART7/8 which are not defined
3. is inverted SBus really connected to RX5? just checking since the other UARTs are iffy and hwdef says there is no TX5 but its shown on pinout
4. if UART1 really is RCin, shouldn't it be in the DMA priority list rather than UART5?
5. if PWM5/6 are not DShot, shouldn't you tag them NODMA?

[andyp1per]

• UART1, RCin, is not pinned out in the images...where is it?

Need new images. It's the CRSF port.

• Images show UART7/8 which are not defined

Need new images.

• is inverted SBus really connected to RX5? just checking since the other UARTs are iffy and hwdef says there is no TX5 but its shown on pinout

That's what the schematic says

• if UART1 really is RCin, shouldn't it be in the DMA priority list rather than UART5?

The DMA mapping works out fine - I checked. You only need to add to DMA_PRIORITY to move things about.

• if PWM5/6 are not DShot, shouldn't you tag them NODMA?

They could be used for dshot. Can't be NODMA because one is the LED which needs DMA.

[Hwurzburg]

put a needsanswer label so this doesnt keep popping up on my radar until new images are provided

[andyp1per]

Silk screen updated @Hwurzburg

[Hwurzburg]

the image has several errors, in addition to my other comments:

• UART5_RX should be UART2_RX
• CAMC should be CAMC/PWM5

[andyp1per]

the image has several errors, in addition to my other comments:

* UART5_RX should be UART2_RX

I've asked them to fix

* CAMC should be CAMC/PWM5

I think this is ok - its a generic picture that covers BF as well which will not have PWM5

[Hwurzburg]

so we should not tell AP users there is a PWM5?otherwise i think the image should show it

[andyp1per]

• UART5_RX should be UART2_RX

README is right, hwdef is wrong - I have fixed

[andyp1per]

Thanks @Hwurzburg

[rmackay9]

this is included in ArduPilot-4.6.0-beta3