falsche Zuweisung

Warum am PC nichts ankommt, steht auf einem anderen Blatt, aber
daß du aus UDR nicht das auslesen kannst, was du vorher reinschreibst,
ist normal.
UDR ist nämlich ein Register, sondern zwei.
In das eine schreibt man rein (das, was gesendet werden soll), aus
dem anderen liest man empfangene Zeichen aus.
Die beiden heißen nur zufällig gleich...

Hi,

a) wäre es schön, wenn du etwas lesbarer schreiben könntest. Je 
fehlerfreier ein Beitrag geschrieben ist, desto mehr Leute werden diesen 
lesen & ev. beantworten.

b) fehlt noch eine ganze Menge zu deinem Code. Was du beim Posten 
weggelassen hast, kann man hier logischerweise nicht erahnen, und meine 
Glaskugel ist gerade zur Reparatur.

Eine sendString sieht z.B. so aus:

1

void SendString(char *string)

2

{

3

  while(*string)

4

  {

5

    while(UART->SR & TX_BUSY);   // Uart Status Register

6

    UART->DATA = *string++;      // Uart Data Register

7

  }

8

}

Damit wartet man darauf, dass der Sendebuffer bereit ist, und schickt 
jeweils das nächste Zeichen eines Strings.

Genaugenommen schreibt man sich eigentlich eine

1

void sendchar(char c);

die dieses Low-Level-Handling übernimmt, und füttert die mit Daten (z.B. 
aus fputc, einer SendString, o.ä.


VG,
/th.

gast schrieb:
> mein UDR kriegt nicht
Dann wohl UART kaaaputtt...

> und bis jetzt weiss ich nicht warum!!!
auch nix wisse, warum.

> Also bitte ich um Hilfe falls jemand einen Fehler in meinem Kode sieht.
Glaskugel putt.

Poste bitte mal den ganzen Code.

VG,
/th.

gast schrieb:

> Probleme: Es erscheint am Terminal PC gar nichst am Bildschirm.

Womit siehst du dir das Übertragene an?

Mit einem Terminalprogramm im ASCII Modus?
Da wunderst es mich dann wenig, dass du nichts siehst, du überträgst ja 
praktisch nur ASCII Zeichen, die keine visuelle Repräsentierung haben 
(mit Ausnahme von 0x21, 0x32, 0x2A, 0x5C und 0x9D. Wobei der Himmel 
alleine weiß, welches Zeichen dein Terminal für 0x9D anzeigen wird)

PS: Die Berechnungen in SendCommand solltest du nochmal durchschauen. 
Insbesondere dann, wenn iNo gleich 1 ist.

PS2: Wenn in einer for-Schleife die sich mit Arrays beschäftigt im 
Bedingungsteil ein <= vorkommt, kann man in 90% aller Fälle davon 
ausgehen, dass da was faul ist. Du hast dir mit der komplizierten 
Formulierung ganz einfach selber ein Eigentor geschossen. Solange du das 
nicht überblicken kannst, quetche nicht alles in eine Berechnung 
zusammen

1

void SendCommand( unsigned char iNo )

2

{

3

  unsigned char i;

4

5

#define BYTES_IN_COMMAND 19

6

7

  // jeweils 19 Bytes sind 1 Kommando

8

  iNo = BYTES_IN_COMMAND * iNo;

9

10

  for  (i = 0; i < BYTES_IN_COMMAND; i++ )

11

  {

12

    cSend = eeprom_read_byte(&eeMenue[ i + iNo ]);

13

    USART_Putc( cSend );

14

  }

15

}

gast schrieb:

> Am Terminalprogramm kriege ich
> <0><0><0><0><0><0><0><0><0><0><0><0><0><0><0><0><0><0><0>
>
> Das heisst mein Terminalprogramm kann meine Zeichen ganz gut übertragen
> oder!!!

Anzeigen, nicht übertragen.
Und du siehst nur, dass es für 0x00 eine Anzeige hat.
Ob und was es aus den restlichen Bytes macht, welche keine anzeigbare 
Darstellung haben, steht in den Sternen.

Aber da du anscheinend sowieso nur daran interessiert bist, welche Bytes 
aus dem EEPROM gelesen werden, warum machst du dir nicht einfach eine 
SendFunktion die ein Byte in seiner HexForm anzeigen kann. Ist doch 
simpel.

1

void USART_Puts( const char * text )

2

{

3

  while( *text )

4

    USART_Putc( *text++ );

5

}

6

7

void USART_PutNibble( unsigned char nibble )

8

{

9

  if( nibble < 0x0A )

10

    USART_Putc( nibble + '0' );

11

  else

12

    USART_Putc( nibble - 10 + 'A' );

13

}

14

15

void USART_PutHex( unsigned char byte )

16

{

17

  USART_Puts( "0x" );

18

  USART_PutNibble( byte >> 4 );

19

  USART_PutNibble( byte & 0x0F );

20

}

21

22

void SendCommand( unsigned char iNo )

23

{

24

  unsigned char i;

25

  unsigned char cSend;

26

27

#define BYTES_IN_COMMAND 19

28

29

  // jeweils 19 Bytes sind 1 Kommando

30

  iNo = BYTES_IN_COMMAND * iNo;

31

32

  for  (i = 0; i < BYTES_IN_COMMAND; i++ )

33

  {

34

    cSend = eeprom_read_byte(&eeMenue[ i + iNo ]);

35

    USART_PutHex( cSend );

36

  }

37

}

gast schrieb:

>>void SendString(char *string)
>>{
>   >while(*string)
>   >{
>    > while(UART->SR & TX_BUSY);   // Uart Status Register
>    > UART->DATA = *string++;      // Uart Data Register
>   >}
>>}
>
> meinst du, dass ich die Funktion brauche !?!

Eine String-Funktion kann man immer brauchen.
Aber die hier passt nicht zu deinen restlichen USART-Funktionen (*).
Am einfachsten und am wenigsten fehlerträchtig ist es normalerweise, 
wenn man die String-Funktion so aufbaut, dass sie die 
Einzelzeichen-Funktion in einer Schleife mit den einzelnen Zeichen 
füttert. Dann sind die Details, wie ein Zeichen versendet wird, nur in 
einer Funktion beisammen und man bekommt keinen Wickel, wenn 
Einzelzeichen-Senden und String-Senden abwechselnd benutzt wird.

> Ich sende keine String!! sondern nur ASCII Zeichen !!

Ein String ist eine Folge von ASCII Zeichen.

Was ist dir lieber

1

  USART_Putc( 'H' );

2

  USART_Putc( 'a' );

3

  USART_Putc( 'l' );

4

  USART_Putc( 'l' );

5

  USART_Putc( 'o' );

oder

1

  USART_Puts( "Hallo" );

Wenn man eine USART hat, braucht man praktisch immer eine Stringausgabe 
in irgendeiner Form. Und sei es nur, dass man Zahlen ausgeben möchte

1

void USART_putInt( int number )

2

{

3

  char Buffer[10];

4

  itoa( number, Buffer, 10 );

5

  USART_Puts( Buffer );

6

}

Sieh diese Funktionen als Baukasten an, die du bei Bedarf benutzt, und 
wo sich eine Funktion auf die anderen stützt. Ganz unten steht die 
Einzelzeichenausgabe. Alle anderen Ausgaben landen letztendlich dort.

Ausserdem sind die Ausgabefunktionen normalerweise nicht sehr lang, 
sodass man davon ausgehen kann, diesen Flash-Speicher praktisch immer 
zur Verfügung zu haben.


(*) Edit:
Ich habe noch nicht mitbekommen, dass du zwischendurch den Ringbuffer 
über Bord geschmissen hast.

gast schrieb:
>>Anzeigen, nicht übertragen.
>>Und du siehst nur, dass es für 0x00 eine Anzeige hat.
>>Ob und was es aus den restlichen Bytes macht, welche keine anzeigbare
>>Darstellung haben, steht in den Sternen.
>
> Danke für die Antwort
>
> eingentlcih möchte ich Am Terminalprogramm das Angezeigt kriegen
> <2><33><1><0><0><0><0><0><0><0><0><0><0><0><0><0><3><157><42>

Na dann mach das doch!

Alledings musst du dir im klaren sein, dass DU dafür verantworltich 
bist, dass die Zahlen richtig kommen, dass die '<' und '>' richtig 
ausgegeben werden.

PS: Ob das so sinnvoll ist, die Anzeige mit Dezimalzahlen zu machen, 
wenn du in deinem C-Programm dieselben Zahlen in Hex-Form eingibst, 
solltest du nochmal überprüfen.

Mann, einige dich jetzt mit dir selber bitte mal, welche Form der 
UART-Routinen du benutzen willst.
Mit Ringbuffer oder ohne.