ATtiny2313 USI und 74HC165

[Mathias (Gast)]

Ich versuche einen 74HC165 über USI einzulesen.

Dies habe ich folgendermassen probiert:

// Liest versetzt
char SPIShiftIn165() {
  USISR = 1 << USIOIF;

  while ((USISR & (1 << USIOIF)) == 0) {
    USICR = (1 << USIWM0) | (1 << USICS1) | (1 << USICLK) | (1 << USITC);
  }
  return USIDR;
}

Dabei wird ein Bit versetzt eingelesen, so wie es die shiftIn() Function 
von Arduino macht. Die habe ich fogendermassen abgespeckt:

// Liest versetzt
char shiftInArduino() {
  uint8_t value = 0;
  uint8_t i;

  for (i = 0; i < 8; ++i) {
    digitalWrite(Clock, true); // <--- beachten
    value |= digitalRead(DataIn) << (7 - i);
    digitalWrite(Clock, false);
  }
  return value;
}

Folgende Funktion arbeitet perfekt. Man beachte dabei das Schreiben von 
Clock.

// arbeitet korrekt
char shiftIn() {
  uint8_t value = 0;
  uint8_t i;

  for (i = 0; i < 8; ++i) {
    value |= digitalRead(DataIn) << (7 - i);
    digitalWrite(Clock, true); // <--- beachten
    digitalWrite(Clock, false);
  }
  return value;
}

Geht dies mit USI auch ?

Im Anhang noch die ganze Source.

[Stefan F. (Gast)]

Mathias schrieb:
> Geht dies mit USI auch ?

Ja schon, aber es wird dir im Vergleich zu deiner Bitbanging Methode 
kaum Vorteile bringen.

Ich würde allerdings digitalWrite() und digitalRead() durch direkte 
Port-Zugriffe ersetzen, das bringt eine ganze Menge:

#define CLOCK_HIGH    PORTB |= (1<<7)    // PB7 Arduino Pin 16 HIGH
#define CLOCK_LOW     PORTB &= ~(1<<7)   // PB7 Arduino Pin 16 LOW
#define READ_IN       (PINB & (1<<5))    // PB5 Arduino Pin 14

Die Bits würde ich so zusammen sammeln:

for (i = 0; i < 8; ++i) 
{
    value = value << 1;
    if (READ_IN) // if input is HIGH
    {
        value |= 1;
    }
    CLOCK_HIGH;
    CLOCK_LOW;
}

[Thomas W. (diddl)]

Stefan ⛄ F. schrieb:
> Die Bits würde ich so zusammen sammeln:
> for (i = 0; i < 8; ++i)
> {
>     value = value << 1;
>     if (READ_IN) // if input is HIGH
>     {
>         value |= 1;
>     }
>     CLOCK_HIGH;
>     CLOCK_LOW;
> }

Sehr schön optimiert.
Der GCC macht daraus einen sehr effizienten Code.

Das erste Shift kann man sich auch noch sparen.
Dann muss man allerdings auf for(;;) verzichten.

[Stefan F. (Gast)]

Thomas W. schrieb:
> Sehr schön optimiert.
> Der GCC macht daraus einen sehr effizienten Code.

Um das zu wissen, muss man allerdings leider ab und zu mal den 
generierten Assembler-Code lesen.

[Mathias (Gast)]

Danke erst mal.

>Ich würde allerdings digitalWrite() und digitalRead() durch direkte
>Port-Zugriffe ersetzen, das bringt eine ganze Menge:
Dies ist mir bekannt, diese beiden haben sehr viel Overhead.
In diesem Beispiel ging es nur um einen Muster-Code.


>Mathias schrieb:
>> Geht dies mit USI auch ?
>>
>>Ja schon, aber es wird dir im Vergleich zu deiner Bitbanging Methode
>> kaum Vorteile bringen.

Wie geht dies mit USI ?

[Stefan F. (Gast)]

Mathias schrieb:
> Wie geht dies mit USI ?

Steht im Datenblatt des Mikrocontrollers. Aber sei gewarnt, die USI 
Schnittstelle ist doof, die benutzt man nur wenn man muss.

[Ingo (Gast)]

Mathias schrieb:
> Wie geht dies mit USI ?
Ich habe da mal was ausgegraben:

uint8_t Usi_Data ( uint8_t Data )
{
  USIDR = Data;
  USISR |= (1<<USIOIF);
  while(!(USISR & (1<<USIOIF))){
    USICR |= (1<<USIWM0) | (1<<USICS1) | (1<<USICLK) | (1<<USITC);
  }
  return USIDR;
}

Setzt voraus, dass die USI-Pins als Ausgang geschaltet sind.

[Mathias (Gast)]

Ingo schrieb:
> Mathias schrieb:
>> Wie geht dies mit USI ?
> Ich habe da mal was ausgegraben:

uint8_t Usi_Data ( uint8_t Data )
 {
   USIDR = Data;
   USISR |= (1<<USIOIF);
   while(!(USISR & (1<<USIOIF))){
     USICR |= (1<<USIWM0) | (1<<USICS1) | (1<<USICLK) | (1<<USITC);
   }
   return USIDR;
 }

> Setzt voraus, dass die USI-Pins als Ausgang geschaltet sind.

Dies ist der gleiche Code, wie von mir, aus dem

USIDR = Data;