>> FifoZeiger = (unsigned int*)(void*)&BusSpiOutput[i];
Dein BusSpiOutput ist ein char, dein FifoZeiger ein integer.
Deklarier den FifoZeiger mal als char*.
Siegfried Saueressig schrieb:
> Das Problem bleibt bestehen.> Wenn ich Bit 7 setze/lösche, wird die nachfolgende Adresse auf 0xFF/0x00> gesetzt.
Ist korrekt bei diesem Code:
> #define SET_BIT(var,bit) ( (var) |= (1<<(bit)) )> #define DEL_BIT(var,bit) ( (var) &= ~ (1<<(bit)) )> ...> ram unsigned char BusSpiOutput[MaxFifoBuffer];> ram unsigned int *FifoZeiger;> ...> FifoZeiger = (unsigned int*)(void*)&BusSpiOutput[i]; #1 #2 #3> ...> SET_BIT(*FifoZeiger,FifoPortBit); #4> ...> DEL_BIT(*FifoZeiger,FifoPortBit); #4
Du nimmst die Adresse auf ein Element eines unsigned char Felds (#1)
Änderst die Adresse in einen unbestimmten Universalzeiger (#2)
Änderst den unbestimmten Zeiger in einen Zeiger auf unsigned int (#3)
Dann manipulierst du Bits an der Stelle auf den der neue Zeiger
gerichtet ist (#4)
Bei Schritt #4 müssen 16-Bits beeinflusst werden!
ADD:
Korrektur s. Lothar
Tipp: Zunächst keine casts einsetzen und Warnungen des Compilers
beachten
Hast du die ganzen casts entfernt?
Das glaub ich dir nicht, dass es dann keine Warnings gibt. Du schmeisst
unsigned char und unsigned int durcheinander, als ob es kein Morgen
gibt!
Du solltest dir eines merken:
Wenn du Adressen oder Pointer zurechtcasten musst, dann ist in 95% aller
Fälle etwas oberfaul.
Siegfried Saueressig schrieb:
> jetzt stehe ich auf dem Schlauch.> keine Warnungen vom Compilers
Kommst du an die Optionen/Kommandozeile des Compilers ran? Steht da
vielleicht eine Option, die (alle) Warnungen abschaltet? Ich würde das
Handbuch des Compilers rauskramen oder online nach Compileroptionen
suchen.
Hallo,
das habe ich vor längerer Zeit aus dem Forum.
FifoZeiger = (unsigned int*)(void*)&BusSpiOutput[i];
hier das Compiler Ergebnis.
----------------------------------------------------------------------
Debug build of project
`E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_tes
t.mcp' started.
Preprocessor symbol `__DEBUG' is defined.
Sat Nov 28 11:14:42 2009
----------------------------------------------------------------------
Clean: Deleting intermediary and output files.
Clean: Deleted file
"E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_mai
n.o".
Clean: Deleted file
"E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_tes
t.cof".
Clean: Deleted file
"E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_tes
t.hex".
Clean: Done.
Executing: "C:\MCC18\bin\mcc18.exe" -p=18F8722 "my_main.c"
-fo="my_main.o" -D__DEBUG -Ou- -Ot- -Ob- -Op- -Or- -Od- -Opa-
MPLAB C18 v3.22 (feature limited)
Copyright 2000-2008 Microchip Technology Inc.
This version of MPLAB C18 does not support the extended mode
and will not perform all optimizations. To purchase a full
copy of MPLAB C18, please contact your local distributor or
visit buy.microchip.com.
Executing: "C:\MCC18\bin\mplink.exe" /l"C:\mcc18\lib" "..\18f8722.lkr"
"my_main.o" /u_CRUNTIME /u_DEBUG /z__MPLAB_BUILD=1 /z__MPLAB_DEBUG=1
/o"my_test.cof" /M"my_test.map" /W
MPLINK 4.22, Linker
Copyright (c) 2008 Microchip Technology Inc.
Errors : 0
MP2HEX 4.22, COFF to HEX File Converter
Copyright (c) 2008 Microchip Technology Inc.
Errors : 0
Loaded
E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_test
.cof.
----------------------------------------------------------------------
Debug build of project
`E:\Pic_C_Sourcecode\Stellpult\StellpultController_18F8722\mytest\my_tes
t.mcp' succeeded.
Preprocessor symbol `__DEBUG' is defined.
Sat Nov 28 11:14:43 2009
----------------------------------------------------------------------
BUILD SUCCEEDED
Gruß Siegfried
AH, da sieht man zumindest den Compiler (MPLAB C18)
Nimm in die Kommandozeile die Option -verbose auf.
Im User's Guide sieht man nicht, welcher Warnlevel default eingestellt
ist und wie man den gewünschten Warnlevel 2 einstellt. Ich vermute, dass
die -w2 Option es tut; könnte aber auch -w=2 sein. (Abschnitt 1.2.2
Displaying Diagnostics)
Kompiliere dann neu mit dieser Zeile im Sourcecode (Verschlag von Karl
heinz): FifoZeiger = /* (unsigned int*)(void*) */ &BusSpiOutput[i];
MPLAB C18 C COMPILER USER'S GUIDE
http://ww1.microchip.com/downloads/en/.../C18_User_Guide_51288j.pdf
Stefan B. schrieb:
> Kompiliere dann neu mit dieser Zeile im Sourcecode (Verschlag von Karl> heinz): FifoZeiger = /* (unsigned int*)(void*) */ &BusSpiOutput[i];
Da muss dann zumindest eine Warnung kommen. Ich würde sogar eher einen
Fehler erwarten, weil die Pointertypen nicht kompatibel sind.
Wenn die Warnung da ist, dann änderst du um auf
1
ramunsignedchar*FifoZeiger;
denn dort steckt das eigentliche Problem (und die Behebung des Problems
macht dann auch die Casts unnötig)
Aber gib auf keinen Fall auf, solange du in der Version mit dem unsigned
int Pointer und den weggelassenen Casts keine Diagostic bekommst (Error
oder Warning). Bei solchen Sachen MUSS der Compiler seinen Senf
dazugeben! Ansonsten sind Fehlern Tür und Tor geöffnet!
Und nein: Casts sind dann keine Lösung. Die Lösung steckt praktisch
immer darin, dass man sich die Datentypen der Pointer ansieht und
angleicht.
Siegfried Saueressig schrieb:
> Hallo,>> das habe ich vor längerer Zeit aus dem Forum.> FifoZeiger = (unsigned int*)(void*)&BusSpiOutput[i];
Das mag sein und im richtigen Zusammenhang kann das auch richtig sein.
Bei dir ist es aber falsch.
Bei dir wäre richtig
FifoZeiger = &BusSpiOutput[i];
und dazu muss FifoZeiger ein Pointer auf unsigned char sein.
Hallo,
hoffe das ich die richtige Infos gefunden habe.
Diagnostic's level
Error and warnings
Default storage class
Auto
Inherit global setting
-Ou- -Ot- -Ob- -Op- -Or- -Od- -Opa-
Als Anhang das Projekt
Gruß Siegfried
PS:
Info, habe weitere Zeilen eingefügt
Solange du dich resistent zeigst, und diesen Unsinn
ram unsigned int *FifoZeiger;
nicht endlich änderst, hat das alles keinen Sinn.
Es wurde dir jetzt schon mindestens 5 mal gesagt, dass die Casts raus
müssen und der Datentyp des Pointer angepasst werden muss.
Schmeiss endlich die Casts raus!
Mit den Casts sagst du sinngemäss dem Compiler:
Es ist mir scheissegal, ob die Datentypen stimmen oder nicht. Halt die
Schnauze und machs einfach!
Und kauf dir ein C-Buch!
Habe jetzt den Rat befolgt.
Was ich nicht verstehe, ist das mit dem
"ram unsigned char *FifoZeiger;"
Ich war der Meinung, das ein Zeiger immer " int " ist.
Gruß Siegfried
> Ich war der Meinung, das ein Zeiger immer " int " ist.schrei
Der Typ des Zeigers gibt an, auf was er Zeigt. Ganz einfach. ein /char*
p;/ zeigt auf ein char und ein /int* p;/ zeigt auf ein int. Und /klaus*
finger;/ zeigt auf einen Klaus.
Und nun kauf dir ein C-Buch, daran wirst du viel freude haben, weil das
Programmieren auf einmal sehr viel einfacher geht!
Siegfried Saueressig schrieb:
> Ich war der Meinung, das ein Zeiger immer " int " ist.
Diese Meinung musst du korrigieren!
Vielleicht hast du die Größe eines Zeigers im Speicher mit dem Typ auf
den er zeigt verwechselt. Bei manchen (!) Systemen ist der
Speicherbedarf eines Zeigers genau so groß wie der Speicherbedarf einer
int Variablen, also z.B. beide 16-Bit.
>> Und kauf dir ein C-Buch!
Das kann ich nur unterstützen ;-)
@ Lothar Miller
bitte erkläre mir, wo ich was unterdrückt habe.
>> Klar, wenn sie mit einem Monster-Cast unterdrückt werden :-o
Habe all meine Projekte mit dem "Project Wizard" erstellt.
Gruß Siegfried
Oh wie schön ist Sprache "C",
schon beim Lesen tut das weh....
Mit "c" schreibt sich ein ganz normaler Kasten,
weil dort die Typen gar nicht passten.
Was glaubt ihr denn, wie mich das freut,
wenn ich nicht lesen muss ein "Void".
Auch das kleine Woertchen "Main"
ist bei mir nicht gern geseh'n.
Wenn ich bestimmen koennte all:
Auf den Schirm käm' nur PASCAL!
In diesem Sinne
Paul
Eine eigene Frage?
Dieser Thread ist über 16 Jahre alt.
Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.
