Skip to content

Update TAKERF722SE #918

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

Closed
wants to merge 2 commits into from
Closed

Update TAKERF722SE #918

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d5c6d06
Add the configuration for baro, default UART, and PINIO1.
YI-BOYANG Sep 29, 2025
cbf76f6
Revert the changes to pinio1
YI-BOYANG Oct 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions configs/TAKERF722SE/config.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,9 @@
#define USE_ACC_SPI_MPU6000
#define USE_ACC_SPI_ICM42605
#define USE_ACC_SPI_ICM42688P
#define USE_ACCGYRO_BMI270
#define USE_BARO
#define USE_BARO_BMP280
#define USE_BARO_DPS310
#define USE_FLASH
#define USE_FLASH_M25P16
#define USE_FLASH_W25N01G
Expand Down Expand Up @@ -103,9 +105,10 @@
#define ADC1_DMA_OPT 0

#define DEFAULT_FEATURES FEATURE_ESC_SENSOR
#define SERIALRX_UART SERIAL_PORT_USART3

// serial 5 1024 115200 57600 0 115200
#define MSP_DISPLAYPORT_UART SERIAL_PORT_USART1
#define SERIALRX_UART SERIAL_PORT_USART2
Copy link
Member

@VitroidFPV VitroidFPV Sep 29, 2025

Choose a reason for hiding this comment

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

Default UART port functions are generally only used for peripherals that are integrated onto the board itself, or BNF quads where the factory setup will consistently use those specific ports. If you cannot ensure consistency, it's best to leave this up to the user or factory setup

Copy link
Contributor Author

@YI-BOYANG YI-BOYANG Sep 29, 2025

Choose a reason for hiding this comment

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

imageIn the design of FC, UART2 is specifically used for RX communication. However, the target has not been modified before.

Copy link
Member

@VitroidFPV VitroidFPV Sep 29, 2025

Choose a reason for hiding this comment

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

If that's how it comes set up in the BNFs that use it, that's probably fine



#define MAG_I2C_INSTANCE I2CDEV_1
#define BARO_I2C_INSTANCE I2CDEV_1
Expand All @@ -116,7 +119,7 @@
#define BEEPER_INVERTED
#define MAX7456_SPI_INSTANCE SPI2
#define PINIO1_CONFIG 129
#define PINIO1_BOX 0
#define PINIO1_BOX 40
Copy link
Contributor

@ot0tot ot0tot Sep 29, 2025

Choose a reason for hiding this comment

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

Why are you changing this? What does PINIO 1 control?

Copy link
Contributor Author

@YI-BOYANG YI-BOYANG Sep 30, 2025

Choose a reason for hiding this comment

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

Because some users have requested that USER1 be used to control the on/off of the 12V power supply.

Copy link
Contributor

@ot0tot ot0tot Sep 30, 2025

Choose a reason for hiding this comment

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

What is PINIO1 connected to on the FC? Can you post the schematic in the Discord?

Copy link
Contributor Author

@YI-BOYANG YI-BOYANG Sep 30, 2025

Choose a reason for hiding this comment

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

QQ_1759201293471 This is the schematic diagram of the PINIO1 connection. It can control a pad to output 12V voltage.

Copy link
Contributor

@ot0tot ot0tot Sep 30, 2025

Choose a reason for hiding this comment

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

So for the past several years the VTX power on this FC was tied to the arm switch? That makes no sense. Have you changed the design?

Copy link
Contributor

@ot0tot ot0tot Sep 30, 2025

Choose a reason for hiding this comment

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

If you are making changes to the FC design, this requires a new target.

Copy link
Contributor Author

@YI-BOYANG YI-BOYANG Oct 1, 2025

Choose a reason for hiding this comment

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

I believe that the new design and the old design can use the same target; there is no need to create a new one. However, the new design is still under planning. If you don't agree with the modifications made by pinio1, I can first cancel them and then we can have a discussion later. We need to add a barometer now.

Copy link
Contributor

@ot0tot ot0tot Oct 3, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Does this FC have a barometer? The product page and images make no mention of it.
https://geprc.com/product/geprc-taker-f722-se-flight-controller/

image

Copy link
Contributor Author

@YI-BOYANG YI-BOYANG Oct 9, 2025

Choose a reason for hiding this comment

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

Our subsequent plan is to add the "spl06-001 baro" to it.

Copy link
Contributor

@ot0tot ot0tot Oct 11, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

  1. The SPL06 is a poor quality baro, use something better.
  2. Any revisions to the FC require a new review. The design as-is does not meet the current standards.

#define PINIO2_BOX 41
#define FLASH_SPI_INSTANCE SPI3
#define GYRO_1_SPI_INSTANCE SPI1