GCC Komisches Verhalten

[Thom (Gast)]

Hallo,

ich habe folgendes Problem, ich übergebe an eine Funktion einen 96 Byte 
großen Buffer, wenn ich jedoch in die Funktion Springe ist dieser immer 
mit 0x00 (1 Byte) übgergeben.

Anbei der Source

int8_t as3911WriteFifo(const uint8_t* values, uint8_t length)
{
    int8_t err;
    uint8_t cmd = AS3911_FIFO_LOAD;
  cmd = 0x80;
    /* make this operation atomic */
    cpu_irq_disable();//IRQ_INC_DISABLE();
    {
    as3911_SPI_Select();
        {
            err = spi_master_transfer(cmd, 1);

            if (ERR_AMS_NONE == err)
            {
                err = spi_master_transfer(&values,  length);
            }
        }    
    cpu_irq_enable();//IRQ_DEC_ENABLE();
        as3911_SPI_DeSelect();

    }
    return err;
}

Ich habe noch 2 Bilder angehänt vom Debuggen wo man es sehen sollte.

[Peter II (Gast)]

Thom schrieb:
> spi_master_transfer(&values,  length);

was macht das & da?

Vermutlich falsch, ich glaube kaum das spi_master_transfer ein Zeiger 
auf Zeiter erwartet.

[Thom (Gast)]

Die Funktion ist bis dato immer so gegangen, ich habe noch einige andere 
befehle die das genau so nutzen...

Anbei der Code des Transfer:

/**
 * \brief Perform SPI master transfer.
 *
 * \param pbuf Pointer to buffer to transfer.
 * \param size Size of the buffer.
 */
int8_t spi_master_transfer(void *p_buf, uint32_t size)
{
  uint32_t i;
  uint8_t uc_pcs;
  static uint16_t data;
  int8_t err;

  uint8_t *p_buffer;

  p_buffer = p_buf;

  for (i = 0; i < size; i++) {
    err = spi_write(SPI_MASTER_BASE, p_buffer[i], 0, 0);
    /* Wait transfer done. */
    while ((spi_read_status(SPI_MASTER_BASE) & SPI_SR_RDRF) == 0);
    err = spi_read(SPI_MASTER_BASE, &data, &uc_pcs);
    p_buffer[i] = data;
  }
  return err;
}

[Thom (Gast)]

Leider wird der Buffer aus dem Bild sentFifo.jpg nicht an die Funktion 
aus dem Bild wFifo übergeben...

[Peter II (Gast)]

Thom schrieb:
> Die Funktion ist bis dato immer so gegangen, ich habe noch einige andere
> befehle die das genau so nutzen...

kann kaum sein, du hast schon ein zeiger und die Funktion braucht einen 
Zeiger. Damit ist das & falsch.

Lass es doch einfach mal weg.

[Max H. (hartl192)]

Thom schrieb:
> spi_master_transfer(cmd, 1);
> spi_master_transfer(&values,  length);
Du kannst nicht einmal einen Zeiger, das andere Mal einen Wert übergeben 
cmd wird als Adresse (0x80) interpretiert.

Mit &values erhältst du einen Zeiger auf den Zeiger uint8_t* values und 
nicht einen Zeiger auf das was im (vermutlich) Array steht.

[Peter II (Gast)]

Nachtrag:

das kann doch alles nicht ordentlich funktionieren.

 cmd = 0x80;
 err = spi_master_transfer(cmd, 1);

hier übergibst du einen Wert, die Funktion braucht aber ein zeiger


 err = spi_master_transfer(&values,  length);
hier übergibst du einen Adresse von einer Zeiger - auch falsch




spi_master_transfer ändert die Daten in p_buffer.
Aber du hast es als const übergeben

const uint8_t* values

[Thom (Gast)]

Wenn ich es ändere bringt es auch nichts...

Mein Problem ist das die Funktion

int8_t as3911WriteFifo(const uint8_t* values, uint8_t length)

schon keine Daten enthält siehe Bild wFifo.jpg dort ist nichts in dem 
Array (bei anderen Funktionen schon) und übergeben an die oben genannte 
Funktion übergebe ich das sentFifo.jpg angezeigte Array...

Alle anderen Funktionen die ich Debugge zeigen mir immer das Quell Array 
an bzw. dessen Daten...

[Peter II (Gast)]

Thom schrieb:
> Mein Problem ist das die Funktion
>
> int8_t as3911WriteFifo(const uint8_t* values, uint8_t length)
>
> schon keine Daten enthält siehe Bild wFifo.jpg dort ist nichts in dem
> Array (bei anderen Funktionen schon) und übergeben an die oben genannte
> Funktion übergebe ich das sentFifo.jpg angezeigte Array...

ich sehen keine Fehler - die Ausgabe ist richtig. Die Debugger kennt die 
länge von values nicht und der erste warte ist 0 - auf beiden 
Screenshots.

[Max H. (hartl192)]

values ist ein kein Array, sondern ein Pointer auf das erste Element 
von, in diesem Fall, as3911StreamBuffer, welches null ist. Damit ist es 
auch vollkommen korrekt, dass *values gleich 0x00 ist.

[Jim M. (turboj)]

Thom schrieb:
>            err = spi_master_transfer(cmd, 1);`
>
>             if (ERR_AMS_NONE == err)
>             {
>                 err = spi_master_transfer(&values,  length);
>             }

Falsch herum.
Bei der Zeile `err = spi_master_transfer(cmd, 1);` muss der Compiler 
auch eine Warning ausspucken. Wenn er das nicht tut, schalte die mal an 
(-Wall -Wextra im GCC).

So herum ist richtig:

err = spi_master_transfer(&cmd, 1);

if (ERR_AMS_NONE == err)
{
  err = spi_master_transfer(values,  length);
}

Falls Du Dich fragst: Warum so herum? Dann solltest Du Dir ein gutes 
Buch über die C Programmiersprache besorgen und mal durchlesen. C ist 
nicht selbsterklärend und auch nicht so leicht erlernbar.