Skip to content

Conversation

arve0
Copy link
Contributor

@arve0 arve0 commented Oct 9, 2014

This will allow restarting bridge connection. Hack for #2028.

if (Bridge.isOk() != true) {
  Serial.println("waiting..");
  delay(60000); // hope for ar9331 booting
}
// restart bridge
Bridge.started = false;
Bridge.begin();

@cmaglie cmaglie added Library: Bridge The Bridge Arduino library Type: Bug feature request A request to make an enhancement (not a bug fix) labels Oct 10, 2014
@facchinm
Copy link
Member

As in #2781, I did not merge this PR because it's more like an hack than a real fix.
In fact, the "XX" command you are sending triggers this function (https://github.com/arduino/YunBridge/blob/master/bridge/packet.py#L98) which launches usr/bin/blink-start (unwanted) and returns 2.
A real "ping" function would be a good improvement but must be implemented by both side of the Bridge; on the python side it will be something like

self.processor.register('Q', PING_Command(self))

I compiled a list of commands already in use https://gist.github.com/facchinm/569c1c8d112d28d8ff73 so if you want to update your merge request dividing it in two parts (one here and one for YunBridge) while keeping the same functionality it would be great!

@facchinm facchinm added Waiting for feedback More information must be provided before we can proceed and removed Type: Bug labels Mar 17, 2015
@facchinm facchinm closed this Mar 17, 2015
@arve0
Copy link
Contributor Author

arve0 commented Mar 17, 2015

Thanks for the feedback. I might look into creating a correct ping, if I find some time.

facchinm referenced this pull request in facchinm/Arduino Mar 17, 2015
to be used in conjuntion with "Bridge: add PING-like function"
in YunBridge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request A request to make an enhancement (not a bug fix) Library: Bridge The Bridge Arduino library Waiting for feedback More information must be provided before we can proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants