Skip to content

Add OmnibusF4 V6 target #4354

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.

Sign up for GitHub

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

Jump to bottom
Closed
asizon wants to merge 9 commits into from
Closed

Add OmnibusF4 V6 target #4354

asizon wants to merge 9 commits into from

Conversation

@asizon
Copy link
Contributor

@asizon asizon commented Feb 12, 2019

I have created target OmnibusF4 V6 as a variant of FireworksV2

@asizon asizon mentioned this pull request Feb 12, 2019
Closed
@asizon
Copy link
Contributor Author

asizon commented Feb 12, 2019

inav_2.0.1_OMNIBUSF4V6.zip
.HEX for test @dl1bg

@shellixyz shellixyz changed the base branch from master to development February 12, 2019 11:12
@asizon
Copy link
Contributor Author

asizon commented Feb 12, 2019 •
edited
Loading

@shellixyz What has happened in this PR?take advantage to put past commits?

@asizon
Copy link
Contributor Author

asizon commented Feb 12, 2019 •
edited
Loading

lol, I understand, i have open PR again master and now is mixed.
It does not matter if the master merged commits merges with the development, right? or should I backtrack?

@shellixyz
Copy link
Collaborator

shellixyz commented Feb 12, 2019 •
edited
Loading

You have to use the development branch as a base. I just changed the destination branch from master to development. Rebase your branch onto development and force push.

@dl1bg
Copy link

dl1bg commented Feb 12, 2019

Thank you, Asizon!
Just updated. The I2C magnetometer (HMC5883) if the BM-880 is not detected.
It works with Andrew's (atobiese's) version
https://www.dronetrest.com/t/i2c-magnetometer-not-working-on-omnibus-f4-v6/7005/11?u=fat

INAV/OMNIBUSF4V6 2.0.1 Feb 12 2019 / 11:41:25 (da78e89)

Did I miss to configure something?

digitalentity
digitalentity reviewed Feb 12, 2019
View reviewed changes
src/main/target/FIREWORKSV2/target.h
#define SERIALRX_UART SERIAL_PORT_USART1
#define SMARTAUDIO_UART SERIAL_PORT_USART4

#define TARGET_CONFIG
Copy link
Member

@digitalentity digitalentity Feb 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not necessary if you don't have target-specific config code.

digitalentity
digitalentity reviewed Feb 12, 2019
View reviewed changes
Copy link
Member

@digitalentity digitalentity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, please add to release.mk list

@asizon asizon closed this Feb 13, 2019
@asizon asizon deleted the omnibusf4v6 branch February 13, 2019 17:48
@dl1bg
Copy link

dl1bg commented Feb 13, 2019

Now tried the two versions back and forth, I2C does not work with this HEX file.
Please tell me if I can help in any way.

@asizon
Copy link
Contributor Author

asizon commented Feb 13, 2019

inav_2.1.0_OMNIBUSF4V6.zip
@dl1bg try it please :)

@asizon
Copy link
Contributor Author

asizon commented Feb 14, 2019

the new PR atempt #4369

@CRISTSC
Copy link

CRISTSC commented Feb 20, 2019 •
edited
Loading

the pin assignment is not entirely right for this board. this is what i discovered with a multi-meter:

resource

IO:

A00: RSSI IN
A01; SERIAL4 UART RX
A02: SERIAL2 UART TX
A03: MOTOR3 OUT
A04: MPU CS
A05: SPI1 SCK
A06: SPI1 MISO
A07: SPI1 MOSI
A08: FREE
A09: SOFTSERIAL1 UART TX/RX
A10: FREE
A11: USB IN
A12: USB OUT
A13: FREE
A14: FREE
A15: OSD CS
B00: MOTOR1 OUT
B01: MOTOR2 OUT
B02: FREE
B03: BARO CS
B04: BEEPER OUT
B05: MOTOR4 OUT
B06: LED1 OUT
B07: CAM_C
B08: I2C1 SCL
B09: I2C1 SDA
B10: SERIAL3 UART TX
B11: SERIAL3 UART RX
B12: FLASH CS
B13: SPI2 SCK
B14: SPI2 MISO
B15: SPI2 MOSI
C00: FREE
C01: ADC CH1
C02: ADC CH2
C03: FREE
C04: FREE
C05: FREE
C06: FREE
C07: SERIAL6 UART RX
C08: MOTOR 5 OUT
C09: MOTOR 6 OUT
C10: SPI3 SCK
C11: SPI3 MISO
C12: SPI3 MOSI
C13: FREE
C14: FREE
C15: FREE
D00: FREE
D01: FREE
D02: SPI_PREINIT CS
D03: FREE
D04: FREE
D05: FREE
D06: FREE
D07: FREE
D08: FREE
D09: FREE
D10: FREE
D11: FREE
D12: FREE
D13: FREE
D14: FREE
D15: FREE

i bold what i discovered wrong so far. can you please make the modifications to recover the full functionality of this board. thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@digitalentity digitalentity digitalentity left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@asizon @shellixyz @dl1bg @CRISTSC @digitalentity @Kurtoid @DzikuVx