Fix for non-root users & SPI #257
Merged
Conversation
This file contains 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
PandorasFox
changed the title
|
Would you mind making all the new mem file strings a define and then use that define in the code instead? I think that would be just a bit more clean. Otherwise, looks good. Thanks. |
|
Yeah, I'll do that in a bit tonight. Should I also add something to the README about needing to be in the |
also add a note to the readme
|
Should be good to go now. I used spaces for alignment in the header file, since that seems to be what's already been done. |
|
Great, thanks. You didn't use tabs in the code did you? I'd prefer to
keep everything as spaces if so. Thanks.
…On Thu, Jan 4, 2018 at 5:57 PM Chris Guillott ***@***.***> wrote:
Should be good to go now. I used spaces for alignment in the header file,
since that seems to be what's already been done.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#257 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIPemyePLCEqBZTHySYHYZhAGXgJ9Mxtks5tHXNkgaJpZM4RRTci>
.
|
|
No tabs in the code, yep. My editor usually warns about mixed whitespace and I didn't see it complaining, so it should be fine. |
|
Oh perfect, thanks. I'll see if I can get this pulled in the morning.
Thanks again for your contribution!
…On Thu, Jan 4, 2018 at 8:33 PM Chris Guillott ***@***.***> wrote:
No tabs in the code - I didn't add any lines aside from the two #defines,
so I only had to align inside those.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#257 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIPemxOqJvxH51N9Wxl65HNsh0uKyIu2ks5tHZgXgaJpZM4RRTci>
.
|
|
No problem, and thanks for the lib! It's pretty cool to have a strip sitting here that I'm controlling purely from userspace now :) |
brindosch
pushed a commit
to hyperion-project/rpi_ws281x
that referenced
this pull request
Apr 5, 2018
* Added delay to ensure reset time being met with low led counts (jgarff#157) * Fix includes as per jgarff#178 (jgarff#189) * clean up doco and strandtest (jgarff#187) * updated documentation on GPIO usage strandtest now defaults to GRB colour order added version.h to gitignore * updated python readme * more README cleanup * added notes on spi max transfer size * Fix freeze after running for long time (jgarff#197) * Whitespace cleanup * Move channel_protocol_time so it handles 32 bit Color LEDs. * Fix a freezing issue when running for a long time. When current_timestamp overflowed 2^64, rolling back to 0 and ws2811->render_wait_until had not, this caused an incredibly long usleep in ws2811_render. * Update README.md (jgarff#191) * Update README.md * updated spi buffer size location * Add -lrt link flag for systems with older glibc (jgarff#195) * Fix 32-bit overflow in get_microsecond_timestamp(). (jgarff#206) On Raspbian systems, the tv_sec member of struct timespec is 32 bits wide. get_microsecond_timestamp() was overflowing in 32-bit arithmetic on this member. This causes the time that it returns to jump ahead by many thousands of years at uptime = 35 minutes, 48 seconds. (While the starting point for CLOCK_MONOTONIC_RAW is unspecified, it seems to track system uptime in practice.) This overflow didn't cause noticeable problems for me--ws2811_render() skips the sleep when it thinks such a huge amount time has elapsed--but it could possibly result in rarely-occurring skipped frames if the WS2811 reset time isn't effectively honored by ws2811_render(). * Add TOOLCHAIN option for cross compilation (jgarff#213) Use this option if you want to cross-compile the library. Typical usage is: scons TOOLCHAIN=arm-linux-gnueabihf * Add -Wextra compile option and adapt code to suppress warnings (jgarff#214) The changes in code are mostly type casts between int ann unsigned. * Add shared library rasbpian package builder. The package is not built by default. To generate the package you need: - The dpkg-deb utility. - Use "scons deb" to generate the package. * Merge user-settable gamma correction into master (jgarff#216) Merge user-settable gamma correction into master * Bug fixes for Stretch and Compute Module support (jgarff#222) * Bug fixes Missing aligned(4) attribute in pcm.h makes the PCM case failing under Stretch, due to a newer version of gcc (6.x). Also fixed a bug in check_hwver_and_gpionum, so original Model A is recognised. * README, using SPI on RPi3 Added note to change the core frequency to 250 MHz when using SPI on RPi 3. * README fix * Add support for Compute Module 1 and 3 * Fix undefined clock_gettime symbol * add missing memory allocation for SPI mode (jgarff#231) * added Ctrl-C handler to turn off all LEDs on exit (jgarff#140) * added Ctrl-C handler to turn off all LEDs on exit * added option -c * Fix for segmentation fault on exit for Python API (jgarff#239) Remove useless __del__ because the cleanup() function gets called by atexit(). Remove ws.ws2811_fini() call which causes the SEGFAULT on exit. Python does not complain about the memory leak in this way. * Add support for new A+ hardware. * Change default DMA channel to safe choice (10) (jgarff#253) add warning on README.md * Add getBrightness for brightness feedback from the buffer (jgarff#248) * Instructions to install the Python library (jgarff#254) Solving issue jgarff#142 * Fix for non-root users & SPI (jgarff#257) * Fix for non-root users can use SPI * use DEFINE'd device strings also add a note to the readme * Basic Golang Examples (jgarff#256) * Added golang example * Added basic README for golang examples. Added os exits for errors during color wipes for golang examples * Fix the default DMA in python code (jgarff#266) DMA 5 causes file system errors on PI 3 * Fixed typo in README.md (jgarff#273) Was perusing recent commits and saw this. * clarify electrical notes (jgarff#264) * Fixed strandtest.py (jgarff#258) * Fixed keyboardinterrupt, indentation inconsistency, & a few other minor bugs. Added shebang. * Fixed KeyboardInterrupt. Again. * KeyboardInterrupt works now. For sure. * Tabs to spaces, removed redundant signal handling * Removed unused sys module import * Added Raspberry Pi 3 B+ (jgarff#276) * Added Pi 3 B+ * Remove unused import
|
Is this has been merged into the master branch yet? |
|
...Yes, that's why this pull says "merged". |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Non-root users can't access
/dev/mem, but users in thegpiogroup can access/dev/gpiomem, which is all that's needed for SPI.Existing functionality shouldn't be impacted, but this definitely does work on my machine with SPI now.