Skip to content

Added GPIO_0 module and decimal delay support#7

Open
shubhi1407 wants to merge 7 commits into
deepakkarki:masterfrom
shubhi1407:master
Open

Added GPIO_0 module and decimal delay support#7
shubhi1407 wants to merge 7 commits into
deepakkarki:masterfrom
shubhi1407:master

Conversation

@shubhi1407
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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