Hilfe bei AVR PCINT

[R. B. (dxx255)]

Hallo
Ich möchte mit einem Attiny 85 den PCINT des Reset Pins nutzen. Die 
Fuses hab ich entsprechend gesetzt. Hier der Code:

#include <avr/io.h>
#include "nrf24.h"
#define F_CPU 1000000L
#include <util/delay.h>
#include <avr/interrupt.h>
uint8_t tx_address[5] = {0xE7,0xE7,0xE7,0xE7,0xE7};
uint8_t messages[2][8]={{0,0,126,126,126,0,0},{0,0,127,127,127,0,0}};//opened closed
int main() {
  nrf24_init();
    PCMSK|=(1<<5);
    GIMSK|=(1<<5);
    DDRB&= ~(1<<5);
    PORTB |= (1<<5);
    /* Channel #2 , payload length: 8 */
      nrf24_config(2,8);

    /* Set the device addresses */
      nrf24_tx_address(tx_address);
        sei();
    while(1) {
    }
        
    return 0;
}
ISR(PCINT0_vect){
  _delay_ms(50);
  nrf24_send(messages[(PINB & (1<<5))>>5]);
    while(nrf24_isSending());
}

Wenn ich aber PB5 an Masse lege wird die ISR immer doppelt ausgeführt. 
Wenn ich die Verbindung wieder trenne nicht. Warum ist das so?

LG

[Marc V. (Firma: Vescomp) (logarithmus)]

Roman B. schrieb:
> Wenn ich aber PB5 an Masse lege wird die ISR immer doppelt ausgeführt.
> Wenn ich die Verbindung wieder trenne nicht. Warum ist das so?

 Welche Verbindung ? Zur Masse oder zu anderem NRF ?
 Und wie meinst du das: "PB5 an Masse lege " ?

[R. B. (dxx255)]

PB5 ist über einen Reed-Schalter mit GND verbunden. Wenn ich den auslöse 
werden zwei Nachrichten versandt. Wenn ich ihn loslasse nur eine. Warum?

[H.Joachim S. (crazyhorse)]

Wird wohl am prellen liegen.

[Marc V. (Firma: Vescomp) (logarithmus)]

Roman B. schrieb:
> werden zwei Nachrichten versandt. Wenn ich ihn loslasse nur eine. Warum?

 Prellen ?

[H.Joachim S. (crazyhorse)]

Weil das zweite Ereignis gespeichert wir, während die erste ISR noch 
läuft.
Beim Loslassen hast du wohl tatsächlich nur eine einzige Flanke, gut 
möglich.

Deine ISR ist an sich Mist, mit delay und zeitaufwändiger Kommunikation.
Kannst ja als erste Massnahme erstmal vor Verlassen der ISR weitere 
anhängige Pinchange-Ints löschen, dann wirds funktionieren. Ist aber 
bestimmt noch nicht die endgültige Lösung :-)

[R. B. (dxx255)]

So gehts:

#include <avr/io.h>
#include "nrf24.h"
#define F_CPU 1000000L
#include <util/delay.h>
#include <avr/interrupt.h>
uint8_t tx_address[5] = {0xE7,0xE7,0xE7,0xE7,0xE7};
uint8_t messages[2][8]={{0,0,126,126,126,0,0},{0,0,127,127,127,0,0}};//opened closed
char state=-1;
int main() {
  nrf24_init();
    PCMSK|=(1<<5);
    GIMSK|=(1<<5);
    DDRB&= ~(1<<5);
    PORTB |= (1<<5);
    /* Channel #2 , payload length: 8 */
      nrf24_config(2,8);

    /* Set the device addresses */
      nrf24_tx_address(tx_address);
        sei();
    while(1) {
      if(state){
        cli();
        nrf24_send(messages[state-1]);
        while(nrf24_isSending());
        state=-1;
        sei();
      }
    }
        
    return 0;
}
ISR(PCINT0_vect){
  _delay_ms(50);
  state=(PINB & (1<<5)>>5)+1;
}

[H.Joachim S. (crazyhorse)]

Durch die Brust ins Auge?

[R. B. (dxx255)]

Was meinst du damit? Es geht doch nicht:(

[R. B. (dxx255)]

Jetzt aber:

#include <avr/io.h>
#include "nrf24.h"
#define F_CPU 1000000L
#include <util/delay.h>
#include <avr/interrupt.h>
uint8_t tx_address[5] = {0xE7,0xE7,0xE7,0xE7,0xE7};
uint8_t messages[2][8]={{0,0,126,126,126,0,0},{0,0,127,127,127,0,0}};//opened closed
int state=-1;
int main() {
  nrf24_init();
    PCMSK|=(1<<5);
    GIMSK|=(1<<5);
    DDRB&= ~(1<<5);
    PORTB |= (1<<5);
    /* Channel #2 , payload length: 8 */
      nrf24_config(2,8);

    /* Set the device addresses */
      nrf24_tx_address(tx_address);
        sei();
    while(1) {
      if(state){
        cli();
        nrf24_send(messages[state-1]);
        while(nrf24_isSending());
        state=-1;
        sei();
      }
    }
        
    return 0;
}
ISR(PCINT0_vect){
  _delay_ms(50);
  if(PINB & (1<<5))state=1;
  else state=2;
}