I think i have attached in the code most of the information required.

Then we moved to I2C command. I think there i have shared the reg and addr values. For the SCL and SDA i dont think that is necessary at all to post which port is or not.
wrong on all counts

the i2c defines and osc speed can be crucial to i2c bus problems
pin and port settings can also play a vital role
the type of vars defined for reg and addr are also important ,for instance if reg is a word var then all bets are off

sorry i tried