inav icon indicating copy to clipboard operation
inav copied to clipboard
verified publisher icon iNavFlight

Add support for SKYSTARSF405V2

Open DusKing1 opened this issue 7 months ago • 7 comments

Vendor confirmed this target code works for them.

DusKing1 avatar May 22 '25 11:05 DusKing1

I've spoken to DusKing about a test sample.

sensei-hacker avatar May 22 '25 13:05 sensei-hacker

  • [x] Samples received
  • [x] Flash firmware
  • [x] Calibrate
  • [x] Orientation matches
  • [x] Gyro working
  • [x] Accel working
  • [x] Baro working
  • [x] Voltage correct
  • [x] Current correct
  • [x] Analog RSSI
  • [x] Mag I2C Bus
  • [x] UART1
  • [x] UART2
  • [x] UART3
  • [x] UART4
  • [x] UART5
  • [x] UART6
  • [x] LEDs working
  • [x] Buzzer working
  • [x] Motor outputs
  • [x] DShot support on m1-4
  • [x] Servo outputs
  • [x] Blackbox
  • [x] Analog Camera working
  • [x] Video Out working
  • [x] OSD working
  • [ ] PINIO1
  • [x] PINIO2

sensei-hacker avatar Jun 19 '25 04:06 sensei-hacker

I sent a PR to add one line to let the ADCs work. https://github.com/DusKing1/inav/pull/168

For the flash chip, I'm getting chip ID 0xFFFFFF Including after commenting out uart 3 since that shares a pin with the SPI flash.

sensei-hacker avatar Jun 20 '25 22:06 sensei-hacker

Thanks. I'll check blackbox.

DusKing1 avatar Jun 21 '25 02:06 DusKing1

Just updated def for SPI3 according to SCH. Thanks for your feedback!

DusKing1 avatar Jun 21 '25 03:06 DusKing1

It looks good now. Just one question - what is PINIO1 supposed to do, please? I had guessed it would turn the 10V on and off, but it doesn't seem to do that.

PINIO2 switches cameras, I see, and that works fine.

sensei-hacker avatar Jun 21 '25 17:06 sensei-hacker

It looks good now. Just one question - what is PINIO1 supposed to do, please? I had guessed it would turn the 10V on and off, but it doesn't seem to do that.

You are right, PINIO1 is to toggle the 10V on and off, I've fixed the pin def for PINIO1. Thanks for such a careful check!

DusKing1 avatar Jun 22 '25 12:06 DusKing1

I did some probing around the PINIO_1 circuit. It looks like there may be an issue there. Unless we hear some good news back from DusKing1, we may remove that from the target.

sensei-hacker avatar Jun 23 '25 04:06 sensei-hacker

Awesome. We'll just want to adjust config.c to match on the PINIO. I don't actually know whether you can have PINIO 2 without having PINIO 1. I can test that later.

sensei-hacker avatar Jun 23 '25 17:06 sensei-hacker

I tested after removing PINIO1 from config.c - It works fine. You CAN have pinio2 without having pinio1.

sensei-hacker avatar Jun 24 '25 17:06 sensei-hacker

@sensei-hacker any update on merging procedure of this PR?

DusKing1 avatar Aug 05 '25 10:08 DusKing1