inav icon indicating copy to clipboard operation
inav copied to clipboard

Added Zerodrag WARP7

Open vishnumda opened this issue 1 year ago • 20 comments

  • Added config.c, target.h, target.c and CMakeLists.txt

vishnumda avatar Jul 30 '24 06:07 vishnumda

For tracking progress

WARP F7

  • [x] Flash firmware
  • [x] Calibrate
  • [x] Orientation matches
  • [x] Gyro working
  • [x] Accel working
  • [x] Baro working
  • [x] Mag working (i2c)
  • [x] Voltage correct
  • [x] Current correct
  • [x] UART1
  • [x] UART2
  • [x] UART3
  • [x] UART4
  • [x] UART5
  • [ ] Camera working
  • [ ] Video Out working
  • [ ] OSD working
  • [ ] LEDs working
  • [ ] Buzzer working
  • [ ] Motor/Servo outputs
  • [x] Blackbox

mmosca avatar Aug 01 '24 12:08 mmosca

An old CMakeCache file was causing the build to fail. It has been removed.

vishnumda avatar Aug 03 '24 09:08 vishnumda

An old CMakeCache file was causing the build to fail. It has been removed.

There should only be the files in the target folder in this pr. there are still a lot of files that are not needed and should not be included.

mmosca avatar Aug 03 '24 09:08 mmosca

Adding some manual screenshots for reference.

image

image

image

mmosca avatar Aug 04 '24 13:08 mmosca

@vishnumda is this T2 pad labeled correctly?

image

Never mind, had a short on my side. :)

mmosca avatar Aug 04 '24 14:08 mmosca

UART6 is missing in the target.

mmosca avatar Aug 04 '24 14:08 mmosca

Board also appears to have a Camera Control pin, but it is missing the definition and the timer for that.

Is P2 supposed to be PINIO output? If so,the PINIO definitions in target.h are missing.

mmosca avatar Aug 04 '24 14:08 mmosca

We are working on it.

vishnumda avatar Aug 07 '24 10:08 vishnumda

Hi. We have pushed our latest changes. It took us some time to test everything. We fixed all of the conflicts and tested all of the functions internally and found them to be working. Please test them from your side and let us know if any changes are needed.

vishnumda avatar Dec 27 '24 06:12 vishnumda

UART6 is missing in the target.

We are only using 5 UARTs. The definitions have been updated. We still get a warning as,

[265/391] Building C object src/main/target/WARPF7/CMakeFiles/WARPF7.elf.dir/__/__/io/serial.c.obj
/home/vishnu/Files/Code/inav/src/main/io/serial.c:83:5: warning: excess elements in array initializer
   83 |     SERIAL_PORT_USART5,
      |     ^~~~~~~~~~~~~~~~~~
/home/vishnu/Files/Code/inav/src/main/io/serial.c:83:5: note: (near initialization for 'serialPortIdentifiers')

I hope this can be ignored.

vishnumda avatar Dec 27 '24 06:12 vishnumda

Can you merge the latest master, to make testing and validation easier?

mmosca avatar Jan 12 '25 22:01 mmosca

All branches have been updated.

vishnumda avatar Jan 13 '25 06:01 vishnumda

The build keeps failing, probably due to the wrong number of uarts selected:

image

mmosca avatar Jan 16 '25 14:01 mmosca

Compilation works for us despite the UART count warning as you can see in the screenshot.

image

vishnumda avatar Jan 17 '25 05:01 vishnumda

Build is still failing: image

Can you fix the build, so we can proceed with the tests?

For the full logs, check: https://github.com/iNavFlight/inav/actions/runs/12903988544/job/36210785483?pr=10262

mmosca avatar Jan 27 '25 09:01 mmosca

Seems like all of the warning are being treated as errors, including the UART count error. Let me fix it.

vishnumda avatar Jan 28 '25 05:01 vishnumda

Add -DWARNINGS_AS_ERRORS=ON to your cmake command line, so you can build it locally with the same settings we build in github.

mmosca avatar Jan 28 '25 09:01 mmosca

We changed the serial port count back to 6 and everything compiles now without any warnings or errors.

image

vishnumda avatar Feb 07 '25 11:02 vishnumda

Was Dshot fixed?

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

Yes, it was fixed it is working.

vishnumda avatar Jun 19 '25 06:06 vishnumda

As:

  • No developer is likely to be able to do any further testing on this FC; and
  • The vendor has stated that they have addressed and tested all previously raised issues.

Unless there are any objections in the next 48 hours I intend to merge this target.

stronnag avatar Jun 27 '25 08:06 stronnag

As:

  • No developer is likely to be able to do any further testing on this FC; and
  • The vendor has stated that they have addressed and tested all previously raised issues.

Unless there are any objections in the next 48 hours I intend to merge this target.

That's a coincidence. As you were writing that comment, they were writing down my address to send a board for testing. I have a set up to do thorough testing very quickly.

sensei-hacker avatar Jun 27 '25 09:06 sensei-hacker