Programm übernimmt Werte nicht

[Tim S. (tommypic)]

Guten Tag,

ich bin gerade dabei das PIC 18 Explorer Board mit dem 
Ethernet-Controller zu verbinden. Nun schreibe ich gerade die 
Initialisierung und möchte die MAC-Addresse auf meinem LCD ausgeben. 
Durch debuggen habe ich herausgefunden das ich vom Ethernet-COntroller 
die richtigen Werte über SPI bekomme, allerdings werden diese nicht in 
meinen Variablen gespeichert und somit wird immer nur 00-00-00-00-00-00 
ausgegeben.

hier mal mein Code:

main-funktion:

void main(void) {
    BYTE mac1 = 0, mac2 = 0, mac3 = 0, mac4 = 0, mac5 = 0, mac6 =0;
    TRISDbits.TRISD7 = 0;
    TRISBbits.TRISB5 = 1;
    LATBbits.LATB5 = 1;
    TRISD = 0x00;
    TRISBbits.TRISB0 = 1;
    initTimer();
    LCD_init();
    resetENC();
    delayms(250);
    setERXST(((0x5fff)-2*1518));
    LATDbits.LATD0 = 1;
    delayms(1000);
    LATDbits.LATD0 = 0;
    while (1) {
        delayms(500);
        LCD_Line1();
        mac1 = getMac1();
        LCD_hex(getMac1());
        LCD_string("-");
        mac2 = getMac2();
        LCD_hex(mac2);
        LCD_string("-");
        LCD_hex(getMac3());
        LCD_string("-");
        LCD_hex(getMac4());
        LCD_string("-");
        LCD_hex(getMac5());
        LCD_string("-");
        LCD_hex(getMac6());
    }
}

getMac-funktion (sind alle gleich bis auf Registeraddressen)

unsigned char getMac1(){
    ByteOp(B3SEL);
    return Byte2Op((BYTE)(RCR|0x04),0x00);
}

SPI-Abfrage:

BYTE Byte2Op(BYTE code, BYTE data) {
    BYTE answer = 0;
    ENC_CS = 0;
    SPI_IF = 0;
    SPI_BUF = code;
    WaitForSend();
    answer = SPI_BUF;
    SPI_BUF = data;
    WaitForSend();
    answer = SPI_BUF;
    ENC_CS = 1;
    return answer;
}

BYTE ist in einer Header-Datei so definiert:

typedef unsigned char           BYTE;

also debuggen lief so ab:

answer bekommt den Wert des Bytes einer Stelle der Mac-Addresse und 
sobald ich das in der Main in eine mac-Variable abspeichern möchte, 
bleibt diese aber auf dem Wert 00...

ich verstehe nicht warum :(

mit freundlichen Grüßen

tim

[holger (Gast)]

>answer bekommt den Wert des Bytes einer Stelle der Mac-Addresse und
>sobald ich das in der Main in eine mac-Variable abspeichern möchte,
>bleibt diese aber auf dem Wert 00...
>
>ich verstehe nicht warum :(

Der ENC hat keine MAC Adresse. Du musst da eine reinschreiben!

[Tim S. (tommypic)]

holger schrieb:
>>answer bekommt den Wert des Bytes einer Stelle der Mac-Addresse
> und
>>sobald ich das in der Main in eine mac-Variable abspeichern möchte,
>>bleibt diese aber auf dem Wert 00...
>>
>>ich verstehe nicht warum :(
>
> Der ENC hat keine MAC Adresse. Du musst da eine reinschreiben!

nein er hat eine mac-addresse, diese kann man überschreiben muss man 
aber nicht.

auszug aus dem datenblatt:

"8.6.1. preprogrammed mac address
as shipped, each ENCX24J600 device has been preprogrammed with a unique 
MAC address. This value is stored in nonvolatile memory and reloaded 
into the MAADR registers after every Power-on and System Reset. The 
factory preprogrammed MAC address is permanent and will be restored to 
the MAC registers after each Reset."

und sonst hätte ich beim Debuggen ja auch den Wert 0x00 in answer 
stehen. dort bekomme ich allerdings 6 unterschiedliche Werte für die 
Bytes, diese decken sich mit der Mac-Addresse die ich habe wenn ich das 
Beispiel-Programm von Microchip verwende

[holger (Gast)]

>nein er hat eine mac-addresse, diese kann man überschreiben muss man
>aber nicht.
>
>auszug aus dem datenblatt:
>
>"8.6.1. preprogrammed mac address
>as shipped, each ENCX24J600 device has been preprogrammed with a unique
>MAC address.

Dann hättest du vieleicht mal sagen sollen um welchen ENC es
sich dreht. Das kann hier ja keiner raten. Genausowenig kann man
raten was die anderen nicht gezeigten Codeteile machen.

Viel Glück noch.

[Tim S. (tommypic)]

holger schrieb:
>>nein er hat eine mac-addresse, diese kann man überschreiben muss
> man
>>aber nicht.
>>
>>auszug aus dem datenblatt:
>>
>>"8.6.1. preprogrammed mac address
>>as shipped, each ENCX24J600 device has been preprogrammed with a unique
>>MAC address.
>
> Dann hättest du vieleicht mal sagen sollen um welchen ENC es
> sich dreht. Das kann hier ja keiner raten. Genausowenig kann man
> raten was die anderen nicht gezeigten Codeteile machen.
>
> Viel Glück noch.

gehts noch einwenig unfreundlicher bitte?! ok die eine angabe vergessen, 
aber ich hatte erwähnt das ich über SPI die richtigen werte bekomme, 
also kann es nicht daran liegen das der ENC keine MAC-Addresse hat!



hier nochmal mein debuggprozess:

mac1 = getMac1();

--> ruft getMac1() auf
mittels

ByteOp(B3SEL);

 wird Bank 3 des ENC selektiert.

return Byte2Op((BYTE)(RCR|0x04),0x00);

--> rufe Byte2Op auf und gebe den Wert zurück.

Byte2Op ruft den Wert eines Bytes einer Stelle in der Mac-Addresse ab!

BYTE Byte2Op(BYTE code, BYTE data) {
    BYTE answer = 0;
    ENC_CS = 0;
    SPI_IF = 0;
    SPI_BUF = code;
    WaitForSend();
    answer = SPI_BUF;
    SPI_BUF = data;
    WaitForSend();
    answer = SPI_BUF; <-- an dieser stelle bekommt answer den wert des      empfangenen bytes
    ENC_CS = 1;
    return answer; <-- Wert wird zurückgegeben
}

unsigned char getMac1(){
    ByteOp(B3SEL);
    return Byte2Op((BYTE)(RCR|0x04),0x00); <-- gibt Wert von answer zurück
}

        delayms(500);
        LCD_Line1();
        mac1 = getMac1(); <-- hier kommt dieser Wert nie an
        LCD_hex(getMac1()); <-- in dieser Codezeile hat mac1 immer noch den wert 0x00
        LCD_string("-");
        mac2 = getMac2(); <-- das gleiche mit dem zweiten byte
        LCD_hex(mac2); <-- mac2 bleibt bei 0x00
        LCD_string("-");
        LCD_hex(getMac3());
        LCD_string("-");
        LCD_hex(getMac4());
        LCD_string("-");
        LCD_hex(getMac5());
        LCD_string("-");
        LCD_hex(getMac6());

denke nicht das noch andere programmteile benötigt werden, falls ja 
bitte sagen

edit:

also der Wert geht beim return der getMac()-Funktion verloren

unsigned char getMac2(){
    BYTE buffer = 0;
    ByteOp(B3SEL);
    buffer = Byte2Op((BYTE)(RCR|0x05),0x00); //hier wird der richtige wert übergeben
    LCD_hex(buffer); //hier wird der wert richtig ausgegeben
    return buffer; //hier zurück gegeben allerdings kommt der wert nie bei mac2 in der main-methode an
}

[Tim S. (tommypic)]

ok hat sich erledigt - mein fehler lag darin das die getMac()-Funktionen 
in einer seperaten .c-datei lagen und diese nicht in der dazu gehörigen 
header-datei definiert worden sind. damit kam wohl der compiler nicht 
klar