C - Casting Verständniss Problem

[Marcus (Gast)]

Was mache ich falsch? Bin gerade fast am heulen kopf gegen wand

void foo( void *dest )
{
  DMA.CH0.DESTADDR0 = (uint8_t)(((uint32_t) dest) & 0xFF);
  DMA.CH0.DESTADDR1 = (uint8_t)(((uint32_t) dest >> 8) & 0xFF);
  DMA.CH0.DESTADDR2 = (uint8_t)(((uint32_t) dest >> 16) & 0xFF);
}

Wenn ich versuche das zu kompilieren, kommt immer ein warning. 
Wahrscheinlich zu Recht..

Severity  Code  Description  Project  File  Line
Warning    cast from pointer to integer of different size [-Wpointer-to-int-cast]  XMEGA DMA  c:\users\marcus\Documents\Atmel Studio\7.0\XMEGA DMA\XMEGA DMA\main.c  14

[STK500-Besitzer (Gast)]

bei deinen Shifts fehlt ein * (Stern)

[Marcus (Gast)]

Ganz sicher???

[Jim M. (turboj)]

Xmega hat IIRC nur 16-bittige Pointer..?

Versuche mal die uint32_t durch uintptr_t zu ersetzen.

[Jim M. (turboj)]

STK500-Besitzer schrieb:
> bei deinen Shifts fehlt ein * (Stern)

Nö. Er versucht auf der Adresse zu rechenen.

[Sam W. (sam_w)]

Das sieht doch eher so aus, als würde bewusst mit den Adressen 
gearbeitet, auf die "dest" zeigt.
Eventuell liegt es einfach daran, dass Zeiger (soweit ich weiß, bitte 
berichtigt mich sonst) normalerweise die Breite der Systemarchitektur 
aufweisen und die kleiner als 32 Bit ist.

Gruß
Sam

[Reiner_Gast (Gast)]

Marcus schrieb:
> Was mache ich falsch?

Hier mal schauen, lesen, verstehen:

https://www.tutorials.at/c/11-zeiger.html

[Marcus (Gast)]

Jim M. schrieb:
> Xmega hat IIRC nur 16-bittige Pointer..?
>
> Versuche mal die uint32_t durch uintptr_t zu ersetzen.

Das sieht schon besser aus.

{
  DMA.CH0.DESTADDR0 = (uint8_t)((uintptr_t) dest) & 0xFF;
  DMA.CH0.DESTADDR1 = (uint8_t)((uintptr_t) dest >> 8) & 0xFF;
  DMA.CH0.DESTADDR2 = (uint8_t)((uintptr_t) dest >> 16UL) & 0xFF;
}

Wenn der XMEGA nur 16 Bittige Zeiger hat, wieso gibt es dann das 
Register "DESTADDR2" ?
Es kommt jedoch jetzt immer noch eine Warnung. Ist ja auch verständlich. 
Er meckert das dass schieben überläuft..

[Tobias B. (Firma: www.elpra.de) (ttobsen)]

Auf was für einem System entwickelst du? 32 oder 64 bit?

Kannst du mal ausgeben was die Größe des void Pointers ist?

sizeof(void*)

Evtl. liegt da der Hund begraben.

[Marcus (Gast)]

Tobias B. schrieb:
> Auf was für einem System entwickelst du? 32 oder 64 bit?
>
> Kannst du mal ausgeben was die Größe des void Pointers ist?
> sizeof(void*)
> Evtl. liegt da der Hund begraben.

Das ist für nen XMEGA, schreibe den Source in C. Kann die Größe jetzt 
leider nicht ausgeben. Mein Rechner ist jedoch ein 64 Bit. Aber das 
sollte damit doch nichts zu tun haben?

[foobar (Gast)]

Dein Problem ist, dass du DMA-Adressen und C-Pointer mischt. Bei einigen 
CPU-Architekturen sind die nicht identisch. Die XMEGA haben (bei deinem 
Compiler) 16-Bit C-Pointer aber 24-Bit DMA-Adressen. Mach dir eine 
Hilfsfunktion, die von C-Ptr nach DMA-Adresse umwandelt - im trivialen 
Fall ist das ein Cast, bei einigen Architekturen ist aber mehr nötig 
(Adressoffsets, etc) ...

typedef uint32_t dmaaddr_t;
#define cptr2dma(cptr) ((dmaaddr_t)(uintptr_t)(cptr))

...
{
  dmaaddr_t addr = cptr2dma(dest);

  DMA.CH0.DESTADDR0 = addr;
  DMA.CH0.DESTADDR1 = addr >> 8;
  DMA.CH0.DESTADDR2 = addr >> 16;
}

[Marcus (Gast)]

foobar schrieb:
> Dein Problem ist, dass du DMA-Adressen und C-Pointer mischt. Bei
> einigen
> CPU-Architekturen sind die nicht identisch. Die XMEGA haben (bei deinem
> Compiler) 16-Bit C-Pointer aber 24-Bit DMA-Adressen. Mach dir eine
> Hilfsfunktion, die von C-Ptr nach DMA-Adresse umwandelt - im trivialen
> Fall ist das ein Cast, bei einigen Architekturen ist aber mehr nötig
> (Adressoffsets, etc) ...
> typedef uint32_t dmaaddr_t;
> #define cptr2dma(cptr) ((dmaaddr_t)(uintptr_t)(cptr))
>
> ...
> {
>   dmaaddr_t addr = cptr2dma(dest);
>
>   DMA.CH0.DESTADDR0 = addr;
>   DMA.CH0.DESTADDR1 = addr >> 8;
>   DMA.CH0.DESTADDR2 = addr >> 16;
> }

Ich habe mir das hier abgesehen..
/*
 * DMAController.cpp
 *
 *  Created on: 16.01.2013
 *      Author: Nicolas Göddel
 *
 * 
https://www.mikrocontroller.net/articles/Speicherdirektzugriff_(DMA)_mit_dem_ATxmega#DMAController.cpp
 */

[René H. (Gast)]

Hi Marcus,

bist noch nicht viel weiter gekommen. Weshalb aber ein neuer Thread?

Vielleicht solltest Du beschreiben was du machen willst.

Grüsse,
René

[Marcus (Gast)]

René H. schrieb:
> Hi Marcus,
>
> bist noch nicht viel weiter gekommen. Weshalb aber ein neuer Thread?
>
> Vielleicht solltest Du beschreiben was du machen willst.
>
> Grüsse,
> René

Hi Rene,

ich versuche ( einen Post über deinen (link) ) die C++ Bibliothek in C 
zu schreiben.

[Marcus (Gast)]

foobar schrieb:
> typedef uint32_t dmaaddr_t;
> #define cptr2dma(cptr) ((dmaaddr_t)(uintptr_t)(cptr))
>
> ...
> {
>   dmaaddr_t addr = cptr2dma(dest);
>
>   DMA.CH0.DESTADDR0 = addr;
>   DMA.CH0.DESTADDR1 = addr >> 8;
>   DMA.CH0.DESTADDR2 = addr >> 16;
> }

Das sieht gut aus. Keine Warnings mehr.
Und jetzt steht dort auch die Adresse der Variablen drin oder deren 
Inhalt?

[René H. (Gast)]

Marcus schrieb:
> foobar schrieb:
>> typedef uint32_t dmaaddr_t;
>> #define cptr2dma(cptr) ((dmaaddr_t)(uintptr_t)(cptr))
>>
>> ...
>> {
>>   dmaaddr_t addr = cptr2dma(dest);
>>
>>   DMA.CH0.DESTADDR0 = addr;
>>   DMA.CH0.DESTADDR1 = addr >> 8;
>>   DMA.CH0.DESTADDR2 = addr >> 16;
>> }
>
> Das sieht gut aus. Keine Warnings mehr.
> Und jetzt steht dort auch die Adresse der Variablen drin oder deren
> Inhalt?

Die Adresse.

Grüsse,
René

[Marcus (Gast)]

>> Das sieht gut aus. Keine Warnings mehr.
>> Und jetzt steht dort auch die Adresse der Variablen drin oder deren
>> Inhalt?
>
> Die Adresse.
>
> Grüsse,
> René

Das mit dem Makro ist echt ne klasse Idee ;) Besten dank!

[Marcus (Gast)]

Habe das gerade mal auf dem Rechner ausprobiert.

#include <stdio.h>
#include <stdlib.h>
#include <stdint.h>
#include <stdbool.h>

typedef uint32_t dmaaddr_t;
#define CPTR2DMA(cptr) ((dmaaddr_t)(uintptr_t)(cptr))

void foo( void *dest )
{
  printf( "%d\r\n" , CPTR2DMA( dest ) & 0xFF );
  printf( "%d\r\n" , ( CPTR2DMA( dest ) >> 8 ) & 0xFF );
        printf( "%d\r\n" , ( CPTR2DMA( dest ) >> 16 ) & 0xFF );
        printf( "%d\r\n" , ( CPTR2DMA( dest ) >> 32 ) & 0xFF );
}


int main(int argc, char *argv[]) 
{
  uint32_t test;
  
  foo( &test );
  
  printf( "Adresse: %d" , &test );
  
  return 0;
}

Wenn ich mit "printf" die ganze Adresse ausgeben möchte, kommt für mich 
eine richtige Adresse raus.

Wenn ich jedoch die Variable der Funktion "foo()" übergebe und das 
zerlege, kommt was ganz anderes raus?!

[foobar (Gast)]

> Wenn ich jedoch die Variable der Funktion "foo()" übergebe und
> das zerlege, kommt was ganz anderes raus?!

Der letzte Shift ist falsch (muss >>24 sein, nicht >>32). Und mit %x 
statt %d fällt das Vergleichen leichter ...

[Marcus (Gast)]

foobar schrieb:
>> Wenn ich jedoch die Variable der Funktion "foo()" übergebe und
>> das zerlege, kommt was ganz anderes raus?!
>
> Der letzte Shift ist falsch (muss >>24 sein, nicht >>32). Und mit %x
> statt %d fällt das Vergleichen leichter ...

Klasse!

void foo( void *dest )
{
  dmaaddr_t addr = CPTR2DMA( dest );
  
  printf( "%x\r\n" , (addr & 0x000000FF) >> 0  );
  printf( "%x\r\n" , (addr & 0x0000FF00) >> 8  );
  printf( "%x\r\n" , (addr & 0x00FF0000) >> 16 );
  printf( "%x\r\n" , (addr & 0xFF000000) >> 24 );
}


int main(int argc, char *argv[]) 
{
  uint32_t test;
  
  foo( &test );
  
  printf( "Adresse: %x" , &test );
  
  return 0;
}

Jetzt haut es hin ;)