Befehl wir nur einmal ausgeführt

[Chunga (Gast)]

Hey Leute!

Ich bräuchte mal etwas Hilfe und hab folgende Fragestellung.

Nachdem ich den Pin A5 einlese, (Taster) wird das Programm wie gewollt 
ausgeführt, doch wenn es EINMAL Fertig abgelaufen ist lässt es sich 
nicht mehr starten. Es funktioniert nur wieder wenn ich das Programm neu 
auf den 18F27J53 lade.

Hier der betrefende Teil des Codes: (CCS Compiler)

// Hauptprogramm
void main() {
  
  HW_init();  
  

  while (true){

      
    if(input(PIN_A5)){ //Vorwärts
    
    delay_ms(2000);

      while(true){
        if(dis >= 20){
          
          doFor();    
          
          doEntf();
    
        }

        else{

          doStop();

              
        }
      delay_ms(80);
      
      }  
    }  
  }    
}

Ich hoffe jemand hat ne idee^^

[Ralf (Gast)]

Du hast zweimal 'while(true)'. So, und jetzt überleg mal, ob die äußere 
Schleife noch jemals drankommt, wenn die innere abgearbeitet wird...

Ralf

[Chunga (Gast)]

Ja die ist ja nicht das Problem!

Das Programm lässt sich selbst dann kein 2tes mal starten wenn ich den 
Pic resete bzw die Stromversorgung kappe.

Trd Danke für die schnelle Antwort!

[Ralf (Gast)]

Kannst du auf dem Ding debuggen? Wenn ja, dann mach doch einfach einen 
normalen Reset, und schau dann mal, ob die Variablen vielleicht 
vermurkst sind.

Ralf

[Teo D. (teoderix)]

Dass Problem liegt dann aber nicht in dem Codeschnippel.
Sei mal nicht so sparsam mit den Informationen!

[Chunga (Gast)]

ne debuggen kann ich leider nich.

Hier mal der ganze Code mit Header!

Ich weis ja, da is noch die ein oder andere Leiche begraben!

#include <18F27J53.h>
#device ADC=12                                // nicht ver?ndern
#fuses  HSPLL,PLLEN,PLL5, NOCPUDIV, NOWDT,NOPROTECT          // nicht ver?ndern
#fuses  ADC12, RTCOSC_INT, DSWDTOSC_T1              // nicht ver?ndern
#use    delay(clock=48000000)                        // nicht ver?ndern
#use   i2c(Master,sda=PIN_B5,scl=PIN_B4,fast=6400)        //I2C eintrag

// F?r Bootloader ------------------------------------------------
#build (reset=0x1000, interrupt=0x1008)  // neue Adressen        // nicht ver?ndern
#org 0x0,0xfff {}            // res. Bereich            // nicht ver?ndern
// ---------------------------------------------------------------


#include <usb_cdc.h>

#define  DS_write 0x20 
#define  DS_read  0x21

//#define forLeft set_pwm6_duty
//#define forRight set_pwm7_duty
//#define backLeft set_pwm9_duty
//#define backRight set_pwm10_duty 

// schreibe Char in USB-Transmitt-Buffer
//void usb_putc(char cc)              // printf(usb_putc,...)
//{
//    while(!usb_cdc_putready());
//  usb_cdc_putc(cc);
//}
// USB -------------------------------------------------------------------------------------------


// globale Variable

int1 toggle=0;
int winkelHB, winkelLB, entfHB, entfLB;
long winkel, dis; 
float div;


//ISR Interrupt Srevice Routine TMR0
#INT_TIMER0
void tmr0_handler() //Name der Routine
{
  set_timer0(18660);    // da ja 1s
  toggle=1;        // 1 -> 1s vorbei
}

void HW_init(){

  //SETUP_TIMER_2(T2_DIV_BY_1,255,1);
  //SETUP_CCP6(CCP_PWM);
  //SETUP_CCP7(CCP_PWM);
  //SETUP_CCP9(CCP_PWM);
  //SETUP_CCP10(CCP_PWM);
  
  i2c_start();          //start 
     i2c_write(DS_write);   //Adresse 0x20, ->Write=0, Read=1 
     i2c_write(0x70);       //Wake up  
    i2c_stop();          //stop 
  
  //Wait for wake up
  delay_ms(1100);
} 


void doEntf(){          //Messvorgang in cm Starten

  i2c_start();
  i2c_write(0xE0);      //slave id (0xE0) und write, LSB = low
  i2c_write(0);
  i2c_write(0x51);      //Entfernung
  i2c_stop();

  delay_ms(65);

  i2c_start();
  i2c_write(0xE0);      //slave id (0xE0) und write, LSB = low
  i2c_write(0x02);         //von register 2 lesen (= entfernung high Byte)
  i2c_stop();    

  i2c_start();
  i2c_write(0xE1);      //slave id (0xE0) und READ, LSB = high
  entfHB = i2c_read(1);    //do ACK
  entfLB = i2c_read(0);     //don't ACK      
    i2c_stop();         //stop

  delay_ms(10);


  dis = make16 (entfHB, entfLB);  
}


void doCalibration(){

  i2c_start();
  i2c_write(DS_write);
  i2c_write(0xE0);          
  i2c_stop();  
}

//I2C Compass einlesen
void doCompass(){
     

  i2c_start();
  i2c_write(DS_write);
  i2c_write(0xC0);          
  i2c_stop();  

  delay_ms(210);

  i2c_start();
  i2c_write(DS_read);   
     winkelHB = i2c_read();
  winkelLB = i2c_read(0);       
     i2c_stop();             

  delay_ms(10);
}


void doFor(){

  output_high(Pin_B6);
  output_low(Pin_B7);  
  output_high(Pin_C7);
  output_low(Pin_C6);    
}

void doBack(){
  
  
  output_low(Pin_B6);
  output_low(Pin_C7);
  output_high(Pin_B6);
  output_high(Pin_C7);  
}

void doRight(){

  output_low(Pin_B6);
  output_high(Pin_B7);
  output_low(Pin_C6);
  output_low(Pin_C7);

}

void doLeft(){

  output_low(Pin_B0);
  output_low(Pin_B1);
  output_high(Pin_B2);
  output_low(Pin_B3);  

}

void doStop(){

  output_high(Pin_B6);
  output_high(Pin_B7);
  output_high(Pin_C6);
  output_high(Pin_C7);

}

// Hauptprogramm
void main() {
  
  HW_init();  
  

  while (true){

      
    if(input(PIN_A5)){ //Vorwärts
    
    delay_ms(2000);

      while(true){
        
        if(dis >= 20){
          
          doFor();    
          
          doEntf();
    
        }

        else{

          doStop();

              
        }
      delay_ms(80);
      
      }  
    }    
  }    
}

[Karl H. (kbuchegg)]

Das kommt mir seltsam vor:

Muss man beim Pic nie definieren, ob ein Hardware-Pin jetzt Eingang oder 
Ausgang sein soll? Ich sehe in deinem ganzen Programm nirgends 
entsprechende Codestellen, die wenigstens ungefähr so aussehen als ob 
sie das tun würden. Oder erledigen das die Funktionen input() bzw. 
output()

Auch: Wie ist denn der Taster hardwaremässig angeschaltet?
Ist sichergestellt, dass
a) der entsprechende Pin tatsächlich auf Eingang gestellt ist
b) der entsprechende Portpin nie offen ist. D.h hast du da einen 
externen Pullup (oder Pulldown) Widerstand drann, der dem Pin auch dann 
einen definierten Pegel gibt, wenn der Taster nicht gedrückt ist?