Skip to content

i2c: set hold time of SDA during transmit to an appropriate value #273

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.

Sign up for GitHub

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

Merged
Wren6991 merged 7 commits into from
Apr 6, 2021
Merged

i2c: set hold time of SDA during transmit to an appropriate value #273

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5b7f33e
i2c: set hold time of SDA during transmit to 2 for TCS34725 color sensor
fivdi Mar 20, 2021
2e5891f
i2c: fix issues in i2c_write_blocking_internal
fivdi Mar 29, 2021
2ab09b2
i2c: rename sda_hold_count to sda_tx_hold_count
fivdi Mar 29, 2021
aa96906
use assert rather than invalid_params_if for internal consistency checks
fivdi Mar 30, 2021
1c71ab6
i2c: use a more appropriate sda tx hold time at higher baudrates
fivdi Mar 30, 2021
5301647
i2c: reduce 120/1e9 to the smallest possible integer numerator and de…
fivdi Mar 31, 2021
ac986f0
Don't use implicit 0 for field LSB
Wren6991 Apr 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions src/rp2_common/hardware_i2c/i2c.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,10 @@ uint i2c_set_baudrate(i2c_inst_t *i2c, uint baudrate) {
i2c->hw->fs_scl_hcnt = hcnt;
i2c->hw->fs_scl_lcnt = lcnt;
i2c->hw->fs_spklen = lcnt < 16 ? 1 : lcnt / 16;
// Set hold time of SDA during transmit to 2 for TCS34725 color sensor
Comment thread
fivdi marked this conversation as resolved.
Outdated
i2c->hw->sda_hold =
I2C_IC_SDA_HOLD_IC_SDA_RX_HOLD_RESET << I2C_IC_SDA_HOLD_IC_SDA_RX_HOLD_LSB |
2 << I2C_IC_SDA_HOLD_IC_SDA_TX_HOLD_LSB;

i2c->hw->enable = 1;
return freq_in / period;
Expand Down