Ausgabe an UART bringt Müll

1

void decode ( uint8_t sc )

2

{

3

  if ( sc != 0xF0 )

4

  {

5

    uart_putc(sc);

6

//    __release=0;      

7

  } else {

8

//    __release=1;

9

  }

10

  return;

11

}

Seit wann hat eine void Funktion ein return?

Peter II schrieb:
> Oliver J. schrieb:
>> Seit wann hat eine void Funktion ein return?
>
> wie willst du sonst eine funktion mal in der mitte abbrechen?

Du hast recht.

Hallo,

das "return" kann durchaus entfernt werden, es ändert aber das 
beschriebene Problem nicht. Sobald ich versuche einen Zwischenstand 
mittels globaler Variable oder localer Variable "static uint8_t release" 
zu speichern, kommt auf dem Terminal nur noch Quatsch an.

Hat noch jemand dazu einen Tipp?

HUE \a

Ich hatte sowas von unrecht.
Hier ist eine Referenz zum return-statement.

http://home.htw-berlin.de/~junghans/cref/SYNTAX/return.html

Justus schrieb:
> sorry, der Link funktioniert nicht.

Bei mir schon.

Hallo,

die Anweisungen "volatile" und das Rücksetzen von "data" haben nicht den 
gewünschten Effekt gebracht. Es kommt immer noch Müll auf dem Terminal 
an.

HUE \a

Also nach weiteren Tests hat es nur mit dem Setzen der Variable 
"_release" zu tun.
Fraglich warum nur das Setzen schon den Müll auf dem Display verursacht. 
Ich habe doch noch nichts verbrochen, was die Ausgabe an das Terminal 
angeht, oder?

HUE \a

Hier ist die Routine "decode"

1

void decode ( uint8_t sc )

2

{

3

    if ( sc != 0xF0 )

4

    {

5

         uart_putc(sc);

6

//      __release=0;            

7

    } else {

8

//      __release=1;

9

    }

10

}

So gehts!

1

void decode ( uint8_t sc )

2

{

3

    if ( sc != 0xF0 )

4

    {   

5

        uart_putc(sc);

6

      __release=0;            

7

    } else {

8

      __release=1;

9

    }   

10

}

So nicht!

HUE \a

Es könnte auch schlicht ein Timing-Problem sein. Der PS/2-Clock ist mit 
10-16.7 kHz ja auch nicht gerade langsam. Es könnte sein, dass es ohne 
die Zuweisungen an __release noch so gerade eben passt, mit ihnen aber 
nicht mehr.

PS: Was soll der Unsinn mit den beiden führenden Unterstrichen? Solche 
Namen sind für den Compiler reserviert.

Wo ist denn in der Funktion "decode" das RETURN"?

Hier mein funktionierender Code mit der lokalen Definition von 
"release".

1

void decode ( uint8_t sc )

2

{

3

    uint8_t release;

4

5

    if ( sc != 0xF0 )

6

    {

7

        uart_putc(sc);

8

        release=0;

9

    } else {

10

        release=1;

11

    }

12

}

Es kann ja nicht ein Timingproblem sein, wenn das schreiben der Variable 
"release" im lokalen Modus funktioniert, und als "static" Variable oder 
als "globale" Variable nicht.

Der Code ist an die AppNote von Atmel zur Abfrage der Tastatur 
angelehnt.

HUE \a

Hue \a schrieb:
> Wo ist denn in der Funktion "decode" das RETURN"?

Ignoriere den Troll doch einfach.

Hue \a schrieb:
> Es kann ja nicht ein Timingproblem sein, wenn das schreiben der Variable
> "release" im lokalen Modus funktioniert, und als "static" Variable oder
> als "globale" Variable nicht.

Doch, kann es, denn als lokale Variable wird sie (und die Zuweisungen an 
sie) komplett wegoptimiert. Packe doch mal ein paar NOPs in die 
funktionierende Version, und schaue was passiert.

Hallo,

vielen Dank für Eure konstruktiven Hinweise. Ich habe mich nun der 
Meinung angeschlossen, dass ich ein Timing Problem habe. Ich werde mich 
nun bemühen, dies zu beheben.

Ein schönes Wochenende...

HUE \a