Port Einleseproblem in ISR und C++

[Gert Birkner (Gast)]

Hallo Leute,
ich habe heute eine ISR in C++ implementiert, die alle 10µs ausgeführt 
werden soll (F_CPU = 16MHz). Die ISR funktioniert soweit ganz gut, aber 
beim Einlesen des PORTB (PINB) reagiert die if nicht, obwohl ich weiß 
(Oszi) dass das PINB1 garantiert 400µs auf 1 ist.

CodeAusschnitt:

void ultrasonic::isrTimer()
{
 switch(m_Status)  {
  case inAktive:
  m_Status = triggerActive;
  (*TriggerPort) |= (1 << TriggerPin);
  break;
 case triggerActive:
  m_Status = awaitingEcho;
  (*TriggerPort) &=  ~(1 << TriggerPin);
  break;
 case awaitingEcho:
  if((*EchoPort) & EchoPin)     // !!!!!!  wird nie true ??? !!!!!!!!
   m_Status = measureActive;
  m_distanceCount=1;
  break;
 case measureActive:
  if((*EchoPort) & EchoPin)
   m_distanceCount++;
  else {
   if (m_Behavior == single) {
    TCCR0B &= ~(1 << CS00);       // Timer stoppen
    TCNT0 = 0;
   } 
   (*TriggerPort) &=  ~(1 << TriggerPin);
   m_Status = inAktive;
   }
  break;
 } // switch
}

bis case awaitingEcho: funktioniert alles sehr gut. Der TriggerPin wird 
gesetzt, beim nächsten Durchgang gelöscht. Und dann sollte auf das Echo 
gewartet werden um die Messzählung zu starten. Aber die if wird nie 
true!

Aufgebaut mit: ATmega1284, die Adressen der Ports stimmen mit dem 
Datenblatt überein. Für das EchoPort wird PINB übergeben und EchoPin = 
1.

Hab ich hier einen Denkfehler?

[Justus S. (jussa)]

Gert Birkner schrieb:
> Hab ich hier einen Denkfehler?

ja, du fragst Pin 0 ab...

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

Gert Birkner schrieb:
> Hab ich hier einen Denkfehler?
 Da ich annehme, dass du die Ports entsprechend definiert hast...

Gert Birkner schrieb:

void ultrasonic::isrTimer()
{
 switch(m_Status)  {
  case inAktive:
  m_Status = triggerActive;
  (*TriggerPort) |= (1 << TriggerPin);
  break;

//* Hast du in diesen 10us, die TriggerPin on war, etwas sinnvoles getan ?
 case triggerActive:
  m_Status = awaitingEcho;
  (*TriggerPort) &=  ~(1 << TriggerPin);    //* Vielleich sollte das erst weiter unten passieren ?
  break;

//* Angenommen, (*EchoPort) & EchoPin) funktioniert (tut es aber nicht)...
 case awaitingEcho:
  if((*EchoPort) & EchoPin)     // !!!!!!  wird nie true ??? !!!!!!!!
   m_Status = measureActive;
  m_distanceCount=1;
  break;

 case measureActive:
  if((*EchoPort) & EchoPin)
   m_distanceCount++;

//* Wenn m_Behavior == single wird der Timer gestoppt, muss erst wieder gestartet werden
  else {
   if (m_Behavior == single) {
    TCCR0B &= ~(1 << CS00);       // Timer stoppen
    TCNT0 = 0;
   }

//* TriggerPin ist schon aus, wenn du hier angelangt bist, wozu (Fehler) ?
   (*TriggerPort) &=  ~(1 << TriggerPin);

//* Nur wenn m_Behavior <> single oder Timer erneut gestartet wird, macht die untere Zeile einen Sinn.
   m_Status = inAktive;
   }
  break;
 } // switch
}

[Daniel A. (daniel-a)]

EchoPort muss volatile sein. Ist er das?

ungetestet:

// variante 1
// header
#define EchoPort PINx
// variante 2
// header
extern volatile unsigned char* EchoPort;
// cpp
volatile unsigned char* EchoPort = &PINx;
// variante 3
// header
static constexpr volatile unsigned char& EchoPort = PINx;
// variante 4
// header
struct Port {
  volatile unsigned char& port;
  volatile unsigned char& pin;
  operator int(){
    return pin;
  }
  Port operator=(int x){
    port=x;
  }
  Port(volatile unsigned char& port,volatile unsigned char& pin) : port(port), pin(pin) {}
};
static constexpr Port EchoPort(PORTx,PINx);
// unendlich viele weitere möglichkeiten...

[Gert Birkner (Gast)]

Vielen Dank,
da hast du natürlich recht. es sollte natürlich
if ((*EchoPort) & (1 << EchoPin))
heißen.
... dummer Fehler ...