Controller startet bei atol neu?

[HaWe]

also passiert der Fehler gar nicht in atol? Sondern in uart_getc() oder uart_gets() ? Oder wie oder was?

[Tux12Fun]

Genau das versuche ich gerade heraus zu finden.
Leider ist das ganze UART Thema für mich auch Neuland.

- - - Aktualisiert - - -

Ich hab schön langsam eine Ahnung, ich glaub ich hab mehr RAM verbraucht als der ATMEL hat und
sobald ich dann irgendwo im RAM schreibe wo ich nicht sein sollte macht es boom

Wenn ich nämlich wie jetzt hier den buffer auf 512 setze geht es noch bei 1024 gibt es undefiniertes verhalten.

Und das Datenblatt bestätigt das wohl gerade
– 1KByte Internal SRAM daran dachte ich wohl nicht.

Klingt das für euch Logisch?

#define MCU  atmega8
#define F_CPU 2000000UL
//-U lfuse:w:0xe2:m -U hfuse:w:0xd9:m
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <inttypes.h>
#include <stdlib.h> 
#include <stdio.h>

#define BT_EN_REGISTER DDRC
#define BT_EN_PORT     PORTC
#define BT_EN_PIN      PC0

#define LED_REGISTER   DDRC
#define LED_PORT       PORTC
#define LED_RED_PIN    PC5
#define LED_GREEN_PIN  PC4
#define LED_BLUE_PIN   PC3

//                                BAUDRATE
#define BAUD_PRESCALE (((F_CPU / (9600 * 16UL))) - 1)	/* Define prescale value */

//Data from App
long pl_valid_time_sec = 0;

void USART_SendString(char *str){
   int i=0;
   while (str[i]!=0){
      UDR = str[i];                    /* Write data to be transmitting in UDR */
      while (!(UCSRA & (1<<UDRE)));    /* Wait until data transmit and buffer get empty */
      i++;
   }
}

void get_BTData(){
   // INIT USART
   UCSRB |= (1 << RXEN) | (1 << TXEN);				/* Enable USART transmitter and receiver */
   UCSRC |= (1 << URSEL)| (1 << UCSZ0) | (1 << UCSZ1);	/* Write USCRC for 8 bit data and 1 stop bit */ 
   UBRRL = BAUD_PRESCALE;							/* Load UBRRL with lower 8 bit of prescale value */
   UBRRH = (BAUD_PRESCALE >> 8);					/* Load UBRRH with upper 8 bit of prescale value */
   
   LED_PORT |= (1<<LED_BLUE_PIN);

   int i=0;
   uint8_t c = '\0';
   uint8_t uart_buffer[1024];
   
   while( i < 512){
      while (!(UCSRA & (1<<RXC))){}
      c = UDR;
      if ( c == '\n' ){ break; }
      uart_buffer[i] = c;
      i++;
   }
   uart_buffer[i]='\0';
   
   USART_SendString("TEST");
   USART_SendString(uart_buffer);
    
   
   char spl_buffer[12];
   spl_buffer[0] = '1';    spl_buffer[1] = '2';      spl_buffer[1] = '\0';
   pl_valid_time_sec = atol(spl_buffer);
   
   _delay_ms(5000);
   LED_PORT &= ~(1<<LED_BLUE_PIN);
}

int main(void){
   // Direction Registers
   DDRB = 0x00;            // B as input
   DDRC = 0x00;            // D as Input
   DDRD = 0x00;            // D as Input
   
   // Set Output Registers
   BT_EN_REGISTER |= (1 << BT_EN_PIN);        // Bluetooth PWR Controll
   LED_REGISTER   |= (1 << LED_RED_PIN);      // LED as output
	LED_REGISTER   |= (1 << LED_GREEN_PIN);    // LED as output
	LED_REGISTER   |= (1 << LED_BLUE_PIN);     // LED as output

   // Ports to LOW (GND no Pullup)
   PORTB = 0x00;
   PORTC = 0x00;
   PORTD = 0x00;
   
   for (int i=0; i < 4; i++){
      LED_PORT ^= (1 << LED_GREEN_PIN);
      _delay_ms(500);
   }
   
   get_BTData();  // INIT MODE
   
   while(1){
       LED_PORT ^= (1 << LED_RED_PIN);
       _delay_ms(500);
   }

}

[Moppi]

Morgen Tux,

Dein Buffer (uint8_t uart_buffer[1024] befindet sich innerhalb "void get_BTData()".
Das Problem gab es hier schon in verschiedenen Fällen, dass so etwas dann Probleme bereitet (Stapelspeicher).
Eventuell bringts Dir was, "uint8_t uart_buffer[1024];" global anzulegen, also außerhalb jeder Funktion, das könnte das Problem beseitigen.


MfG

[shedepe]

Mal noch so als idee: Du hast da einen 1kB großen Buffer -> Das ist ganz schön viel für einen kleinen Atmega. Überleg dir doch mal ob du die Daten nicht. Stückchenweise oder sogar Interrupt basiert verarbeiten kannst.

[Siro]

Hallo Tux12Fun,

der ATMega8 hat nur 1 KByte RAM
Diesen Speicher teilen sich Stack und Daten.
Je mehr Speicher für den Stack angelegt wird, desto weniger bleibt also für den RAM übrig.

Es ist unmöglich einen Puffer mit 1024 Bytes anzulegen
und schon garnicht gleich zweie davon:

char uart_buffer[UART_IN_BUFFER_LEN];
char spl_buffer[UART_IN_BUFFER_LEN];

Normalerweise merkt das ein Compiler und gibt eine entsprechende Fehler oder Warnmeldung raus.
Da diese Arrays aber bei deinem Code lokal innerhalb der Funktion get_BTData(char mode) definiert sind,
ist es vermutlich für den Compiler nicht eindeutig, weil der Speicher ja nicht ständig,
sondern nur innerhalb der Funktion reserviert und auch wieder verworfen wird.

Ich würde testweise mal vorschlagen die Arrays ausserhalb der Funktion zu platzieren
und neu zu kompilieren. Meiner Meinung nach müsste es dann eine entsprechende Fehlermeldung geben.

Du brauchts auch eher unwahrschienlich diese riesiegen Puffer von 512 oder sogar 1024 Bytes.
Ich komme meistens mit 16 bis 32 Bytes für meine serielle Schnittstelle aus.

Das Verhalten der Software bei solch einem Problem ist völlig zufällig und da kann man sich wirklich nen Wolf suchen.
Wie HaWe auch gleich schrieb: Code auf das minimalste kürzen.
Deine Idee mit den verschiedenen Optimierungen des Compilers ist übrigens eine sehr gute Testmethode,
das mache ich auch oft bei unerklärlichem Verhalten. Der Code sollte aber immer mit allen Optimierungen laufen.
Zumindest ist das generell mein oberstes Gebot.

Siro

[Tux12Fun]

Vielen Dank,

genau das war auch das Problem, jetzt habe ich meine buffer schön kompakt auf 100 Byte begrenzt und bin total happy.

Danke für die ganze Unterstützung bei der Fehlersuche.

Nun habe ich noch ein kleines Timer Problem. Dazu mache ich aber ein neues Thema auf.