i2c is not very well designed

[hmaarrfk]

The function should be more "stateless"

The user should always have to provide:

1. The slave address
2. The slave register size (0 for no slave register)
3. The slave register MSB/LSB
4. The slave register itself
5. The data size
6. The data transfer direction MSB/LSB
7. The data to send itself as a single number

[hmaarrfk]

One potential issue is that there is a 64 character buffer somewhere in the pipeline. I've found that communication with more than 64 characters is often finicky.