- LiTime Speicher und Akkus         
Seite 1 von 2 12 LetzteLetzte
Ergebnis 1 bis 10 von 13

Thema: Controller startet bei atol neu?

  1. #1
    Neuer Benutzer Öfters hier
    Registriert seit
    22.12.2019
    Beiträge
    19

    Unglücklich Controller startet bei atol neu?

    Anzeige

    Powerstation Test
    Hallo,

    vielleicht kann mir jemand helfen. Ich bin echt am verzweifeln.
    Ich habe ein bisschen C Programmiert und bin auf ein Problem gestoßen.
    Ich weiß echt nicht mehr weiter und finde einfach den Fehler nicht.

    Sobald ich in der Funktion get_BTData(char mode) eine atoi/atol oder scanf Funktion einbaue,
    löst der ATMEGA8 bei USART_Init(9600); einen reboot aus.

    Das sehe ich daran, dass mein Programm folgendes verhalten zeigt.

    OHNE ATOL
    ==========
    LED blinkt grün 2x
    LED blinkt blau 2x
    LED geht aus und leuchtet dann durch gehend blau.

    -- Der ATMEGA wartet auf Kommandos vom BT Module


    Jetzt ändere ich nur eine Zeile (143 in der main.c) pl_valid_time_sec = atol(spl_buffer);
    ===========================
    LED blinkt grün 2x
    LED blinkt blau 2x
    LED blinkt grün 2x
    LED blinkt blau 2x
    LED blinkt grün 2x
    LED blinkt blau 2x
    ....


    Ich versteh die Welt nicht mehr, baue ich die atol Zeile wieder aus, geht es wieder.
    Ich habe auch schon an eine Compiler Optimierung oder einen Speicher Überlauf gedacht.
    Aber müsste da nicht GCC eine Warnung ausgeben.

    Es gab auch keine Änderung wenn ich zwischen -os oder -o2 wechsel. ( avr-gcc (Gentoo 8.3.0-r1 p1.1) 8.3.0)

    Vielleicht habt ihr noch eine Idee der ATMEGA8A läuft bei 2Mhz das BT Module ist ein HC-05

    main.c
    Code:
    #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>
    
    #include <avr/eeprom.h>
    #define EEPROM_DEF 0xFF
    
    #include "USART_RS232_H_file.h"	/* include USART library */
    
    #define UART_IN_BUFFER_LEN 512
    
    #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
    
    uint8_t gEi_initCode EEMEM = 123;
    
    long      gl_remaining_time = 15;
    char      gc_intrusion_detected = 0;
    char      gi_wakeup_pressed =0;
    char*     gs_verificationCode = "ddd";
    
    //Data from App
    char pc_code_sement[UART_IN_BUFFER_LEN];
    long pl_valid_time_sec = 0;
    long pl_lock_time_sec  = 0;
    int  pi_multiplikator  = 0;
    
    
    char bitRead(char *x, char n) {
       return (*x & (1 << n)) ? 1 : 0;
    }
    
    
    void my_sleep(uint16_t sec){
       for (unsigned short int i =0; i < sec; i++){
          _delay_ms(1000);
       }
    }
    
    uint8_t uart_getc(uint8_t timeout){
       uint32_t i_timeout = timeout;
       uint32_t runntime = 0;
        while (!(UCSRA & (1<<RXC))){   // warten bis Zeichen verfuegbar
           runntime++;
           if (runntime >= F_CPU){
              runntime=0;
              i_timeout--;
              if (i_timeout == 0){
                 return '\0';
              }
           }
        }  
        return UDR;                   // Zeichen aus UDR an Aufrufer zurueckgeben
    }
    
    void uart_gets( char* Buffer, uint32_t MaxLen, uint32_t timeout ){
      uint8_t NextChar;
      uint32_t StringLen = 0;
      NextChar = uart_getc(timeout);         // Warte auf und empfange das nächste Zeichen
      while( NextChar != '\n' && StringLen < MaxLen - 1 ) { // Sammle solange Zeichen, bis: entweder das String Ende Zeichen kam oder das aufnehmende Array voll ist
        *Buffer++ = NextChar;
        StringLen++;
        NextChar = uart_getc(timeout);
    
      }
      *Buffer = '\0';             // Noch ein '\0' anhängen um einen Standard C-String daraus zu machen
    }
    
    
    long getCurrentVoltage(void){
      long adc_result = 0;  //Gesamtergebnis der Messung des ADC
      ADMUX |= (1<<REFS0) | (1<<REFS1) ; //VCC als Referenzspannung für den AD-Wandler
      ADMUX |= (1<<MUX3) | (1<<MUX2) | (1<<MUX1);  //1.3V Referenzspannung als Eingang für ADC 
      _delay_ms(2000);  //warten bis sich die Referenzspannung eingestellt hat
      ADCSRA |= (1<<ADEN);   //ADC aktivieren
      for (int i=0; i < 11; i++){
          ADCSRA |= (1<<ADSC);  //Messung starten
          while(!(ADCSRA&(1<<ADIF))){
             //warten bis Messung beendet ist
          }
          //Ergebnisse des ADC zwischenspeichern. Wichtig: zuerst ADCL auslesen, dann ADCH
          if (i != 0){  adc_result = adc_result + ADCW; }
          ADCSRA |= (1<<ADIF); //Bit Freigeben
      }
      return adc_result;
    }
    
    
    
    void get_BTData(char mode){
       char uart_buffer[UART_IN_BUFFER_LEN];
       // Enable Power for Bluetooth
       BT_EN_PORT &= ~(1<<BT_EN_PIN);
       
       for(int i=0; i<4; i++){
           LED_PORT ^= (1 << LED_BLUE_PIN);
           _delay_ms(500);
       }
    
       USART_Init(9600);                      /* initialize USART with 9600 baud rate */
       LED_PORT |= (1<<LED_BLUE_PIN);
    
       uart_gets(uart_buffer,UART_IN_BUFFER_LEN,60);
       if (uart_buffer[0] == '\0'){                                                                               // TIMEOUT
       }else if ( (mode=='I') && (uart_buffer[0] == 'I') && (uart_buffer[1] == 'N') && (uart_buffer[2] == 'I') && (uart_buffer[3] == 'T') && (uart_buffer[4] == ':') ){    // INIT CODE
          gEi_initCode = atoi(uart_buffer+5);
          eeprom_write_byte(&gEi_initCode, gEi_initCode); // schreiben
          USART_SendString("KSR_IC:InitCode Saved");     // KeySaveResponse_InformationCode
    
       }else if ( (mode=='I') && (uart_buffer[0] == 'S') && (uart_buffer[1] == 'E') && (uart_buffer[2] == 'T') && (uart_buffer[3] == 'K') && (uart_buffer[4] == ':') ){    // SET KeySafe Code
          //Read Init Code from EEPROM
          gEi_initCode = eeprom_read_byte(&gEi_initCode);
        
          char spl_buffer[UART_IN_BUFFER_LEN];
          uint8_t ele = 0;
          // Split up
          uint32_t ii=0;
          for (uint32_t i=5; i < UART_IN_BUFFER_LEN; i++){
             pc_code_sement[i-5] = uart_buffer[i];           //Copy Code
             spl_buffer[ii] = uart_buffer[i];
             
             if ((ele == 0) && (uart_buffer[i] == ',')){
                spl_buffer[ii] = '\0';
                //pl_valid_time_sec = atol(spl_buffer);
                ele++;
                ii=0;
             }else if ((ele == 1) && (uart_buffer[i] == ',')){
                spl_buffer[ii] = '\0';
                //pl_lock_time_sec = atol(spl_buffer);
                ele++;
                ii=0;
             }else if ((ele == 2) && (uart_buffer[i] == ',')){
                spl_buffer[ii] = '\0';
                //pi_multiplikator=atoi(spl_buffer);
                ele++;
                ii=0;
                break;
             }
             ii++;
          }
          
          snprintf(uart_buffer, sizeof uart_buffer, "KSR_RT:%ld\nKSR_ES:%ld\nKSR_ID:%s\n", gl_remaining_time,getCurrentVoltage(),gs_verificationCode); 
          USART_SendString(uart_buffer);
       }else if ( (uart_buffer[0] == 'G') && (uart_buffer[1] == 'E') && (uart_buffer[2] == 'T') && (uart_buffer[3] == 'S') && (uart_buffer[4] == ':') ){    // GET Status
          snprintf(uart_buffer, sizeof uart_buffer, "KSR_RT:%ld\nKSR_ES:%ld\nKSR_ID:%s\n", gl_remaining_time,getCurrentVoltage(),gs_verificationCode);  /
          USART_SendString(uart_buffer);
       }else{     
          USART_SendString("KSR_EC:UnknwonCommand");   
       }
       _delay_ms(5000);
       LED_PORT &= ~(1<<LED_BLUE_PIN);
       BT_EN_PORT |= (1<<BT_EN_PIN);
    }
    
    
    
    int main(void){
       // Save Power
       ACSR = 0x80;            // Disable Analogcomparator
       ADCSRA = 0;             // Disable ADC
       
       // 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;
          
           LED_PORT ^= (1 << LED_GREEN_PIN);
           _delay_ms(500);
                  LED_PORT ^= (1 << LED_GREEN_PIN);
           _delay_ms(500);
                  LED_PORT ^= (1 << LED_GREEN_PIN);
           _delay_ms(500);
                  LED_PORT ^= (1 << LED_GREEN_PIN);
           _delay_ms(500);
    
       get_BTData('I');  // INIT MODE
       
       
       while(1){
           LED_PORT ^= (1 << LED_GREEN_PIN);
           _delay_ms(500);
       }
       
    }
    Eingebunden ist noch die
    * USART_RS232_C_file.c
    https://www.electronicwings.com/user...ce_with_atmega

  2. #2
    HaWe
    Gast
    kürze dein Programm doch bitte mal auf die minimalste Version zurück, in der sich der Fehler gerade noch zeigen lässt - alles andere an Codezeilen löschen.

  3. #3
    Neuer Benutzer Öfters hier
    Registriert seit
    22.12.2019
    Beiträge
    19
    Hallo HaWe,

    ich habe mich gleich ans Werk gemacht und den Code gekürzt das verhalten ist noch 1:1 das gleiche.
    Schon mal vielen Dank, dass du dir den Quellcode an siehst und mir hilfst.

    Code:
    #define MCU  atmega8
    #define F_CPU 2000000UL
    
    #include <avr/io.h>
    #include <avr/interrupt.h>
    #include <util/delay.h>
    #include <inttypes.h>
    #include <stdlib.h> 
    #include <stdio.h>
    #include "USART_RS232_H_file.h"	/* include USART library */
    
    #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
    
    //Data from App
    long pl_valid_time_sec = 0;
    
    uint8_t uart_getc(void){
        while (!(UCSRA & (1<<RXC))){}   // warten bis Zeichen verfuegbar
        return UDR;                   // Zeichen aus UDR an Aufrufer zurueckgeben
    }
    
    void uart_gets( char* Buffer, uint32_t MaxLen){
      uint8_t NextChar;
      uint32_t StringLen = 0;
    
      NextChar = uart_getc();         // Warte auf und empfange das nächste Zeichen
      while( NextChar != '\n' && StringLen < MaxLen - 1 ) { // Sammle solange Zeichen, bis: entweder das String Ende Zeichen kam oder das aufnehmende Array voll ist
        *Buffer++ = NextChar;
        StringLen++;
        NextChar = uart_getc();
      }
      *Buffer = '\0';             // Noch ein '\0' anhängen um einen Standard C-String daraus zu machen
    }
    
    
    void get_BTData(char mode){
       char uart_buffer[512];
    
       for(int i=0; i<4; i++){
           LED_PORT ^= (1 << LED_BLUE_PIN);
           _delay_ms(500);
       }
    
       USART_Init(9600);                      /* initialize USART with 9600 baud rate */
       LED_PORT |= (1<<LED_BLUE_PIN);
    
       uart_gets(uart_buffer,512);
       
       char spl_buffer[512];
       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('I');  // INIT MODE
       
       while(1){
           LED_PORT ^= (1 << LED_RED_PIN);
           _delay_ms(500);
       }
    
    }
    Wenn ich wüsste was hier bei der Lib an BAUD steht, könnte ich die ganze Lib auch noch ausbauen
    Der ATMEGA läuft bei 2Mhz
    Code:
    void USART_Init(unsigned long BAUDRATE)				/* USART initialize function */
    { 
    	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 */
    }

  4. #4
    HaWe
    Gast
    UART und LED-Code auch mal rauslöschen. wenn atol eine runtime-Exception auslöst, müsste es davon unabhängig sein.
    Stattdessen könntest du Zeichen aus UART auch durch const cstrings testweise ersetzen.

    On success, the function returns the converted integral number as a long int value.
    If no valid conversion could be performed, a zero value is returned.
    If the converted value would be out of the range of representable values by a long int, it causes undefined behavior. See strtol for a more robust cross-platform alternative when this is a possibility.
    http://www.cplusplus.com/reference/cstdlib/atol/
    Geändert von HaWe (06.04.2020 um 22:57 Uhr)

  5. #5
    Neuer Benutzer Öfters hier
    Registriert seit
    22.12.2019
    Beiträge
    19
    Ich habe jetzt

    uint8_t uart_getc(void){
    void uart_gets( char* Buffer, uint32_t MaxLen){

    Auskommentiert und den Aufruf USART_Init(9600); uart_gets(uart_buffer,512);

    jetzt bleibt die Blaue LED an. -> Also ok.

    Die Led brauche ich damit ich mit bekomme was das Programm tut.


    Wenn ich den Init wieder aktiviere geht es auch noch beim uart_gets(uart_buffer,512); habe ich wieder den Fehler.

  6. #6
    HaWe
    Gast
    das deutet für mich darauf hin, dass der UART-buffer aus uart_getc() oder uart_gets() einen ungültigen Wert liefern könnte, der eine Exception auslöst. Ich verstehe hier aber deinen Code zugegebenermaßen nicht.
    Kannst du diesen UART-Wert auslesen, anzeigen (Display?) und checken, bevor du ihn an atol übergibst?
    Geändert von HaWe (06.04.2020 um 23:06 Uhr)

  7. #7
    Neuer Benutzer Öfters hier
    Registriert seit
    22.12.2019
    Beiträge
    19
    Aber ich gebe ja atol inzwischen schon einen fixen String und nichts mehr aus den UART Buffer.

  8. #8
    HaWe
    Gast
    also passiert der Fehler gar nicht in atol? Sondern in uart_getc() oder uart_gets() ? Oder wie oder was?

  9. #9
    Neuer Benutzer Öfters hier
    Registriert seit
    22.12.2019
    Beiträge
    19
    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?

    Code:
    #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);
       }
    
    }

  10. #10
    Erfahrener Benutzer Robotik Einstein
    Registriert seit
    18.03.2018
    Beiträge
    2.643
    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



Seite 1 von 2 12 LetzteLetzte

Ähnliche Themen

  1. ISS: Tierbeobachtungssystem Icarus startet
    Von Roboternetz-News im Forum Neuigkeiten / Technik-News / Nachrichten / Aktuelles
    Antworten: 0
    Letzter Beitrag: 10.07.2019, 11:10
  2. [ERLEDIGT] PC startet nicht von CD/DVD
    Von oderlachs im Forum Offtopic und Community Tratsch
    Antworten: 6
    Letzter Beitrag: 08.01.2013, 20:24
  3. PC startet nur manchmal....
    Von Paulschen97 im Forum PC-, Pocket PC, Tablet PC, Smartphone oder Notebook
    Antworten: 3
    Letzter Beitrag: 12.12.2012, 14:12
  4. Controller, Board, Servo Controller fuer den Spiderbot?
    Von Kiraminh im Forum Microcontroller allgemeine Fragen/Andere Microcontroller
    Antworten: 1
    Letzter Beitrag: 11.12.2007, 17:38
  5. AVR startet bei Interrupt neu
    Von Jaecko im Forum Basic-Programmierung (Bascom-Compiler)
    Antworten: 2
    Letzter Beitrag: 25.02.2007, 15:37

Berechtigungen

  • Neue Themen erstellen: Nein
  • Themen beantworten: Nein
  • Anhänge hochladen: Nein
  • Beiträge bearbeiten: Nein
  •  

LiFePO4 Speicher Test