Warum funktioniert dieses programm nicht

[Thomas (Gast)]

#include <p18f1320.h>
#pragma config OSC = HS             //32MHZ
#pragma config PWRT = ON            //Power Up Timer
#pragma config WDT = OFF            //Watchdog Timer

volatile unsigned char marke=0;



void main(void)
{
 ADCON1 = 0x7f;
 TRISB=0b11110000;
 TRISA=0b11111;
 LATB=0x00;

 while(1){
    if(PORTAbits.RA0) marke=1; else marke=0;
    if(marke) LATB++;
}
}
ich dachte, wenn ich an porta bit 0 eine H anlege, dann muss an dem port 
B der wert hochgezählt werden, also vom 0 bis 255, aber alle leds sind 
auf low. ist das programm falsch???

[Kai F. (kai-)]

kleiner Tipp fürs nächste Mal:
wenn du den verwendeten Controller in den Threadtitel schreibst, muss 
nicht jeder draufklicken. Ich kenne mich mit diesem µC nämlich zB gar 
nicht aus

[Flo (Gast)]

TRISA=0b11111;
das sieht nicht gut aus, mach mal
TRISA=0b00011111;
draus, damits ein 8bit-Wert wird.

[Thomas (Gast)]

ok probiere gleich aus danke

[Thomas (Gast)]

nein, leider hat nicht geholfen, das selbe ergebnis

[Christian H. (netzwanze)]

Bist Du sicher, dass Du bei 32MHz noch ein Hochzählen beobachten kannst?

[Thomas (Gast)]

na ja habe schon 4MHZ eingebaut, aber portb ausgänge sind "visuell" auf 
low, habe momentan nichts zum messen, wenn es funktionieren würde dann 
müssen die "visuell" auf H sein oder?

[Thomas (Gast)]

könnte es sein, dass die if anweisung falsch ist?

[hallo (Gast)]

42

[Rene K. (draconix)]

Selbst bei 4Mhz... Das sind immerhin 4.000.000 Anweisungen pro sekunden, 
da kann man ganz leicht mal 256 übersehen ;) Mach in deine While 
Anweisung ein definiertes warten rein und schau ob es dann klappt.

Gehe ich richtig in der Annahme das es sich um einen PIC handelt?!

[Thomas (Gast)]

ja das ist pic18f1320 und ent. umgebung ist mplab c18

[Edi R. (edi_r)]

Ich kenne den c18 nicht. Ist "LATB" wirklich Port B, sowohl als Eingang 
als auch als Ausgang?

[Thomas (Gast)]

so wie ich es verstanden habe, LATB ist ausgang, PORTB ist eingang, mehr 
kann ich auch nicht dazu sagen ;)

[Edi R. (edi_r)]

Bei "LATB++" wird erst aus LATB gelesen, dann um eins erhöht, dann auf 
LATB zurückgeschrieben. Wenn LATB nicht der Eingang von Port B ist, wird 
irgendwas anderes inkrementiert und auf dem Port ausgegeben.

Probier's mal mit "LATB = PORTB+1" statt "LATB++".

[Thomas (Gast)]

nein geht nicht, habe probiert, der reagiert nicht auf dise anweisung 
if(marke) nicht, aber warum, die variable ist doch global definiert???

[Thomas (Gast)]

diese anweisung arbeitet nicht: if(PORTAbits.RA0) marke=1; else 
marke=0;, der setzt die variable marke nicht auf 1; aber warum???

[Michael L. (hasimaus)]

Hallo Thomas,

zuerst mal: Ich kenn' den PIC auch nicht.

Aber, bei Dir scheinen beide "IF" Abfragen nicht richtig zu 
funktionieren. Nicht alle Kompiler verstehen "<>0" als "TRUE" und "0" 
als "FALSE". Somit wird die "IF" Abfrage auf "0" nicht unbedingt korrekt 
verarbeitet.

Versuch's mal mit "if(PORTAbits.RA0 == 1)".
Zudem scheint mir die Einführung der Variable "marke" redundant, da Du 
ja gleich

if(PORTAbits.RA0 == 1) LATB++;

machen kannst.

Gruß

[Thomas (Gast)]

danke für den vorschlag, habe gerade ausprobiert, leider das gleiche 
ergebnis, die variable "marke" ist in dem fall redundant, gebe ich zu, 
aber ich brauche die später

[Edi R. (edi_r)]

Es wird Zeit, den Fehler systematisch einzukreisen. Mein Vorschlag für 
einen ersten Schritt:

  while(1){
    LATB++;
  }

um erst mal zu sehen, ob das Hochzählen und das Ausgeben auf dem Port 
funktioniert.

[Thomas (Gast)]

habe ausprobiert, hochzählen ohne if anweisung geht

[Thomas (Gast)]

also ausgabe am port B funktioniert mit dem code:

while(1){
    LATB++;
  }

[Edi R. (edi_r)]

Nächster Schritt:

 while(1){
    if(PORTAbits.RA0) LATB++;
 }

um zu sehen, ob der Eingang abgefragt wird.

[Thomas (Gast)]

while(1){
    if(PORTAbits.RA0) LATB++;
 }
funzt nicht!!! keine ahnung warum????

[Edi R. (edi_r)]

Es kann entweder an der Hardware liegen (falscher Portpin, Chip defekt) 
oder an der Software. Falls es die Software ist, kann man das weiter 
einkreisen.

Nächster Schritt:

 while(1){
    LATB = PORTA;
 }

Damit müsste man am RB0 sehen, was an RA0 geändert wird.

[Thomas (Gast)]

keine reaktion auf dem port B egal ob ich die 1 anlege oder nicht

[Thomas (Gast)]

ich denke, dass es mit dem compiler probleme gibt, da ich zwei bausteine 
habe von der sorte pic18f1320, un bei den beiden das gleiche

[Thomas (Gast)]

habe compiler neu installiert auf dem anderen rechner das gleiche 
ergebnis, wie kann ich die eingänge beim pic abfragen, wiso geht das 
nicht????

[Hey Mann (Gast)]

mach doch mal

while(1){
  LATB=PORTA;
}

und schau was passiert, wenn du PORT A bit 0 high oder low setzt.

[pic pig (Gast)]

Schon einmal im single step versucht, die
Portänderung nachzuvollziehen ?
Da müsstest Du doch sehen, ob eine Änderung
feststellbar ist im Eingang.

[Thomas (Gast)]

ich bekomme ein ergebnis am portb, nur wenn ich porta als ausgang 
definiere, keine ahnung warum, was soll das, aber bin schritt weiter, 
danke dir

[Luke patrick D. (lpdunwell)]

du hast vermutlich den ADC aktiv auf PORTA...

[edit]
Aus Datenblatt:

Note: On a Power-on Reset, RA3:RA0 are
configured as analog inputs and read as
‘0’. RA4 is always a digital pin.
[/edit]

[Thomas (Gast)]

wird der nicht mit ADCON1 = 0x7f; abgeschaltet??

[Luke patrick D. (lpdunwell)]

oh ja stimmt

[Hey Mann (Gast)]

mit ADCON0.ADON=0 wird er abgeschaltet

[Thomas (Gast)]

also lansamm möchte ich einen hammer nehmen und alles um mich herum 
schön klein machen

[Michael L. (hasimaus)]

Hallo,

nochmal kurz zurückgespult.

Du willst Port B für die Ausgabe eines 8Bit Wertes benutzten.
Mit TRISB=0b11110000; setzt Du aber nur die unteren vier Bit auf 
Ausgang, glaube ich.
Zudem lass ich kürzlich in einem anderen Thread, dass 0bxxxxxxx für 
Binärdarstellung nicht von allen Kompilern verstanden wird.
Versuch mal
   TRISB=0x00;
Damit werden alle Bits als Ausgänge definiert.
Versuch mal
   TRISA=0x1F ;
Damit sind die untern 5Bits Eingäng

Gruß

[Michael L. (hasimaus)]

@Hey Mann
Ich vermute, dass durch "ADCON0.ADON=0" nur das Modul selbst 
abgeschaltet wird, die I/O Konfiguration über "ADCON1 = 0x7f;" muß 
trotzdem noch erfolgen.
So geht's jedenfalls bei den meisten uC.

Gruß

[Luke patrick D. (lpdunwell)]

also

volatile unsigned char marke=0;

main( void )
{
  //Configure port A
  TRISA=0xff;
  //Configure port B
  TRISB=0x00;
  
  //Initialize port B
  LATB=0x00;

  ADCON = 0x7f;
  
  while(1)
  {
    if(PORTA & 0x01) marke=1; else marke=0;
    if(marke) LATB++;
  }

}

funktioniert bei mir...

[Thomas (Gast)]

also noch mal, hier ist das programm:

volatile unsigned int marke=0;
#define A1 PORTAbits.RA1
void main(void)
{
 TRISB=0x00;
 TRISA=0x1f;
 ADCON1 = 0x7f;           // Alle Ausgänge/Eingänge auf Digital I/O 
setzen
 ADCON0bits.ADON=0;
 LATB=0x00;
 LATA=0x00;
 while(1){
  if(PORTAbits.RA0){
  LATB++;
    }
}

}

wenn ich porta bitstelle null mit H belege, muss doch der µC am PORTB 
hochzählen, oder??? da passiert nichts alle led's sind dunkel, wenn ich 
eine bitkombination an dem PORTB anlege z.b. LATB=0x05 es funktioniert, 
keine ahnung was da noch falsch sein kann?

[Michael L. (hasimaus)]

@Luke patrick Dunwell

Entschuldige bitte, wenn ich so kleinlich bin, aber muß es statt

   ADCON = 0x7f;

nicht

   ADCON1 = 0x7f;

heißen ? Geht das dann auch noch bei Dir ?
Gruß

[Thomas (Gast)]

welchen µc hast du?

[Thomas (Gast)]

ja ich habe ADCON1! mal ausprobieren, warte

[Thomas (Gast)]

ne habe jetzt im datenblatt gesehen war ADCON1 register

[sven (Gast)]

Stimmt. Laut Datenblatt heißen die ADCON0, ADCON1, und ADCON2.

[Michael L. (hasimaus)]

@Thomas
Hab' mir gerade mal den Kompiler Users Guide Nummer_5-mäßig reingezogen.
Danach bestätigt sich meine erste Vermutung, dass Bit-Werte mit 
0bxxxxxxx nicht möglich sind.
Es scheint jedoch möglich, über "0" bzw "<>0" die IF Abfrage zu 
kontrollieren. Trotzdem möchte ich vorschlagen, die Abfrage so zu ändern 
wie von Luke vorgeschlagen.
Nur so nebenbei, gibt's einen Schaltplan ?

Gruß

[Luke patrick D. (lpdunwell)]

Michael Leusink schrieb:
> @Luke patrick Dunwell
>
> Entschuldige bitte, wenn ich so kleinlich bin, aber muß es statt
>
>    ADCON = 0x7f;
>
> nicht
>
>    ADCON1 = 0x7f;
>
> heißen ? Geht das dann auch noch bei Dir ?
> Gruß

ist eine sache der defines bei mir.

[Marcus (Gast)]

Michael Leusink schrieb:
> ... Nicht alle Kompiler verstehen "<>0" als "TRUE" und "0"
> ... als "FALSE". Somit wird die "IF" Abfrage auf "0" nicht unbedingt
> korrekt

Doch!

 
if ( PORTAbits.RA0 ) {

}

ist gültiger Code. Wer's expliziter mag, schreibt:

 
if ( PORTAbits.RA0 == 1 ) {

}

beide Ausdrücke evaluieren identisch.


Michael Leusink schrieb:
> Hab' mir gerade mal den Kompiler Users Guide Nummer_5-mäßig reingezogen.
> Danach bestätigt sich meine erste Vermutung, dass Bit-Werte mit
> 0bxxxxxxx nicht möglich sind.

Ich weiss ja nicht, welchen Users-Guide Du gelesen hat, der vom MCC18 
war's definitv nicht. Der (MCC18) kann's nämlich.


@ThreadOpener:

Für solche geschichten sind LEDs als debugging-output aufgrund der hohen 
Taktfrequenzen denkbar ungeeignet.
Definiere Dir für den MPLAB Simulator einen Stimulus für RA0 und bring' 
das Ganze erstmal im Debugger zum laufen.

Wenn es im Debugger läuft kannst Du dich voll und Ganz auf Deine 
HArdware konzentrieren. Ist sichergestellt, daß du den PIN auf 
ordentliche Pegel ziehst ?

Viel Erfolg, nur die Ruhe bewahren!

[Michael L. (hasimaus)]

@Markus
Danke für den Hinweis, hab ich wohl beim schnellen Querlesen doch 
übersehen.

@Thomas
Nachdem soviele geholfen haben und Hinweise gaben, was ist den aus dem 
Programm geworden ?

Gruß

[Thomas (Gast)]

das programm funktioniert, danke an alle, habe hardwarefehler gehabt, 
das war das problem, aber nach dem richtigen hinweis von Markus konnte 
ich es koregieren. noch mal danke an alle!!! und wünsche allen ein 
schönes fest!!!