I2C CONFUSION, Help needed please

[FromTheCockpit]

Have a look at my real term serial monitor. I am actually getting a lot of unexpected data which I don't understand how is getting stored in just one byte. Though among all this, I can see '2E' (but it is swapped around so 'E2'). Note sure if I am trying to convince myself or the data is rubbish.

[FromTheCockpit]

Better in nibble setting. I can see '2E' repeated a few times.

[LinkMTech]

My bad, change the Read statement:

I2CREAD SDA,SCL,ADDR,[VALUE] ',Fail

[FromTheCockpit]

Still getting the same with the following code:

Value Var 	BYTE
Addr	Var	byte
Loc 	Var	Byte
'---------------------------------------------------------
Pause 500
	DEBUG "Start......",13,10
Value=0
Loc=0

Main:
High Led : Pause 500: Low Led : PAUSE 500
	Addr=$36
	I2CWRITE SDA,SCL,ADDR,[LOC],Fail1
	Addr=$37 : pause 10 ' Sometimes it is failing, leaving pause 10 makes it better.
	I2CREAD SDA,SCL,ADDR,[VALUE] ,Fail2
	PAUSE 25
	DEBUG  Value,13,10
	PAUSE 250
	DEBUG "Done....",13,10
WHILE 1 : WEND
Goto Main

Fail1:
while 1:	toggle portc.2 : pause 50 : wend 
goto main

Fail2:
WHILE 1
	High Portc.2 : Pause 250 : Low PortC.2 : Pause 250
WEND

[LinkMTech]

What if you assign the Read/Write addresses before hand?

Value Var 	BYTE
Addr	Var	byte
Loc 	Var	Byte

'---------------------------------------------------------
Pause 500
	DEBUG "Start......",13,10
Value=0
Loc=0
WR_Add CON $36
RD_Add CON $37

Main:
High Led : Pause 500: Low Led : PAUSE 500
	'Addr=$36
	I2CWRITE SDA,SCL,WR_Add,[LOC],Fail1
	'Addr=$37 : pause 10 ' Sometimes it is failing, leaving pause 10 makes it better.
	I2CREAD SDA,SCL,RD_Add,[VALUE] ,Fail2
	PAUSE 25
	DEBUG  Value,13,10
	PAUSE 250
	DEBUG "Done....",13,10
WHILE 1 : WEND
Goto Main

Fail1:
while 1:	toggle portc.2 : pause 50 : wend 
goto main

Fail2:
WHILE 1
	High Portc.2 : Pause 250 : Low PortC.2 : Pause 250
WEND

Question: Where is the "1" monitored at in these statements? Never seen it done this way before.

WHILE 1: WEND

[FromTheCockpit]

The manual says do not use constants for address.
Though I tried the above but the result is the same.

While 1 : Wend - I just put this just like that to halt the code.

[LinkMTech]

Oh yeah, no Constants.
Then give it to him straight!

Have you tried clearing the LOC and Value variables at the beginning of each loop just in case? Start with a clean slate every time.

Main:
    Loc=0
    Value=0
High Led : Pause 500: Low Led : PAUSE 500
    I2CWRITE SDA,SCL,$36,[LOC],Fail1
    I2CREAD SDA,SCL,$37,[VALUE] ,Fail2
    PAUSE 25
    DEBUG  Value,13,10
    PAUSE 250
    DEBUG "Done....",13,10
    
WHILE 1 : WEND
Goto Main

Fail1:
while 1:	toggle portc.2 : pause 50 : wend 
goto main

Fail2:
WHILE 1
	High Portc.2 : Pause 250 : Low PortC.2 : Pause 250
WEND

You're so close, I can smell it!
Or was that someone in the room?