In I2CDirect, in none of the 6 methods with an offset parameter, the offset parameter is referenced anywhere in the body

[stefanhaustein]

The length parameter seems to be ignored a lot, too

[stefanhaustein]

Might make sense to run a tool for unused parameters and similar issues over the whole ffm codebase?

[FDelporte]

Hi, I'm trying to understand how this I2C implementation works. I reworked the internal methods in I2CDirect to include an offset parameter, so they are flagged as unused by SonarQube. This doesn't happen in the override methods...

Draft pull request is here #605
This will need more work as it doesn't fix this problem yet, it only makes it more visible at this moment.
@DigitalSmile can you assist me what more changes are needed?