-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
tests: i2c: i2c_target_api: added single mode testcase #72802
base: main
Are you sure you want to change the base?
tests: i2c: i2c_target_api: added single mode testcase #72802
Conversation
I would not recommend to enable test this way, as the board to board loopback testing is not that reliable, instead can you enable i2c_ram test through the ardiono_i2c? but this is not a blocking for this PR. so my recommandation would be to add the mimxrt1170_evk/mimxrt1176/cm7 and mimxrt1170_evk/mimxrt1176/cm4 in this PR as well. |
@CZKikin, please address your CI issues |
a bit premature on the +1, ci needs to be fixed
@CZKikin add mimxrt1170_evk/mimxrt1176/cm7 overlay, but do not remove the mimxrt1170_evk@B/mimxrt1176/cm7 |
ccafabc
to
1652832
Compare
Some drivers do not support dual mode testcase. It would be great to test both signle and dual mode, even when the drivers do support this feature. Signed-off-by: Krystof Sadlik <krystof.sadlik@nxp.com>
board testing is PASS for all supporting platforms, but need fix the CI issue @CZKikin |
1652832
to
d93911d
Compare
@CZKikin you can resolve the compliance error locally by amending your commit to use the same author name as the Signed-off entry, just run the following:
|
Some drivers do not support dual mode testcase. It would be great to test both signle and dual mode, even when the drivers do support this feature.