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

Fixes issue #1533 on Contiki #50

Merged
merged 2 commits into from
Feb 25, 2016
Merged

Conversation

PalmaITEM
Copy link
Contributor

The issue in contiki-os/contiki#1533

Is solved by adding the delay in the "invoke_bootloader" after the entire invoking sequence.

The issue in contiki-os/contiki#1533

Is solved by adding the delay in the "invoke_bootloader" after the entire invoking sequence
@g-oikonomou
Copy link
Contributor

Sorry, I was perhaps not clear in the Contiki discussion.

We need both delays - the first one exists for different reasons

@alignan
Copy link

alignan commented Feb 25, 2016

Yes, we need both of them. I can confirm this works.

The explanation is simple: the on-board PIC detects the RTS/DTS sequence, waits a bit, then replicates onto the CC2538's PA3/RST. We need this delay that was introduced by the VM USB forwarding and later by the intelhex and python-magic.

Keep the original sleep as it was needed for different reasons.
@PalmaITEM
Copy link
Contributor Author

Understood, I fixed the pull. Sorry for mixing things.
(I didn't know if it was best just to close the request, I'll leave it to you)

g-oikonomou added a commit that referenced this pull request Feb 25, 2016
Fixes issue #1533 on Contiki
@g-oikonomou g-oikonomou merged commit 0ec4033 into JelmerT:master Feb 25, 2016
g-oikonomou added a commit that referenced this pull request Feb 25, 2016
@PalmaITEM PalmaITEM deleted the patch-1 branch February 25, 2016 19:40
@PalmaITEM
Copy link
Contributor Author

Hi @g-oikonomou ,
Hi @alignan ,

I have an update on this issue/pull request.

Using a fresh ubuntu machine (native, not a VM), the same problem appeared.

Solutions:

  • Increasing just slightly the sleeps (to 0.005 has worked flawlessly so far)
  • Keeping the sleeps as they were but installing python-magic improves but it still fails now and then (it seems non-deterministic and sometimes it even works without the second sleep, sometimes it doesn't, so I guess it's still needed)

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.

None yet

3 participants