Bitfeld definition

SPDR=
((hc595_1.led1)+(hc595_1.led2*2)+(hc595_1.led3*4)+(hc595_1.led4*8)+(hc59 
5_1.led5*16)+(hc595_1.led6*32)+(hc595_1.led7*64)+(hc595_1.led8*128));
   SPDR=
((hc595_1.led1<<0)|(hc595_1.led2<<1)|(hc595_1.led3<<2)|(hc595_1.led4<<3) 
|(hc595_1.led5<<4)|(hc595_1.led6<<5)|(hc595_1.led7<<6)|(hc595_1.led8<<7) 
);

eins on beiden... keine ahnung ob es einfacher geht

So ginge es einfacher:

union {
  unsigned char Byte;
  struct {
    unsigned char led1:1;     // Bit 0
    unsigned char led2:1;     // Bit 1
    unsigned char led3:1;     // ...
    unsigned char led4:1;
    unsigned char led5:1;
    unsigned char led6:1;
    unsigned char led7:1;
    unsigned char led8:1;
  } Bitfeld;
} hc595_1;

Bit setzen:

hc595_1.Bitfeld.led1 = 1;

Byte beschreiben:

hc595_1.Byte = 0;


Man könnte sich natürlich auch mit den Bitoperationen von C
auseinandersetzen, dann wäre der ganze Aufriss nicht erforderlich.

Also mit Bitfeldern gibt das einen ziemlichen Codewust.

Deshalb arbeiten C-Programmierer lieber mit Masken und seltener mit
Bitfeldern.

Du hast 4 + 11 + 12 = 27 LEDs, das kann man bequem in ein long (32 Bit)
packen und die 4 Bytes schiebt man dann raus.

// min = 0..59, hour = 1..12

unsigned long led_bits( unsigned char min, unsigned char hour )
{
  unsigned long ledbits = 0;
  unsigned long bitmask;
  unsigned char i;

  i = min % 5;
  for( bitmask = 1<<0; i--; bitmask += bitmask ) // bit 0..3
    ledbits += bitmask;

  min /= 5;
  for( bitmask = 1<<4; min--; bitmask += bitmask ) // bit 4..14
    ledbits += bitmask;

  for( bitmask = 1<<15; hour--; bitmask += bitmask ) // bit 15..26
    ledbits += bitmask;

  return ledbits;
}


Peter

Hallo Peter,

zuerst mal vielen Dank.
> gibt es etwas, was Du nicht weißt ? <

Peter :

Wie muß ich diese Funktion ( ist doch eine, nicht?) einfügen?
Ich hab schon einiges versucht, aber irgendwie geht das nicht.

Ich muß die Funktion doch auch irgendwie aufrufen, nicht?
beim aufruf meckertr der Compiler,
irgendetwas mit "too far Arguments"

Könntest Du mir evtl. mal beschreiben, wie ich am besten eine Uhr
höchzähle, ich denke so :

und diese dann
1. an die 6 /Segmentanzeigen sende
2. wie das mit deinem Tip gehen soll


uint_8 Sekunden;
uint_8 Minuten;
uint_8 Stunden;
uint_8 Temp_ziffer;

prog_char
Segmentfolge[]={0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F,0x00,} 
;

void
main(void)
{
 if(Sekunden > 59)
 {
 Sekunden = 0;
 Minuten ++;
 }
  if(Minuten > 59)
  {
  Minuten = 0;
  Stunden ++;
  }
   if(Stunden > 12)
   {
   Stunden = 0;
   }
}

//Timer0 Overflow für Multiplexen der 7Segmentanzeigen

SIGNAL(SIG_OVERFLOW0)
{
if(Temp_ziffer == 6)
  {
  Temp_ziffer = 0;
  }
  Temp_ziffer++;      // Zifferauswahl erhöhen

switch (Temp_ziffer)
  {             //welche Ziffer soll angesteuert werden
   case 1 :
  PORTC= 0x01;           //1. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zl_minute);
  break;
   case 2 :
  PORTC= 0x02;           //2. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zh_minute);
         break;
   case 3 :
  PORTC= 0x04;           //3. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zl_minute5);
        break;
   case 4 :
  PORTC= 0x08;           //4. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zh_minute5);
        break;
   case 5 :
  PORTC= 0x10;           //5. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zl_stunde)
  break;
   case 6 :
  PORTC= 0x20;           //6. Ziffer
  PORTD = ~ PRG_RDB(Segmentfolge + Zh_stunde);

  // Nullstelle Stunde Zehner unterdrücken
  if (Zh_stunde == 0)
    {
    PORTD = ~ PRG_RDB(Segmentfolge + 11);
    }
  break;
  }
}

In der Interruptroutine steht allerdings noch einiges falsches,
da ich im Moment ja noch die Stellen einzeln hochzähle.
Das muß ja auch noch geändert werden, da ich ja die Minuten und Stunden
von 0 bis 59 zählen soll.

Wie ???

Gruß Toby

Hallo Nochmal,

ich habe es nun etwas hin bekommen.

Der Versuch macht schlau !!!

Anliegend meine bisherigen Programmier Künste.

Peter :

Mit den Minuten und 5er Minuten klappt einwandfrei! DANKE.

Nur sind bei Stunden == 0 alle LEDs aus, das ist ok.
Bei Stunde == 1 sind Bit 15 bis 31 an ?!
Bei Stunde == 2 geht Bit 16 aus.
Bei Stunde == 3 gehen Bit 16 und 17 aus.
...
Bei Stunde == 12 gehen Bit 16 bis 26 aus.
Bei Stunde == 13 ist Stunde == 0 und es sind wieder Bits 15 bis 31 an.

Da ich deine rechnung noch nicht so ganz verstehe, könntest du mir da
weiterhelfen?

Anliegend mein Code für WinAvr.

Gruß Toby

Hallo Nochmal.

Einen kleinen nachtrag :

Wenn ich Minuten statt 4 mal 5 mal schiebe, erhalte ich zwischen den
4Minuten Leds und den 11 5Minuten Leds eine Lücke von einer Led.

for( bitmask = 1<<4; minute--; bitmask += bitmask ) // bit 4..14
    ledbits += bitmask;

for( bitmask = 1<<5; minute--; bitmask += bitmask ) // bit 4..14
    ledbits += bitmask;

daher machte ich dies mal mit den Stunden:

Ich schiebe also 16 mal:
Die LEDs bleiben dunkel.

wenn ich nun aber nur 14 mal schiebe, zeigen die stunden korrekt an!
Allerdings überschneiden sich die Stunden dann mit der 11. 5er Minute.

for( bitmask = 1<<14; stunde--; bitmask += bitmask ) // bit 15..26
    ledbits += bitmask;

Muß man evtl. im Bezug auf die Stunden die Minuten mit bedenken?

Gruß Toby