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

Add support for B+ #48

Closed
wants to merge 3 commits into from
Closed

Add support for B+ #48

wants to merge 3 commits into from

Conversation

Kixunil
Copy link

@Kixunil Kixunil commented Mar 31, 2015

No description provided.

@richardghirst
Copy link
Owner

Thanks!

Note this patch seems to remove the ability for it to work on 'rev 2'
boards (board_rev() can now only return 0, 1, or 3). I have changes
pending in this area for Pi2B, so board_rev() is going to change anyway (so
don't worry about updating your patch).

On Tue, Mar 31, 2015 at 8:50 AM, Martin Habovštiak <[email protected]

wrote:


You can view, comment on, or merge this pull request online at:

#48
Commit Summary

  • Added support for B+

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#48.

@Kixunil
Copy link
Author

Kixunil commented Apr 2, 2015

YAW, I forgot the last "else". Fixed it now.

@richardghirst
Copy link
Owner

Martin, it took me a long time, but I eventually integrated B+ support based on your patch, thanks!

@Kixunil
Copy link
Author

Kixunil commented Oct 6, 2015

Thank you! BTW did you try it for Pi2? Last time I tried (before your commit; didn't have time to try again), it froze.

@richardghirst
Copy link
Owner

It should work fine with the Pi2 now - it does for me, anyway.

On Tue, Oct 6, 2015 at 4:30 PM, Martin Habovštiak [email protected]
wrote:

Thank you! BTW did you try it for Pi2? Last time I tried (before your
commit; didn't have time to try again), it froze.


Reply to this email directly or view it on GitHub
#48 (comment).

@Kixunil Kixunil closed this Dec 8, 2015
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