Entprellen Flankenerkennung

[kopr (Gast)]

habe schon einige alte Beiträge zu diesem thema gelesen. bin Anfänger 
wie sich sicherlich versteht.
Das kritische scheint zu sein:
Die Entprellung geschieht dabei durch die ganze Laufzeit des Programms.
für kurze Laufzeiten benötigt man evtl ein delay, bei langen laufzeiten 
wird ein Drücken verschluckt.

Es war aber für mich persönlich die am einfachsten zu verstehende 
routine, zumal ich kein AVR habe und es mich überfordert, die anderen 
routinen auf meine Plattform anzupassen.

Was würdet ihr mir empfehlen?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

kopr schrieb:
> Was würdet ihr mir empfehlen?
Nimm die bekannten Routinen von PeDa.
Versuche zu verstehen, was die machen und lerne daraus.

[kopr (Gast)]

Bitte entschuldigt die Rechtschreibung. Habe das grade mit einem Anflug 
von Emotionen geschrieben

[kopr (Gast)]

Lothar M. schrieb:
> Nimm die bekannten Routinen von PeDa.
> Versuche zu verstehen, was die machen und lerne daraus.

da scheiterts nicht nur am Verständnis, sondern eben auch daran, dass 
ich nicht weiß, wie ich den Code an meine Plattfrom anzupassen habe

[c-hater (Gast)]

kopr schrieb:

> Was würdet ihr mir empfehlen?

Dein Verständnis durch Lernen hinreichend zu verbessern, so dass du in 
den Status gelangst, auch die bewährten "komplizierten" Entprellroutinen 
zu verstehen. Die sind nämlich letztlich auch ziemlich trivial...

> zumal ich kein AVR habe und es mich überfordert, die anderen
> routinen auf meine Plattform anzupassen.

Wenn du nicht einmal diesen trivialen Scheiß an eine andere Plattform 
anpassen kannst, bist du halt einfach noch nichtmal andeutungsweise ein 
Programmierer.

Du kannst das entweder durch LERNEN ändern oder dir einfach ein Hobby 
suchen, für das deine Kompetenzen ausreichen. Briefmarkensammeln z.B. 
soll sehr enspannend sein. Aber: Es gibt auch dort viel zu lernen...

[kopr (Gast)]

bin grade auf Folgendes gestoßen:L

Beitrag "Entprellen für Anfänger"

fürs Auge auf jeden Fall angenehmer. tut mir lied, wenn euch das zu 
simpel gestrickt ist. Jetzt kommt der Punkt mit dem Anpassen

[kopr (Gast)]

c-hater schrieb:
> durch LERNEN ändern

ich möchte grade lernen, wie ich das anpasse =) dieses eine Mal mit 
einer Abkürzung an zig Büchern vorbei...

[Carl D. (jcw2)]

kopr schrieb:
> c-hater schrieb:
>> durch LERNEN ändern
>
> ich möchte grade lernen, wie ich das anpasse =) dieses eine Mal mit
> einer Abkürzung an zig Büchern vorbei...

Benutz PeDas Routine in C. Dann must du keine Neuimplementierung machen, 
wenn du mal die HW-Plattform wechselst.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Was ist denn das für eine geheimnisvolle Plattform? Anstatt da so ein 
Wischiwaschi zu schreiben, hättest du ganz emotionslos mal schreiben 
können, worum es geht.
Eins ist klar, die einzige Anpassung betrifft die I/O des verwendeten 
Controllers.

[c-hater (Gast)]

kopr schrieb:

> ich möchte grade lernen, wie ich das anpasse =) dieses eine Mal mit
> einer Abkürzung an zig Büchern vorbei...

Für Lernen gibt es leider keine Abkürzungen. Das muß man immer selbst 
tun. Ob man dafür 'zig Bücher braucht oder nicht, ist individuell 
verschieden. Aber nach meiner Erfahrung sind zur Übertragung von 
logischen Konzepten keine Massen an Büchern nötig. Eventuell nötiges 
Faktenwissen kann allerdings nur aus Büchern kommen (oder vergleichbaren 
Medien).

[kopr (Gast)]

Matthias S. schrieb:
> Was ist denn das für eine geheimnisvolle Plattform?

msp 430

Matthias S. schrieb:
> Eins ist klar, die einzige Anpassung betrifft die I/O des verwendeten
> Controllers.

#include <avr/io.h>
#define F_CPU 9.6e6
#include <util/delay.h>

was ist mit den includes. ist in den Definitionen etwas drin, was ich 
brauche? Habe beim MSP entsprechend andere Defintionen. Was gibts da zu 
beachten?

[Thomas (Gast)]

c-hater schrieb:
> Für Lernen gibt es leider keine Abkürzungen. Das muß man immer selbst
> tun. Ob man dafür 'zig Bücher braucht oder nicht, ist individuell
> verschieden. Aber nach meiner Erfahrung sind zur Übertragung von
> logischen Konzepten keine Massen an Büchern nötig. Eventuell nötiges
> Faktenwissen kann allerdings nur aus Büchern kommen (oder vergleichbaren
> Medien).

ah ja, da ist ja unser frisch geouteter Doktor

[Joachim B. (jar)]

kopr schrieb:
> dass
> ich nicht weiß, wie ich den Code an meine Plattfrom anzupassen habe

wir wissen ja noch nicht mal deine Plattform!

Beim ESP32 kann ich durchaus einen 10ms Timerinterrupt einrichten und 
dann PeDdas Sourcecode in C nehmen.

Aber wie immer gilt step by step lernen und MACHEN

[Thomas (Gast)]

Joachim B. schrieb:
> wir wissen ja noch nicht mal deine Plattform!

hab doch geschrieben, es ist ein msp430

[kopr (Gast)]

Thomas schrieb:
> Joachim B. schrieb:
>> wir wissen ja noch nicht mal deine Plattform!
>
> hab doch geschrieben, es ist ein msp430

genau ein Msp 430. Hattest du die Frage nach den Definitionen gesehen

[c-hater (Gast)]

Thomas schrieb:

> ah ja, da ist ja unser frisch geouteter Doktor

Das war ein Fake-Posting, da segelte jemand unter falscher Flagge. Nach 
der Logik des Textes könnte es Herr Drexler gewesen sein, allerdings 
halte ich den nicht für auch nur annähernd blöd genug, sich eine 
derartige Blöße zu geben. Es wird also ein böswilliger Störer gewesen 
sein, sowas diese unsägliche MaWin-Imitation, die immer mal wieder 
auftaucht.

Also ganz klar: Ich habe keinen Doktortitel. Ich hätte das im Thread 
selber klargestellt, wenn nicht irgendeiner der "weisen" Moderatoren 
beschlossen hätte, ihn zu schliessen, als es gerade begann, spannend zu 
werden...

[Joachim B. (jar)]

Thomas schrieb:
> hab doch geschrieben, es ist ein msp430

kopr schrieb:
> genau ein Msp 430

2 Namen, ein User?

egal

Timernutzung, sieht dem AVR und ESP verdammt ähnlich, also kein 
Hexenwerk!
https://www.medit.hia.rwth-aachen.de/fileadmin/MSP430Buch/msp_html_buchse15.html

und auch mit Beispiele in C was PeDas C-Code einfügen einfach macht,
also erst mal eine LED anschalten, ein Port einlesen usw.

[Thomas (Gast)]

sry Leute, sollte "hat" geschrieben heißen

[kopr (Gast)]

Joachim B. schrieb:
> imernutzung

was hat der Debounce-Makro mit Timernutzung zu tun?

[kopr (Gast)]

timernutzung etc sollte beim MSP kein problem sein

[Axel S. (a-za-z0-9)]

kopr schrieb:
> habe schon einige alte Beiträge zu diesem thema gelesen. bin Anfänger
> wie sich sicherlich versteht.

Ich verstehe nur, daß du ein Troll sein willst.
Immerhin hast du den Wochentag korrekt gewählt.
+1

> Das kritische scheint zu sein:
> Die Entprellung geschieht dabei durch die ganze Laufzeit des Programms.

Nein.

> für kurze Laufzeiten benötigt man evtl ein delay

Nein.

> bei langen laufzeiten wird ein Drücken verschluckt.

Nein.

> Es war aber für mich persönlich die am einfachsten zu verstehende
> routine, zumal ich kein AVR habe und es mich überfordert, die anderen
> routinen auf meine Plattform anzupassen.

Du sprichst in Rätseln. Was willst du gelesen haben?

> Was würdet ihr mir empfehlen?

Die Tür zu schließen. Von außen. Leise.

"Vielen Dank für die Beachtung aller Sicherheitsmaßnahmen!"

[A. S. (Gast)]

kopr schrieb:
> Was würdet ihr mir empfehlen?

Schreib mal, was Du hast und kannst.

Wenn Du z.B.
- eine Taste auslesen kannst
- einen SysTicker oder eine zyklische Funktion hast
- nur den Druck (als die erste Flanke) erkennen willst

dann geht es relativ einfach gebastelt:

/* Block, der zyklisch, z.B. jede ms aufgerufen wird */
extern int key1Active(void); /* 1 wenn gedrückt, sonst 0 */
#define PRELLZYKLEN 20 /* entsprechend 2, wenn alle 10ms */
static int key1busy;

    if(key1Active())
    {
        if(!key1busy)  /* kurz für key1busy==0 */
        {
            /* neuer Tastendruck, hier Deinen Code einfügen */
            ...
        }    
        key1busy=PRELLZYKLEN;
    }
    else
    {
        if(key1busy>0) {key1busy--;}
    }

/* alternativ mit SysTicker, mit Aufruf öfter als Prellzeit */
extern int key1Active(void); 
extern unsigned int SysTicker; /* erhöht sich um 1 jede MS */

#define PRELLZEIT 20 
static int key1busy;
static unsigned int keyTime; 

    if(key1Active())
    {
        if(!key1busy) 
        {
            /* neuer Tastendruck, hier Deinen Code einfügen */
            ...
        }
        key1busy=1;
        keyTime=SysTicker;
    }
    else
    {
        if(SysTicker-keyTime > PRELLZEIT) {key1busy=0;}
    }

Der Code ist nicht schön, nicht getestet, sollte aber prinzipiell und 
ohne Verzögerung funktionieren. Er entprellt, filtert aber keine 
Störungen (Spikes) heraus. Zudem braucht er für jede Taste entsprechende 
Variablen. Das spielt am Anfang aber alles keine Rolle. Wenn PeDas Code 
0,01% an Platz und Zeit braucht, braucht dieser <0,1%.

[kopr (Gast)]

A. S. schrieb:
> dann geht es relativ einfach gebastelt:

danke ! ich habe zu wenig Ahnung um das zu beurteilen: ist das eine 
angemessene Lösung?

[kopr (Gast)]

A. S. schrieb:
> einen SysTicker oder eine zyklische Funktion hast

in meinem Fall würde ich diese Funktion durch einen Timer immer wieder 
aufrufen lassen. Ich weiß nämlich nicht, wie die Programmlaufzeiten bei 
mir sind

[kopr (Gast)]

extern int key1Active(void);

wie kann ich das auf eins setzten?
ich dachte erst an soetwas:

extern int key1Active(void)
{
 if (P2IFG & T_Setzen)
   key1Active=1;
  else
  {_NOP();}
  
}

bekomme ich fehlermeldungen

[kopr (Gast)]

extern int key1Active(void)
{
 if (P2IFG & T_SET)
   return 1;
  else
  {_NOP();}
  
}

[A. S. (Gast)]

kopr schrieb:
>

> extern int key1Active(void)
> {
>  if (P2IFG & T_SET)
>    return 1;
>   else
>   {_NOP();}
> 
> }
>

else return 0;

Mein Beispiel ist ok, aber wenn Du nicht gerade von Pascal kommst, frag 
ruhig öfter nach.

[kopr (Gast)]

das ist mein Code. es funktioniert nicht. Bevor ich return 0 eingefügt 
habe, hat er das Drücken des Tasters erkannt

#include "msp430f5529.h"            // Einbinden der Definitionen
#define  LED_X     0x01     // LED (rot) an P1.0
#define PRELLZYKLEN 20 /* entsprechend 2, wenn alle 10ms */
#define  T_SET     0x02     // Taster an P2.1

 extern void INIT_UART1(void);
 extern void TX_String(unsigned char *cx);
static int key1busy;
extern int key1Active(void); /* 1 wenn gedrückt, sonst 0 */
int main( void )
{
    WDTCTL = WDTPW + WDTHOLD;   // WatchDogTimer abschalten
   INIT_UART1 ();
 // -- INIT_Port1
  P1SEL &= ~(LED_X);            // Beide sind BIN-IO       
  P1DIR |= LED_X;               // LED    = BIN-OUT
  P1OUT &= ~(LED_X);            // LED_X = AUS (weil H-aktiv)
      // INIT_Port_2
    P2SEL  &= ~(T_SET);           // Funktion ist BIN-IO
    P2DIR  &= ~(T_SET);           // Taster ist BIN-IN
    P2IES  |= T_SET;              // Int. bei neg. Flanke (H-->L)
    P2IE   |= (T_SET);           // Beim System.Start noch nicht frei gegeben
    P2IFG  &= ~(T_SET);           // Alte Fl. Ereignis löschen
    //
    P2REN |= T_SET;               // Intern.Widerstand = aktiv
    P2OUT |= T_SET;               // Als PullUp eingestellt
    
  TB0CTL = TBCLR;
  TB0CTL = TBSSEL_2 + ID_0 + MC_1;
  TB0CCR0 = 10480;    // 10 ms
  TB0CCTL0 = CCIE;
 
_BIS_SR(GIE); 

  while(1) {_NOP();} 
}


#pragma vector = TIMER0_B0_VECTOR
__interrupt void TimerB0_0_ISR ()
{
  /* Block, der zyklisch, z.B. jede ms aufgerufen wird */


    if(key1Active())
    {
        if(!key1busy)  /* kurz für key1busy==0 */
        {
            /* neuer Tastendruck, hier Deinen Code einfügen */
             TX_String("\nLED\n");
             P1OUT ^= (LED_X);
        }    
        key1busy=PRELLZYKLEN;
    }
    else
    {
        if(key1busy>0) {key1busy--;}
    }
}

extern int key1Active(void)
{
 if (P2IFG & T_SET)
   return 1;
  else
  {
   return 0;
  }
}

[kopr (Gast)]

meine Frage ist auch, ob es in ordnung ist, ein Drücken durch die 
Int.-Flag abzufragen. Alternativ stünde ja die Flankenerkennung zur 
Verfügung

[Joachim B. (jar)]

kopr schrieb:
> b es in ordnung ist, ein Drücken durch die
> Int.-Flag abzufragen

nicht wenn Taster prellen und das tun die Meisten, die arme CPU kommt ja 
aus den IRQ kaum noch raus wenn diese andauernd durch prellende Taster 
im µs bis ms Takt eintrudeln! Wann soll dann die CPU ihren echten 
Aufgaben nachgehen wenn immer Fehlalarme kommen?

Lese endlich mal Tastenentprellung im Timer IRQ von Dannegger!

[kopr (Gast)]

Joachim B. schrieb:
> nicht wenn Taster prellen und das tun die Meisten, die arme CPU kommt ja
> aus den IRQ kaum noch raus wenn diese andauernd durch prellende Taster
> im µs bis ms Takt eintrudeln! Wann soll dann die CPU ihren echten
> Aufgaben nachgehen wenn immer Fehlalarme kommen?

wie hat sich A. S. (achs) das dann vorgtestellt?

Joachim B. schrieb:
> Lese endlich mal Tastenentprellung im Timer IRQ von Dannegger!

habe ich. ich würde debounce-makro auf meine Plattform anpassen. Kannst 
du mir dabei helfen?

kopr schrieb:
> Matthias S. schrieb:
>> Eins ist klar, die einzige Anpassung betrifft die I/O des verwendeten
>> Controllers.#include <avr/io.h>
> #define F_CPU 9.6e6
> #include <util/delay.h>
> was ist mit den includes. ist in den Definitionen etwas drin, was ich
> brauche? Habe beim MSP entsprechend andere Defintionen. Was gibts da zu
> beachten?

das war meine erste Frage

[A. S. (Gast)]

Du darfst den Code nicht im Interrupt laufen lassen!

Der Code gehört in die while-schleife im main!

Im Interrupt (z.b. jede ms) nur den systicker++.

(Und ich vergaß: systicker mit volatile)

[A. S. (Gast)]

Also:

kopr schrieb:
> while(1) {_NOP();}
> }

stattdessen, wenn in einer Datei:

static int key1Active(void); 
static volatile unsigned int SysTicker; 
#define PRELLZEIT 20 /* bzw (20/x), wenn die Interruptroutine alle x ms aufgerufen wird. */
...

int main(void)
{
    ...
    //while(1) {_NOP();}
    while(1) 
    {
    /* alternativ mit SysTicker, mit Aufruf öfter als Prellzeit */
    static int key1busy;
    static unsigned int keyTime; 

        if(key1Active())
        {
            if(!key1busy) 
            {
                /* neuer Tastendruck, hier Deinen Code einfügen */
                TX_String("\nLED\n");
                P1OUT ^= (LED_X);
            }
            key1busy=1;
            keyTime=SysTicker;
        }
        else
        {
            if(SysTicker-keyTime > PRELLZEIT) {key1busy=0;}
        } 
        ...  
        /* weiterer Code, der zyklisch notwendig ist ... */ 
    }
}

#pragma vector = TIMER0_B0_VECTOR
__interrupt void TimerB0_0_ISR ()
{
    SysTicker++;
}

static int key1Active(void)
{
    if (P2IFG & T_SET) {return 1;}
    return 0;
}

Und gerade wenn Du blutiger Anfänger bist, und verschiedene Fragen hast, 
dann poste doch bitte unter einem Namen. Hätten wir gewusst, dass die 
TX_String geschichte von Dir ist, wäre niemand auf die Idee gekommen, 
dass Du PeDas Code verstehen kannst.

Anmerkung: key1Active kannst Du auch ganz weglassen und direkt 
einsetzen:

//  if(key1Active())
    if(P2IFG & T_SET)

[kopr (Gast)]

danke dass du dir die Mühe gemacht hast. die Routine funktioniert bei 
leider nicht. Weder toggelt die LED, noch werden die Strings ausgegeben.

A. S. schrieb:
> static int key1Active(void)
> {
>     if (P2IFG & T_SET) {return 1;}
>     return 0;
> }

hab dort auch vorsichtshalber ein

 P2IFG &= ~(T_SET);                          // Flanken Ereignis löschen

eingefügt

[Stefan F. (Gast)]

Wieso ist eigentlich Taster-Entprellung plötzlich das Thema Nummer eins 
hier? Für mich steht der Troll-Alarm bereits auf 80%.

[A. S. (Gast)]

kopr schrieb:
> danke dass du dir die Mühe gemacht hast. die Routine funktioniert bei
> leider nicht. Weder toggelt die LED, noch werden die Strings ausgegeben.


Funktioniert der Code denn vorher? Oder wenn Du die Funktionen so 
aufrufst?

Was funktioniert denn, und was nicht? Wenn Du nur die LED hast, dann 
mache z.B. folgendes:

Statt Tastenauswertung ein sinnvolles Bit vom SysTicker (mit welcher 
Rate läuft er bei Dir denn jetzt, weisst Du das schon?)

static int key1Active(void)
{
    if(SysTicker & 1024) {return 1;}
    return 0;
}

Wenn dann die LED sekündlich toggelt, dann laufen zumindest SysTicker 
und die Auswertung. Wenn die LED nur alle 10s toggelt, dann passe das 
entsprechend an

Und gebe den String einmal vor "while(1)" aus, dann weisst Du, ob Du da 
versehentlich irgendwas zerschossen hast.

Und kennst Du die Funktion TX_String? Wenn die blockierend läuft, dann 
hätte die nie in einer Interruptroutine sein dürfen. Daher nochmal:

A. S. schrieb:
> Schreib mal, was Du hast und kannst.


kopr schrieb:
> hab dort auch vorsichtshalber ein
>  P2IFG &= ~(T_SET);                          // Flanken Ereignis löschen
> eingefügt

und warum hast Du das vorher nicht gebraucht? Dieser Teil kommt von Dir, 
nicht von mir. Poste mal den ganzen Code. Gerne als Anhang.

[kopr (Gast)]

#include "msp430f5529.h"            // Einbinden der Definitionen
#define  LED_X     0x01     // LED (rot) an P1.0
#define PRELLZYKLEN 20 /* entsprechend 2, wenn alle 10ms */
#define  T_SET     0x02     // Taster an P2.1

 extern void INIT_UART1(void);
 extern void TX_String(unsigned char *cx);
static int key1busy;
extern int key1Active(void); /* 1 wenn gedrückt, sonst 0 */
int main( void )
{
    WDTCTL = WDTPW + WDTHOLD;   // WatchDogTimer abschalten
   INIT_UART1 ();
 // -- INIT_Port1
  P1SEL &= ~(LED_X);            // Beide sind BIN-IO       
  P1DIR |= LED_X;               // LED    = BIN-OUT
  P1OUT &= ~(LED_X);            // LED_X = AUS (weil H-aktiv)
      // INIT_Port_2
    P2SEL  &= ~(T_SET);           // Funktion ist BIN-IO
    P2DIR  &= ~(T_SET);           // Taster ist BIN-IN
    P2IES  |= T_SET;              // Int. bei neg. Flanke (H-->L)
    P2IE   |= (T_SET);           // Beim System.Start noch nicht frei gegeben
    P2IFG  &= ~(T_SET);           // Alte Fl. Ereignis löschen
    //
    P2REN |= T_SET;               // Intern.Widerstand = aktiv
    P2OUT |= T_SET;               // Als PullUp eingestellt
    
  TB0CTL = TBCLR;
  TB0CTL = TBSSEL_2 + ID_0 + MC_1;
  TB0CCR0 = 10480;    // 10 ms
  TB0CCTL0 = CCIE;
 
_BIS_SR(GIE); 

  while(1) {_NOP();} 
}


#pragma vector = TIMER0_B0_VECTOR
__interrupt void TimerB0_0_ISR ()
{
  /* Block, der zyklisch, z.B. jede ms aufgerufen wird */


    if(key1Active())
    {
        if(!key1busy)  /* kurz für key1busy==0 */
        {
            /* neuer Tastendruck, hier Deinen Code einfügen */
             TX_String("\nLED\n");
             P1OUT ^= (LED_X);
        }    
        key1busy=PRELLZYKLEN;
    }
    else
    {
        if(key1busy>0) {key1busy--;}
    }
}

extern int key1Active(void)
{
 if (P2IFG & T_SET)
   return 1;
  else
  {
   _NOP();
  }
}

der code hat das Drücken des tasters erkannt

[kopr (Gast)]

A. S. schrieb:
> Und gebe den String einmal vor "while(1)" aus, dann weisst Du, ob Du da
> versehentlich irgendwas zerschossen hast.

habe ich, das passt

A. S. schrieb:
> Statt Tastenauswertung ein sinnvolles Bit vom SysTicker (mit welcher
> Rate läuft er bei Dir denn jetzt, weisst Du das schon?)

ist damit die 25 MHz der CPU gemeint?

A. S. schrieb:
> Wenn dann die LED sekündlich toggelt, dann laufen zumindest SysTicker
> und die Auswertung. Wenn die LED nur alle 10s toggelt, dann passe das
> entsprechend an

toggelt garnicht

A. S. schrieb:
> Und kennst Du die Funktion TX_String? Wenn die blockierend läuft, dann
> hätte die nie in einer Interruptroutine sein dürfen. Daher nochmal:

hatte in anderen Programmen kein Problem mit String in ISR

[kopr (Gast)]

Stefan ⛄ F. schrieb:
> Wieso ist eigentlich Taster-Entprellung plötzlich das Thema Nummer eins
> hier? Für mich steht der Troll-Alarm bereits auf 80%.

meinst du deswegen:
Beitrag "Debounce-Makro von Peter Dannegger - MSP430"

ist kein Troll, schade das du mir da nicht hilfst. Hätte Hilfe bitter 
nötig

[A. S. (Gast)]

kopr schrieb:
> der code hat das Drücken des tasters erkannt

Und wie sieht Dein Code aus, der nicht funktioniert?

Übrigens, das _NOP hat hier keine Funktion. Du kannst den else Zweig 
weglassen. Dann siehst Du auch, dass die Funktion ohne Tastendruck 
nichts zurückliefert (kein return hat), was

a) immer eine Warnung geben sollte (hast Du noch mehr Warnungen?)
b) beliebigen Müll zurückgeben kann.

kopr schrieb:
> extern int key1Active(void)
> {
>  if (P2IFG & T_SET)
>    return 1;
>   else
>   {
>    _NOP();
>   }
> }

[Stefan F. (Gast)]

kopr schrieb:
> schade das du mir da nicht hilfst

Ich halte mich da lieber raus, weil ich Peters Code noch nie brauchte. 
Noch programmiere solche Sachen selber.

[kopr (Gast)]

A. S. schrieb:
> a) immer eine Warnung geben sollte (hast Du noch mehr Warnungen?)

ne habe keine Warnungen, mit return 0 hat der den Taster nicht mehr 
erkannt. Echt komisch.

Im Anhang der Code, der nicht funktioniert

[kopr (Gast)]

Stefan ⛄ F. schrieb:
> ch halte mich da lieber raus, weil ich Peters Code noch nie brauchte.
> Noch programmiere solche Sachen selber.

naja, es geht um die Frage, wie ich den Code von der einen Plattform auf 
die andere Übertrage. Mir macht das Schwierigkeiten

[Stefan F. (Gast)]

kopr schrieb:
> ne habe keine Warnungen

Dann aktiviere die mal. Du wirst sie brauchen.

[A. S. (Gast)]

kopr schrieb:
> ne habe keine Warnungen, mit return 0 hat der den Taster nicht mehr
> erkannt. Echt komisch.
>
> Im Anhang der Code, der nicht funktioniert

Nein, nicht komisch. Da sind mehrere Stellen, die einfach Müll sind. 
Sowohl in Deinem ersten, als auch in diesem Code. Von daher ist es sehr 
wahrscheinlich, dass er auch nicht tut, was er soll. Schalte Warnungen 
ein.

kopr schrieb:
> extern int key1Active(void)
> {
>  if (P2IFG & T_SET)
>    return 1;
>   else
>   {
>    _NOP();
>   }
> }

Hier passiert müll. Vermutlich liefert der Tastendruck nicht "T_SET" 
gesetzt, weil er z.B. an Pin3 ist und nicht Pin1. Dein Code funktioniert 
nämlich dann trotzdem. Ein return 0 repariert diesen Fehler. Aber wenn 
der Fehler das Feature war...

> if (P2IFG & T_SET) {return 1;  P2IFG &= ~(T_SET); }  // Flanken Ereignis löschen
Verstehst Du, was da passieren soll? Versuche das selbst zu ergründen 
und schalte die Warnungen ein, bis Du das verstehst. Das sind wirkliche 
Grundlagen, da ist Tastenentprellung Hightech gegen.

Und Frage Dich, was an der Routine anders ist, dass Du es im 
funktionierenden Code nicht gebraucht hast.

Bringe die Tastenauswertung zum laufen, dann wird der Rest 
funktionieren. Notfalls schalte die LED ein, wenn die Taste gedrückt 
ist, und aus, wenn sie es nicht ist

int key1Active(void)
{
    if (P2IFG & T_SET)
    {
        P1OUT |= (LED_X);
        P2IFG &= ~(T_SET); /* falls notwendig */ 
        return 1;
    }
    else
    {
        P1OUT &= ~(LED_X);
    }
    return 0;
}

[kopr (Gast)]

ja ich habe eine warnung ohne return haha. Aber in dem Code (Anhang) 
sind keine Warnungen

[kopr (Gast)]

okay ich überarbeite alles

[kopr (Gast)]

#include "msp430f5529.h"            // Einbinden der Definitionen
#define  LED_X     0x01     // LED (rot) an P1.0
#define PRELLZYKLEN 20 /* entsprechend 2, wenn alle 10ms */
#define  T_SET     0x02     // Taster an P2.1

 extern void INIT_UART1(void);
 extern void TX_String(unsigned char *cx);
static int key1busy;
extern int key1Active(void); /* 1 wenn gedrückt, sonst 0 */
int main( void )
{
    WDTCTL = WDTPW + WDTHOLD;   // WatchDogTimer abschalten
   INIT_UART1 ();
 // -- INIT_Port1
  P1SEL &= ~(LED_X);            // Beide sind BIN-IO       
  P1DIR |= LED_X;               // LED    = BIN-OUT
  P1OUT &= ~(LED_X);            // LED_X = AUS (weil H-aktiv)
      // INIT_Port_2
    P2SEL  &= ~(T_SET);           // Funktion ist BIN-IO
    P2DIR  &= ~(T_SET);           // Taster ist BIN-IN
    P2IES  |= T_SET;              // Int. bei neg. Flanke (H-->L)
    P2IE   |= (T_SET);           // Beim System.Start noch nicht frei gegeben
    P2IFG  &= ~(T_SET);           // Alte Fl. Ereignis löschen
    //
    P2REN |= T_SET;               // Intern.Widerstand = aktiv
    P2OUT |= T_SET;               // Als PullUp eingestellt
    
  TB0CTL = TBCLR;
  TB0CTL = TBSSEL_2 + ID_0 + MC_1;
  TB0CCR0 = 10480;    // 10 ms
  TB0CCTL0 = CCIE;
 
_BIS_SR(GIE); 

  while(1) {_NOP();} 
}


#pragma vector = TIMER0_B0_VECTOR
__interrupt void TimerB0_0_ISR ()
{
  /* Block, der zyklisch, z.B. jede ms aufgerufen wird */


    if(key1Active())
    {
        if(!key1busy)  /* kurz für key1busy==0 */
        {
            /* neuer Tastendruck, hier Deinen Code einfügen */
             TX_String("\nLED\n");
             P1OUT ^= (LED_X);
        }    
        key1busy=PRELLZYKLEN;
    }
    else
    {
        if(key1busy>0) {key1busy--;}
    }
}

extern int key1Active(void)
{
 if ((P2IN & 0x02) == 1)
   return 1;
  else
  {
   return 0;
  }
}

die Flanke wurde in anderen programmen immer gefunden, also kann es 
nicht daran liegen. Es funktioniert immer noch nicht

[A. S. (Gast)]

kopr schrieb:
> if ((P2IN & 0x02) == 1)

Die Zeile ist auch falsch.
Entweder
if ((P2IN & 0x02) == 0x02)

Oder

if ((P2IN & 0x02))

[kopr (Gast)]

A. S. schrieb:
>> if ((P2IN & 0x02) == 1)

oder

A. S. schrieb:
> if ((P2IN & 0x02))

wo ist der Unterschied?

[Joachim B. (jar)]

kopr schrieb:
> habe ich. ich würde debounce-makro auf meine Plattform anpassen. Kannst
> du mir dabei helfen?

fange erst mal an das eine LED leuchtet
Dann sehe zu das dein IRQ Programm alle 10ms aufgerufen wird!

Baustelle, arbeite gerade daran, aber das lief in anderen Programmen 
schon.

Es fehlt noch Port einlesen, Variablen zur Timerzählung, jede IRQ 
Zählung löst in der main loop eine andere Aktion aus.

Etwas musst du dir auch schon selber erarbeiten, es ist dein Chip und 
Beispiel Script zum Timmer für diesen nannte ich schon.

[kopr (Gast)]

vielen Dank Joachim, ich habe inzwischen eine funktionierende, eigene 
Routine

[kopr (Gast)]

kopr schrieb:
> A. S. schrieb:
>>> if ((P2IN & 0x02) == 1)
>
> oder
>
> A. S. schrieb:
>> if ((P2IN & 0x02))
>
> wo ist der Unterschied

auch das hat sich geklärt

[Joachim B. (jar)]

kopr schrieb:
> vielen Dank Joachim, ich habe inzwischen eine funktionierende, eigene
> Routine

auch OK
jeder wie er mag!