PCF 8591 auslesen

[PcVirus]

Hi,
ich habe mir eine Funktion geschrieben um 2 PCF 8591 auszulesen.
Die Werte auf meinem Display sind vom PCF2 alle 4 Eingänge bei 1 und beim PCF1 schwanken sie zwischen 12 und 60. Die ist so wenn keine Spannung an einem Eingang angelegt ist. Sobald ich +5V anlege, gibt es einen kleinen Funken. Könnt ihr mal rüberschauen, ob das Programm soweit in Ordnung ist?

#include <avr/io.h>
#include <util/delay.h>
#include <wintek2704.c>		//LCD Lib
#include <stdint.h>			//Integer datentypen
#include <stdlib.h>			//versch. Funktionen
#include "i2cmaster.h"		//I2C Lib von Peter Fleury

#define PCF2  0x9E     // Adresse von PCF2
#define PCF1  0x90     // Adresse von PCF1

//Liniensensor defines
#define	 L1	1		
#define	 L2	2
#define	 L3	3
#define	 L4	4
#define	 L5	5
#define	 L6	6
#define	 L7	7
#define	 L8	8

char * int_to_string(uint16_t integer);
uint8_t readLine(uint8_t sensor);
void sleep(uint16_t sleep_ms);

int main(void)
{
	i2c_init();                                // init I2C interface
	lcd_init(LCD_DISP_ON);			//Display initialisieren
	while(1)
	{
	    lcd_clrscr();
		lcd_gotoxy(0,0);
		lcd_puts(int_to_string(readLine(L1)));
		
		lcd_gotoxy(10,0);
		lcd_puts(int_to_string(readLine(L2)));
		
		lcd_gotoxy(0,1);
		lcd_puts(int_to_string(readLine(L3)));
		
		lcd_gotoxy(10,1);
		lcd_puts(int_to_string(readLine(L4)));
		
		lcd_gotoxy(0,2);
		lcd_puts(int_to_string(readLine(L5)));
		
		lcd_gotoxy(10,2);
		lcd_puts(int_to_string(readLine(L6)));
		
		lcd_gotoxy(0,3);
		lcd_puts(int_to_string(readLine(L7)));
		
		lcd_gotoxy(10,3);
		lcd_puts(int_to_string(readLine(L8)));
		
		sleep(100);
		
	}
	return 0;
}

char * int_to_string(uint16_t integer)
{
	static char return_string[50];
	itoa(integer,return_string,10);
	return return_string;
}

void sleep(uint16_t sleep_ms) 
{
	uint16_t i=0;
	while(i<sleep_ms)
	{
		_delay_ms(1);
		i++;
	}
}

//Funktion liest die Liniensensoren L1-L8 aus
uint8_t readLine(uint8_t sensor)
{
	uint8_t i2c_status = 0;
	uint8_t value = 0;
    
	if(sensor <= 4)		//Wenn	Sensoren 1 - 4, dann PCF1
	{
		i2c_status = i2c_start(PCF1+I2C_WRITE);       // I2C startcondition senden, danach kann geschrieben werden
	}
	else				//PCF2
	{
		i2c_status = i2c_start(PCF2+I2C_WRITE);       // I2C startcondition senden, danach kann geschrieben werden
	}
	//Status überprüfen
	if(i2c_status)			//Wenn 1, dann liegt ein Fehler vor
	{
		value = 0;		//Bei Fehler ist value 0
		i2c_stop();		//I2C Bus freigeben
	}
	else
	{
		//Command zum PCF1 senden, jenachdem welcher Sensor gewählt wurde
		if(sensor==L1)		
		{
			i2c_write(0x00);                       // Commando zum PCF senden. (Analogwert lesen, kanal0)
		}
		else if(sensor==L2)
		{
			i2c_write(0x01);                       // Commando zum PCF senden. (Analogwert lesen, kanal1)
		}
		else if(sensor==L3)
		{
			i2c_write(0x02);                       // Commando zum PCF senden. (Analogwert lesen, kanal2)
		}
		else if(sensor==L4)
		{
			i2c_write(0x03);                       // Commando zum PCF senden. (Analogwert lesen, kanal3)
		}
		
		if(sensor <= 4)		//Wenn	Sensoren 1 - 4, dann PCF1
		{
			i2c_rep_start(PCF1+I2C_READ);       // Erneute startcondition mit nachfolgendem lesen
		}
		else
		{
			i2c_rep_start(PCF2+I2C_READ);       // Erneute startcondition mit nachfolgendem lesen
		}
		
		//3 von 4 mal lesen
		for(uint8_t i=0; i<3; i++)	
		{
			value += i2c_readAck();
		}
		value += i2c_readNak();	//4. mal lesen, NAK
		i2c_stop();
		value /= 4;
		value++;		//+1, damit value immer > 0 und somit ein Fehler erkennbar ist (value = 0) 			
	}
	return value;
}

MfG Jörn

[PcVirus]

Ich hab den Fehler jetzt selbst gefunden. Die Variable value war einfach zu klein.