Go Down

Topic: DS18B20 auf Atmega8 (Read 8703 times) previous topic - next topic

skorpi080

Ziel ist es, den DS18B20 auf einem Atmega 8 auslesen und den Wert per Modbus zu senden.
Hat hier jemand ein Sketch, der unter 4000 Bytes ist?
Hab Dallas Lib alle Beispiele geschaut, passen grad noch so auf den Atmega8, ich muss aber noch einige Taster auswerten und die Modbus Lib ist auch noch drauf.
Momentan ohne den DS18B20 hab ich 4kB.
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

Peter_n

When all the code is in the sketch, what was the total size after compiling ?

The new Arduino IDE 1.5.7 BETA has a new compiler. I was able to shrink the code by 25% with an extra compiler flag.

skorpi080

Ich teste mit diesem Sketch:

Code: [Select]
#include <OneWire.h>
OneWire  ds(10);  // on pin 10

void setup(void) {
  Serial.begin(9600);
}

void loop(void) {
  byte i;
  byte present = 0;
  byte type_s;
  byte data[12];
  byte addr[8];
  float celsius, fahrenheit;
 
  if ( !ds.search(addr)) {
    Serial.println("No more addresses.");
    Serial.println();
    ds.reset_search();
    delay(250);
    return;
  }
 
  Serial.print("ROM =");
  for( i = 0; i < 8; i++) {
    Serial.write(' ');
    Serial.print(addr[i], HEX);
  }

  if (OneWire::crc8(addr, 7) != addr[7]) {
      Serial.println("CRC is not valid!");
      return;
  }
  Serial.println();

  // the first ROM byte indicates which chip
  switch (addr[0]) {
    case 0x10:
      Serial.println("  Chip = DS18S20");  // or old DS1820
      type_s = 1;
      break;
    case 0x28:
      Serial.println("  Chip = DS18B20");
      type_s = 0;
      break;
    case 0x22:
      Serial.println("  Chip = DS1822");
      type_s = 0;
      break;
    default:
      Serial.println("Device is not a DS18x20 family device.");
      return;
  }

  ds.reset();
  ds.select(addr);
  ds.write(0x44,1);         // start conversion, with parasite power on at the end
 
  delay(1000);     // maybe 750ms is enough, maybe not
  // we might do a ds.depower() here, but the reset will take care of it.
 
  present = ds.reset();
  ds.select(addr);   
  ds.write(0xBE);         // Read Scratchpad

  Serial.print("  Data = ");
  Serial.print(present,HEX);
  Serial.print(" ");
  for ( i = 0; i < 9; i++) {           // we need 9 bytes
    data[i] = ds.read();
    Serial.print(data[i], HEX);
    Serial.print(" ");
  }
  Serial.print(" CRC=");
  Serial.print(OneWire::crc8(data, 8), HEX);
  Serial.println();

  // convert the data to actual temperature

  unsigned int raw = (data[1] << 8) | data[0];
  if (type_s) {
    raw = raw << 3; // 9 bit resolution default
    if (data[7] == 0x10) {
      // count remain gives full 12 bit resolution
      raw = (raw & 0xFFF0) + 12 - data[6];
    }
  } else {
    byte cfg = (data[4] & 0x60);
    if (cfg == 0x00) raw = raw << 3;  // 9 bit resolution, 93.75 ms
    else if (cfg == 0x20) raw = raw << 2; // 10 bit res, 187.5 ms
    else if (cfg == 0x40) raw = raw << 1; // 11 bit res, 375 ms
    // default is 12 bit resolution, 750 ms conversion time
  }
  celsius = (float)raw / 16.0;
// fahrenheit = celsius * 1.8 + 32.0;
// Serial.print("  Temperature = ");
// Serial.print(celsius);
//  Serial.print(" Celsius, ");
// Serial.print(fahrenheit);
// Serial.println(" Fahrenheit");
}


Was kann da noch gelöscht werden?
Da wird die Adresse vom DS18B20 ermittelt, ich würde die auch selber eintragen, wenns viel speicher einnimmt.
Leider kenne ich mich mit dem nicht aus.
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs


Was kann da noch gelöscht werden?


Sobald Du in Deinem Sketch eine einzige Gleitkommeberechnung drin hast, wird eine ca. 1,4 KB große Gleitkomma-Library zu Deinem Programm dazugelinkt.

Wenn Du in einem Programm ohne Gleitkommarechnungen auskommst, wird das Programm gleich ca. 1,4 KB kleiner.

Temperaturen könnte man z.B. statt als Gleitkommazahl als Integerwert behandeln, z.B. als ganze Zahl "Temperatur in Zehntelgrad" oder bei höheren Anforderungen als ganze Zahl "Temperatur in Hundertstelgrad".

Dein Temperatursensor macht das intern ja auch: Seine Temperaturen sind intern dargestellt nur 9, 10, 11 oder 12 einzelne Bits und keine Gleitkommazahl.

skorpi080

#4
Sep 21, 2014, 03:40 pm Last Edit: Sep 21, 2014, 03:51 pm by skorpi08 Reason: 1
Mir würde auch reichen wenn es ohne komma wäre.
Also ganze Zahl, 19,5 Grad == 195 oder 1950

Hier wäre der ganze Code, für Atmega8:

3.762 Bytes ohne DS18B20
5.060 Bytes mit DS18B20
Mit Arduino IDE 1.5.7:
Der Sketch verwendet 4.754 Bytes (61%) des Programmspeicherplatzes. Das Maximum sind 7.680 Bytes.
Globale Variablen verwenden 276 Bytes des dynamischen Speichers.

Code: [Select]
#include <SimpleModbusSlave.h>
#include <OneWire.h>
/* The modbus_update() method updates the holdingRegs register array and checks communication.

   Note: 
   The Arduino serial ring buffer is 64 bytes or 32 registers.
   Most of the time you will connect the arduino to a master via serial
   using a MAX485 or similar.

   In a function 3 request the master will attempt to read from your
   slave and since 5 bytes is already used for ID, FUNCTION, NO OF BYTES
   and two BYTES CRC the master can only request 58 bytes or 29 registers.

   In a function 6 request the master will attempt to write to your
   slave and since a 9 bytes is already used for ID, FUNCTION, ADDRESS,
   NO OF REGISTERS, NO OF BYTES and two BYTES CRC the master can only write
   54 bytes or 27 registers.

   Using a USB to Serial converter the maximum bytes you can send is
   limited to its internal buffer which differs between manufactures.
*/
/***************************************************
* / Input/output pins
/***************************************************/

#define PIN_TASTER       9
OneWire  ds(A3);  // on pin 10

#define ON  8
#define OFF 7

#define BUTTON_DEBOUNCE_DELAY 200
long lastDebounceTime = 0;

/***************************************************
* / Modbus globals
/***************************************************/
#define MB_TXENPIN   A2
#define MB_SLAVE_ID  1
#define MB_BAUDRATE  9600
#define MB_PARITY  SERIAL_8N2
/*   Valid modbus byte formats are:
     SERIAL_8N2: 1 start bit, 8 data bits, 2 stop bits
     SERIAL_8E1: 1 start bit, 8 data bits, 1 Even parity bit, 1 stop bit
     SERIAL_8O1: 1 start bit, 8 data bits, 1 Odd parity bit, 1 stop bit
*/

//////////////// registers of your slave ///////////////////
enum
{     
  // just add or remove registers and your good to go...
  // The first register starts at address 0
  celsius,
  Licht,       
  HOLDING_REGS_SIZE // leave this one
  // total number of registers for function 3 and 6 share the same register array
  // i.e. the same address space
};

unsigned int holdingRegs[HOLDING_REGS_SIZE]; // function 3 and 6 register array
////////////////////////////////////////////////////////////

void setup()
{
    // configure pins
  pinMode(PIN_TASTER, INPUT);
  pinMode(ON, OUTPUT); 
  pinMode(OFF, OUTPUT);
 
  modbus_configure(&Serial, MB_BAUDRATE, MB_PARITY, MB_SLAVE_ID, MB_TXENPIN, HOLDING_REGS_SIZE, holdingRegs);
  // modbus_update_comms(baud, byteFormat, id) is not needed but allows for easy update of the
  // port variables and slave id dynamically in any function.
  modbus_update_comms(MB_BAUDRATE, MB_PARITY, MB_SLAVE_ID);

}

void loop()
{

byte i;
  byte present = 0;
  byte type_s =0;
  byte data[12];
  byte addr[8] = {0x28, 0x8F, 0x34, 0x85, 0x04, 0x00, 0x00, 0x8D};
  float celsius, fahrenheit;


  ds.reset();
  ds.select(addr);
  ds.write(0x44,1);         // start conversion, with parasite power on at the end
 
  delay(1000);     // maybe 750ms is enough, maybe not
  // we might do a ds.depower() here, but the reset will take care of it.
 
present = ds.reset();
  ds.select(addr);   
  ds.write(0xBE);         // Read Scratchpad

  Serial.print("  Data = ");
  Serial.print(present,HEX);
  Serial.print(" ");
  for ( i = 0; i < 9; i++) {           // we need 9 bytes
    data[i] = ds.read();
    Serial.print(data[i], HEX);
    Serial.print(" ");
  }

  // convert the data to actual temperature

  unsigned int raw = (data[1] << 8) | data[0];
  if (type_s) {
    raw = raw << 3; // 9 bit resolution default
    if (data[7] == 0x10) {
      // count remain gives full 12 bit resolution
      raw = (raw & 0xFFF0) + 12 - data[6];
    }
  } else {
    byte cfg = (data[4] & 0x60);
    if (cfg == 0x00) raw = raw << 3;  // 9 bit resolution, 93.75 ms
    else if (cfg == 0x20) raw = raw << 2; // 10 bit res, 187.5 ms
    else if (cfg == 0x40) raw = raw << 1; // 11 bit res, 375 ms
    // default is 12 bit resolution, 750 ms conversion time
  }
  celsius = (float)raw / 16.0;
 
 
  // modbus_update() is the only method used in loop(). It returns the total error
  // count since the slave started. You don't have to use it but it's useful
  // for fault finding by the modbus master.
    modbus_update();
 
   if ((digitalRead(PIN_TASTER) == HIGH) && ((millis() - lastDebounceTime) > BUTTON_DEBOUNCE_DELAY))
  {
    // invert LED state
    holdingRegs[Licht] = !holdingRegs[Licht];

    lastDebounceTime = millis();
  }

    if (holdingRegs[Licht]==1){
      // ON //
  digitalWrite(ON,1);
  digitalWrite(OFF,0);
  delay(10);
  digitalWrite(ON,0);
  digitalWrite(OFF,0);
}
  else{
    // OFF //
  digitalWrite(ON,0);
  digitalWrite(OFF,1);
  delay(10);
  digitalWrite(ON,0);
  digitalWrite(OFF,0);
}

}



Da ist immer noch mit Serial.print, wie kann man es ohne dem machen?
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs


Mir würde auch reichen wenn es ohne komma wäre.
Also also ganze Zahl, 19,5 Grad == 195 oder 1950


Also ich sehe nur eine einzige Gleitkommadivision in Deinem Code:
Code: [Select]
celsius = (float)raw / 16.0;

Deine Division durch 16 ist quasi eine Multiplikation mit 0.0625, weil 1/16=0.0625

Daher kannst Du die Temperatur ganz einfach als "long" Wert in Zehntausendstel Grad bekommen, wenn Du Deinen raw-Wert mit 625 multiplizierst.

Code: [Select]
long celsius10000= raw*625L;

Also 19,5°C wären dann 195000 als long.
Mit der richtigen Rundung und einer Integer-Division durch ein Vielfaches von 10 kannst Du die Zehntausendstel Grad dann auch wieder runterrechnen auf einen Integer in Zehntelgrad oder Hundertstelgrad. Und das Programm sollte deutlich kleiner als vorher sein, weil die Math-Library für Gleitkommaarithmetik nicht mit eingebunden werden braucht.

skorpi080

Auch wenn ich
Code: [Select]
float celsius;
celsius = (float)raw / 16.0;


auskommentiere, bleibts bei 5060 Bytes

Mit der Arduino IDE 1.5.7 kann ich wohl nicht über USBasp uploaden.
Steht als Fehler:
Quote
Fehler beim Hochladen: Fehlender Konfigurationsparameter 'upload.tool'
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs


Auch wenn ich
Code: [Select]
float celsius;
celsius = (float)raw / 16.0;


auskommentiere, bleibts bei 5060 Bytes


Du bindest ja auch noch eine Third-Party Magic-Library ein:
Code: [Select]
#include <SimpleModbusSlave.h>
Wie sieht es denn in dieser Library mit Gleitkommazahlen aus?

skorpi080

Gute Frage, hab mich damit nicht befasst.
Dann wirds wohl schon da genutzt.

Code: [Select]
  delay(1000);     // maybe 750ms is enough, maybe not
  // we might do a ds.depower() here, but the reset will take care of it.
 
present = ds.reset();
  ds.select(addr);   
  ds.write(0xBE);         // Read Scratchpad

  Serial.print("  Data = ");
  Serial.print(present,HEX);
  Serial.print(" ");
  for ( i = 0; i < 9; i++) {           // we need 9 bytes
    data[i] = ds.read();
    Serial.print(data[i], HEX);
    Serial.print(" ");
  }


kann man es nicht ohne delay machen?
Wozu ist present? Habs mal gelöscht, wird aber wohl für noch irgendwas gebraucht. Dacht ist einfach nur ne anzeige für   Serial.print("  Data = ");.
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs

#9
Sep 21, 2014, 04:29 pm Last Edit: Sep 21, 2014, 04:54 pm by jurs Reason: 1

kann man es nicht ohne delay machen?


Natürlich kannst Du auch einen DS18B20 auch ohne delay abfragen.

Aber Du mußt ihm trotzdem Zeit zum Messen geben.
Mindestens 750 Millisekunden pro Messung (oder etwas mehr).

Dazu müßtest Du den Messvorgang dann in zwei Vorgänge aufteilen:
1.) Messung vom Sensor anfordern
2.) Nach frühestens 750 ms (oder etwas mehr): Messwert abholen

Was Du in den 750 Millisekunden dazwischen machst, ist dem Sensor egal. Du kannst das Programm entweder für die Zeit mit delay() blockieren. Oder irgendwas anderes machen. Aber Du kannst den angeforderten Messwert nicht vom Sensor abholen, bevor der Sensor nicht mit der angeforderten Messung fertiggeworden ist.

Nachtrag noch wegen der scheinbar gleichbleibenden Dateigröße:
Die zusätzlich eingebundene Library scheint keine Gleitkommarechnungen durchzuführen, aber Du in Deinem Test-Sketch hast zwar die Berechnung hingeschrieben:
Code: [Select]
celsius = (float)raw / 16.0;
aber der so errechnete Wert wird im Programm hinterher nirgends mehr verwendet!
Dadurch optimiert der Compiler dann natürlich die Berechnung weg, und wenn er feststellt, dass keine Gleitkommarechnung notwendig ist, weil diese wegoptimiert wurde, wird dann auch kein Gleitkommacode mehr ins Programm eingebunden.

Du mußt den Wert auch tatsächlich verwenden und zum Beispiel auf Serial ausgeben, damit die Rechnung NICHT aus dem Code wegoptimiert wird. Also z.B.:
Code: [Select]
celsius = (float)raw / 16.0;
Serial.println(celsius);

Dann kann die Rechnung nicht wegoptimiert werden und dann wird auch die Gleitkomma-Library automatisch mit in den Code eingebunden.

skorpi080

Eigentlich dachte ich dass der Wert über Modbus gesendet wird?!
Code: [Select]
//////////////// registers of your slave ///////////////////
enum
{     
  celsius,
  Licht,       
  HOLDING_REGS_SIZE // leave this one
  // total number of registers for function 3 and 6 share the same register array
  // i.e. the same address space
};


Alles was bei enum drin steht, wird auch übertragen.
Somit wäre der Wert auch in "Verwendung".
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs


Eigentlich dachte ich dass der Wert über Modbus gesendet wird?!
Code: [Select]
//////////////// registers of your slave ///////////////////
enum
{     
  celsius,
  Licht,       
  HOLDING_REGS_SIZE // leave this one
  // total number of registers for function 3 and 6 share the same register array
  // i.e. the same address space
};


Alles was bei enum drin steht, wird auch übertragen.
Somit wäre der Wert auch in "Verwendung".


???

Deine obenstehende enum-Definition ist eine Umschreibung für sowas:
Code: [Select]
//////////////// registers of your slave ///////////////////
#define  celsius 0
#define  Licht 1
#define  HOLDING_REGS_SIZE 2


Das Aufzählungskonstante "celsius" hat mit der später in der loop-Funktion deklarierten Gleitkomma-Variablen celsius nichts zu tun. Das kannst Du auch testen, indem Du mal die eine oder andere "celsius" Deklaration vom Namen her änderst.

skorpi080

Naja, etwas falsch ausgedrückt.
So wäre es korrekter:
holdingRegs[celsius] == celsius;

Hab grad Problem mit der Übertragung, kommen ständig Fehler obwohl nix übertragen werden sollte
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

skorpi080

#13
Sep 21, 2014, 06:57 pm Last Edit: Sep 21, 2014, 07:31 pm by skorpi08 Reason: 1
mit
Code: [Select]
long celsius1= raw*625L;
holdingRegs[celsius] =celsius1;  

5.144 Bytes
bekomme ich

Quote
18:37:13.178 INFO  runtime.busevents[:26] - Light_GF_Corridor_Wardrobe state updated to OFF
18:37:13.178 INFO  runtime.busevents[:26] - Temperature_GF_Corridor state updated to 10981
18:37:14.233 INFO  runtime.busevents[:26] - Light_GF_Corridor_Ceiling state updated to OFF

Wenn man mit dem Finger auf dem DS18B20 drauf hält, steigt die Zahl.

mit
Code: [Select]

float celsius1;
celsius = (float)raw / 16.0;
holdingRegs[celsius] =celsius1;

5.718 Bytes
bekomme ich

Quote
18:38:52.016 INFO  runtime.busevents[:26] - Light_GF_Corridor_Wardrobe state updated to OFF
18:38:52.017 INFO  runtime.busevents[:26] - Temperature_GF_Corridor state updated to 27
18:38:53.069 INFO  runtime.busevents[:26] - Light_GF_Corridor_Ceiling state updated to OFF

Hier bleibt es allerdings durchgehend bei 27.

mit
Code: [Select]
  long celsius1= raw / 16;
  holdingRegs[celsius] =celsius1;


bekomme ich 22
Quote
19:31:04.125 INFO  runtime.busevents[:26] - Temperature_GF_Toilet state updated to 22
19:31:05.174 INFO  runtime.busevents[:26] - Light_GF_Corridor_Ceiling state updated to OFF
19:31:05.731 INFO  runtime.busevents[:26] - Light_GF_Corridor_Wardrobe state updated to OFF
19:31:05.731 INFO  runtime.busevents[:26] - Temperature_GF_Corridor state updated to 65267


Hab ichs richtig mit dem delay für den DS18B20 gemacht?
Code: [Select]
#include <SimpleModbusSlave.h>

/* The modbus_update() method updates the holdingRegs register array and checks communication.

  Note:  
  The Arduino serial ring buffer is 64 bytes or 32 registers.
  Most of the time you will connect the arduino to a master via serial
  using a MAX485 or similar.

  In a function 3 request the master will attempt to read from your
  slave and since 5 bytes is already used for ID, FUNCTION, NO OF BYTES
  and two BYTES CRC the master can only request 58 bytes or 29 registers.

  In a function 6 request the master will attempt to write to your
  slave and since a 9 bytes is already used for ID, FUNCTION, ADDRESS,
  NO OF REGISTERS, NO OF BYTES and two BYTES CRC the master can only write
  54 bytes or 27 registers.

  Using a USB to Serial converter the maximum bytes you can send is
  limited to its internal buffer which differs between manufactures.
*/
/***************************************************
* / Input/output pins
/***************************************************/
#include <OneWire.h>
OneWire  ds(A3);  // on pin 10
#define PIN_TASTER       9

#define ON  8
#define OFF 7

#define BUTTON_DEBOUNCE_DELAY 900
long lastDebounceTime = 0;

/***************************************************
* / Modbus globals
/***************************************************/
#define MB_TXENPIN   A2
#define MB_SLAVE_ID  2
#define MB_BAUDRATE  9600
#define MB_PARITY  SERIAL_8N2
/*   Valid modbus byte formats are:
    SERIAL_8N2: 1 start bit, 8 data bits, 2 stop bits
    SERIAL_8E1: 1 start bit, 8 data bits, 1 Even parity bit, 1 stop bit
    SERIAL_8O1: 1 start bit, 8 data bits, 1 Odd parity bit, 1 stop bit
*/

//////////////// registers of your slave ///////////////////
enum
{    
 // just add or remove registers and your good to go...
 // The first register starts at address 0
 Licht,
celsius,  
 HOLDING_REGS_SIZE // leave this one
 // total number of registers for function 3 and 6 share the same register array
 // i.e. the same address space
};

unsigned int holdingRegs[HOLDING_REGS_SIZE]; // function 3 and 6 register array
////////////////////////////////////////////////////////////

void setup()
{
   // configure pins
 pinMode(PIN_TASTER, INPUT);
 pinMode(ON, OUTPUT);  
 pinMode(OFF, OUTPUT);
 
 modbus_configure(&Serial, MB_BAUDRATE, MB_PARITY, MB_SLAVE_ID, MB_TXENPIN, HOLDING_REGS_SIZE, holdingRegs);
 // modbus_update_comms(baud, byteFormat, id) is not needed but allows for easy update of the
 // port variables and slave id dynamically in any function.
 modbus_update_comms(MB_BAUDRATE, MB_PARITY, MB_SLAVE_ID);

}

void loop()
{  
 
   modbus_update();
 byte i;
 byte present = 0;
 byte type_s =0;
 byte data[12];
 byte addr[8] = {0x28, 0x97, 0xCC, 0x84, 0x04, 0x00, 0x00, 0x60};
 ds.reset();
 ds.select(addr);
 ds.write(0x44,1);         // start conversion, with parasite power on at the end
 
//  delay(1000);     // maybe 750ms is enough, maybe not
 // we might do a ds.depower() here, but the reset will take care of it.
 if (((millis() - lastDebounceTime) > BUTTON_DEBOUNCE_DELAY))
 {
present = ds.reset();
 ds.select(addr);    
 ds.write(0xBE);         // Read Scratchpad
 Serial.print(present,HEX);
 for ( i = 0; i < 9; i++) {           // we need 9 bytes
   data[i] = ds.read();
   Serial.print(data[i], HEX);
 }

 // convert the data to actual temperature

 unsigned int raw = (data[1] << 8) | data[0];
 if (type_s) {
   raw = raw << 3; // 9 bit resolution default
   if (data[7] == 0x10) {
     // count remain gives full 12 bit resolution
     raw = (raw & 0xFFF0) + 12 - data[6];
   }
 } else {
   byte cfg = (data[4] & 0x60);
   if (cfg == 0x00) raw = raw << 3;  // 9 bit resolution, 93.75 ms
   else if (cfg == 0x20) raw = raw << 2; // 10 bit res, 187.5 ms
   else if (cfg == 0x40) raw = raw << 1; // 11 bit res, 375 ms
   // default is 12 bit resolution, 750 ms conversion time
 }
 float celsius1;
celsius1 = (float)raw / 16.0;
 //long celsius1= raw*625L;
 holdingRegs[celsius] =celsius1;
 }

 // modbus_update() is the only method used in loop(). It returns the total error
 // count since the slave started. You don't have to use it but it's useful
 // for fault finding by the modbus master.
 
 
  if ((digitalRead(PIN_TASTER) == HIGH) && ((millis() - lastDebounceTime) > BUTTON_DEBOUNCE_DELAY))
 {
   // invert LED state
   holdingRegs[Licht] = !holdingRegs[Licht];

   lastDebounceTime = millis();
 }

   if (holdingRegs[Licht]==1){
     // ON //
 digitalWrite(ON,1);
 digitalWrite(OFF,0);
 delay(10);
 digitalWrite(ON,0);
 digitalWrite(OFF,0);
}
 else{
   // OFF //
 digitalWrite(ON,0);
 digitalWrite(OFF,1);
 delay(10);
 digitalWrite(ON,0);
 digitalWrite(OFF,0);
}

}



Kann man hier das Serial.print irgendwie rausnehmen? oder stört es bei der Übertragung nicht?
Code: [Select]
present = ds.reset();
 ds.select(addr);    
 ds.write(0xBE);         // Read Scratchpad
 Serial.print(present,HEX);
 for ( i = 0; i < 9; i++) {           // we need 9 bytes
   data[i] = ds.read();
   Serial.print(data[i], HEX);
 }
playground.arduino.cc/PmWiki/433Mhz-Funkmodule
[Arduino IDE] Feinheiten und Tricks  forum.arduino.cc/index.php?topic=632403.0

jurs


mit
Code: [Select]
long celsius1= raw*625L;
holdingRegs[celsius] =celsius1;   



Wie wäre es denn damit, symbolische Konstanten stets in GROSSBUCHSTABEN zu schreiben, und das bei enum-Konstanten genau so wie bei #define-Konstanten zu handhaben, damit Du nicht zwischen Konstantennamen und Variablennamen durcheinander kommst?

Code: [Select]

//////////////// registers of your slave ///////////////////
enum
{     
  // just add or remove registers and your good to go...
  // The first register starts at address 0
  CELSIUS,
  LICHT,       
  HOLDING_REGS_SIZE // leave this one
  // total number of registers for function 3 and 6 share the same register array
  // i.e. the same address space
};


Dann kommt man beim Schreiben von Code nicht so leicht durcheinander, wo eine symbolische Konstante und wo eine Variable verwendet wird.


Hab ichs richtig mit dem delay für den DS18B20 gemacht?

Keine Ahnung, ich finde, es sieht irgendwie merkwürdig aus.


Hier bleibt es allerdings durchgehend bei 27.


Dass Dein Register-Array als Integer-Array deklariert ist, ist dir aber schon klar:
Code: [Select]
unsigned int holdingRegs[HOLDING_REGS_SIZE];
Oder?
Und wenn Du eine Gleitkommazahl an einen Integer-Wert zuweist, bekommt der Integer nur den ganzzahligen Anteil. Also überträgst Du so wie Du es machst auch stets nur einen Integer-Wert, nämlich die Vorkommastellen der Temperatur.

Vielleicht doch die Temperatur lieber als Integer in Zehntelgrad übertragen?

Go Up