Kan inte läsa från AD omvandlaren!

fatpo83 · Inlägg av **fatpo83** » 14 augusti 2005, 17:25:02

Jag får konstiga tecken på LCDn när jag försöker läsa från 10bit ADn. Endast AN0 används i pic18f2455. Har jag gjort nåt fel eller? Jag har även försökt läsa av endast ADRESH och ADRESL som är 8bit var och skriva ut i LCDn men samma fel.

Kod: Markera allt

void main (void)									
{
	
	int tal,talH,talL; 
	char data[];
		

	TRISB= 0x00;
	PORTB= 0x00;

	PORTA= 0x00;
	TRISA= 0x03;	//b0000011, RA1 och RA0 input
	ADCON0 = 0x00 ; 	//channel AN0, AD off 
	ADCON1 = 0x0D;     // VSS,VDD, AN0 AN1
	ADCON2 = 0xB7;     //b10110111, right justified, 16TAD, FRC
	ADCON0 = 0x01 ;	//AD ON 

	OpenXLCD(FOUR_BIT & LINES_5X7);	// configure external LCD
	WriteCmdXLCD(BLINK_OFF);	
	putrsXLCD( "1" );  

	while(1) 		
	{

		PORTBbits.RB7 = 1;		//LED indicator
		pic_ad_read(0);
		tal	=	 ADRESL;		/* Getting HSB of CCP1 */
		tal	+= (ADRESH << 8);		/* Getting LSB of CCP1 */
		talH=ADRESH;
		talL=ADRESL;

		delay250Ms(2);

		itoa(tal,data); 			// convert integer to string
		
		WriteCmdXLCD(lcd_rad2);
		putsXLCD(data);										
		PORTBbits.RB7 = 0;	
		delay250Ms(2);
	
	}
}

void pic_ad_read(unsigned char channel)
{
	char i;
	ADCON0 = 0x01; 			//reset channels

	ADCON0 |= (channel<<2);

	for(i=0;i<50;i++);
	ADCON0bits.GO = 1;		//start the conversion
	while(ADCON0bits.GO == 1);
}

Icecap · Inlägg av **Icecap** » 14 augusti 2005, 17:44:05

Tydligen har du väl fel med itoa().

$tiff · Inlägg av **$tiff** » 14 augusti 2005, 17:55:16

ska det inte vara:

putsXLCD(itoa(tal,data));

när du ska sriva ut strängen? Så är det i GCC iaf.

sodjan · Inlägg av **sodjan** » 14 augusti 2005, 18:59:31

Kanske att det kan underlätta att hjälpa om man vet vad de "konstiga tecken" du får på LCDn är för något...

fatpo83 · Inlägg av **fatpo83** » 14 augusti 2005, 19:32:25

Nej, itoa() fungerar fint. Jag har tidigare omvandlat tal (int, char) med den funktionen och sparat i en string (här "data"). Sen är det bara att skriva ut den med "putsXLCD(data)".

sodjan, de tecken jag får är typ vågräta linjer med en pixels mellanrum (LCDn är en 5x7). Förstår du?

Det konstiga är också att det första tecknet som skrivs på LCD är vanlig siffra som ändras med samplingen, men de övriga är som beskrivet ovan.

Jag har väl inte gjort fel på initieringen av ADn elller?

Erik_Lind · Inlägg av **Erik_Lind** » 14 augusti 2005, 20:34:11

Hur stor blir egentligen:

Kod: Markera allt

char data[];

Bör du inte speca hur mycket utrymme den skall tilldelas?
Kan vara så att det inte får plats med ett \0 islutet på strängen och då skriver kanske din LCD rutin ut diverse annat som ligger i minnet.

sodjan · Inlägg av **sodjan** » 14 augusti 2005, 20:48:19

> "vågräta linjer med en pixels mellanrum..."

Anledningen att jag frågade var att man kanske via teckentabellen för LCD'n kan räkna ut bakvägen vad som faktiskt skickas ut till den.

Är det rätt *antal* tecken, eller kommer det jycket mer än vad det ska ?
Kan du skriva ut någonting alls på LCD'n från samma kod ?
T.ex ettan i "putrsXLCD( "1" );", kommer den ut ?
Hur mycket debugging har du gjort ?
Har du verifierat itoa och LCD rutinen genom att ge "tal" ett hårdkodat värde (alltså inte hämtat från ADC'n) ?
Det är ju centralt att ta reda på om det är ADC'n eller LCD'n du har problem med !!
Har du provat att göra ett enkelt assembler program för att läsa ADC för att verifiera att de delarna är OK ?
Har du kollat assembler koden som genereras för att verifiera att C kompilatorn gör vad du tror ?
Vad är det för C kompilator ?
Är "for(i=0;i<50;i++);" för ADC'ns aqcuisition time (borde i och för sig räcka...) ?

fatpo83 · Inlägg av **fatpo83** » 14 augusti 2005, 20:59:18

Felet ligger inte i "char data[]"...det som läggs in dit är 10bitars data fast konverterad till sträng.

Antalet tecken som skrivs på LCDn är rätt, alltså 10st. Anledningen till att jag lade till "putrsXLCD( "1" );" är just att kolla att LCD inte är död. Så det fungerar klockrent. Självaste rutinen "itoa" fungerar fint som jag skrev tidigare.

"for(i=0;i<50;i++);" har jag lagt till i efterhand för säkerhetskull, men annars finns ju "Tad" tiden definierade i ADCON registren. Kompilatorn är C18.

Jag kanske borde ta en titta på assembler koden som genereras.

sodjan · Inlägg av **sodjan** » 14 augusti 2005, 21:23:07

10 tecken ? Borde det inte vara max 4 tecken ?
"0000" till "1023" eller något i den stilen ?

Vad förväntar du dig på LCD'n ?
Alltså för att det skall vara "rätt" ?

Jag kanske har missförstått hur itoa() fungerar...

fatpo83 · Inlägg av **fatpo83** » 14 augusti 2005, 22:12:55

Jag har nu gjort en egen funktion som fixar det. orkar inte felsöka längre.

PS. Det kanske inte ens är värt att använda de inbyggda funktionerna i C18? Trots att man går efter manualen....

tack för alla svar!

sodjan · Inlägg av **sodjan** » 14 augusti 2005, 22:24:32

Vadå "fixar det" ? Vad är "det" ?

Vad är det du har ersatt med en egen funktion ?

Vilken funktion i C18 är det som inte är värd att använda ? Och varför inte ?

Vad är det i manualen som inte verkar stämma (det är vad jag läser mellan rederna att du tycker) ?

Det är lite dålig stil (efter ha fått flera att lägga tid på att hjälpa till) inte lämna en förklaring till vad som var orsaken till dina problem med koden...

PS : Jag förstår fortfarande inte varför det skulle bli 10 tecken på LCDn.

fatpo83 · Inlägg av **fatpo83** » 15 augusti 2005, 21:46:41

"det" är själva strängkonverteringen (i C18 itoa) felet ligger. Den sträng som konverterats kan inte skrivas ut till lcdn med putsXLCD rutinen; det är just då man får konstiga tecken (men ibland funkar det faktiskt typ 1 på 10) .

Jag skrev ihop nåt snabbt och vips så funkade allt utmärkt:

Kod: Markera allt

void lcd_dec_byte(int val)				

{
	int d,digits=4;
	if (digits > 3)
	{
		d=val/1000;
		val=val%1000;
		WriteDataXLCD(num_to_char(d));
	}
	if (digits > 2)
	{
		d=val/100;
		val=val%100;
		WriteDataXLCD(num_to_char(d));
	}
	if (digits > 1)									
	{
		d=val/10;
		val=val%10;
		WriteDataXLCD(num_to_char(d));
	}
	if (digits > 0)									
	{
		WriteDataXLCD(num_to_char(val));		
	}
}

[/code]

AndLi · Inlägg av **AndLi** » 15 augusti 2005, 22:02:42

vad gör if satserna för nytta?
digits är ju alltid 4! Altså körs alla alltid...

data[] borde definitivt sättas till en längd, om inte minst för att göra koden tydlig, man har ju inte en susning om den är 10 bytes eller 200 bytes när man vill banta programmet.

fatpo83 · Inlägg av **fatpo83** » 16 augusti 2005, 00:13:20

Jag är mycket medveten om att if satserna inte hjälper (här). Jag testade nämligen också med olika "digits" först.

Visst ska man alltid begränsa längden, här till data[5], men det funkar ändå...men visst ska man det!

sodjan · Inlägg av **sodjan** » 16 augusti 2005, 00:40:08

> "och vips så funkade allt utmärkt..."

OK. Fint !
Och du får ut 10 tecken på LCD'n ?

Du såg inget "mönster" i de ADC värden där itoa varianten fungerade ?
Det måste ju vara *något* som gör att det fungerar ibland och ibland inte...