inav icon indicating copy to clipboard operation
inav copied to clipboard

Kakute H743 Wing Target doesn't match Manufacturer documentation

Open paul-fornage opened this issue 1 year ago • 0 comments

Current Behavior

The Kakute H743 Wing target in iNav lists the Pitot sensor as being on I2C bus 1, but HolyBro says to use I2C bus 2.

From src/main/target/KAKUTEH7WING/target.h

// *************** I2C /Baro/Mag *********************
#define USE_I2C
#define USE_I2C_DEVICE_1
#define I2C1_SCL PB8
#define I2C1_SDA PB7

#define USE_I2C_DEVICE_2
#define I2C2_SCL PB10
#define I2C2_SDA PB11

#define USE_I2C_DEVICE_4
#define I2C4_SCL PD12
#define I2C4_SDA PD13

#define USE_BARO
#define BARO_I2C_BUS BUS_I2C4
#define USE_BARO_BMP280
#define USE_BARO_MS5611
#define USE_BARO_DPS310
#define USE_BARO_SPL06

#define USE_MAG
#define MAG_I2C_BUS BUS_I2C1
#define USE_MAG_ALL

#define TEMPERATURE_I2C_BUS     BUS_I2C1
#define PITOT_I2C_BUS           BUS_I2C1

#define USE_RANGEFINDER
#define RANGEFINDER_I2C_BUS BUS_I2C1

// *************** UART *****************************

Manufacturer Manual says I2C bus 1 is just for compass, and other peripherals should use I2C bus 2.

Also the wiring diagram uses I2C bus 2: image

Steps to Reproduce

Just read the target at src/main/target/KAKUTEH7WING/target.h.

Expected behavior

I would expect the iNav mapping and wiring diagram to match the iNav target

Suggested solution(s)

I don't know. This could be fixed easily by changing a couple '1's to '2's, but this risks breaking existing builds and causing a few headaches. I don't know if this is worth the headache new builds will have to deal with.

Additional context

  • FC Board name and vendor: Kakute H743 Wing
  • INAV version string: (The current source code)

paul-fornage avatar Jan 12 '24 20:01 paul-fornage