[PIC24] error: syntax error before '.' token

[Andreas G. (beastyk)]

Moin Leute,

ich hab hier ein Problem und komm einfach nicht weiter, kann sich mal 
jemand die Codeschnipsel ansehen und mir vielleicht sagen was ich da 
falsch gemacht habe?!?

Build Output:

mfrc522.c:39: error: syntax error before '.' token
mfrc522.c: In function 'MFRC522':
mfrc522.c:40: error: number of arguments doesn't match prototype
mfrc522.h:266: error: prototype declaration
mfrc522.c:50: warning: 'return' with no value, in function returning non-void
Halting build on first failure as requested.

mfrc522.c:

//include basic header definition
#include "p24FJ256GA106.h"
#include "GenericTypeDefs.h"
#include "stdbool.h"
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "OnePIC_int.h"
#include "OnePIC_lcd.h"
#include "OnePIC_switch.h"
#include "mfrc522.h"
#include "TimeDelay.h"
#include <spi.h>

#define CS_PIN_OUT     TRISBbits.TRISB15
#define CS_PIN       LATBbits.LATB15
#define RST_PIN_OUT    TRISBbits.TRISB13
#define RST_PIN       LATBbits.LATB13 

 * Prepares the output pins.
 */

//PIC24F SS(out)-> MFRC522's SPI slave select input (RB15/PTP6, NSS, active low)
//PIC24F reset(out)-> MFRC522's reset and power down input (RB13/PTP4, NRSTPD, ??active low??)
BYTE MFRC522(BYTE CS_PIN, BYTE RST_PIN)    
{
  // Set the chipSelectPin as digital output, do not select the slave yet
  _chipSelectPin = CS_PIN;//set _chipSelectPin to CS_Pin -> LATBbits.LATB15
  CS_PIN_OUT = 0;      //Set TRISBbits.TRISB15 to output
  _chipSelectPin = 1;    // LATBbits.LATB15=1 
  
  // Set the resetPowerDownPin as digital output, do not reset or power down.
  _resetPowerDownPin = RST_PIN; //Set _resetPowerDownPin to RST_PIN -> LATBbits.LATB13
  RST_PIN_OUT = 0;    //Set TRISBbits.TRISB13 to output
  _resetPowerDownPin = 0;  // LATBbits.LATB13=0
 return;
}

mfrc522.h:

  BYTE MFRC522(BYTE CS_PIN, BYTE RST_PIN);

Ich hoffe das langt an Code und ihr könnt mir helfen!

best wishes
dat
Beast

[g457 (Gast)]

>
>  * Prepares the output pins.
>  */
   ^^
HTH

[Rufus Τ. F. (rufus)]

#define CS_PIN_OUT     TRISBbits.TRISB15
#define CS_PIN       LATBbits.LATB15
#define RST_PIN_OUT    TRISBbits.TRISB13
#define RST_PIN       LATBbits.LATB13 

...

BYTE MFRC522(BYTE CS_PIN, BYTE RST_PIN)

Was soll eine Funktion (oder ein Funktionsprototyp) mit konstanten 
Werten anstelle von Variablen als Funktionsargumenten anfangen?

Was Du da machst enstpricht dem hier:

#define WERT_A 5
#define WERT_B 6

BYTE Funktion(BYTE WERT_A, BYTE WERT_B) 
...

Damit kann Dein Compiler aus guten Gründen nichts anfangen.

[Andreas G. (beastyk)]

Danke an euch Beiden,

bin gerade am rumgrübeln und lese noch etwas über "extern" und 
"volatile" bei Variablen.
Ich hab also nur dem Lat und Tris einen neuen Namen gegeben und das sagt 
aber nichts darüber aus ob da etwas auf 0 oder 1 gesetzt wurde.
Das mach ich ja anscheinend mit/über _chipSelectPin und 
_resetPowerDownPin.
Also wäre es doch angebrachter das ich diese Werte in der Funktion 
übergebe.

Also wenn ich alles so ändere dann meckert mein Linker mit mir rum, ich 
hätte in der main.c und mfrc522.c Variablen doppelt definiert (multiple 
definition) (hier von _chipSelectPin, _resetPowerDownPin und 
uid)...darum lese ich gerade etwas in der FAQ hier im Forum, auch wenn 
es für AVR ist.

Ist der Weg richtig?

Gruß
das C Tutorial lesende
Beast

[Rufus Τ. F. (rufus)]

Mit "volatile" und "extern" hat Dein Problem nichts zu tun.

Warum versuchst Du Deiner Funktion überhaupt etwas zu übergeben, wenn Du 
doch gleichzeitig in Deiner Funktion mit Konstanten arbeitest?

void MFRC522(void)    
{
  // Set the chipSelectPin as digital output, do not select the slave yet
  _chipSelectPin = CS_PIN;       //set _chipSelectPin to CS_Pin -> LATBbits.LATB15
  CS_PIN_OUT = 0;                //Set TRISBbits.TRISB15 to output
  _chipSelectPin = 1;            // LATBbits.LATB15=1 
  
  // Set the resetPowerDownPin as digital output, do not reset or power down.
  _resetPowerDownPin = RST_PIN;  //Set _resetPowerDownPin to RST_PIN -> LATBbits.LATB13
  RST_PIN_OUT = 0;               //Set TRISBbits.TRISB13 to output
  _resetPowerDownPin = 0;        // LATBbits.LATB13=0
}

Das erste void rührt daher, daß Deine Funktion gar nichts zurückgibt - 
das leere return am Ende ist daher auch überflüssig.

Der Funktionsprototyp in Deiner Headerdatei sieht dann so aus:

void MFRC522(void);

Vermutlich haben Deine Variablen _chipSelectPin und resetPowerDownPin 
auch keinen weiteren Nutzen, so daß sich Deine Funktion reduzieren lässt 
auf das hier:

void MFRC522(void)    
{
  // Set the chipSelectPin as digital output, do not select the slave yet
  CS_PIN_OUT = 0;   //Set TRISBbits.TRISB15 to output
  CS_PIN = 1;       // LATBbits.LATB15=1 
  
  // Set the resetPowerDownPin as digital output, do not reset or power down.
  RST_PIN_OUT = 0;  //Set TRISBbits.TRISB13 to output
  RST_PIN = 0;      // LATBbits.LATB13=0
}

[Andreas G. (beastyk)]

nabend Rufus,

im Grunde benutz ich _chipSelectPin und _resetPowerDownPin später in den 
anderen Funktionen wenn es um die Kommunikation mit dem MFRC522 über SPI 
geht, aber du hast natürlich Recht wenn du darauf hinweist das mfrc522 
als Funktion eigentlich nur die Anfangsbedingungen festlegt.
Die Library war im Grunde für den Arduino geschrieben und im 
Orignialcode stand:

MFRC522::MFRC522(  byte chipSelectPin,    ///< Arduino pin connected to MFRC522's SPI slave select input (Pin 24, NSS, active low)
          byte resetPowerDownPin  ///< Arduino pin connected to MFRC522's reset and power down input (Pin 6, NRSTPD, active low)
        )

Das MFRC522:: hab ich für eine Eigenart des Arduino gehalten, vielleicht 
so etwas wie eine übergeordnete Instanz oder sowas.
Alle anderen Funktionen fingen auch so an, aber ohne das MFRC522:: 
machten die Funktionen halt genausoviel Sinn. Hab aber gesehen das die 
Orignialfunktion kein Return zurückgibt, anscheinend hat Arduino das 
nicht so mit dem void....
Mal abgesehen davon darf man wohl auch 2 mal PCD_WriteRegister und 
PCD_ReadRegister als Funkionennamen verwenden, wobei einmal nur ein Byte 
und das andere Mal mehrer Bytes geschrieben/gelesen werden.

Ich danke dir für die Ruhe die du bewahrst beim erklären ;o)

Gruß
dat
Beast

[Andreas G. (beastyk)]

Andreas G. schrieb:
> Das MFRC522:: hab ich für eine Eigenart des Arduino gehalten, vielleicht
> so etwas wie eine übergeordnete Instanz oder sowas.

Wie gut das ich mich so garnicht gut auskenne.
C++: class MFRC522 -> MFRC522::MFRC522(...,...) -> 
EineKlasse::eineFunktion