Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Si5324 I2C low level is too high #68

Closed
marmeladapk opened this issue Feb 3, 2020 · 6 comments
Closed

Si5324 I2C low level is too high #68

marmeladapk opened this issue Feb 3, 2020 · 6 comments
Milestone

Comments

@marmeladapk
Copy link
Member

Si5324 seems not to be able to properly pull SDA low. Tested with FPGA and with MMC talking to Si5324. Other than that Si5324 is reacting properly.

We modified pullup resistors on both sides to 10k. It didn't help.

Screenshot on CPU pullup resistors:

tek00013

Screenshot on I2CMUX4 bus:

tek00012

@gkasprow
Copy link
Member

gkasprow commented Feb 3, 2020

It looks like one of the ICs was switched to fast mode (push-pull IOs) or MMC has wrong IO mode ( push-pull enabled )

@gkasprow
Copy link
Member

gkasprow commented Feb 3, 2020

observe SDA before and after the I2C mux

@jbqubit jbqubit added this to the v1.0 milestone Feb 3, 2020
@kaolpr
Copy link
Member

kaolpr commented Feb 4, 2020

Si5324 seems not to be able to properly pull SDA low. Tested with FPGA and with MMC talking to Si5324. Other than that Si5324 is reacting properly.

We modified pullup resistors on both sides to 10k. It didn't help.

Screenshot on CPU pullup resistors:

tek00013

Screenshot on I2CMUX4 bus:

tek00012

@gkasprow These are signals before mux (CPU pullup resistors) and after (I2CMUX4).

@marmeladapk
Copy link
Member Author

I shorted FPGA bus with Si5324 bus, ACK is still >1V. Both muxes are held in reset.

@marmeladapk
Copy link
Member Author

marmeladapk commented Feb 4, 2020

Just as a precaution I removed muxes and left only 10kOhm pullup (one per line). Still the same.

Hmm, low level on SCL is quite high - around 800mV.
image

@marmeladapk
Copy link
Member Author

Found it!

FPGA I2C bus is also connected to the CPU (LPC1776).

CPU was configured differently than on Sayma: IOCON_MODE_INACT on Metlino, IOCON_OPENDRAIN_EN in Sayma. After changing this, Si5324 works in ARTIQ as @sbourdeauducq tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants