Fix issue #3 - 'naive' I2C transaction in pimoroniTrackball::changed() caused it to fail on ESP32 and maybe other targets
Fix compiler warnings for use of TwoWire.requestFrom(int, int, int)
Fix some style issues and tidy up
Tested on ESP32-S2, ESP8266 and ATmega32u4