Soft SPI 24 Bit

[Steffen (Gast)]

Hallo zusammen,

ich bin eben am verzewifeln 24bit am Stück per SPI zu übertragen. Dazu 
habe habe ich folgenden SoftSPI genutzt. Es geht bis 16bit ohne 
Probleme, dann eher nicht mehr. Ich nutze zum Senden einen Tiny85. Der 
Empfänger der Daten ist ein DAC, der aber nur einen DI, SCK und CS 
Eingang hat. d.h. DO oder MISO gibt es nicht.

Wo liegt mein Fehler?

uint32_t spi_tx(uint32_t data)
{  unsigned char i;

  cbi(spi_PORT, CS);
  for (i=0; i<24; i++)
  {
    if (data&0x7FFFFF)
    sbi(spi_PORT, SDI);
    else
    cbi(spi_PORT, SDI);
    data<<=1;
    sbi(spi_PORT, SCK);
    asm("nop");
    asm("nop");
    cbi(spi_PORT, SCK);
  }
  sbi(spi_PORT, CS);
  return data;
}

Aufrufen tu ich das ganze mit:

spi_tx(0x30FE00);

[Jim M. (turboj)]

Darüber solltest Du nochmal nachdenken:

if (data&0x7FFFFF)

Hint: In C ist true alles was ungleich 0 (Null) ist.

[Trumpelstielzchen (Gast)]

Steffen schrieb:
> if (data&0x7FFFFF)
soll vermutlich heißen

if (data&0x800000)

[Dr. Sommer (Gast)]

Steffen schrieb:
> if (data&0x7FFFFF)

Was soll das denn und wie soll das mit 16 Bit funktionieren?? Mit 
0x800000 sollte es besser gehen...

[Dr. Sommer (Gast)]

Steffen schrieb:
> data<<=1;

PS: Eventuell kann der Compiler das besser optimieren wenn du 
stattdessen
data = (data << 1) & 0xFFFFFF;
und am Anfang der Funktion einmal:
data = data & 0xFFFFFF;
schreibst. Probier das mal aus, ob da die unnötige Berechnung des 4. 
Registers verschwindet...

[Steffen (Gast)]

Dr. Sommer schrieb:
> 0x800000

jo das war es... vielen Dank :-)

[eProfi (Gast)]

void spi_tx(uint32_t data){
  uint32_t mask=0x800000;
  cbi(spi_PORT, CS);
  do{
    if(data&mask)sbi(spi_PORT, SDI);
    else         cbi(spi_PORT, SDI);
    sbi(spi_PORT, SCK);
    asm("nop");asm("nop");
    cbi(spi_PORT, SCK);
  }while((mask>>=1)!=0);
  sbi(spi_PORT, CS);
}

Oder noch besser jedes Byte einzeln rausschieben,
dann muss man nicht jedes mal jedes Mal 4 Bytes schieben.

union{
  uint32_t d32;
  uint8_t  d08[4];
} u;

void spi_tx(uint32_t data){
  uint8_t mask=0x80; u.d32=data;
  cbi(spi_PORT, CS);
  do{
    if(u.d08[2]&mask)sbi(spi_PORT, SDI);
    else             cbi(spi_PORT, SDI);
    sbi(spi_PORT, SCK);
    asm("nop");asm("nop");
    cbi(spi_PORT, SCK);
  }while((mask>>=1)!=0);

  mask=0x80; do{
    if(u.d08[1]&mask)sbi(spi_PORT, SDI);
    else             cbi(spi_PORT, SDI);
    sbi(spi_PORT, SCK);
    asm("nop");asm("nop");
    cbi(spi_PORT, SCK);
  }while((mask>>=1)!=0);

  mask=0x80; do{
    if(u.d08[0]&mask)sbi(spi_PORT, SDI);
    else             cbi(spi_PORT, SDI);
    sbi(spi_PORT, SCK);
    asm("nop");asm("nop");
    cbi(spi_PORT, SCK);
  }while((mask>>=1)!=0);
  sbi(spi_PORT, CS);
}

Warum gibst Du data wieder mit return zurück?