Arduino ADC liest Unsinn

[someone (Gast)]

Ich will einen Arduino Uno als einfachen logger verwenden (mit 
SD-shield).

Das schreiben auf die SD Karte funktioniert, nur der ADC ließt offenbar 
Blödsinn.

#include <SPI.h>
#include <SD.h>

const int chipSelect = 4;
  File dataFile;

uint16_t *buff;
uint8_t buffcnt;
void setup() {
  // put your setup code here, to run once:
  buff=(uint16_t*)malloc(128);
  buffcnt=0;
  SD.begin(chipSelect);
 
  

}
void save(){
  dataFile=SD.open("log.dat", FILE_WRITE);
  for(int i=0;i<64;i++){
    dataFile.println(buff[i]);
    }
  dataFile.close();
  }

void loop() {
  uint16_t voltage=analogRead(0);
  buff[buffcnt]=voltage;
  if(buffcnt=64){
    buffcnt=0;
    save();
    delay(10000);
    
  }
  
  
  
  

}

Zu Testzwecken habe ich A0 hart auf GND gelegt und erhalte recht 
zufällig anmutende Werte zwischen 0 und 2^16 wobei 0 deutlich öfter 
auftritt als durch eine Gleichverteilung zu erklären wäre, aber deutlich 
<10% der Messwerte sind unter 128, exakt 0 vielleicht jeder 50.

[Mein grosses V. (vorbild)]

someone schrieb:
> if(buffcnt=64)
            ^
            ^
            ^

[Sascha (Gast)]

Schaltplan und die Setup-Routine vom ADC bitte.

[Sascha (Gast)]

2^16 ist bei einem 10 Bit ADC übrigens recht schwer.

[M. K. (sylaina)]

someone schrieb:
> Zu Testzwecken habe ich A0 hart auf GND gelegt und erhalte recht
> zufällig anmutende Werte zwischen 0 und 2^16

Spricht für floaten. Sicher, dass du eine gute elektrische Verbindung 
zum Kanal 0 hast? Ich denke du hast hier einen Wackler.

[Heiko G. (heikog)]

someone schrieb:

> void loop() {
>   uint16_t voltage=analogRead(0);
>   buff[buffcnt]=voltage;
>   if(buffcnt=64){
>     buffcnt=0;
>     save();
>     delay(10000);
>
>   }

Ist das der aktuelle Sourcecode? Da fehlt noch ein ++, dadurch bleibt 
bufcnt immer auf 0 stehen (bzw. wird durch if(buffcnt=64) immer auf 64 
gehalten. Dadurch sollte immer nur der zufällige Inhalt des Buffers 
zyklisch ausgegeben werden.

[W.G. (Gast)]

Heiko G. schrieb:
> someone schrieb:
>
>> void loop() {
>>   uint16_t voltage=analogRead(0);
>>   buff[buffcnt]=voltage;
>>   if(buffcnt=64){
>>     buffcnt=0;
>>     save();
>>     delay(10000);
>>
>>   }
>
> Ist das der aktuelle Sourcecode? Da fehlt noch ein ++, dadurch bleibt
> bufcnt immer auf 0 stehen (bzw. wird durch if(buffcnt=64) immer auf 64
> gehalten. Dadurch sollte immer nur der zufällige Inhalt des Buffers
> zyklisch ausgegeben werden.

Außerdem

>>   if(buffcnt=64){

soll wohl heißen

>>   if(buffcnt==64){