Pages: [1]   Go Down
Author Topic: Arduino Official Wifi shield on Arduino 328 - Multibyte write and close issues  (Read 2068 times)
0 Members and 1 Guest are viewing this topic.
Offline Offline
Newbie
*
Karma: 0
Posts: 46
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Hello,

I have been having issues converting over a sketch from the ethernet shield over to the wifi shield. Particularly with multibyte writes and closing the connection. I modified the WebServer sketch which exposes these issues(see below). I suspect that they are due to some issues with the SPI driver. If you stay under about 50 bytes on the multibyte write you are fine. If you goto 90 you might be OK. If you goto 100 bytes forget about it. Size of the file doesn't seem to be a factor. As for the connection, sometimes it closes properly, sometimes it closes too soon, sometimes it doesn't close at all. Arduino could you fix these issues? Also, could you give people a way to identify the connection(in the ethernet an wifi libraries), specifically the remoteport?

Thanks,

-diesel



/*
  Web  Server
 
 A simple web server that shows the value of the analog input pins.
 using a WiFi shield.
 
 This example is written for a network using WPA encryption. For
 WEP or WPA, change the Wifi.begin() call accordingly.
 
 Circuit:
 * WiFi shield attached
 * Analog inputs attached to pins A0 through A5 (optional)
 
 created 13 July 2010
 by dlf (Metodo2 srl)
 modified 31 May 2012
 by Tom Igoe
 */
#include <SPI.h>
#include <WiFi.h>
#include <SdFat.h>

char ssid[] = "ssid";      //  your network SSID (name)
char key[] = "key";   // your network password
int keyIndex = 1;                 // your network key Index number (needed only for WEP)

int status = WL_IDLE_STATUS;

WiFiServer server(80);

SdFat sd;
SdFile file;
uint8_t buf[201];

void setup() {
  pinMode(10,OUTPUT);
  digitalWrite(10,HIGH);
  sd.init(SPI_FULL_SPEED,4);
  //Initialize serial and wait for port to open:
  Serial.begin(9600);
  while (!Serial) {
    ; // wait for serial port to connect. Needed for Leonardo only
  }
  
  // check for the presence of the shield:
  if (WiFi.status() == WL_NO_SHIELD) {
    Serial.println("WiFi shield not present");
    // don't continue:
    while(true);
  }
  
  // attempt to connect to Wifi network:
  while ( status != WL_CONNECTED) {
    Serial.print("Attempting to connect to SSID: ");
    Serial.println(ssid);
    // Connect to WPA/WPA2 network. Change this line if using open or WEP network:    
    //status = WiFi.begin(ssid, pass);
status = WiFi.begin(ssid, keyIndex, key);
    // wait 10 seconds for connection:
    delay(10000);
  }
  server.begin();
  // you're connected now, so print out the status:
  printWifiStatus();
}


void loop() {
  // listen for incoming clients
  WiFiClient client = server.available();
  if (client) {
    Serial.println("new client");
    // an http request ends with a blank line
    boolean currentLineIsBlank = true;
    while (client.connected()) {
      if (client.available()) {
        char c = client.read();
        Serial.write(c);
        // if you've gotten to the end of the line (received a newline
        // character) and the line is blank, the http request has ended,
        // so you can send a reply
        if (c == '\n' && currentLineIsBlank) {
          
          // send a standard http response header
          client.println("HTTP/1.1 200 OK");
          client.println("Content-Type: text/html");
          //client.println("Connnection: close");
          client.println();
          client.println();
          
          pinMode(10,OUTPUT);
          digitalWrite(10,HIGH);
          sd.chdir("/");
          file.open("page.htm",O_READ); ///page file was a basic html file with lorem ipsum, 1kb-4kb
          uint16_t x = file.fileSize();
          uint16_t y=100;  /// *********************************************bytes to write each time thru loop
          while(x){
          if(x<y) {y=x;}
          pinMode(10,OUTPUT);
          digitalWrite(10,HIGH);
          file.read(buf,y);    
          pinMode(4,OUTPUT);
          digitalWrite(4,HIGH);
          client.write(buf,y);///**********************************************************problem
          x-=y;
          Serial.write(buf,y);///**********************************************************debugging
          Serial.print("x=");
          Serial.println(x,DEC);
          }
          file.close();
          //client.println("<!DOCTYPE HTML>");
          //client.println("<html>");
          //client.println("</html>");
           break;
        }
        if (c == '\n') {
          // you're starting a new line
          currentLineIsBlank = true;
        }
        else if (c != '\r') {
          // you've gotten a character on the current line
          currentLineIsBlank = false;
        }
      }
    }
    // give the web browser time to receive the data
    delay(1);
      // close the connection:
      client.stop();
      Serial.println("client disonnected");
  }
}


void printWifiStatus() {
  // print the SSID of the network you're attached to:
  Serial.print("SSID: ");
  Serial.println(WiFi.SSID());

  // print your WiFi shield's IP address:
  IPAddress ip = WiFi.localIP();
  Serial.print("IP Address: ");
  Serial.println(ip);

  // print the received signal strength:
  long rssi = WiFi.RSSI();
  Serial.print("signal strength (RSSI):");
  Serial.print(rssi);
  Serial.println(" dBm");
}
« Last Edit: October 08, 2012, 07:21:52 pm by diesel » Logged

Seattle, WA USA
Offline Offline
Brattain Member
*****
Karma: 601
Posts: 48556
Seattle, WA USA
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Code:
          pinMode(10,OUTPUT);
You should NOT be diddling with the mode of pin 10 in loop. It MUST be an output pin at all times.

The SD and WiFi libraries should take care of activating the correct slave select pin, so there is no reason for you to be diddling with pin 10 at all. Pin 4, too.

Which WiFi library are you using?
Logged

Offline Offline
Newbie
*
Karma: 0
Posts: 46
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Paul,

I removed the additional digitalWrite and pinmode commands, same results. I downloaded the WiFi code from the github repository https://github.com/arduino/wifishield/tree/master/libraries. I tried sketch on ides 1.0 and 1.01 with win xp mach. Have you tried this sketch, does it work for you? I have noticed that this sketch seems to work best with chrome on ff it doesn't seem to catch the end of the request.

I would try uploading the firmware, but I am unsure how to do so. I read online.... "we made updating its firmware very easy, just connect a usb cable to the shield". So, what is the rest of the process?

I noticed that I had a couple misprints on my previous post. Under setup, instead of digitalWrite(10, OUTPUT), it should be digitalWrite(10, HIGH) and the sketch was the WebServer sketch not the WEP sketch. I edited the post with these changes.

I don't think this has anything to do with this, but I noticed that the MAC address is incorrectly labeled on the board.

Thanks,

-diesel
« Last Edit: October 08, 2012, 07:16:10 pm by diesel » Logged

Seattle, WA USA
Offline Offline
Brattain Member
*****
Karma: 601
Posts: 48556
Seattle, WA USA
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Quote
I don't this has anything to do with this, but I noticed that the MAC address is incorrectly labeled on the board.
What does this mean? The MAC address on the sticker on the board is the address you are supposed to put in the sketch.

Code:
          //client.println("<!DOCTYPE HTML>");
         //client.println("<html>");
          // add a meta refresh tag, so the browser pulls again every 5 seconds:
          //client.println("<meta http-equiv=\"refresh\" content=\"5\">");
          // output the value of each analog input pin
          //for (int analogChannel = 0; analogChannel < 6; analogChannel++) {
          //  int sensorReading = analogRead(analogChannel);
          //  client.print("analog input ");
          //  client.print(analogChannel);
         //   client.print(" is ");
          //  client.print(sensorReading);
         //   client.println("<br />");     
         // }
          //client.println("</html>");
Is it really necessary to post all this commented out code? Delete it!
Logged

Offline Offline
Newbie
*
Karma: 0
Posts: 1
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

I'm having the same problem with close issues. I'm working with a sketch that acts as a telnet server to implement a CLI to control a roomba. The WiFi shield closes the TCP connection after about 20 seconds of inactivity, the ethernet shield does not.

The code at GitHub doesn't show a version number, so I guess I'll have to really dig to see if upgrading firmware (nowhere near as easy as advertised) will help. The documentation is pretty sparse in many respects.
Logged

Offline Offline
Newbie
*
Karma: 0
Posts: 46
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Paul,

I removed most of the commented lines. You use the mac address on the ethernet shield not on the wifi shield.

Thanks,

-diesel
Logged

0
Offline Offline
Newbie
*
Karma: 0
Posts: 20
Arduino rocks
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Hey,

+1 for me - same here with an UNO R3.
Small files are fine - bigger files are a problem... No chance to get the write to SD Card.

Any news about that?

For the firmware update on the wifi shield this should help you: https://github.com/arduino/wifishield/blob/master/firmware/scripts/ArduinoWifiShield_upgrade.sh

best regards
-andreas
Logged

Offline Offline
Newbie
*
Karma: 0
Posts: 46
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

Andreas,

Yes, there is an issue for sure. When I reduced the bytes per write(50 or less) it seemed to work better.  I tried a range of file sizes, 1kb-~20kb, in my testing, it didn't seem to effect the results.

I didn't try writing, so no comment on that.

Thanks for the link, but still not sure of the process on updating. Do you use avrdude to compile and upload? Did you update the firmware and still have problems? If you did, I will wait to for the Arduino guys to update the firmware.

Thanks,

-diesel
Logged

Oslo, Norway
Offline Offline
Newbie
*
Karma: 0
Posts: 10
@jenschr
View Profile
WWW
 Bigger Bigger  Smaller Smaller  Reset Reset

@diesel look here for details on updating? http://arduino.cc/forum/index.php/topic,130124.msg980076.html#msg980076
Logged

Offline Offline
Newbie
*
Karma: 0
Posts: 46
View Profile
 Bigger Bigger  Smaller Smaller  Reset Reset

jensa,

Thanks for the link on updating the firmware. Did the update help with communication issues with the shield?

Thanks,

-diesel
Logged

Pages: [1]   Go Up
Jump to: