Bitshift für Schieberegister C-Code

[Michael I. (inglinm)]

Hallo Zusammen

Ich habe mir die lcd-routines umgeschrieben um ein lcd mit ks00066 mit 
drei ainschlüssen via 74hc164 anzusteuern.

Beim angefügten c-code bin ich nicht ganz sicher ob er sinngemäss 
korrekt ist (compilieren geht).

Idee Das Datenbit wird gesetzt oder auch nicht, dann wird der bitshift 
port aktiviert und danach beide ports wieder null gesetzt, das ganze 
achtmal.

funktioniert das so? hab leider im moment kein zugriff zur hardware um 
zu testen... vielen dank für anregungen und tipps

static void lcd_out( uint8_t data )
{  
  for ( uint8_t i=0; i<8; i++ )
    {           
      LCD_PORT |= (((data>>i)&1)<<LCD_DB);         // DB Bit setzen 
    
      LCD_PORT |= (1<<SRSHIFT);                      // Bitshift in SR
      _delay_us( SR_ENABLE_US );                     // kurze Pause
      LCD_PORT &= ~((1<<SRSHIFT)|(1<<LCD_DB));       // Data Pin und SR Pin Null setzen
  }   
    
}

[holger (Gast)]

>funktioniert das so? hab leider im moment kein zugriff zur hardware um
>zu testen... vielen dank für anregungen und tipps

Na dann warte halt bis die Hardware wieder zur
Verfügung steht. Wenn es Probleme gibt fragst du noch mal.

[Thomas E. (thomase)]

Michael I. schrieb:
> via 74hc164 anzusteuern

Schlechte Wahl.
Während des Schiebens werden die Bits sofort ausgegeben. Das könnte dein 
Display ganz schön durcheinander bringen.
Nimm' lieber ein 74HC595. Damit kannst du erst die Daten reinschieben 
und dann alle auf einmal auf die Ausgänge legen.

Michael I. schrieb:
> Idee Das Datenbit wird gesetzt oder auch nicht, dann wird der bitshift
> port aktiviert und danach beide ports wieder null gesetzt, das ganze
> achtmal.
Schlechte Idee.
Bessere Idee: SPI.

mfg.

[Michael I. (inglinm)]

Wieso ist HC164 eine schlechte Wahl? es macht ja nichts wenn die Daten 
sofort ausgegeben werden oder? soviel ich weiss werden die doch erst mit 
dem enable puls übernommen...

[Thomas E. (thomase)]

Michael I. schrieb:
> soviel ich weiss werden die doch erst mit dem enable puls übernommen...
Aber nur wenn der direkt vom Controller kommt.

mfg.

[Peter D. (peda)]

Thomas Eckmann schrieb:
> Michael I. schrieb:
>> via 74hc164 anzusteuern
>
> Schlechte Wahl.

Nö, der ist optimal.
Du brauchst nur 3 Leitungen zum MC:
- 164-Data/LCD-RS
- 164-Clock
- LCD-E


Thomas Eckmann schrieb:
> Nimm' lieber ein 74HC595.

Macht die Sache nur deutlich komplizierter.


Peter

[holger (Gast)]

http://frank.bol.ucla.edu/AVR2LCD.htm

[Michael I. (inglinm)]

Ja das kommt er. 1 Pin wird für enable gebraucht, 1 Pin für Bitshift, 
und der dritte Pin für setzen der bits im Register, sowie als befehls 
oder dateneingabe ans lcd...
wollte nicht gleich die ganze posten (um nicht alle abzuschrecken)

Im Anhang nun die modifizierten files...

Danke für den Typ mit SPI, aber gehöre eher noch in die 
Anfängerkategorie und werde mich später mal damit auseinandersetzen

[Michael I. (inglinm)]

>
> Nö, der ist optimal.
> Du brauchst nur 3 Leitungen zum MC:
> - 164-Data/LCD-RS
> - 164-Clock
> - LCD-E
>
>
> Thomas Eckmann schrieb:
>> Nimm' lieber ein 74HC595.
>
> Macht die Sache nur deutlich komplizierter.
>
>
> Peter

Genau so ist es gedacht. beim link von holger ist auch noch die 
verdrahtung aufgezeichnet (bei mir einfach übers kreuz) da lcd_out wohl 
anders ist...

doch zur ursprünglichen frage, tut der code was er soll?

[Thomas E. (thomase)]

Michael I. schrieb:
> doch zur ursprünglichen frage, tut der code was er soll?
Ja.

mfg.

[Lukas K. (carrotindustries)]

Beim Elektor gab's sowas auch mal.
Dort war die E-Leitung einfach das letzte Bit im Schieberegister.
Im übrigen sind Operationen im Stil 1<<i auf AVRs 'teuer'; es ist besser 
pro interation eine Variable einfach um 1 weiterzuschieben und ein 
festes Bit abzufragen.
Für 16 bit kann das dann so aussehen:

void dac_raw(uint16_t b) {
  DAC_PORT &= ~(1<<CS_P);
  unsigned char i = 16;
  while (i > 0) {
    DAC_PORT |= (1<<SCK_P);
    if(b&(1<<15)) {
      DAC_PORT |= (1<<D_P);
    }
    else {
      DAC_PORT &= ~(1<<D_P);
    }
    b <<= 1;
    i--;
    _delay_us(5);
    DAC_PORT &= ~(1<<SCK_P);
    _delay_us(5);
    
  }
  DAC_PORT |= (1<<CS_P);
}

[Michael I. (inglinm)]

Betreffend optimierung meinst du sowas oder?

static void lcd_out( uint8_t data )
{  
  for ( uint8_t i=0; i<8; i++ )
    {           
      LCD_PORT |= ((data&1)<<LCD_DB);         // DB Bit setzen 
    
      LCD_PORT |= (1<<SRSHIFT);                      // Bitshift in SR
      _delay_us( SR_ENABLE_US );                     // kurze Pause
      LCD_PORT &= ~((1<<SRSHIFT)|(1<<LCD_DB));       // Data Pin und SR Pin Null setzen
      data >>= 1 }   
    
}