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

PPP can't work on atm128-based platforms #32

Closed
tinyos-issues opened this issue Dec 21, 2012 · 3 comments
Closed

PPP can't work on atm128-based platforms #32

tinyos-issues opened this issue Dec 21, 2012 · 3 comments

Comments

@tinyos-issues
Copy link
Member

Original author: qy.n...@gmail.com (April 08, 2011 10:09:13)

What steps will reproduce the problem?

  1. apps that use ppp protocol such as 'tos/lib/ppp/tests/Ipv6' can't compile on atm128-based platforms(mica-serial,iris), due to the lack of AlarmMilli32C/CounterMilli32C/PlatformLedC/avr_stdio.h.
  2. On avr-based platforms, the mote resets when printf() is called, because the putchar() function in PppPrintfP only works for msp430.

What version of the product are you using? On what operating system?

tinyos-main svn r5536, IRIS platform, Linux

Please provide any additional information below.

I attached the patch which makes ppp works on IRIS as I tested, hope to solve the problem.

Original issue: http://code.google.com/p/tinyos-main/issues/detail?id=29

@tinyos-issues
Copy link
Member Author

From pabigot@gmail.com on April 17, 2011 01:04:30
I'm ok with the changes to PPP proper (viz., the init method), but would prefer that whoever's responsible for mica be the one who determines whether to add PlatformLed components to that platform. That can proceed without requiring anything in ppp.

I'll look into the printf stuff but don't believe things like avr_studio.h should be cloned into lib/ppp. It may be necessary to add -I$(TOSDIR)/lib/printf to the Makefiles.

@tinyos-issues
Copy link
Member Author

From pabigot@gmail.com on April 17, 2011 11:19:01
I've extracted the mica changes, and submitted those for somebody else to review as ticket 32.

I've refactored the printf infrastructure and provided a solution that seems to isolate platform-specific material to a single component in lib/printf. That's submitted for review as ticket 33. (It is possible that changes to that infrastructure will be required as I am unable to test it on the mica platform.)

Integration of those two patches should resolve this ticket.

@tinyos-issues
Copy link
Member Author

From pabigot@gmail.com on May 15, 2011 15:52:14
With completion of issue 33, I'm going to call this issue fixed. Actual completion should depend only on issue 32.

mmaroti added a commit to mmaroti/tinyos-main that referenced this issue Jan 22, 2013
…d solution,

since it creates a 32-bit milli timer running on the MCU clock. Somethin else
is needed for micaz and iris, but this is a stop gap solution.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant