Code fuer effiziente Schieberegisteransteuerung

[sammy (Gast)]

Hi, ich habe zwei 74HC164 Schieberegister (Serial In, Parallel Out) an
einem 2313 haengen.
Ich komme mit der effizienten Ansteuerung allerdings nicht so ganz
klar, das braucht alles zu viele Programmzeilen.
Ich braeuchte eine Moeglichkeit es moeglichst effizient anzusteuern,
wenn ich z.B. die Bitfolge 101010101 am Ausgang haben moechte, oder
weiss, das z.B. die ersten 4 oder letzten 4 Ausgaenge des
Schieberegisters eine 1 haben sollen....

Habt ihr da so was auf Anhieb parat?

[Sebastian__ (Gast)]

kopiere doch dein Datenbyte in ein Register, zb r16, jetzt kannst du mit
jedem Clock impuls den du erzeugst das register mit lsr nach rechts
schieben, nach jedem shift fragst du dann das bit 0 von r16 mit (sbrs
r16,0) ab und gibst endweder ein low oder ein high auf den IO port.
nach 8 mal shiften bist du feritg.


MfG
Sebastian

[Peter D. (peda)]

Um sagen zu können, was an Deinem Code nicht effizient ist, müßtest Du
ihn doch erstmal hier rein stellen (als Dateianhang).

Oder hast du etwa gar keinen und willst nur das fertige Programm.


Peter

[sammy (Gast)]

Code habe ich schon, halt auf meine typische "fuer den PC schnell
zusammengebastelt Art" programmiert....


/*
;  PB0 : Schiebe1 RESET
;  PB1 : Schiebe1 CLOCK
;  PB2 : Schiebe1 DATA
;  PB3 : LED TOO HIGH Max
;  PB4 : LED TOO LOW Max
*/

static void led_schiebe_hochzaehlen(void)
{
    sbi(PORTB,1);// Schieberegister hochzaehlen
    cbi(PORTB,1);
    cbi(PORTB,0);
}


/*
static void led_ansteuerung(unsigned short int amount) // 10 LEDs
ansteuern
{
  sbi(PORTB,0); //Schieberegister resetten
  cbi(PORTB,0);
    if (amount<6) // TOO HIGH LEDS
    {
      if (amount==5)
      {
        sbi(PORTB,3);
      }else{cbi(PORTB,3);}
      if (amount>=4)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=3)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=2)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=1)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
    }
    else if (amount>5) //TOO LOW LEDS
    {
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      led_schiebe_hochzaehlen();
      if (amount>=6)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=7)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=8)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount>=9)
      {
        sbi(PORTB,1);
      }
      led_schiebe_hochzaehlen();
      if (amount==10)
      {
        sbi(PORTB,4);
      }else{cbi(PORTB,4);}
    }
}

[Peter D. (peda)]

So richtig kann ich nicht erkennen, was Dein Code machen soll.


Anbei eine Routine zum Ausgeben eines Bytes.
Für 2* 74HC164 hintereinander must Du 2 Bytes raussschieben, d.h. diese
Routine 2* aufrufen.

Resetten brauchst Du nicht, nach 16 Takten sind doch eh alle alten Bits
rausgeschoben.

Sieht so aus, als ob Du mit PB3 und PB4 noch ein extra Latch
ansteuerst. Dann nimm doch besser gleich den 74HC595.
Auch reicht ein Latchsignal aus, um alle 16 Bits gleichzeitig zu
übernehmen.


Peter

[Minos (Gast)]

also laut doc unter den faqs soll man das so machen:

PORTB &= (unsigned char)~(1<<DMOSI);

da der operand ~ ein 16 bit aus der anweisung macht.

sollte der compiler nicht wenigstens nen warning rauswerfen?

[Peter D. (peda)]

"sollte der compiler nicht wenigstens nen warning rauswerfen?"

Warum, das Ergebnis stimmt doch auch bei 16 Bit.

Allerdings sollte ein 8Bit Compiler die Operation auch auf 8Bit
optimieren, wenn der Zieloperand nur 8Bit ist. Und es macht wenig Sinn,
das High Byte zu berechnen und es dann niemandem zuzuweisen.

Aber im Zweifelsfalle gilt natürlich immer "nimm lieber ein cast
mehr", schaden kanns ja nicht.


Peter

[sammy (Gast)]

Gibt es zu dem ganzen irgendwo ein gutes Tutorial, steige so durch die
Befehle nicht ganz durch. :-/

[Peter D. (peda)]

Du solltest Dir ein C-Lehrbuch beschaffen, wo die Operatoren
(+,-,|,&,||,&&,<<,~ usw.) alle erklärt sind.


Bestimmt findest Du auch entsprechende Seiten im Web.


Peter

[Minos (Gast)]

>> "sollte der compiler nicht wenigstens nen warning rauswerfen?"
> Warum, das Ergebnis stimmt doch auch bei 16 Bit.

ich habe mal für kurze zeit mit dem wickenhäuser flash_m1 und dem
compiler gearbeitet. der hat zumindest nen warning generiert.

[Matthias (Gast)]

Hi

http://www.schellong.de/c.htm ist garnicht schlecht.

Matthias

[Peter D. (peda)]

@Minos

"der hat zumindest nen warning generiert."


Und wie lautet diese ?


Peter

[Minos (Gast)]

test.c 49: Warning: overflow in converting constant expression from
'int' to 'unsigned char'

[Peter D. (peda)]

@Minos,

das darf aber nur dann kommen, wenn Du für DMOSI Werte >7 einsetzt,
also versuchst, einen nicht existenten 9. Portpin anzusprechen.

Ansonsten ist so eine Warnung schlichtweg falsch und der Compiler
Schrott.

Eine 1 um 7 geschiftet ist nur 128 und das paßt noch dicke in ein
unsigned char.


Peter

[Minos (Gast)]

yo haste recht. ich arbeite eh nicht mehr mit dem und das ganze war nur
mal zum antesten! ich halte eh die ganze reihe 80CXXX für veraltet.