Align i2c with ack/busy/alert improvements

alex96295 commented 2 years ago

Overview

Align udma_subsystem with the improved capability of an i2c master device to receive an error (nack) from an i2c slave device after a write:

nack register is held high until further read and does not last for a clock cycle only
busy, alert and error logic is wired in udma_i2c (it was unconnected)
busy and alert receive the same priority of nack, i.e. they are kept at 1 until the nack is recognized, and an error is raised

This branch has been tested with the pulp-runtime based i2c master regression test in pulp-open master branch. Yet, the test still does not check for nack from a slave device; it has been executed to check against regression at the moment.
The ack check needs to be added in the sw drivers and properly tested

alex96295 commented 2 years ago

We remove WIP with the uda-i2c new version and I think we can merge

alex96295 commented 2 years ago

@bluewww I rebased and aligned with the new version of udma_i2c, released last week with i2c fixes

We are now able to merge this