AVR stürzt ab nach PCINT

[Mr. Fehler (Gast)]

Hallo,

hat jmd. eine Ahnung, woran es liegen könnte, das meine AVR beim Starten 
immer so 2-3x abstürzen direkt nach dem Start, nachdem ich PCINT 
aktiviert habe?!

Ich setze den Pin auf Eingang, dann PCINT aktiv (während dessen liegt 
dort bereits ein 20khz Signal an). Dann einige Sachen später mache ich 
sei().

Woran liegt das?!

[Bastian W. (jackfrost)]

Ohne dein Program zu kennen kann dir keiner helfen.

Gruß JackFrost

[tremolo (Gast)]

Schaltplan und Code bitte. Sonst hilft nur die Glaskugel. Die sagt 
Kurzschluss oder Abblockkondensatoren vergessen.

[Dieter F. (Gast)]

Mr. Fehler schrieb:
> Woran liegt das?!

Keine ISR definiert.

[H.Joachim S. (crazyhorse)]

Dieter F. schrieb:
> Keine ISR definiert.

Mr. Fehler schrieb:
> Dann einige Sachen später mache ich
> sei().

Vorher dürfte das also nichts machen.
Aber ohne Programm gehts hier nicht weiter.

[Rätsel Rater (Gast)]

Mr. Fehler schrieb:
> das meine AVR beim Starten
> immer so 2-3x abstürzen

Ich würde erst mal wissen wollen woran man erkennt das
der Controller "abstürzt".

Am Bluescreen?
Ach so, hat keinen Screen ... hmmmm ....

[Karl (Gast)]

Mr. Fehler schrieb:
> hat jmd. eine Ahnung, woran es liegen könnte, das meine AVR beim Starten
> immer so 2-3x abstürzen direkt nach dem Start, nachdem ich PCINT
> aktiviert habe?!

Hast Du einen Programmer angeschlossen?

[Mr. Fehler (Gast)]

Hallo,

kein Programmer angeschlossen, 100n habe ich natürlich und woran man das 
merkt, ganz einfach:

Es wird vor dem ganzen Init die IOs eingestellt, und 2 Ausgangspinne auf 
5V geschaltet. Und da hängen LEDs dran über einen Mosfet. Und die 
flackern halt 2-3x bevor alles normal läuft. Man sieht es auch am Osci.


ISR habe ich natürlich.


In der ISR ist eine recht große inline Funktion, das geht nicht anders. 
Ich habe das mal schrittweise durchgetestet, die Variablen sind alle 
volatile. Ich poste später das Programm noch.

[Mr. Fehler (Gast)]

By the way: Was mir aufgefallen ist: Wir der controller initialisiert 
ohne das das 20khz Signal anliegt, dann stürzt er nicht ab.

[Wolfgang (Gast)]

Mr. Fehler schrieb:
> In der ISR ist eine recht große inline Funktion, das geht nicht anders.

Und die braucht maximal wie lange?

[Mr. Fehler (Gast)]

Das steht in der ISR drin um das Signal zu detectieren:

//get and reset the timer ovf bits
  if (decoder.dcc.state.systemBits.timer_status)  decoder.dcc.state.systemBits.timer_status = FALSE;
  
  //dcc is now high
  if (DCC_IS_HIGH && !decoder.dcc.state.systemBits.cnt_starts)
  {
    TCNT0 = NULL;
    decoder.dcc.state.systemBits.cnt_starts = TRUE;
  }
  else if (!DCC_IS_HIGH && decoder.dcc.state.systemBits.cnt_starts)
  {            
    if (decoder.dcc.dcc_stretched_0)  decoder.dcc.dcc_cnted_time = 0xFF;
    else                decoder.dcc.dcc_cnted_time = TCNT0;
    
    decoder.dcc.dcc_stretched_0 = FALSE;
    decoder.dcc.state.systemBits.cnt_starts = FALSE;
    decoder.dcc.state.systemBits.timer_status = TRUE;
  }
  else if (decoder.dcc.state.systemBits.cnt_starts && TCNT0 >= 220)   //detect streched 0
  {
    decoder.dcc.dcc_stretched_0 = TRUE;
  }

if (decoder.dcc.sysCond.systemBits.init_time_over && !decoder.timer.timer_univ[start_switching_timer])
  {
    decoder.signalstate = signalstate_stop;
    decoder.signalstate_last = signalstate_stop;
    decoder.dcc.sysCond.systemBits.init_time_over = FALSE;
  }

  
  //noch kein ganzes packet bekommen?
  if (decoder.dcc.state.systemBits.timer_status && !decoder.dcc.state.systemBits.received_all_valid)
  {
    if (!decoder.dcc.state.systemBits.start_detected)  //find beginning!
    {
      //cnt all 1 bits up to get a 0 and cnter >= 10 ==> Start!
      if (decoder.dcc.dcc_cnted_time <= decoder.dcc.dcc_time_logic_one)  decoder.dcc.cnt_bit_pos++;
      else //0 is detected
      {
        //start is detected!! //more than 9 1's bits? ==> must be the start!
        if (decoder.dcc.cnt_bit_pos > 9)
        {
          decoder.dcc.cnt_byte_pos = NULL;
          decoder.dcc.state.systemBits.start_detected = TRUE;
          decoder.dcc.state.systemBits.get_data_byte = TRUE;
          
          for (uint8_t i = NULL; i < 15; i++)  decoder.dcc.RX[i] = NULL;
        }
        
        decoder.dcc.cnt_bit_pos = NULL;
      }
    }
    else if (decoder.dcc.state.systemBits.get_data_byte)  //byte transfer
    {
      if (decoder.dcc.dcc_cnted_time <= decoder.dcc.dcc_time_logic_one)  decoder.dcc.RX[decoder.dcc.cnt_byte_pos] |= (1<<(7 - decoder.dcc.cnt_bit_pos));
      
      decoder.dcc.cnt_bit_pos++;
      
      
      //complete byte detected, now check the divisionbit
      if (decoder.dcc.cnt_bit_pos > 7)
      {
        decoder.dcc.cnt_bit_pos = NULL;
        decoder.dcc.state.systemBits.get_data_byte = FALSE;
      }
    }
    else  //get the division bit, 0 = divide ==> next bytes comes, 1 = end (then check the data)
    {      
      if (decoder.dcc.dcc_cnted_time <= decoder.dcc.dcc_time_logic_one)   //division bit is 1 ==> end transmission
      {
        //check parity, if good ==> you can work with data
        decoder.dcc.RX[decoder.dcc.cnt_byte_pos + 1] = decoder.dcc.RX[decoder.dcc.cnt_byte_pos - 1];
        for (uint8_t i = NULL; i < (decoder.dcc.cnt_byte_pos - 1); i++)  decoder.dcc.RX[decoder.dcc.cnt_byte_pos + 1] ^= decoder.dcc.RX[i];
        
        //calculated parity is same as transmitted ==> data is correct! Now check the 2 0815 states (empty data and decoder restart)
        if (decoder.dcc.RX[decoder.dcc.cnt_byte_pos] == decoder.dcc.RX[decoder.dcc.cnt_byte_pos + 1])
        {  
          //set instruktion pointer!
          //where is the real data?! (only for locomotives)
          if (DCC_IS_NORMAL_ADR_SHORT)  decoder.dcc.operation_instr_pos = 1;
          else              decoder.dcc.operation_instr_pos = 2;
          
          decoder.dcc.state.systemBits.operation_dcc = TRUE;
          decoder.dcc.state.systemBits.received_mode = 2;  //deceoder reset as init
          
          for (uint8_t i = NULL; i < decoder.dcc.cnt_byte_pos; i++)  //check if every byte is 0
            if (decoder.dcc.RX[i])  decoder.dcc.state.systemBits.received_mode = 1;  //not every 0's ==> check if every 2nd byte is 0
          
          for (uint8_t i = NULL; i < decoder.dcc.cnt_byte_pos; i+=2)  //check if every 2nd byte is 0
            if (decoder.dcc.RX[i])  decoder.dcc.state.systemBits.received_mode = NULL;  //not every 2nd byte is 0 ==> normal op
          
          
          //check the states
          if (decoder.dcc.state.systemBits.received_mode == 1)    //idle mode received
          {
            decoder.dcc.dcc_reset_state = NULL;
            decoder.dcc.state.systemBits.start_detected = FALSE;
          }
          else if (decoder.dcc.state.systemBits.received_mode == 2)  //reset mode received
          {
            decoder.dcc.dcc_reset_state++;
            decoder.dcc.state.systemBits.start_detected = FALSE;
            
            if (decoder.dcc.dcc_reset_state >= 8)  dcc_reset_decoder();
          }
          else  //check normal op or programming mode
          {
            decoder.dcc.dcc_reset_state = NULL;
            
            //cv programming (service mode)
            if (DCC_IS_SERVICE_MODE)    
            {          
              dcc_handle_service_mode();
              decoder.dcc.sysCond.systemBits.init_time_over = FALSE;  
            }
            else if (DCC_IS_POM_MODE_ASSCES(DCC_16BIT_ADDRESS(decoder.system.switch1.switch_addr_hi, decoder.system.switch1.switch_addr_lo)))  
            {
              dcc_handle_service_POM_mode();  
              decoder.dcc.sysCond.systemBits.init_time_over = FALSE;  
            }
            else
              decoder.dcc.state.systemBits.received_all_valid = TRUE;
          }
        }
        
        decoder.dcc.state.systemBits.start_detected = FALSE;
      }
      else //got next byte
        decoder.dcc.cnt_byte_pos++;
      
      
      decoder.dcc.state.systemBits.get_data_byte = TRUE;
    }
  }

[Karl M. (Gast)]

Hallo Mr. Fehler,

leider kann man die PCINT intitialisierung nicht sehen.

Meistens wird der 8er Block PCINTx Eingänge einem falschen PCINTy 
Interrupt Requenst zugeordnet.

Ich denke hier liegt auch dein Fehler.

[Mr. Fehler (Gast)]

Ich denke nicht (und hoffe).

Angeschlossen an PB3 an einem Attiny85

Aus der init funktion:

  decoder.peripherie.dcc_ack_pin = pin_ack;
  decoder.peripherie.dcc_pin = pin_dcc;
  decoder.peripherie.dcc_dir = dir_dcc;
  decoder.peripherie.dcc_port = port_dcc;
  decoder.peripherie.ack_dir = dir_ack;
  decoder.peripherie.ack_port = port_ack;
  decoder.peripherie.dcc_pin_reg = pin_reg;
  
  
  //Timer init with 1ms/ovf (only for timing, e.g can dimming (soft)
  //Für den Timer (interrupt)
  TCCR1 |= (1<<CS11)|(1<<CS12);    //prescaler 32
  TIMSK |= (1<<TOIE1);  //TOIE ==> ~1 kHz
  
  //timer for dcc detection
  TCCR0B |= (1<<CS01);    //prescaler 8

#if (DCC_DETECTION_INTERRUPT)  //PCINT3
  PCMSK |= (1<<PCINT3);
  GIMSK |= (1<<PCIE);  
#endif

sei();

[Karl M. (Gast)]

Hallo,

was soll das sein ?

decoder.peripherie.dcc_ack_pin = pin_ack;
  decoder.peripherie.dcc_pin = pin_dcc;
  decoder.peripherie.dcc_dir = dir_dcc;
  decoder.peripherie.dcc_port = port_dcc;
  decoder.peripherie.ack_dir = dir_ack;
  decoder.peripherie.ack_port = port_ack;
  decoder.peripherie.dcc_pin_reg = pin_reg;

Wo stehen die AVR Register und ist der Interrupt Vector belegt worden?

[Mr. Fehler (Gast)]

Karl M. schrieb:
> Hallo,
>
> was soll das sein ?decoder.peripherie.dcc_ack_pin = pin_ack;
>   decoder.peripherie.dcc_pin = pin_dcc;
>   decoder.peripherie.dcc_dir = dir_dcc;
>   decoder.peripherie.dcc_port = port_dcc;
>   decoder.peripherie.ack_dir = dir_ack;
>   decoder.peripherie.ack_port = port_ack;
>   decoder.peripherie.dcc_pin_reg = pin_reg;Wo stehen die AVR Register
> und ist der Interrupt Vector belegt worden?

Das ist von der oberen Funktion wo die entsprechenden PORT und Pin und 
PIN übergeben werden. Ist das wichtig/interessant?! Das ist nur PORTB, 
PBx und PINB

Das ist der Vector:

ISR(PCINT0_vect)
{
   //hier steht das von oben als inline void
}

[B. P. (skorpionx)]

Das habe ich mit PIC erlebt.:

Unbenutzte Pins vom Mikroprozessor als Fehlerquelle.

[Cerberus (Gast)]

Ich würde erst mal die Betriebsspannung des AVR stabilisieren
BEVOR man ein Signal auf einen Eingang gibt. Sonst können durch
die interne Schutzbeschaltung die seltsamsten Dinge passieren,
z.B. das so ein Signal wie eine Fremdversorgung wirkt, aber die
ist eben nicht stabil.

[Mr. EEPROM (Gast)]

Ich habe tatsächlich alle benutzt

[Mr. EEPROM (Gast)]

Das kann gut sein, dass Signal liegt aber immer an das kann ich gar 
nicht ändern. Ich kann es mal testen aber im Betrieb geht das nicht 
anders. Evtl pin als Ausgang warten dann Eingang