Skip to content
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

[TARGET REQUEST] SKYSTARSF7HDPRO (BMI270) #733

Closed
seb3876 opened this issue Nov 25, 2021 · 37 comments
Closed

[TARGET REQUEST] SKYSTARSF7HDPRO (BMI270) #733

seb3876 opened this issue Nov 25, 2021 · 37 comments
Labels

Comments

@seb3876
Copy link

seb3876 commented Nov 25, 2021

Hi,

anyone could send me target for SKYSTAR F7HDPRO?

Thanks a lot!

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Nov 25, 2021

Needs tested!
Check gyro orientation, motor arrangement, flash, osd, vtx control, everything else if possible please update here with feedback that it works or needs improvement.
PROPS OFF FOR SAFETY

EmuFlight_0.4.0_SKSTF7HDPRO_Build_c8cc50dde.zip

(edit: hex updated for barometer, osd, battery, usb connect)

@BeauBrewski BeauBrewski added the in-testing-needs-testing needs testing before merge consideration label Nov 25, 2021
@seb3876
Copy link
Author

seb3876 commented Nov 26, 2021

Thanks for Target

1/ the connection usb don't run: impossible to connect with computer but via wifi it's ok.
2/ Board and alignment don't ok, i must put -90 to the yaw degrees
3/ there are a lot of options missing in the osd

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Nov 26, 2021

1/ the connection usb don't run: impossible to connect with computer but via wifi it's ok. 2/ Board and alignment don't ok, i must put -90 to the yaw degrees 3/ there are a lot of options missing in the osd

  1. I'll look into that. Seem to be having some USB issues with different targets right now. But the target is working and not bricking the board, so that is good.
  2. Alignment is set to the manufacturer recommendations according to BetaFlight configuration. Different builds will have different alignment however. Will double check.
  3. Try turning on "Expert Mode" for more OSD options. It will still have different OSD options than BF however.

@seb3876
Copy link
Author

seb3876 commented Nov 27, 2021

after many test, baro not recognized.

Could you do something ?

@BeauBrewski
Copy link
Collaborator

What brand/kind of baro? Internal/External?

@seb3876
Copy link
Author

seb3876 commented Nov 27, 2021

the baro is integrated on fc

@BeauBrewski
Copy link
Collaborator

Did it work before with BF?
Do you have a manufacturer web address for your FC?
Do you have a BF dump of your FC before flashing to Emu?

The SKYSTARS F722HD PRO fc I saw said no baro http://skystars-rc.com/Products/ESC/2021/0620/93.html
image

If yours has one, then it must be a different FC but same target, so I will need a copy of that web address and a copy of your DUMP

@seb3876
Copy link
Author

seb3876 commented Nov 27, 2021

the baro worked on betaflight

the manufacturer web adress you cited is the good page web but i have a baron on betaflight, I find it surprising that the web adress does not specify a baro

this is the dump in betaflight in attached file

dump all betaflight.txt

@BeauBrewski
Copy link
Collaborator

Thank you for that!

@BeauBrewski
Copy link
Collaborator

@seb3876 can I get a copy of your Emuflight dump as well for comparison?

@seb3876
Copy link
Author

seb3876 commented Dec 2, 2021

hi,

this is the dump in Emuflight in attached file:

DUMP ALL.txt

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Dec 2, 2021

Okay, I think that helped me fix it. I thought I had all the barometer stuff installed already, but your Emu dump said otherwise. I updated the hex linked above, but am including it again here to be easy. Try it and let me know if it works. Do not copy any diff/dump info onto this flash until you verify that the Baro is detected, please.

EmuFlight_0.4.0_SKYSTARSF7HDPRO_Build_b87117f08.zip

@seb3876
Copy link
Author

seb3876 commented Dec 3, 2021

i tested the new target:
1/ no baro
2/ no information on the osd
3/ always difficult to connect the fc on emuflight
4/ capa information always 0

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Dec 3, 2021

@seb3876 Thank you again for your patience and help. I spotted my code errors and you may have helped me narrow down the issue with targets connecting poorly with USB. I think I fixed OSD, battery, and Baro issues, as before they weren't showing up in the Dump properly, but now they are.

EmuFlight_0.4.0_SKSTF7HDPRO_Build_8582a8fbe.zip

@seb3876
Copy link
Author

seb3876 commented Dec 5, 2021

@BeauBrewski thank you for your work!

all the previous points are resolved but now the drone turns over as if a propeller was upside down

can you do something?

thanks you

@BeauBrewski
Copy link
Collaborator

@seb3876 send me a copy of your newest Dump and I'll see if a setting is off.
In the meantime, double check your board orientation in comparison to the animated drone in the configurator. Double check your props. Double check motor direction. Double check that normal/reversed props setting is correct.

@seb3876
Copy link
Author

seb3876 commented Dec 6, 2021

@BeauBrewski this is my new DUMP in attached file.
i must put FC -90 to the yaw degrees but quad fly not in right way

Dump.txt

@BeauBrewski
Copy link
Collaborator

@seb3876 Something is seriously not right with that dump. Looks all glitchy. Either need to try again, or do a reflash with full chip erase.

@seb3876
Copy link
Author

seb3876 commented Dec 9, 2021

i tried a full erase, new configuration and dump all.

See in attached file.

Thank You

DUMP ALL EMUFLIGHT.txt

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Dec 10, 2021

@seb3876 I think I spotted an issue between the BF and Emu dumps you've given me. You don't have the radio set up the same on Emuflight. Make sure your channel map is correct on the Receiver tab and that the Roll/Pitch/Throttle/Yaw move correctly when radio connected (will probably have to plug in battery to quad when testing with radio). And that the animated quad moves accordingly when sticks are moved.

You can also copy the AUX settings from the BF dump and paste them into CLI of Emuflight then type save. That should get your controls matching.
image

image

@seb3876
Copy link
Author

seb3876 commented Dec 12, 2021

@BeauBrewski The settings are good. I don't understand why they don't match.

@BeauBrewski
Copy link
Collaborator

@seb3876 Hmm. I tested on a different target changing the CLI Aux settings and it did not appear to make a difference, so that is not the problem. Sorry about that. It was the only noticeable difference I saw in the dumps.

I reworked the gyro code and have a new test file.

EmuFlight_0.4.0_SKSTF7HDPRO_Build_c8cc50dde.zip

@seb3876
Copy link
Author

seb3876 commented Dec 14, 2021

@BeauBrewski thank you very much !

@BeauBrewski
Copy link
Collaborator

@seb3876
Did the newest build ever end up working correctly? I know this was being a headache.
Feedback is much appreciated for as many features as possible or available:
✔️/❌ Flash
✔️/❌ Gyro
✔️/❌ Accelerometer
✔️/❌ Motors
✔️/❌ Onboard ADC
✔️/❌ OSD (analog)
✔️/❌ Smart Audio controls (OSD and CLI)
✔️/❌ Alarm switch
✔️/❌ Barometer
✔️/❌ Blackbox
✔️/❌ LED
✔️/❌ GPS
Or any other features or issues you notice

@BeauBrewski
Copy link
Collaborator

Above target only works with older boards with MPU6xxx gyro.

Will need to add BMI270 support when available.

@BeauBrewski BeauBrewski changed the title [TARGET REQUEST] SKYSTARSF7HDPRO [TARGET REQUEST] SKYSTARSF7HDPRO (BMI270) May 21, 2023
@BeauBrewski
Copy link
Collaborator

Working on test target
#890

@BeauBrewski
Copy link
Collaborator

Needs tested!
Working SKYSTARSF7HDPRO target (name below is shortened for reasons). Bench tested and works on BMI270 flight controllers. Needs flight tested and other features.

EmuFlight_0.4.2_SKSTRF7HDPRO_BMI270_2b5d8a19c.zip

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented May 29, 2023

**Note: Had USB connect issues--would take 3+ tries to connect into Emu Config. Shortened target name and USB connection is now fast and first try.
Gyro works-8k/8k pidloop running at 2-4% CPU. Blackbox appears detected. Will update more once installed on quad.

@nerdCopter
Copy link
Member

does the folder-name need shortening or only the target.h's USBD_PRODUCT_STRING ? or both?

@nerdCopter
Copy link
Member

src/main/pg/board.h:26:#define MAX_BOARD_NAME_LENGTH 20 🤔

@BeauBrewski
Copy link
Collaborator

I'm not sure, but I shortened both. SKYSTARSF7HDPRO isn't longer than 20 🤷

@BeauBrewski
Copy link
Collaborator

From FB page:
"Οκ the hex file is working !!!! No problem at all . I hover for 1 min and no hot motors"

@BeauBrewski
Copy link
Collaborator

Update from FB Page:
Everything works great no problem at all !!! but when I enable the led strips feature 2 of my motors stop working
😭

@nerdCopter
Copy link
Member

maybe: TIM_USE_PPM or _ANY, rather than TIM_USE_LED i saw this some place recently

@BeauBrewski
Copy link
Collaborator

maybe: TIM_USE_PPM or _ANY, rather than TIM_USE_LED i saw this some place recently

That got the motors working, but now LED's don't. 😞

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Jun 5, 2023

Suspecting there might be an issue with LED code/drivers and how it does timers. Tried without drivers, with light_ws2811strip.c, and with light_ws2811strip_hal.c drivers. Tried light_ws2811strip_stdperiph.c but would not compile as its for F4, F3, and F1 boards only. Either LEDs don't work, or motors 2 and 3 don't work. Everything works fine in BF however.

Edit: One thing I noted as well is when setting up LED, they stayed white and would not change color when saved.

@nerdCopter
Copy link
Member

target resolved by #943 , therefore closing ticket.
any lingering problems unknown at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants