Add support for the two new 4 tile mic array boards#21
Open
mbruno-xmos wants to merge 8 commits intoxmos:masterfrom
Open
Add support for the two new 4 tile mic array boards#21mbruno-xmos wants to merge 8 commits intoxmos:masterfrom
mbruno-xmos wants to merge 8 commits intoxmos:masterfrom
Conversation
Added wscript
* xmos/master: Added waf build script cleaned up some of the mic_input test code added a README to mic_input test and added additional functionality changed the mic_input test to report abs max diff in gain as the result rather then something else. added precision improvement along with overflow protection. added negative correlation detection to mic_array switched mic_input test condition to mmse from rms error lots of maths improvements to mic_input test plus added script to run it added new xn for mic_input and removed the xscope output update to mic_input test
…of 10's of nanoseconds (#3) The mabs button and LED server was updating the LEDs every MIN_POLL_TIME_US reference clock ticks. But, the macro name implies that it should be updating the LEDs every MIN_POLL_TIME_US microseconds, rather than reference clock ticks. So, this fixes that. This also fixes an issue with the BGA167 boards where the MABS server is broken when the core clock is 500 MHz, because the timer is firing so rapidly it's never able to service any other events.
pthedinger
suggested changes
Jul 3, 2018
Contributor
pthedinger
left a comment
pthedinger
left a comment
There was a problem hiding this comment.
Generally looks fine to me. Just need to update the CHANGELOG to keep that in line with the version number.
| p_buttons :> button_val; | ||
| while(1){ | ||
| // #pragma ordered | ||
| //[[ordered]] |
Contributor
There was a problem hiding this comment.
If this is commented out it seems better to simply delete it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.