Bitmanipulationsproblem

[TM F. (p_richner)]

Hallo zusammen

Ich habe eine Soft-SPI-Datenleitung, bei der ich das Eingangssignal(8 
Bit) auslesen möchte. Das Signal wird korrekt zum uC übertragen, jedoch 
weiss ich nicht, was ich in der Auswertung falsch mache, dass ich das 
Signal nicht richtig lese.

Datenpin ist auf PC1.
Clock auf PC0(der funktioniert)

Der Code:

// Switch Data pin to input
    DDRC &= ~(1<<SPI_SOFT_DATA);

    unsigned char data_buffer;
    
    // Receive the data byte
    for(loop = 0, mask = 0x80, data_buffer=0;loop<8;loop++,mask = mask >> 1;)
    {
        SPI_SOFT_CLK_HIGH();
        // If data input is high, set bit
        if(PORTC & 0x02)
            data_buffer |=  mask;

        SPI_SOFT_CLK_LOW();    
    }
    SPI_CSX_HIGH();
    // Switch Data pin to output
    DDRC |= (1<<SPI_SOFT_DATA);
    return data_buffer;

Ich hoffe, ihr könnt mir helfen, den Knoten zu lösen.

MfG

[Rufus Τ. F. (rufus)]

Du solltest vor dem Durchlaufen Deiner Schleife sicherstellen, daß 
"data_buffer" mit 0 initialisiert ist. Da das eine automatische Variable 
zu sein scheint (d.h. sie liegt auf dem Stack), ist sie das nämlich 
nicht, und Du mischst Deine Nutzdaten mit dem "Rauschen", das da schon 
in der Variablen drinsteht.

[TM F. (p_richner)]

Das habe ich gemacht, ergibt jedoch keine Änderung.

[Der Gast (Gast)]

Welcher Prozessor ?
PORTC wirklich Register für Input ?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Der Gast schrieb:
> PORTC wirklich Register für Input ?
Mit hinreichender Wahrscheinlichkeit sollte es das PINC Register sein...

[KU (Gast)]

TM F. schrieb:
> Ich hoffe, ihr könnt mir helfen, den Knoten zu lösen.

Vielleicht bist du zu schnell beim lesen - du sagst ja nicht welcher 
Baustein mit SPI das ist. Näheres steht im Datenblatt bzgl. des Timings.

Deine For-Schleife ist schrecklich unübersichtlich so.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

KU schrieb:
> du sagst ja nicht welcher Baustein mit SPI das ist.
Das kann jeder sein. Es ist ein Software-SPI...

> Deine For-Schleife ist schrecklich unübersichtlich so.
Und der letzte Strichpunkt ist unnötig ;-)
Ich hätte sie so geschrieben:

    for (loop=0, mask=0x80, data_buffer=0;   loop<8;   loop++, mask=mask>>1)

[Stefan F. (Gast)]

Ja, du musst PINC lesen.
Ich finde, du solltest auch eine gewisse Zeit abwarten, zwischen dem 
Takt und dem Einlesen des Pins.

Vorschlag, um deine for-Schleife besser lesbar zu machen:

    uint8_t data_buffer=0;
    for(int i=0; i<8; i++)
    {
        SPI_SOFT_CLK_HIGH();
        _delay_us(1);
        data_buffer <<= 1;
        if(PINC & 0x02)
        {
            data_buffer+=1;
        }
        SPI_SOFT_CLK_LOW();    
    }

Aber das ist letztendlich Geschmackssache.

[TM F. (p_richner)]

Lothar M. schrieb:
> Mit hinreichender Wahrscheinlichkeit sollte es das PINC Register sein...

Mit der änderung von PORTC zu PINC funktioniert es.

Habe mich mal wieder selber gelinkt.

Stefan U. schrieb:
> Ich finde, du solltest auch eine gewisse Zeit abwarten, zwischen dem
> Takt und dem Einlesen des Pins.

Das Datenbit wird schon bei der fallenden Flanke des Clocks geschrieben, 
deshalb ist es nichts zeitkritisches.

Danke für die Antworten

[Thomas E. (picalic)]

Stefan U. schrieb:
> uint8_t data_buffer=0;

die Initialisierung kann man sich hier auch noch sparen, da am Schluss 
sowieso alle 8 Datenbits 'rausgeschoben worden sind.