C: Pointerübergabe klappt nicht

[Marc (Gast)]

Hallo ihr Experten!

Ich hab ein Problem mit der Übergabe eines Pointers:

void TWI_Start_Transceiver_With_Data( unsigned char *msg, unsigned char msgSize)
{

  /*Write address and data */
  do
  {
    /* Write a byte */
    PORTB &=~(1<<SCL);            // Pull SCL low
    USIDR = *(msg++);            // Setup data
    TWI_Master_Transfer(tempUSISR_8bit);  // Send 8 bits on bus

    /* Clock and verify (N)ACK from slave */
    DDRB &=~(1<<SDA);            // Enable SDA as input
    if(TWI_Master_Transfer(tempUSISR_1bit) & 0x01) 
    {
      // NACK
    }
  }while( --msgSize);
}

... wird aufgerufen von...

#include <avr/io.h>
#include <avr/interrupt.h> 
#include <stdint.h>
#include "TWI_Master.h"

volatile uint8_t newuartdata;
volatile unsigned char uartin;

////////////////////////////////////////////////////////////////////////////////////////
// Hauptprogramm
////////////////////////////////////////////////////////////////////////////////////////

int main (void)
{
  cli();
  // Initialisierungen
  Ports_Init();
  Uart_Init();
  TWI_Master_Init();

  sei();
  // Hauptschleife
     while(1){
    if(newuartdata){
      newuartdata = 0;
      TWI_Start_Transceiver_With_Data((unsigned char)uartin,1);
    }
     }
     return 0;
}

////////////////////////////////////////////////////////////////////////////////////////
// Interrupt Service Routinen
////////////////////////////////////////////////////////////////////////////////////////

ISR(USART_RX_vect){
  uartin = UDR;
  newuartdata = 1;
}

Nun das Problem: wenn ich "uartin" mit einem Inhalt von zB 0x02 übergebe 
ist der Inhalt von "USIDR" dann immernoch der alte. Wo liegt der Fehler?

MfG Marc

[Karl H. (kbuchegg)]

Marc schrieb:

> Nun das Problem: wenn ich "uartin" mit einem Inhalt von zB 0x02 übergebe
> ist der Inhalt von "USIDR" dann immernoch der alte. Wo liegt der Fehler?

In dem Fall: Das du gecastet hast und die Fehlermeldung des Compilers 
nicht überdacht hast. (*)

      TWI_Start_Transceiver_With_Data( &uartin, 1 );

(*)
PS: Wenn der Compiler eine Warnung ausgibt, nach dem Motto "... makes 
Pointer from Integer without a Cast", dann ist ein Cast nicht immer die 
richtige Lösung. Manchmal muss man auch einfach nur die Adresse der 
Variablen übergeben, weil die empfangende Funktion eine Adresse haben 
möchte, von der sie sich die Daten holen kann, bzw. in die sie ihre 
Ergebnisse zurückschreibt.

[Marc (Gast)]

Danke Karl Heinz!

Manchmal ists doch einfach... In meiner Vorlage wurde dort ein Array 
verwendet und da brauchts natürlich kein & davor !

Aber der Compiler meldet jetzt :

../RS232_TWI.c:47: warning: passing argument 1 of 'TWI_Start_Transceiver_With_Data' discards qualifiers from pointer target type

Was bedeutet das?

[Karl H. (kbuchegg)]

Marc schrieb:

> Was bedeutet das?

uartin ist volatile.

Das geht aber verloren, wenn du einfach nur die Adresse an die Funktion 
übergibst.

Im Grunde läuft es darauf hinaus, dass du dir überlegen musst, in welche 
Schwierigkeiten TWI_Start_Transceiver_With_Data kommen wird, wenn sich 
die Daten (auf die es einen Pointer bekommt) verändern, während die 
Funktion arbeitet.

[Marc (Gast)]

Ok alles klar! Danke dir!