Portenta H7 - 3x I2C bus

[RoboCode23]

The Portenta has 3x I2C bus and only 2 are defined in the libraries...

added and modified howmany from 2 to 3 in pins_arduino.h

#define WIRE_HOWMANY		3
#define I2C_SDA2			(PH_12)
#define I2C_SCL2			(PH_11)

and added into Wire.h and Wire.cpp

#if WIRE_HOWMANY > 2
arduino::MbedI2C Wire2(I2C_SDA2, I2C_SCL2);
#endif

makes all 3 work

[facchinm]

Hi @RoboCode23 ,
would you mind submitting a pull request so we can properly track the authorship of this fix? Thanks a lot!

[RoboCode23]

Hi @RoboCode23 , would you mind submitting a pull request so we can properly track the authorship of this fix? Thanks a lot!

@facchinm Sure, first time I found out something useful, so I didn't know how this works on GitHub - pull request added, don't know if properly...

[cHemingway]

I have also seen this issue, its particurally impactful on the Portenta Machine Control, where the front Groove connector is labelled "SCL, SDA" on the silkscreen and connected to PH_8, PH_7 (bus I2C3 on the schematic)

These are not not listed in pins_arduino.h for that board, and so there is no matching Wire instance by default.

[cHemingway]

@facchinm , chasing this, would it help if I submitted the same pull request as RoboCode23 so this could be merged and the "missing" I2C port added?

[RoboCode23]

Hello, yes, sorry i'm overwhelmed with other work, if somebody else submits this that be great! RoboCode23 Robert Kotlaba *Virtual Garage s.r.o.**www.301.cz <http://www.301.cz> & www.virtual-garage.eu <http://www.virtual-garage.eu> * +420 606 171 143 <+420%20606%20171%20143> V Sadech 1229 43401 Most Czech Republic IC: 06860885

…

On Fri, 25 Feb 2022 at 16:32, Chris Hemingway ***@***.***> wrote: @facchinm <https://github.com/facchinm> , chasing this, would it help if I submitted the same pull request as RoboCode23 so this could be merged and the "missing" I2C port added? — Reply to this email directly, view it on GitHub <#332 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AUOAE3RD6L4YLBHPEWK2P6DU46OJ3ANCNFSM5EUTQYMQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[cHemingway]

I have opened #413 and listed you as the author in the commit

[Jenifen]

PH_8, PH_7 still not listed in pins_arduino.h?
@cHemingway

[thechordmaster]

Any update on this? Have they been updated please?

[pennam]

PR #413 was merged in march 2022 so this change should be included in the core releases since a long time.