PIC EEPROM Hilfe C

[andreas (Gast)]

Hallo,

Ich habe ein iButton Schloss gebaut, auf Basis des PIC16F628A
Damit können 4 Ausgänge geschalten werden, für jeden dieser Ausgänge 
können 4 Schlüssel gespeichert werden( ein iButton benötigt 8 Byte) der 
PIC hat 128 Byte EEPROM Speicher, somit sind also insgesamt 16 Schlüssel 
möglich.
Angelernt werden die Schlüssel mittels 4 Taster, für jeden Ausgang ein 
Taster, also Taster 1 wird gedrückt, angesteckter Schlüssel gespeichert, 
Ausgang 1 geschaltet, das gleiche für die jeweils anderen 
Taster/Ausgänge.

Funktioniert soweit auch alles einwandfrei, nur das Problem ist wenn die 
ersten 4 Speicherplätze belegt sind und ich versuche einen 5. Schlüssel 
auf dem jeweiligen Ausgang zu speichern springt er automatisch auf den 
Speicherbereich des nächsten Ausgangs, und dementsprechend wird auch der 
nächste Ausgang geschaltet, das soll aber nicht so sein !

Ich möchte, dass wenn die 4 Speicherplätze belegt sind, kein weiterer 
Schlüssel mehr für den jeweiligen Ausgang gespeichert werden kann.
Also wirklich nur fix 4 für jeden Kanal.

Da ich von C Programmierung nicht wirklich viel Ahnung habe, hat 
vielleicht jemand eine Idee wie man den Code entsprechend ändern könnte.
Danke !

Anbei der Code Teil fürs Speichern und suchen im EEPROM.

[Max H. (hartl192)]

andreas schrieb:
> Funktioniert soweit auch alles einwandfrei, nur das Problem ist wenn die
> ersten 4 Speicherplätze belegt sind und ich versuche einen 5. Schlüssel
> auf dem jeweiligen Ausgang zu speichern springt er automatisch auf den
> Speicherbereich des nächsten Ausgangs

  for(bytes = 0; bytes < 8; bytes++)
  {
    eeprom_write(((keys * 8) + bytes), pData[bytes]);
  }

Dieser Teil wird auf jeden Fall ausgeführ, auch wenn oben alle 
Platzebesetzt waren. Ich würde davor noch ein return einbauen wenn alle 
Platze voll sind.



andreas schrieb:
> von C Programmierung nicht wirklich viel Ahnung
Noch ein Tipp:
mMn hast du zu viele Funktionalitäten in diese Funktionen gepackt, 
dadurch wird der Code unübersichtlich. Ich würde für ein paar eigene 
Funktionen schreiben und diese dann von WriteEEPROM aufrufen lassen.
Ich würde z.B. eine Funktion writeKey, readKey, keyEmpty schreiben:

writeKey(uint8_t *key, uint8_t key_number)
{
  uint8_t byte_counter = 0;
  key_number *= 8;
  for( ; byte_counter < 8; byte_counter++)
    eeprom_write(key_number++, *key++);
}

readKey(uint8_t *key, uint8_t key_number)
{
  uint8_t byte_counter = 0;
  key_number *= 8;
  for( ; byte_counter < 8; byte_counter++)
    *key++ = eeprom_read(key_number++);
}

uint8_t keyEmpty(uint8_t *key)
{
  uint8_t byte_counter = 0;
  for( ; byte_counter < 8; byte_counter++)
  {
    if(*key++ != 0xFF)
      return 0;
  }   
  return 1;
}

Danach in der Funktion WriteEEPROM in etwa so verwenden:

uint8_t WriteEEPROM(uint8_t *new_key, uint8_t out_number) 
{
  out_number *= 4;
  uint8_t key_counter = 0;
  uint8_t read_key[8];
  for( ; key_counter < 4; key_counter++)
  {
    readKey(read_key, out_number)
    if(keyEmpty(read_key)
    {
      writeKey(out_number, new_key);
      return 1; // New key saved
    }
    else
      out_number++;
  }
  return 0; // No free space for the key
}

P.S. Wenn du die *.C Endung behältst funktioniert hier das Syntax 
Highlighting.

[andreas (Gast)]

Hallo Max,
Ja, mein Code ist wirklich überladen, das stimmt schon, bin aber schon 
froh das es überhaupt funktioniert.
Max H. schrieb:
•

  for(bytes = 0; bytes < 8; bytes++)
  {
    eeprom_write(((keys * 8) + bytes), pData[bytes]);
  }

> Dieser Teil wird auf jeden Fall ausgeführ, auch wenn oben alle
> Platzebesetzt waren. Ich würde davor noch ein return einbauen wenn alle
> Platze voll sind.

leider bringe ich das mit dem Return nicht zusammen, dann wird überhaupt 
nichts mehr gespeichert.

for(keys; keys < keysl; keys++)
  {
  for(bytes = 0; bytes < 8; bytes++)
    {
    if(eeprom_read((keys * 8) + bytes) == 0xFF)
      counter++;
    }

  if(counter == 8) break;
  else counter = 0;
  }

Ich denke hier müsste irgendwo ein Return rein ?

Dein Code Vorschlag sieht auf jeden Fall gut aus, ich weiß im Moment nur 
noch nicht wie ich den einbauen kann.

[Max H. (hartl192)]

andreas schrieb:
> Ich denke hier müsste irgendwo ein Return rein ?
Versuch mal nach der for(keys; keys < keysl; keys++) schleife ein

if(!counter)
  return 0;

einzubauen.

Oder du machst die ganze Schleife so:

  for(keys; keys < keysl; keys++)
  {
    for(bytes = 0; bytes < 8; bytes++)
    {
      if(eeprom_read((keys * 8) + bytes) == 0xFF)
        counter++;
    }

    if(counter == 8)  // If a space is free
    {
      //Write the key to the EEPROM
      for(bytes = 0; bytes < 8; bytes++)
        eeprom_write(((keys * 8) + bytes), pData[bytes]);
      return 1;
    }
    else 
      counter = 0;
  }
  return 0;
}

[andreas (Gast)]

So:
•

for(keys; keys < keysl; keys++)
  {
if(!counter)
  return 0;

dann wird überhaupt nichts mehr gespeichert.

[Max H. (hartl192)]

andreas schrieb:
> for(keys; keys < keysl; keys++)
>   {
> if(!counter)
>   return 0;
> dann wird überhaupt nichts mehr gespeichert.
Was auch logisch ist und weswegen ich
>> nach der for(keys; keys < keysl; keys++) Schleife
und nicht in der Schleife geschrieben habe.

P.S. Wenn du konsequent gleich einrücken und die geschwungene Klammern 
gleich setzten würdest, würde man Anfang und Ende der Schleifen auch 
leichter erkennen.

[andreas (Gast)]

Max H. schrieb:
> Oder du machst die ganze Schleife so:



•

  for(keys; keys < keysl; keys++)
  {
    for(bytes = 0; bytes < 8; bytes++)
    {
      if(eeprom_read((keys * 8) + bytes) == 0xFF)
        counter++;
    }

    if(counter == 8)  // If a space is free
    {
      //Write the key to the EEPROM
      for(bytes = 0; bytes < 8; bytes++)
        eeprom_write(((keys * 8) + bytes), pData[bytes]);
      return 1;
    }
    else 
      counter = 0;
  }
  return 0;
}

Ja, das funktioniert !!!, Klasse
Vielen Dank !

[andreas (Gast)]

Max H. schrieb:
> Was auch logisch ist und weswegen ich
>>> nach der for(keys; keys < keysl; keys++) Schleife
> und nicht in der Schleife geschrieben habe.

Ich habe das in unterschiedlichen Versionen auch danach versucht, nur, 
mehr als 1 Key wurde da auch nicht gespeichert.

Ja, ja, die Klammern..., ich weiß.

Ich bedanke mich herzlich für deine Geduld und Hilfe,
Gruß, Andreas.