24bit wert in 3 bytes aufteilen

[Oliver F. (ollif)]

Hallo,

ich möchte einen LTC2756 18 Bit DAC über SPI an einem Atmega32 
ansteuern.

Initialisieren funktioniert Ausgabe von Werten mittels Byteweiser 
Übergabe
funktioniert auch (high mid Low)

Wenn ich die untenaufgeführte Funktion in einer Schleife von 0 bis 2^18 
-1 aufrufe geht alles gut bis zu dem Wert 65536.

Hier springt meine Spannung wieder auf 0.

Welchen Fehler mache ich hier?

uint16_t SPIWrite24(uint32_t Data)
{

  uint16_t b1=Data&0xff;
  uint16_t b2=(Data>>8)&0xff;
  uint16_t b3=(Data>>16)&0xff;
  uint16_t b4=(Data>>24)&0xff;
  

  SPDR = b2;              
  while(!(SPSR & (1<<SPIF)));          
  
  SPDR = b3;              
  while(!(SPSR & (1<<SPIF)));          
  
  SPDR = b4;              
  while(!(SPSR & (1<<SPIF)));        

  return SPDR;
}

Vielen Dank im Vorraus.


Gruß

Oliver

[Wolfgang (Gast)]

Oliver F. schrieb:
> Wenn ich die untenaufgeführte Funktion in einer Schleife von 0 bis 2^18
> -1 aufrufe geht alles gut bis zu dem Wert 65536.

Und wie sehen Schleife und zugehörige Variablendeklarationen aus?

[lambda (Gast)]

Wie sieht der Parameter aus, den du der Funktion übergibst? Aktuell 
sendest du nämlich nur 2. bis 4. Byte deines 32-Bit-Parameters und das 
auch in der vermutlich falschen Reihenfolge.
Dass der Chip noch einen Controllcode zusammen mit dem DAC-Wert möchte 
beachtest du in deiner Schleife?

Es würde übrigends reichen, für b1 bis b4 uint8_t zu nehmen, dann kannst 
du dir auch das &0xFF sparen um auf 8 Bit zu kürzen.

[Oliver F. (ollif)]

HAllo

schon mal Danke,


hier ist ein weitere Code Auschnitt:

Ich durchalaufe hier die Tabelle von 0 -71 was bis zum wert 61860 auch 
funktionier dann fangt die Spannung wieder bei O an bei 131072 passiert 
das wieder.

Es muss irgendwie an der Konvertierung liegen

//Wertetabelle Steuerspannung

volatile uint32_t cvs[] = {  4096, 4340, 4598, 4871, 5161, 5468, 5793, 6138, 6502, 6889, 7299, 7733,
      8192, 8679, 9195, 9742, 10322, 10935, 11586, 12275, 13004, 13778, 14597, 15465,
      16384,17358, 18390, 19484, 20644, 21870, 23171, 24550, 26008, 27555, 29194, 30930,
      32768, 34715, 36779, 38968, 41288, 43739, 46341, 49100, 52016, 55110, 58387, 61860,
      65536, 69429, 73558, 77936, 82576, 87478, 92682, 98200, 104032, 110219, 116773, 123719,
      131072, 138858, 147116, 155871, 165151, 174955, 185363, 196399, 208064, 220437, 233545, 247438,
      262143};


//initialisierung DA
void DAC_init()
{
  //Span input register range set to unipolar 0V to Vref
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  SPIWrite8(32);//command: 
  SPIWrite8(0);
  SPIWrite8(0); //span  0V to Vref
  SPIWrite8(0);
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  
  //Code input Register code Set to mid scale (Vref/2)
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  SPIWrite8(48);
  SPIWrite8(128);
  SPIWrite8(0);
  SPIWrite8(0);
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  
  //update code and range
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  SPIWrite8(64);
  SPIWrite24(0);
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  
  //
}


void DACWrite(uint32_t data)
{
  //Write Data
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  SPIWrite8(48);
  SPIWrite24(data);
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  
  //update code and range
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  SPIWrite8(64);
  SPIWrite24(0);
  
  PORTB &= ~(1 << DD_SS);
  PORTB |= (1 << DD_SS);
  PORTB &= ~(1 << DD_SS);
  
}

int main(void)
{

  SPI_init();

  DAC_init();



  // Durchläuft Steuerspannung für 72 Tasten 6 Oktaven

  for (int i = 0;i<72;i++)
  {
    DACWrite(cvs[i]);
    Lcd4_Clear();
    Lcd4_Set_Cursor(1,0);
    fprintf(stderr, "i =%d \n",i);
      
    Lcd4_Set_Cursor(2,0);
    fprintf(stderr, "i = %ld \n", cvs[i]);
      
    _delay_ms(500);
  }



}

Gruß

Danke

[lambda (Gast)]

Also soweit ich das beim überfliegen sehe schreibst du erst den 
Steuercode 48 = 0011 0000 als 8 Bit Wert, soweit korrekt. Allerdings 
macht deine Funktion SPIWrite24() für mich wenig Sinn. Die sollte doch 
eigentlich den Parameter passend aufbereiten und dann raussenden?
Von den restlichen 24 Bit die nach dem Steuercode zu senden sind, sind 
die ersten 18 der Wert für den DAC und danach folgen noch 6 Bit 
Don't-care. Du sendest allerdings die niederwertigsten 8 Bit garnicht 
welche in b1 liegen. Danach folgt dann in der falschen Reihenfolge der 
Rest, das heißt die letzten 6 Bit von b4 werden vom DAC dann auch 
ignoriert.

Dass da überhaupt etwas sinnvolles rauskommt ist finde ich echt 
erstaunlich. Ich würde dir empfehlen erstmal den Code dahingehend zu 
bearbeiten, dass der Wert korrekt rausgesendet wird, vermutlich erledigt 
sich dadurch dann auch das von dir beobachtete Fehlverhalten.

Nachdem deine Eingangswerte maximal 18 Bit haben würde ich in der 
Funktion SPIWrite24() die Daten erstmal um 6 nach links shiften und dann 
die unteren 3 Byte beginnend mit dem höchstwertigsten versenden.

Also sowas wie
Data = Data << 6;
 while(!(SPSR & (1<<SPIF)));
SPDR = (Data >> 16);
 while(!(SPSR & (1<<SPIF)));
SPDR = (Data >> 8);
 while(!(SPSR & (1<<SPIF)));
SPDR = Data&0xFF;

[Wolfgang (Gast)]

Oliver F. schrieb:
> Es muss irgendwie an der Konvertierung liegen

Tut es wohl auch.

Deine Konstanten sollten vom Datentyp unsigned /long/ sein, also z.B. 
165151UL

[Oliver F. (ollif)]

Hallo Lambda,

Das war es!

Jetzt klappt es.

Vielen Dank jetzt kann ich weiter an meinem Synthesizer werkeln.


Gruß

Oliver