CC1101 + Arduino + panstamp libary fehler

[Jannis P. (janpeter)]

Moin!
Ich versuche derzeit meinen CC1101 zu flashen. Der Sender hat folgenden 
Code:

#include "EEPROM.h"
#include "cc1101.h"

CC1101 cc1101;

#define LEDOUTPUT 7

// counter to get increment in each loop
byte counter;
byte b;
byte syncWord = 199;

void blinker(){
     digitalWrite(LEDOUTPUT, HIGH);
     delay(100);
     digitalWrite(LEDOUTPUT, LOW);
     delay(100);
}

void setup()
{
 Serial.begin(38400);
 Serial.println("start");

 // setup the blinker output
 pinMode(LEDOUTPUT, OUTPUT);
 digitalWrite(LEDOUTPUT, LOW);

 // blink once to signal the setup
 blinker();
 
 // reset the counter
 counter=0;
 Serial.println("initializing...");
 // initialize the RF Chip
 cc1101.init();
 cc1101.reset();
 cc1101.setSyncWord(&syncWord, false);
 cc1101.setCarrierFreq(CFREQ_868);
 cc1101.disableAddressCheck();
 
 Serial.print("CC1101_PARTNUM "); //cc1101=0
 b=cc1101.readReg(CC1101_PARTNUM, CC1101_STATUS_REGISTER);
 Serial.println(b);
 b=cc1101.readReg(CC1101_VERSION, CC1101_STATUS_REGISTER);
 Serial.print("CC1101_VERSION "); //cc1101=4
 Serial.println(b);  
 Serial.print("CC1101_MARCSTATE ");
 Serial.println(cc1101.readReg(CC1101_MARCSTATE, CC1101_STATUS_REGISTER) & 0x1f);
 
 Serial.println("device initialized");
}

void send_data() {
CCPACKET data;
 data.length=1;
 byte blinkCount=counter++;
 data.data[0]=blinkCount;
 Serial.print("CC1101_MARCSTATE ");
s Serial.println(cc1101.readReg(CC1101_MARCSTATE, CC1101_STATUS_REGISTER) & 0x1f);
 if(cc1101.sendData(data)){
   Serial.print("sent ok :)");
   blinker();
 }else{
   Serial.print("sent failed :(");
   blinker();
   blinker();
 }  
}

void loop()
{
 send_data();
 Serial.println("loop done");
 delay(1000);
}

Wenn ich auf Flashen klicke kommt er aber nicht mal durchs kompillieren. 
Folgende Fehlermeldung:

Sketch wird kompiliert...
"C:\Users\Jan\Desktop\arduino-1.8.0\hardware\tools\avr/bin/avr-g++" -c -g -Os  -std=gnu++11 -fpermissive -fno-exceptions -ffunction-sections -fdata-sections -fno-threadsafe-statics -MMD -flto -mmcu=atmega328p -DF_CPU=16000000L -DARDUINO=10800 -DARDUINO_AVR_NANO -DARDUINO_ARCH_AVR   "-IC:\Users\Jan\Desktop\arduino-1.8.0\hardware\arduino\avr\cores\arduino" "-IC:\Users\Jan\Desktop\arduino-1.8.0\hardware\arduino\avr\variants\eightanaloginputs" "-IC:\Users\Jan\Desktop\arduino-1.8.0\hardware\arduino\avr\libraries\EEPROM\src" "-IC:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp" "C:\Users\Jan\AppData\Local\Temp\arduino_build_460993\sketch\CC1101-Senden.ino.cpp" -o "C:\Users\Jan\AppData\Local\Temp\arduino_build_460993\sketch\CC1101-Senden.ino.cpp.o"
In file included from C:\Users\Jan\Documents\Arduino\CC1101-Senden\CC1101-Senden.ino:3:0:

C:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp/cc1101.h: In function 'void setup()':

C:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp/cc1101.h:306:10: error: 'byte CC1101::readReg(byte, byte)' is private

     byte readReg(byte regAddr, byte regType);

          ^

CC1101-Senden:44: error: within this context

  b=cc1101.readReg(CC1101_PARTNUM, CC1101_STATUS_REGISTER);

                                                         ^

In file included from C:\Users\Jan\Documents\Arduino\CC1101-Senden\CC1101-Senden.ino:3:0:

C:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp/cc1101.h:306:10: error: 'byte CC1101::readReg(byte, byte)' is private

     byte readReg(byte regAddr, byte regType);

          ^

CC1101-Senden:46: error: within this context

  b=cc1101.readReg(CC1101_VERSION, CC1101_STATUS_REGISTER);

                                                         ^

In file included from C:\Users\Jan\Documents\Arduino\CC1101-Senden\CC1101-Senden.ino:3:0:

C:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp/cc1101.h:306:10: error: 'byte CC1101::readReg(byte, byte)' is private

     byte readReg(byte regAddr, byte regType);

          ^

CC1101-Senden:50: error: within this context

  Serial.println(cc1101.readReg(CC1101_MARCSTATE, CC1101_STATUS_REGISTER) & 0x1f);

                                                                        ^

C:\Users\Jan\Documents\Arduino\CC1101-Senden\CC1101-Senden.ino: In function 'void send_data()':

CC1101-Senden:61: error: 's' was not declared in this scope

 s Serial.println(cc1101.readReg(CC1101_MARCSTATE, CC1101_STATUS_REGISTER) & 0x1f);

 ^

Bibliothek EEPROM in Version 2.0 im Ordner: C:\Users\Jan\Desktop\arduino-1.8.0\hardware\arduino\avr\libraries\EEPROM  wird verwendet
Bibliothek panstamp im Ordner: C:\Users\Jan\Desktop\arduino-1.8.0\libraries\panstamp (legacy) wird verwendet
exit status 1
within this context

Kann mir jemand helfen was ich falsch mache? Googlen hat leider kein 
Erfolg gebracht :/
Danke!

[Arduinoversager (Gast)]

Jannis P. schrieb:
> Ich versuche derzeit meinen CC1101 zu flashen.

Jannis P. schrieb:
> Kann mir jemand helfen was ich falsch mache?

Nein, ich kann dir nicht helfen. Denn diesen Baustein
kann man nicht flashen da er gar kein Flash hat.

Aber vielleicht kannst du die Features hochskillen dann
kann man ihn vielleicht doch flashen.

[Kaj (Gast)]

Was der Fehler ist, steht da doch:

error: 'byte CC1101::readReg(byte, byte)' is private

Du benutzt eine private Funktion ausserhalb der Klasse.

[Jannis P. (janpeter)]

Kaj schrieb:
> Was der Fehler ist, steht da doch:error: 'byte
> CC1101::readReg(byte, byte)' is private
>
> Du benutzt eine private Funktion ausserhalb der Klasse.

Hey danke für die Antwort!
D.h.:
in den dateien

#include "EEPROM.h"
#include "cc1101.h"

die ich aus der Libary ziehe ist dann irgendwas falsch deklariert bzw 
nicht passend für meinen Sketch?
Ich schau mal ob ich ne andere Version der Libary finde oder das 
bearbeiten kann.

[aufmerksamer Leser (Gast)]

Jannis P. schrieb:
> D.h.:
> in den dateien
>
> #include "EEPROM.h"
> #include "cc1101.h"
>
> die ich aus der Libary ziehe ist dann irgendwas falsch deklariert bzw
> nicht passend für meinen Sketch?

Dein Sketch benutzt Funktionen, die nur bibliotheksintern verwendet 
werden sollen. Das, was Du da abbilden willst, muss anders gehen. Am 
besten die Doku zur Bibliothek noch mal lesen und den Sketch umschreiben 
...

Natürlich kannst Du diese Funktion in der cc1101.h auch aus "private" in 
den Bereich "public" ziehen, das ist aber ein übler Hack.

[Nixversteher (Gast)]

aufmerksamer Leser schrieb:
> Am besten die Doku zur Bibliothek noch mal lesen und den
> Sketch umschreiben

Copy & Paste Maker verstehen nicht eine Doku zur Bibliothek,
sie wissen auch nichts von einer Klasse, geschweige denn
was Public oder Private ist. Deswegen ist jeder Ansatz, Hilfe
zur Selbsthilfe zu geben, sinnlos. Das sollte bereits aus
dem ersten Satz erkennbar sein:

Jannis P. schrieb:
> Ich versuche derzeit meinen CC1101 zu flashen.

[Kaj (Gast)]

Jannis P. schrieb:
> die ich aus der Libary ziehe ist dann irgendwas falsch deklariert bzw
> nicht passend für meinen Sketch?
Nun, das kann man jetzt sehen wie man will.

Entweder man sieht es so wie du, naemlich dass in dem Code der "Lib" 
etwas falsch ist. Oder man betrachtet es so, dass Du die "Lib" falsch 
benutzt, naemlich nicht so, wie sich der Entwickler der "Lib" das 
gedacht hat.
Letzteres ist die verbreitetere Ansicht da man davon ausgehen kann, dass 
sich der Entwickler der "Lib" (hoffentlich(!)) etwas dabei gedacht hat.

Um das Problem zu verstehen muesstest du dich mal ganz kurz mit OOP, C++ 
und Klassen beschaeftigen.


Ich hab mal kurz in die Headerdatei reingeschaut:
https://github.com/raimue/arduino/blob/master/libraries/panstamp/cc1101.h

Hiernach ist die entsprechende Funktion public und sollte aufrufbar 
sein.
Abgesehen davon ist das Teil aber, wie bei nahezu allem Arduino-Code, 
einfach ein grauss...

[Kaj (Gast)]

Ich hab deinen Code gerade mal versucht zu compilieren...

s Serial.println(cc1101.readReg(CC1101_MARCSTATE, CC1101_STATUS_REGISTER) & 0x1f);

Ich weiss nicht was das 's' da am Zeilen anfang macht, aber das gehoert 
da nicht hin. Wenn ich das 's' da entferne dann compiliert der Code bei 
mir. Gibt zwar einen haufen Warnungen, aber es compiliert.

[Jannis P. (janpeter)]

Kaj schrieb:

> Oder man betrachtet es so, dass Du die "Lib" falsch
> benutzt, naemlich nicht so, wie sich der Entwickler der "Lib" das
> gedacht hat.

Kann durchaus sein, da die panstamp Lib ja eigentlich für das 
Panstamp/Stick board gedacht ist. Jedoch nutzen viele diese Libary für 
ihren CC1101 also sollte das auch gehen irgendwie. Also ist es wohl der 
Code. Dieser scheint jedoch bei anderen Usern zu funktionieren. z.B. bei 
dir.


> Hiernach ist die entsprechende Funktion public und sollte aufrufbar
> sein.
> Abgesehen davon ist das Teil aber, wie bei nahezu allem Arduino-Code,
> einfach ein grauss...

Ja dieses erstellen der Sketche im ArduinoEditor ist schon etwas schräg 
irgendwie weil man nur eine Datei hat und irgendwie keine Übersich wie 
bei z.B. Eclipse usw.

> Ich weiss nicht was das 's' da am Zeilen anfang macht, aber das gehoert
> da nicht hin. Wenn ich das 's' da entferne dann compiliert der Code bei
> mir. Gibt zwar einen haufen Warnungen, aber es compiliert.

Keine Ahnung woher das "s" kommt. Kompillieren tut er bei mir trotzdem 
nicht :/

Fehlermeldungen sind die gleichen, nur die mit dem "s" fällt weg 
natürlich.

Danke aber schonmal für die Hilfe!