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

I2C3_SCL and I2C3_SDA pins are mixed up #9750

Merged
merged 1 commit into from Jun 21, 2020

Conversation

supersidor
Copy link

according to datasheet of STM32F405xx chip PA8 is I2C3_SCL and PC9 is I2C3_SDA
Currently they are mixed up.
I use MATEK F405-OSD with external barometer and this fix solves my issues

datasheet

@mikeller
Copy link
Member

mikeller commented May 2, 2020

@supersidor: Thanks for this - this target is a legacy target that is not released any more, as it has been superseded by the Unified Target in https://github.com/betaflight/unified-targets/blob/master/configs/default/MTKS-MATEKF405STD.config - interestingly this target does not even specify pins for I2C 3. If they are connected to pads then maybe they should be added?

@supersidor
Copy link
Author

supersidor commented May 2, 2020

PA8 and PC9 have pads on flight controller - they are S4 and S6 pads on PCB. I just checked this for F405-OSD, F405-STD and F405-CTR versions.
I guess it would be nice to have I2C 3 declared but you have more expertise

@mikeller
Copy link
Member

mikeller commented May 2, 2020

@SnowLeopardFPV
Copy link

SnowLeopardFPV commented May 15, 2020

These default I2C 3 resource mapping changes are now preventing motor 4 from spinning if any ESC protocol other than DSHOT is used on the Matek F405-STD boards. The MOTOR 4 resource (pad S4) is already assigned to pin C09 and the MOTOR 6 resource (pad S6) is already assigned to pin A08. So the new I2C 3 resource mapping assignments appear to have caused some kind of resource conflict, but only when using the non-DSHOT ESC protocols.

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

Successfully merging this pull request may close these issues.

None yet

3 participants