[i2c] sda interference during host enable

lowRISC / opentitan

OpenTitan: Open source silicon root of trust

https://www.opentitan.org

Apache License 2.0

2.55k stars 759 forks source link

[i2c] sda interference during host enable #15131

Open tjaychen opened 2 years ago

tjaychen commented 2 years ago

the current i2c assumes that when the host function is enabled, if sda is already low, then it is registered as bus interference. @a-will pointed out that is not true in a multi-host situation.

Since the current i2c explicitly points out it does not support multi-host, this is not something that needs to be urgently fixed.

Filing this away as something to be improved in the future when multi-host is added.

tjaychen commented 1 year ago

since this is actually only needed for multi-host, which we do not support, i'm kicking this to backlog. Let me know if anyone objects.

msfschaffner commented 10 months ago

Multi-host is currently not planned for PROD, hence leaving this in the backlog.

CC @moidx @cfrantz

hcallahan-lowrisc commented 6 months ago

Out of scope for D2 signoff, part of D3.