SI7021 I2C pin setup (SOLVED)

tttadam

@robert-hh Okay, I changed the pins to 5 and 4. I am still getting the same error, but now it takes some time, like a second or so. Before that a I got the error immediately.

Traceback (most recent call last):
  File "main.py", line 16, in <module>
  File "/flash/lib/SI7021.py", line 31, in readTemp
  File "/flash/lib/SI7021.py", line 28, in write_command
OSError: I2C bus error

pin5 = Pin(Pin.exp_board.G5,mode= Pin.OPEN_DRAIN, pull= Pin.PULL_UP)
pin4 = Pin(Pin.exp_board.G4,mode= Pin.OPEN_DRAIN, pull= Pin.PULL_UP)
bus = I2C(0, I2C.MASTER, baudrate=100000, pins=(pin5,pin4))
# bus = I2C(0, I2C.MASTER, baudrate=100000, pins=("P8","P7"))
sensor = SI7021(i2c=bus)
print(sensor.readTemp())
print(sensor.readRH())

line16 is this: print(sensor.readTemp())

robert-hh

@tttadam G5 and G4 aka P13 and P14 are input only ports. I wonder that it does not give an error immedetaly when declaring. Why don't you use the default pins G16 and G17 aka "P9" for SDA and "P10" for SCL, at least for a first test?
Edit: You do not have to initialize the pins yourself. In the i2c constructor, you can write ... pins=("P9", "P10") or pins=("G16", "G17").

tttadam

@robert-hh Yes You are right, I should thought about this earlier.
So I modified my code like this. Something is still wrong, because I still get the same error.
My pins are correct, now but I took a picture just in case.

sensor = SI7021(i2c=I2C())
print(sensor.readTemp())
print(sensor.readRH())

layout

tttadam

@tttadam said in SI7021 I2C pin setup:

Looks like my picture didn't make it.
Here is the url: [https://imgur.com/a/j8fPQxZ](link url)

robert-hh

@tttadam According to the picture, your wiring seems wrong. You connected SDA to P10 and SCL to P9.

tttadam

@robert-hh I switched them back and for forth. Maybe a I took the picture on the wrong state but I tried both way (just to make sure).

robert-hh

@tttadam In order to test the communication, do some basics like:

from machine import I2C
i2c = I2C(0, I2C.MASTER)
i2c.scan()

That already should return the adress of the connected device. If that works, then you know at least that the I2C connection is fine and which address your device is using.

tttadam

@robert-hh said in SI7021 I2C pin setup:

i2c.scan()
This is what it returns:

[64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120]

Eric73

@tttadam Strange things, i have a look to your wire connexion, i hope my eyes are wrong but it seem you power your sensor with VIN (approx 5V), but your device SI7021 have +3.6V maximum power supply voltage so +5V can destroy your device.

If i take time to watch your wire it because typically on a scan i2c your device seem to ACK all address starting by 0x40 (64) and this can be happen in case of bad gnd connexion, please check your power supply +3.3V and GND for your SI7021.
If you have an oscilloscope for probin SDA and SCL this will be very helpfull.

robert-hh

@tttadam Definitely Vcc should be 3.3 V. And I cannot tell whether pull-up resistors are present on the board. Adding some with a value of about 4.7 - 10 kOhm should not hurt.

tttadam

@eric73 Well, you are right. I gave the sensor 5v instead of 3.3. I think I killed it.. well least a learned something. I also tried with 10k resistor but still getting I2C bus error...
Thanks for the help anyway.

robert-hh

@tttadam did you try swapping sda and scl again when running with 3.3v?

tttadam

@robert-hh Yes.

for i2c.scan() i get back this: [64], but I2C bus error for print(sensor.readTemp())

robert-hh

@tttadam So it means that at least the i2c protocol is now working. 64 or 0x40 ist the proper i2c address of the device. Try to reduce the baud rate.

tttadam

@robert-hh

I created a small code so it looped through all tha baudrates. Maybe a little overkill.
Off topic, but the code sometimes freezed for a half a minute or so. It was because I was created to much i2c object, and a garbage collector kicked in? Can I prevent that behavior?
Also If you can give any feedback for my code I would appreciate it very much.

Anyway this code was unsuccessful. I was unable to find a the correct baud rate, I got the same I2C bus error always.

import pycom
from machine import I2C, Pin
from SI7021 import SI7021
import time

pin9 = Pin("P9",mode= Pin.OPEN_DRAIN, pull= Pin.PULL_UP)
pin10 = Pin("P10",mode= Pin.OPEN_DRAIN, pull= Pin.PULL_UP)
expledpin = Pin('G16', mode=Pin.OUT)
start = True
buttonpin = Pin('G17', mode= Pin.IN, pull=Pin.PULL_UP)
while start:
    if buttonpin() == 0:
        start = False
    

with open("/sd/data.csv", 'a')as ds: 
    expledpin.value(1)
    for baud in range(1, 115200):
        pycom.rgbled(0x0000FF)
        bus = I2C(0, I2C.MASTER, baudrate=baud, pins=(pin9,pin10))
        # bus = I2C(0, I2C.MASTER, baudrate=100000, pins=("P8","P7"))
        sensor = SI7021(i2c=bus)
        try:
            print(sensor.readTemp()) 
            ds.write("\n"+str(sensor.readTemp())) 
            pycom.rgbled(0x00FF00)
        except OSError as er:
            pycom.rgbled(0xFF0000)
            expledpin.value(0)
            ds.write(str(repr(er))+str(baud))
            expledpin.value(1)

        time.sleep(0.001)
        pycom.rgbled(0x000000)
    
    ds.close()

robert-hh

@tttadam You do not have to re-create the i2c object. You can call i2c.init() to change the parameters. And I still do not know why you create the Pin objects separately. You just have to name them when creating the I2C object:

bus = I2C(0, I2C.MASTER, baudrate=baud, pins=("P9","P10"))
The init method of i2c will initialize the Pin objects.

tttadam

@robert-hh

Okay I modified the code as you recomended it.
But I got the same error on every baudrate.
Here is the csv, but nothing intresting in it: http://www.filedropper.com/data_11

import pycom
from machine import I2C, Pin
from SI7021 import SI7021
import time

expledpin = Pin('G16', mode=Pin.OUT)
buttonpin = Pin('G17', mode= Pin.IN, pull=Pin.PULL_UP)
i2c = I2C(0, pins=("P9","P10"))
start = True
while start:
    if buttonpin() == 0:
        start = False
    

with open("/sd/data.csv", 'a')as ds: 
    expledpin.value(1)
    for baud in range(1, 115200):
        pycom.rgbled(0x0000FF)
        i2c.init(I2C.MASTER, baudrate=baud)
        sensor = SI7021(i2c=i2c)
        try:
            print(sensor.readTemp()) 
            ds.write("\n\r;;"+str(sensor.readTemp())) 
            pycom.rgbled(0x00FF00)
        except OSError as er:
            pycom.rgbled(0xFF0000)
            expledpin.value(0)
            ds.write("\n\r"+str(repr(er))+";"+str(baud)+";")
            expledpin.value(1)

        time.sleep(0.001)
        pycom.rgbled(0x000000)
    
    ds.close()

robert-hh

@tttadam That#s strange, especially since i2c.scan() works, which also needs I2c communication. You could try to add an i2c.scan in the test after the i2c.init, just to confirm that it works. And yes, you do not need every baud rate. You can go back to a value like 100000.
I faintly recall a link to the SI7021 driver you are using, but cannot find it any more.

Eric73

@robert-hh link you looking for is in tilte https://forum.pycom.io/topic/648/si7021-i2c-sensor not ?

@ttadam Have you add pysical pull up resistor on SDA and SCL to 3.3V (4.7K or 1K value) ? Even if you set software pull up to the pin definitions, when you call i2c.init i dont know if pin your specify are not setup as pure opendrain by i2c library, so perhaps a test can with pull up can be done ?

tttadam

@eric73 I am using a 1k resistor both on SDA and SCL.
The program takes much more time to run because of the i2c.scan().... But I will write ehen I have the result.

Explore Pybytes | Official Documentation | Report a Firmware Bug/Issue | GitHub

SI7021 I2C pin setup (SOLVED)

Pycom on Twitter