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.

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

Jump to bottom

applet.selftest: fixes for revC0 and later #278

Closed
wants to merge 3 commits into from
Closed

applet.selftest: fixes for revC0 and later #278

wants to merge 3 commits into from

Conversation

TomKeddie
Copy link
Contributor

@whitequark
Copy link
Member

That doesn't actually fix the false positives.

@TomKeddie
Copy link
Contributor Author

Agreed, I'll read the pr more closely.

@TomKeddie
Copy link
Contributor Author

@whitequark I'm not sure exactly what more we can do here without more control over the ice40 pullups (do we have this now?).

pins-pull already does a walking ones test of the pullups.

I: g.cli: running handler for applet 'selftest'
I: g.applet.internal.selftest: running self-test mode pins-pull
D: g.applet.internal.selftest: pins-pull: oe=0000000000000000 o=0000000000000000 oe_pull=0000000000000000 o_pull=0000000000000000 i=0000000000000000
D: g.applet.internal.selftest: pins-pull: oe=1111111111111111 o=1111111111111111 oe_pull=1111111111111111 o_pull=1111111111111111 i=1111111111111111
D: g.applet.internal.selftest: pins-pull: oe=0000000000000001 o=0000000000000001 oe_pull=0000000000000001 o_pull=1111111111111111 i=0000000000000001
D: g.applet.internal.selftest: pins-pull: oe=0000000000000010 o=0000000000000010 oe_pull=0000000000000010 o_pull=1111111111111111 i=0000000000000010
D: g.applet.internal.selftest: pins-pull: oe=0000000000000100 o=0000000000000100 oe_pull=0000000000000100 o_pull=1111111111111111 i=0000000000000100
D: g.applet.internal.selftest: pins-pull: oe=0000000000001000 o=0000000000001000 oe_pull=0000000000001000 o_pull=1111111111111111 i=0000000000001000
D: g.applet.internal.selftest: pins-pull: oe=0000000000010000 o=0000000000010000 oe_pull=0000000000010000 o_pull=1111111111111111 i=0000000000010000
D: g.applet.internal.selftest: pins-pull: oe=0000000000100000 o=0000000000100000 oe_pull=0000000000100000 o_pull=1111111111111111 i=0000000000100000
D: g.applet.internal.selftest: pins-pull: oe=0000000001000000 o=0000000001000000 oe_pull=0000000001000000 o_pull=1111111111111111 i=0000000001000000
D: g.applet.internal.selftest: pins-pull: oe=0000000010000000 o=0000000010000000 oe_pull=0000000010000000 o_pull=1111111111111111 i=0000000010000000
D: g.applet.internal.selftest: pins-pull: oe=0000000100000000 o=0000000100000000 oe_pull=0000000100000000 o_pull=1111111111111111 i=0000000100000000
D: g.applet.internal.selftest: pins-pull: oe=0000001000000000 o=0000001000000000 oe_pull=0000001000000000 o_pull=1111111111111111 i=0000001000000000
D: g.applet.internal.selftest: pins-pull: oe=0000010000000000 o=0000010000000000 oe_pull=0000010000000000 o_pull=1111111111111111 i=0000010000000000
D: g.applet.internal.selftest: pins-pull: oe=0000100000000000 o=0000100000000000 oe_pull=0000100000000000 o_pull=1111111111111111 i=0000100000000000
D: g.applet.internal.selftest: pins-pull: oe=0001000000000000 o=0001000000000000 oe_pull=0001000000000000 o_pull=1111111111111111 i=0001000000000000
D: g.applet.internal.selftest: pins-pull: oe=0010000000000000 o=0010000000000000 oe_pull=0010000000000000 o_pull=1111111111111111 i=0010000000000000
D: g.applet.internal.selftest: pins-pull: oe=0100000000000000 o=0100000000000000 oe_pull=0100000000000000 o_pull=1111111111111111 i=0100000000000000
D: g.applet.internal.selftest: pins-pull: oe=1000000000000000 o=1000000000000000 oe_pull=1000000000000000 o_pull=1111111111111111 i=1000000000000000
I```

@whitequark
Copy link
Member

Measure capacitance by observing decay time. You are not required to implement this.

@TomKeddie
Copy link
Contributor Author

Ok, thanks - I'll just close this.

@TomKeddie TomKeddie closed this Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@TomKeddie @whitequark