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.

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

Added GPIO_0 module and decimal delay support #7

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

shubhi1407
Copy link

Corrected previous errors.

  1. Indentation and proper coding style according to repo
  2. All directories clean
  3. Only 4 files changed.

@shubhi1407 shubhi1407 changed the title Added GPIO_0 module support Added GPIO_0 module and decimal delay support Mar 30, 2015
@shubhi1407
Copy link
Author

I have added uS delay support to the exisiting WAIT instruction [ no need for WAITuS :) ]
What has been done :

  1. Add regex to tokenize float ie WAIT 26.345 becomes token ['WAIT', 'INT', 'FLOAT']
  2. Parse WAIT 26.345 to 64 bit instruction with OPCODE 23. lower two bytes of second word has uS delay data ie 345 uS. ( WAIT 26 would have been parsed to usual 32 bit instruction ie no change would have occured )
  3. Modify pru0 firmware to handle uS delays.

In action 👍

img_0170
img_0171
img_0172
img_0173

@jadonk
Copy link
Contributor

jadonk commented Mar 30, 2015

This looks pretty good to me. Can we get a top-level test bench that gives some reasonable code-coverage?

@shubhi1407
Copy link
Author

@jadonk Sorry, but what exactly does that mean ?
More examples to test nothing is broken ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants