C: Problem mit Schieberegister 74HC595 (Ausgänge immer HIGH)

[Michael N. (garril)]

Hallo,

ich möchte mit einem Attiny45 ein Schieberegister ansteuern.
Leider bekomme ich das noch nicht ganz gebacken...
Und zwar habe ich an jedem der 8 Ausgangspins des Registers eine LED 
hängen.
Aktuell leuchten immer alle LEDs. Selbst wenn ich vorher nichtmal 
serpa_init() oder serpa_out() aufrufe.
Liegt es womöglich an meiner Verkabelung?
PB3=STCP/RCK
PB4=CLK/SCK/SHCP
PB5=SER/DS
MR direkt auf +5V
OE direkt auf GND

Hier mein C-Code (wahrscheinlich aber nicht das Problem, siehe unten):

/* Quelle: http://www.rn-wissen.de/index.php/Portexpander_am_AVR#Mit_SPI-Hardware_2 */

/* SERiell nach PArallel (serpa) via Software */
#include <avr/io.h>
#include "serpa.h"

#define SET(port,b) (port) |= (1<<(b))
#define CLR(port,b) (port) &= ~(1<<(b))

/* Array fuer die Daten */
unsigned char serpa[SERPA_SIZE];

void serpa_init (void)
{
    /* Verwendete Ports auf OUT */
  /* PB3=STCP/RCK, PB4=CLK/SCK/SHCP, PB5=SER/DS
  MR auf +5V
  OE auf GND */
  DDRB = (1 << DDB3) | (1 << DDB4) | (1 << DDB5);

    /* SCR und RCK auf definierten Level HIGH */
    SET (PORTB, PB4);
    SET (PORTB, PB3);
}

void serpa_out (void)
{
    unsigned char anz = SERPA_SIZE;
    unsigned char* serp = serpa+SERPA_SIZE;

    do
    {
        unsigned char bits;
        unsigned char data = *--serp;

        /* 8 Bits pro Byte rausschieben */
        for (bits = 8; bits > 0; bits--)
        {
            CLR (PORTB, PB5);
            if (data & 0x80)
            {
                SET (PORTB, PB5);
            }

            data <<= 1;
            /* Strobe an SCK schiebt Daten im Gaensemarsch */
            /* um 1 Position weiter durch alle Schieberegister */
            CLR (PORTB, PB4);
            SET (PORTB, PB4);
        }
    }
    while (--anz > 0);

    /* Strobe an RCK bringt die Daten von den Schieberegistern in die Latches */
    CLR (PORTB, PB3);
    SET (PORTB, PB3);
}

int main(void) {
  serpa_init();
  while (1) {
    serpa[0]=0b10000001;
    serpa_out();
  }
}

Da es aber unabhängig vom Programm zu sein scheint, tippe ich eher auf 
einen Verkabelungsfehler.
Passt das mit MR auf 5V und OE auf GND (habe da einfach jeweils eine 
Drahtbrücke gesteckt)?

Vielen Dank für die Hilfe.

[Krapao (Gast)]

AVR-Tutorial: Schieberegister

> Passt das mit MR auf 5V und OE auf GND (habe da einfach jeweils eine
> Drahtbrücke gesteckt)?

Ja.

Wie ist SERPA_SIZE definiert?
Dein main() geht von 1 aus.

> Und zwar habe ich an jedem der 8 Ausgangspins des Registers eine LED
> hängen.
> Aktuell leuchten immer alle LEDs. Selbst wenn ich vorher nichtmal
> serpa_init() oder serpa_out() aufrufe.
> Liegt es womöglich an meiner Verkabelung?

Wohlmöglich, leider sehen wir die nicht, weil der Schaltplan fehlt.

Bist du sicher, dass das richtige Programm im AVR ist und du nicht 
ständig das falsche HEX-File flashst ("beliebtes" AVR-Studio Problem).

[Michael N. (garril)]

SERPA_SIZE habe ich auf 1 gestellt.

Die passende Datei wird auch übertragen
(mit der zusätzlichen serpa.h muss ich ja nichts mehr machen, wenn ich 
die als zusätzliches File dazugelegt habe und der Compiler auch keine 
Fehler auswirft, oder?)

Habe den Controller gerade auch nochmal komplett gelöscht und jetzt nur 
unter "Flash" das Hexfile draufgeladen (Controller war vorher nämlich 
bei einem anderen Textprojekt im Einsatz)

Habe eben auch einen Schaltplan entworfen und ihn angehängt.

Wie gesagt: Es leuchten einfach alle LEDs...
(Später ist normal eh nur eine an, weshalb die hier direkt am 74HC595 
hängen)

[spess53 (Gast)]

Hi

Wirklich PB5 an SER. Das ist normalerweise der RESET. Wenn du nicht die 
RSTDISBL-Fuse gesetzt hast ist das kein IO. Allerdings funktioniert dann 
ISP nicht mehr.

MfG Spess

[Krapao (Gast)]

Bzw. du taktest stets den Zustand am RESET Pin in das '595, also stets 
HIGH und damit ist das Leuchten aller LED logisch.

[Mikel (Gast)]

SET (PORTB, PB4);
verschieb  das mal am Anfang der Schleife, und wenn die schleife beendet 
wurd.

     for (bits = 8; bits > 0; bits--)
        {
            SET (PORTB, PB4);
            CLR (PORTB, PB5);
            if (data & 0x80)
            {
                SET (PORTB, PB5);
            }

            data <<= 1;
            /* Strobe an SCK schiebt Daten im Gaensemarsch */
            /* um 1 Position weiter durch alle Schieberegister */
            CLR (PORTB, PB4);
          
        }
    }
    while (--anz > 0);
     SET (PORTB, PB4);

[Michael N. (garril)]

Oh, den Reset-Widerstand im Schaltplan ganz vergessen.
Aso das hab ich garnicht bedacht (außerdem hängt dann SER ja immer über 
den Widerstand an +5V...)
umsteck und umprogrammier
Hach, ja...ES GEHT :)

Dankeschön. Ich hätte jetzt noch ewig gesucht^^
Schönen Sonntag noch an alle :)