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

Contiki support for the TI CC32xx #1369

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

Conversation

bjoernrennfanz
Copy link

No description provided.

Björn Rennfanz and others added 30 commits December 19, 2014 12:06
…now configurable with contiki-conf.h.

Drivers for UART now fully implemented.
…compile without RTOS - Support. Work in progress on wireless lan driver of CC3200.
…XL headers. Fixed a bug in startup_gcc.c, that causes heavy stack usage. Fixed compile errors on wifi drivers.
…mmits)

Squashed commits:

[c8aacb6] Sollte so funktionieren... der LED-Treiber
… needed to be reviewed according to our Two-Man-Rule)
@alignan
Copy link
Member

alignan commented Nov 26, 2015

Great contribution!

I don't have any board to test with unfortunately, maybe @jonnteolsson could test

@jonnteolsson
Copy link
Contributor

@alignan please send me an email and I'll fix a board for you!

@tonnenpinguin
Copy link
Contributor

If I am not mistaken common practice is to not include ide-specific (project-) files.

@@ -38,8 +38,10 @@
*/

#include "contiki.h"
#include "dev/leds.h"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any specific reason to include dev/leds here? (or the newline in line 44)


while(1)
{
// Reset watchdog
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix code style here

@alignan
Copy link
Member

alignan commented Jun 6, 2016

I would also run uncrustify to fix code style

@bjoernrennfanz
Copy link
Author

Should i rebase the CC32xx code to the current master or do you that?

@alignan
Copy link
Member

alignan commented Jun 6, 2016

Why should I rebase this for you? be glad I'm reviewing! 😃

Rebase and fix conflicts and code style please.

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

Successfully merging this pull request may close these issues.

6 participants