EEPROM will sich nicht schreiben/lesen lassen

[Christop (Gast)]

Hallo....

ich habe eine PCB mit einem ATMega328. Dieser soll "default" Werte im 
EEPROM haben. Zugleich soll ein weiterer EEPROM Bereich zum Speichern 
der Werte da sein..

So das man bei einem Reset wieder die "default" Werte aus dem EEPROM 
holt.

Ok.

Jetzt habe ich in der main.c folgenden Inhalt....

#include "main.h"
#include "timer.h"
#include "twislave.h"

int main (void) {

  // Write Default EEPROM if not exist
  
  if (getEEDefaultExist() == 0) {
    writeDefaultEEPROM();
    updateEEDefaultExist();
  }
   
  i2c_readaddr_init();
  init_twi_slave(i2c_readaddr());

  // Init PWM
  pwm_init();
  init_pwmmodes();

  // Init Timer
  timer_init();

  // I2C Data
  init_address();

   // Init Data from EEPROM at First Start
   pwmchangemode = getEEPWMChangeMode();
   
   pwmchangetime = getEEPWMSoftModeChangeTime();
   pwmchangetimeR = getEEPWMSoftModeChangeTimeR();
   pwmchangetimeG = getEEPWMSoftModeChangeTimeG();
   pwmchangetimeB = getEEPWMSoftModeChangeTimeB();
  pwmchangetimeW = getEEPWMSoftModeChangeTimeW();
  pwmsoftlimit = getEEPWMSoftLimit();

............

Dazu habe ich den eeprom.c datei, diese sieht wie folgt aus:

#include "main.h"

uint8_t eeDefaultExist EEMEM             = 99;

// PWM Change Mode { 0 = Hard, 1 = Soft}
uint8_t eePWMChangeMode EEMEM           = 100;

// PWM Softmode Changetime
uint8_t eePWMSoftModeChangeTime EEMEM       = 101;

// PWM Softmode Changetime RED Channel
uint8_t eePWMSoftModeChangeTimeR EEMEM     = 102;
// PWM Softmode Changetime GREEN CHANNEL
uint8_t eePWMSoftModeChangeTimeG EEMEM     = 103;
// PWM Softmode Changetime BLUE CHANNEL
uint8_t eePWMSoftModeChangeTimeB EEMEM     = 104;
// PWM Softmode Changetime WHITE CHANNEL
uint8_t eePWMSoftModeChangeTimeW EEMEM     = 105;

// PWM Max Value (Softlimit for PWM)
uint8_t eePWMSoftLimit EEMEM             = 106;

uint8_t eeWs2812Count EEMEM             = 107;

void writeDefaultEEPROM(void) {
  updateEEPWMChangeMode(1);
  updateEEPWMSoftModeChangeTime(200);
  updateEEPWMSoftModeChangeTimeR(200);
  updateEEPWMSoftModeChangeTimeG(200);
  updateEEPWMSoftModeChangeTimeB(200);
  updateEEPWMSoftModeChangeTimeW(200);
  updateEEWS2812LedCount(10);
  updateEEPWMSoftLimit(255);
}

uint8_t getEEDefaultExist(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eeDefaultExist);
  return retByte;
}

uint8_t updateEEDefaultExist(void) {
  eeprom_update_byte(&eeDefaultExist, 1);
}


uint8_t getEEPWMChangeMode(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMChangeMode);
  return retByte;
}

uint8_t updateEEPWMChangeMode(uint8_t newval) {
  eeprom_update_byte(&eePWMChangeMode, newval);
}



uint8_t getEEPWMSoftModeChangeTime(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftModeChangeTime);
  return retByte;
}

uint8_t updateEEPWMSoftModeChangeTime(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftModeChangeTime, newval);
}



uint8_t getEEPWMSoftModeChangeTimeR(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftModeChangeTimeR);
  return retByte;
}

uint8_t updateEEPWMSoftModeChangeTimeR(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftModeChangeTimeR, newval);
}



uint8_t getEEPWMSoftModeChangeTimeG(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftModeChangeTimeG);
  return retByte;
}

uint8_t updateEEPWMSoftModeChangeTimeG(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftModeChangeTimeG, newval);
}



uint8_t getEEPWMSoftModeChangeTimeB(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftModeChangeTimeB);
  return retByte;
}

uint8_t updateEEPWMSoftModeChangeTimeB(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftModeChangeTimeB, newval);
}



uint8_t getEEPWMSoftModeChangeTimeW(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftModeChangeTimeW);
  return retByte;
}

uint8_t updateEEPWMSoftModeChangeTimeW(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftModeChangeTimeW, newval);
}



uint8_t getEEPWMSoftLimit(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eePWMSoftLimit);
  return retByte;
}

uint8_t updateEEPWMSoftLimit(uint8_t newval) {
  eeprom_update_byte(&eePWMSoftLimit, newval);
}



uint8_t getEEWS2812LedCount(void) {
  uint8_t retByte;
  retByte = eeprom_read_byte (&eeWs2812Count);
  return retByte;
}
uint8_t updateEEWS2812LedCount(uint8_t newval) {
  eeprom_update_byte(&eeWs2812Count, newval);
}

Jedoch sind die Werte leer... obwohl ich diese am Anfang der main immer 
erst schreibe..

also so...

#include "main.h"
#include "timer.h"
#include "twislave.h"

int main (void) {

  // Write Default EEPROM if not exist
  
  //if (getEEDefaultExist() == 0) {
  //  writeDefaultEEPROM();
  //  updateEEDefaultExist();
  //}
   
  i2c_readaddr_init();
  init_twi_slave(i2c_readaddr());

  // Init PWM
  pwm_init();
  init_pwmmodes();

  // Init Timer
  timer_init();

  // I2C Data
  init_address();

   // Init Data from EEPROM at First Start
  
   pwmchangemode = 1;
   
   pwmchangetime = 15;
   pwmchangetimeR = 15;
   pwmchangetimeG = 15;
   pwmchangetimeB = 15;
  pwmchangetimeW = 15;
  pwmsoftlimit = 250;
.............
.............

Vielleicht kann mir da ja der ein oder andere weiter helfen :)

[oweiowei (Gast)]

Christop schrieb:
> Jetzt habe ich in der main.c folgenden Inhalt....

Ersten gehört so ein langer Soure-Text in einen Anhang (siehe:
Wichtige Regeln - erst lesen, dann posten!), zweitens ist das
Program nicht vollständig sodass wir nicht beurteilen können
ob du alles richtig machst.

[Christop (Gast)]

Ja...

hier mal der komplette Code

https://github.com/pihome-dev/tempfolder

[Harald (Gast)]

Code habe ich mir nicht angeschaut, aber für die Analyse gibt es 
Logic-Analyzer <10€ und leistungsstarke Software für 0€. Dank 
entsprechender Decoder für nahezu jeden Speicherbaustein in dieser 
Software bekommt man genau angezeigt, wo das Problem liegt.

Darüber hinaus kann der Schreibschutz ein Problem darstellen. Fast immer 
gibt es einen Pin, der muss natürlich richtig stehen. Weiterhin gibt es 
auch manchmal ein Configuration-Word in den EEP-Commands, das nochmals 
einen Schreibschutz beinhalten kann.

[holger (Gast)]

>aber für die Analyse gibt es
>Logic-Analyzer <10€ und leistungsstarke Software für 0€

Na dann erklär mal wie man den Logic-Analyzer an den
Datenbus im ATMega zu seinem im Gehäuse verbauten EEPROM
anschliesst.

[Dumpfbacke (Gast)]

Externes EEPROM.

Wer lesen kann ist klar im Vorteil.
Holger offensichtlich nicht.

[holger (Gast)]

>   // Init Data from EEPROM at First Start
>
>   pwmchangemode = 1;

Man kann Werten im EEMEM nicht einfach im laufenden Programm
per Compiler einen Wert wie bei einer Variablen im RAM zuweisen
und dann denken dass das im EEPROM landet. Dazu musst du schon
irgendeine EEPromWrite Funktion benutzen müssen.

[holger (Gast)]

>Externes EEPROM.

Wo steht da oben irgendwas von externem EEPROM?

[oweiowei (Gast)]

holger schrieb:
> Wo steht da oben irgendwas von externem EEPROM?

Das darf man annehmen wenn er TWI Code und I2C Adressen verwendet.

[oweiowei (Gast)]

oweiowei schrieb:
> Das darf man annehmen wenn er TWI Code und I2C Adressen verwendet.

Wenn sich der TO allerdings dessen nicht bewusst ist dann
ist Hopfen und Malz verloren und ich nehme alles zurück und
behaupte das Gegenteil.

[Harald (Gast)]

holger schrieb:
> Wo steht da oben irgendwas von externem EEPROM?

Christop schrieb:
> Zugleich soll ein weiterer EEPROM Bereich zum Speichern
> der Werte da sein..

Zusätzlich die Verwendung von I2C im Code. Damit war es allem Anschein 
nach klar. Falls es doch anders gemeint war, Pech

[holger (Gast)]

>> Wo steht da oben irgendwas von externem EEPROM?
>
>Das darf man annehmen wenn er TWI Code und I2C Adressen verwendet.

Nein darf man nicht. Am I2C kann sonst was hängen. EEMEM ist jedenfalls
kein EEPROM was am I2C angeschlossen ist.

[holger (Gast)]

>Dieser soll "default" Werte im
>EEPROM haben. Zugleich soll ein weiterer EEPROM Bereich zum Speichern
>der Werte da sein..

>Zusätzlich die Verwendung von I2C im Code. Damit war es allem Anschein
>nach klar. Falls es doch anders gemeint war, Pech

Ich würde das so interpretieren das die "default" Werte in einem Bereich 
im EEPROM liegen der nie verändert wird. Dann gibt es noch einen Bereich 
wo man seine veränderlichen Werte speichert. Und das ganze im internen 
EEPROM vom ATMega. I2C komplett Latte.

[Holger L. (max5v)]

Erst einmal könnte man die update Funktionen ohne return in der eeprom.c 
auf void anstelle von uint8_t setzen.

Zu dem sollte man auch darauf achten, das es sich wirklich um einen 
ATmega328 handelt und nicht um die P oder PB Version, das kann schnell 
mal zu unerwünschtem Verhalten führen.

Und zuletzt,

if (getEEDefaultExist() == 0) {
    writeDefaultEEPROM();
    updateEEDefaultExist();
  }

Sicher das der Speicherinhalt eines "jungfräulichem" EEprom 0x00 ist und 
nicht 0xFF?
Dieses hier wäre evtl. einen Versuch wert:

if (getEEDefaultExist() != 1) {
    writeDefaultEEPROM();
    updateEEDefaultExist();
  }

[Stefan F. (Gast)]

> uint8_t eePWMChangeMode EEMEM = 100;

Das ist eine Integer Variable, die der Compiler in die .eeprom Sektion 
schreibt. Diese Information kann (muss nicht und ist auch eher ein 
Sonderfall) dazu verwendet werden, dass EEPROM zusammen mit dem FLASH zu 
initialisieren. Mehr kann man damit nicht anfangen.

Das C-Programm hat keinen direkten zugriff auf das EEPROM. Diese 
Variable kannst du also weder lesen noch schreiben. Sie haben aus Sicht 
des C Programm eine Adresse aber keinen Wert. Zugriff auf das EEPROM 
bekommst du nur indirekt über die eeprom_xxxx_xxxx() Funktionen.

Also z.B.:

> retByte = eeprom_read_byte (&eePWMChangeMode);

Hier liest du ein Byte aus dem EEPROM an irgendeiner Adresse, die der 
Compiler für eePWMChangeMode vorgegeben hat. eePWMChangeMode enthält 
also gar nicht den Wert von "PWMChangeMode" sondern nur dessen Adresse 
im EEPROM (nicht im RAM!).

Das gleiche Problem hast du auch mit allen anderen Variablen. Reduziere 
dein Beispielprogramm demnächst bitte auf möglichst wenige Zeilen, die 
das Problem noch demonstrieren. Dieses Programm sollte dann aber so 
vollständig sein, dass es compilierbar ist.

Änderungsvorschlag:

Anstelle von
> uint8_t eePWMChangeMode EEMEM = 100;

schreibst du

#define eePWMChangeMode 1

Damit legst du fest, dass der Wert für PWMChangeMode im EEprom an 
Adresse 1 gespeichert werden soll. Die nächste Variable kommt auf 
Adresse 2, dann 3, usw. So kannst du selber vorgeben, was wo hin kommt 
und musst dich nicht mehr oder weniger auf zufällige Vorgaben des 
Compilers verlassen die sich möglicherweise mal unerwartet ändern.

Holger L. schrieb:
> Sicher das der Speicherinhalt eines "jungfräulichem" EEprom 0x00 ist und
> nicht 0xFF?

Es ist 0xFF, guter Hinweis. Das wird der eigentliche Knackpunkt sein.

[Stefan F. (Gast)]

> Ich würde das so interpretieren das

Ich glaube, der TO hat sein eigenes Programm nicht verstanden oder 
abgeschrieben.

Die Default werte stehen nicht im EEPROM, sondern im Quelltext. Und zwar 
in der Funktion writeDefaultEEPROM().

Das Programm schreibt diese Werte erstmalig ins EEPROM, wenn es danach 
aussieht, dass es noch nicht initialisiert wurde. Die Werte von den 
ganzen EEMEM Variablen werden überhaupt nicht verwendet. Es werden nur 
deren Adressen verwendet.

[Pete K. (pete77)]

Und die Return-Werte der Funktionen werden nicht ausgewertet.
Programmieren anno 2020 ....

[Christoph (Gast)]

Also wie schon richtig vermutet hat es mit dem externen eeprom nichts zu 
tun.

Wenn man sich den Code anschaut, sollte das aber klar sein.

Wie schon erkannt, soll der atmega, falls noch nicht vorhanden die 
default Werte schreiben. Userdaten werden woanders geschrieben.

Ich habe den eeprom Teil aus dem tutorial. Dort wird write, Update etc 
erklärt. Auch das Lesen aus dem eeprom.

Was ist nun bei mir anders als dem Code vom tutorial?

[Stefan F. (Gast)]

Christoph schrieb:
> Was ist nun bei mir anders als dem Code vom tutorial?

1.) Wleches Tutorial?
2.) Vergleiche selbst. Die Knackpunkte wurden dir genannt.

[Moot S. (mootseeker)]

Harald schrieb:
> Code habe ich mir nicht angeschaut, aber für die Analyse gibt es
> Logic-Analyzer <10€ und leistungsstarke Software für 0€.

Wo gibt es anständige Logic-Analyzer unter 10€ ?

[Stefan F. (Gast)]

Moot S. schrieb:
> Wo gibt es anständige Logic-Analyzer unter 10€ ?

Bei Amazon, Ebay, Aliexpress.

Ob sie "anständig" sind, ist Ansichtssache. Ich bin mit meinem 
zufrieden. Für das Geld man es selber nicht besser machen. Auch nicht 
für doppelt so viel Geld.