STM32F105R8T6 + USART2

[Jan H. (janiiix3)]

Nabend.

Habe heute mein China Board bekommen (CAN + USART).
Drauf bestückt ist ein 8MHz Quarz.

Als Herzstück ist ein STM32F105R8T6 bestückt. Das ganze habe ich mit 
EmBitz compiliert.

Wenn ich jetzt via. USART was ausgeben möchte, kommen nur Sonderzeichen.
Vermutlich ist die Frequenz im Projekt nicht richtig eingestellt, so das 
die Baudrate überhaupt nicht richtig berechnet wird.

Oder sieht hier jemand einen fatalen Fehler, was die USART 
Initalisierung oder auch das senden betrifft?

void _uartInit(void)
{
    USART_InitTypeDef usart1_init_struct;
    GPIO_InitTypeDef gpioa_init_struct;

    RCC_APB2PeriphClockCmd(RCC_APB2Periph_AFIO | RCC_APB2Periph_GPIOA, ENABLE);

    RCC_APB1PeriphClockCmd(RCC_APB1Periph_USART2, ENABLE);

    /* GPIOA PIN9 alternative function Tx */
    gpioa_init_struct.GPIO_Pin = GPIO_Pin_2;
    gpioa_init_struct.GPIO_Speed = GPIO_Speed_10MHz;
    gpioa_init_struct.GPIO_Mode = GPIO_Mode_AF_PP;
    GPIO_Init(GPIOA, &gpioa_init_struct);
    /* GPIOA PIN9 alternative function Rx */
    gpioa_init_struct.GPIO_Pin = GPIO_Pin_3;
    gpioa_init_struct.GPIO_Speed = GPIO_Speed_10MHz;
    gpioa_init_struct.GPIO_Mode = GPIO_Mode_IN_FLOATING;
    GPIO_Init(GPIOA, &gpioa_init_struct);

    USART_Cmd(USART2, ENABLE);

    usart1_init_struct.USART_BaudRate = 9600;
    usart1_init_struct.USART_WordLength = USART_WordLength_8b;
    usart1_init_struct.USART_StopBits = USART_StopBits_1;
    usart1_init_struct.USART_Parity = USART_Parity_No ;
    usart1_init_struct.USART_Mode = USART_Mode_Rx | USART_Mode_Tx;
    usart1_init_struct.USART_HardwareFlowControl = USART_HardwareFlowControl_None;
    /* Configure USART1 */
    USART_Init(USART2, &usart1_init_struct);
}

void _uartChar(char c, USART_TypeDef *uart)
{
    while ((uart->SR & USART_SR_TXE) == 0);
    uart->DR = c;
}

void _uartStr(char *str, USART_TypeDef *uart)
{
    while(*str)
    {
        while ((uart->SR & USART_SR_TXE) == 0);
        _uartChar(*str++,uart);
    }
}

int main(void)
{
    _canInit();
    _ledInit();
    _uartInit();

    SystemInit();
    //SysTick_Config(2500000);

  while(1)
  {
    _uartStr("Herzlich Willkommen\r\n",USART2);

  }
}

[Jim M. (turboj)]

Das SystemInit() sollte man als erstes machen, dort wird IIRC der Quarz 
eingeschaltet und die Clocks umprogrammiert.

Wenn man das nach der UART Initialisierung macht, sind dessen Baudraten 
Einstellungen hinterher falsch.

[Jan H. (janiiix3)]

Jim M. schrieb:
> Das SystemInit() sollte man als erstes machen, dort wird IIRC der Quarz
> eingeschaltet und die Clocks umprogrammiert.
>
> Wenn man das nach der UART Initialisierung macht, sind dessen Baudraten
> Einstellungen hinterher falsch.

Ich bin mir nicht sicher, wie er konfiguriert ist. Läuft er jetzt mit 
25MHz?
Benutze die Standart Treiber von ST.

[Johannes S. (Gast)]

da wo SystemInit definiert ist sollte es auch etwas wie 
'SystemCoreClock' geben, das kannst du ja über die serielle ausgeben 
(wenn sie funktioniert).

[Jan H. (janiiix3)]

Johannes S. schrieb:
> da wo SystemInit definiert ist sollte es auch etwas wie
> 'SystemCoreClock' geben, das kannst du ja über die serielle ausgeben
> (wenn sie funktioniert).

Also in der Headerdatei steht nicht viel..

/******************************************************************************
 * @file:    system_stm32f10x_cl.h
 * @purpose: CMSIS Cortex-M3 Device Peripheral Access Layer Header File for the
 *           ST STM32 Connectivity Line Device Series
 * @version: V1.01
 * @date:    22. Jul. 2009
 *----------------------------------------------------------------------------
 *
 * Copyright (C) 2009 ARM Limited. All rights reserved.
 *
 * ARM Limited (ARM) is supplying this software for use with Cortex-Mx
 * processor based microcontrollers.  This file can be freely distributed
 * within development tools that are supporting such ARM based processors.
 *
 * THIS SOFTWARE IS PROVIDED "AS IS".  NO WARRANTIES, WHETHER EXPRESS, IMPLIED
 * OR STATUTORY, INCLUDING, BUT NOT LIMITED TO, IMPLIED WARRANTIES OF
 * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE APPLY TO THIS SOFTWARE.
 * ARM SHALL NOT, IN ANY CIRCUMSTANCES, BE LIABLE FOR SPECIAL, INCIDENTAL, OR
 * CONSEQUENTIAL DAMAGES, FOR ANY REASON WHATSOEVER.
 *
 ******************************************************************************/


#ifndef __SYSTEM_STM32F10x_CL_H
#define __SYSTEM_STM32F10x_CL_H

#ifdef __cplusplus
 extern "C" {
#endif

extern uint32_t SystemFrequency;    /*!< System Clock Frequency (Core Clock)  */


/**
 * Initialize the system
 *
 * @param  none
 * @return none
 *
 * @brief  Setup the microcontroller system
 *         Initialize the Embedded Flash Interface,  initialize the PLL and update the SystemFrequency variable
 */
extern void SystemInit (void);

#ifdef __cplusplus
}
#endif

#endif /* __SYSTEM_STM32F10x_CL_H */

[Christopher J. (christopher_j23)]

Das haben Header (zumindest bei C) so an sich, dass da nur die 
Deklarationen drin stehen. Für die Implementierung der Funktionen würde 
ich mal in der entsprechenden .c-Datei nachsehen.

Normalerweise (jedenfalls war das mal von ARM so vorgesehen) wird die 
SystemInit auch noch vor der eigentlichen main aufgerufen. Entsprechende 
Hinweise liefert der Startup-Code. Findest du dort ein "bl SystemInit" 
vor einem "bl main" kannst du dir den Aufruf in der main sowieso sparen.

[Jan H. (janiiix3)]

Christopher J. schrieb:
> Das haben Header (zumindest bei C) so an sich, dass da nur die
> Deklarationen drin stehen. Für die Implementierung der Funktionen würde
> ich mal in der entsprechenden .c-Datei nachsehen.
>
> Normalerweise (jedenfalls war das mal von ARM so vorgesehen) wird die
> SystemInit auch noch vor der eigentlichen main aufgerufen. Entsprechende
> Hinweise liefert der Startup-Code. Findest du dort ein "bl SystemInit"
> vor einem "bl main" kannst du dir den Aufruf in der main sowieso sparen.

Wo sehe ich denn nun die aktuelle Frequenz mit der Initalisiert wird?

[A.. P. (arnonym)]

In deinem Projektverzeichnis sollte müsste noch eine gleichnamige 
Quelldatei liegen (in deinem Fall also system_stm32f10x_cl.c). Darin 
sind dann die Werte enthalten, die mit SystemInit() in die 
Controller-Register geschrieben werden.

Diese Datei kann man mit einem einfachen Excel-Tool erzeugen oder mit 
dem Programm STMCubeMX. Beides gibts bei ST auf der jeweiligen 
Produktseite für deinen Controller zum Download.

Im Anhang siehst du eine solche Datei, welche den Systemtakt einstellt 
(exemplarisch für einen STM32F2).

Gruß

[Jan H. (janiiix3)]

A.. P. schrieb:
> In deinem Projektverzeichnis sollte müsste noch eine gleichnamige
> Quelldatei liegen (in deinem Fall also system_stm32f10x_cl.c). Darin
> sind dann die Werte enthalten, die mit SystemInit() in die
> Controller-Register geschrieben werden.
>
> Diese Datei kann man mit einem einfachen Excel-Tool erzeugen oder mit
> dem Programm STMCubeMX. Beides gibts bei ST auf der jeweiligen
> Produktseite für deinen Controller zum Download.
>
> Im Anhang siehst du eine solche Datei, welche den Systemtakt einstellt
> (exemplarisch für einen STM32F2).
>
> Gruß

Diese Datei die du meinst, muss ich mir erst noch erstellen?

[Dumpfbacke (Gast)]

Jan H. schrieb:
> Diese Datei die du meinst, muss ich mir erst noch erstellen?

Er hat doch gerade erst geschrieben:

A.. P. schrieb:
> In deinem Projektverzeichnis sollte müsste noch eine gleichnamige
> Quelldatei liegen

[Jan H. (janiiix3)]

Dumpfbacke schrieb:
> Jan H. schrieb:
>> Diese Datei die du meinst, muss ich mir erst noch erstellen?
>
> Er hat doch gerade erst geschrieben:
>
> A.. P. schrieb:
>> In deinem Projektverzeichnis sollte müsste noch eine gleichnamige
>> Quelldatei liegen

Diese gibt es eben nicht.

[A.. P. (arnonym)]

Jan H. schrieb:
> Diese Datei die du meinst, muss ich mir erst noch erstellen?

Wie gesagt: entweder, sie ist da, oder du musst sie noch erstellen. Wenn 
du dein Programm kompilieren kannst, wird sie wohl schon da sein und du 
musst überprüfen, was drin steht und bei Bedarf auf den korrekten Wert 
einstellen.

Darüber hinaus muss USART_Cmd() nach USART_Init() aufgerufen werden. Zur 
besseren Orientierung findet man in der Standard Peripherals Library zu 
jeder Peripherie eine kurze Anleitung, wie man sie richtig 
initialisiert. So z.B. für den USART für einen STM32F1xx:

===============================================================================
                       ##### How to use this driver #####
 ===============================================================================
    [..]
        (#) Enable peripheral clock using RCC_APB2PeriphClockCmd(RCC_APB2Periph_USART1, ENABLE)
            function for USART1 or using RCC_APB1PeriphClockCmd(RCC_APB1Periph_USARTx, ENABLE)
            function for USART2 and USART3.
        (#) According to the USART mode, enable the GPIO clocks using 
            RCC_AHBPeriphClockCmd() function. (The I/O can be TX, RX, CTS, 
            or and SCLK). 
        (#) Peripheral's alternate function: 
            (++) Connect the pin to the desired peripherals' Alternate 
                 Function (AF) using GPIO_PinAFConfig() function.
            (++) Configure the desired pin in alternate function by:
                 GPIO_InitStruct->GPIO_Mode = GPIO_Mode_AF.
            (++) Select the type, pull-up/pull-down and output speed via 
                 GPIO_PuPd, GPIO_OType and GPIO_Speed members.
            (++) Call GPIO_Init() function.        
        (#) Program the Baud Rate, Word Length , Stop Bit, Parity, Hardware 
            flow control and Mode(Receiver/Transmitter) using the SPI_Init()
            function.  
        (#) For synchronous mode, enable the clock and program the polarity,
            phase and last bit using the USART_ClockInit() function.  
        (#) Enable the NVIC and the corresponding interrupt using the function 
            USART_ITConfig() if you need to use interrupt mode.   
        (#) When using the DMA mode: 
            (++) Configure the DMA using DMA_Init() function.
            (++) Active the needed channel Request using USART_DMACmd() function.   
        (#) Enable the USART using the USART_Cmd() function.   
        (#) Enable the DMA using the DMA_Cmd() function, when using DMA mode.   
    [..]
            Refer to Multi-Processor, LIN, half-duplex, Smartcard, IrDA sub-sections
            for more details.

Die genauen Werte für z.B. Konstanten oder Funktionsnamen musst du 
natürlich in deiner speziellen Version nachsehen, hier unterscheiden 
sich die Bezeichnungen bei den STMs von Familie zu Familie.

Gruß

[Jan H. (janiiix3)]

A.. P. schrieb:
> In deinem Projektverzeichnis sollte müsste noch eine gleichnamige
> Quelldatei liegen (in deinem Fall also system_stm32f10x_cl.c). Darin
> sind dann die Werte enthalten, die mit SystemInit() in die
> Controller-Register geschrieben werden.
>
> Diese Datei kann man mit einem einfachen Excel-Tool erzeugen oder mit
> dem Programm STMCubeMX. Beides gibts bei ST auf der jeweiligen
> Produktseite für deinen Controller zum Download.
>
> Im Anhang siehst du eine solche Datei, welche den Systemtakt einstellt
> (exemplarisch für einen STM32F2).
>
> Gruß

Woher weißt du das sie genau so heißen muss?

[A.. P. (arnonym)]

Weil es eine übliche Konvention bei C ist, dass eine Quell- und die 
zugehörige Header-Datei denselben Namen haben. Datei.c und Datei.h eben.

Da du aber schon so fragst und du scheinbar keine Probleme hast deinen 
Code zu kompilieren, solltest du dich mal auf die Suche machen, wo denn 
in deinem Code sonst noch die Werte versteckt sein könnten, die 
normalerweise in der system_xxxxx.c Datei stehen. Selbstverständlich 
kann der Code für die Initialisierung des Systemtakts in jeder 
x-beliebigen Datei mit unscheinbarem Namen stehen, solange die Header 
eingebunden werden - das wäre aber grober Unfug ;)

Wenn du einen Editor mit vernünftiger Suchfunktion hast, dann kannst du 
ja einfach mal in deinem Projektverzeichnis nach Zeichenketten wie 
"RCC->CR", "RCC->PLLCFGR" oder einfach "SystemCoreClock" suchen. Das 
sind nämlich so ein paar Zuweisungen, die passieren müssten, wenn der 
Takt konfiguriert ist. Die Datei, in der das steht ist vielleicht auch 
die Datei, die du suchst.

Viel, viel einfacher wäre es aber, einfach das Clock Configuration Tool 
von ST runterzuladen, dir eine saubere Datei zu erzeugen und sich nicht 
weiter darum zu kümmern. In den letzten 30 Minuten hättest du dir eine 
solche Datei ein Dutzend Mal erzeugt :D

Gruß

[holger (Gast)]

>Woher weißt du das sie genau so heißen muss?

Ist doch egal. Mach ne Volltextsuche nach SystemInit
in deinem Projekt. Und wenn du gleich das ganze Projekt und nicht
nur Codeschnipsel geliefert hättest, wäre dein Problem längst gelöst.

Was ich noch so weiß ist das diese SystemInits oft für die EVAL
Boards ausgelegt sind. Da ist ein 25MHz Quarz drauf. Das passt dann
nicht zu deinem 8MHz Quarz.

Kompletten Code zeigen dürfte hier am schnellsten zur Lösung
beitragen.

[Jan H. (janiiix3)]

A.. P. schrieb:
> Wenn du einen Editor mit vernünftiger Suchfunktion hast, dann kannst du
> ja einfach mal in deinem Projektverzeichnis nach Zeichenketten wie
> "RCC->CR", "RCC->PLLCFGR" oder einfach "SystemCoreClock" suchen. Das
> sind nämlich so ein paar Zuweisungen, die passieren müssten, wenn der
> Takt konfiguriert ist. Die Datei, in der das steht ist vielleicht auch
> die Datei, die du suchst.
>
> Viel, viel einfacher wäre es aber, einfach das Clock Configuration Tool
> von ST runterzuladen, dir eine saubere Datei zu erzeugen und sich nicht
> weiter darum zu kümmern. In den letzten 30 Minuten hättest du dir eine
> solche Datei ein Dutzend Mal erzeugt :D
>
> Gruß

Kannst du mir mal einen Link zum Tool posten?
Ist es das "STM32CubeMx"?

[A.. P. (arnonym)]

holger schrieb:
> Was ich noch so weiß ist das diese SystemInits oft für die EVAL
> Boards ausgelegt sind. Da ist ein 25MHz Quarz drauf. Das passt dann
> nicht zu deinem 8MHz Quarz.

Ja, scheint tatsächlich häufig so zu sein. Deshalb sage ich ja: einfach 
diese Datei einmal selbst erzeugen (dauert doch nur 1 Minute) und fortan 
glücklich sein :)

[Jan H. (janiiix3)]

A.. P. schrieb:
> holger schrieb:
>> Was ich noch so weiß ist das diese SystemInits oft für die EVAL
>> Boards ausgelegt sind. Da ist ein 25MHz Quarz drauf. Das passt dann
>> nicht zu deinem 8MHz Quarz.
>
> Ja, scheint tatsächlich häufig so zu sein. Deshalb sage ich ja: einfach
> diese Datei einmal selbst erzeugen (dauert doch nur 1 Minute) und fortan
> glücklich sein :)

Ja krass. Geht! Scheint so als würde "EmBitz" etwas falsch machen beim 
Projekt erstellen.

[A.. P. (arnonym)]

Jan H. schrieb:
> Kannst du mir mal einen Link zum Tool posten?

http://www.st.com/content/st_com/en/products/microcontrollers/stm32-32-bit-arm-cortex-mcus/stm32-mainstream-mcus/stm32f1-series/stm32f105-107/stm32f105r8.html

Das ist die Produktseite für deinen Controller. Ab da musst du schon 
selbst was tun. Einfach mal nach Clock Configuration Tool suchen. Kommt 
meistens als Zip-Datei, obwohl nur ein Excel-Sheet und eine Lizenzdatei 
drinliegen.

[Oliver P. (ollus)]

> Ja krass. Geht! Scheint so als würde "EmBitz" etwas falsch machen beim
> Projekt erstellen.

Nein, EmBitz kann ja Deine Hardware nicht kennen. Man kann da Quarze von 
3-25MHz dranhängen.

[A.. P. (arnonym)]

Jan H. schrieb:
> Ja krass. Geht! Scheint so als würde "EmBitz" etwas falsch machen beim
> Projekt erstellen.

Ich würde diesem EmBitz (kenne das Tool nicht) nicht grundsätzlich 
unterstellen, dass es was falsch macht. Wahrscheinlich hat es nur eine 
Standardvorlage für ein solches Projekt enthalten, wo eben der Takt für 
einen 25 MHz Quarz konfiguriert wird. Daher plädiere ich immer dafür, 
nachzusehen, was unter der Haube tatsächlich vor sich geht und nicht 
einfach alles schlucken, was einem vor die Füße geschmissen wird. Dann 
können solche Problemchen gar nicht erst entstehen.

[Jan H. (janiiix3)]

A.. P. schrieb:
> Jan H. schrieb:
>> Ja krass. Geht! Scheint so als würde "EmBitz" etwas falsch machen beim
>> Projekt erstellen.
>
> Ich würde diesem EmBitz (kenne das Tool nicht) nicht grundsätzlich
> unterstellen, dass es was falsch macht. Wahrscheinlich hat es nur eine
> Standardvorlage für ein solches Projekt enthalten, wo eben der Takt für
> einen 25 MHz Quarz konfiguriert wird. Daher plädiere ich immer dafür,
> nachzusehen, was unter der Haube tatsächlich vor sich geht und nicht
> einfach alles schlucken, was einem vor die Füße geschmissen wird. Dann
> können solche Problemchen gar nicht erst entstehen.

In diesem Fall also mit diesem Controller, war es ziemlich un 
übersichtlich. Bei anderen Controller Familien, war das deutlich 
einfacher festzustellen, wo und welche Taktquelle ich gerade einstelle.

[Jan H. (janiiix3)]

Habe mir gerade mal die Frequenz ausgeben lassen..

typedef struct
{
  uint32_t SYSCLK_Frequency;  /*!< returns SYSCLK clock frequency expressed in Hz */
  uint32_t HCLK_Frequency;    /*!< returns HCLK clock frequency expressed in Hz */
  uint32_t PCLK1_Frequency;   /*!< returns PCLK1 clock frequency expressed in Hz */
  uint32_t PCLK2_Frequency;   /*!< returns PCLK2 clock frequency expressed in Hz */
  uint32_t ADCCLK_Frequency;  /*!< returns ADCCLK clock frequency expressed in Hz */
}RCC_ClocksTypeDef;

        RCC_ClocksTypeDef SysClk;

        char buff[20];
        itoa(SysClk.SYSCLK_Frequency,buff,10);
        _uartStr(buff,USART2);
        _uartStr("\r\n",USART2);

Ergebniss .: 536936408 Hz

Da kommt eine ziemlich krumme Zahl raus? Ist das jetzt wirklich die 
Systemfrequenz?

[A.. P. (arnonym)]

Du kannst doch nicht einfach eine Variable vom Typ eines Structs 
anlegen, ohne vorherige Initialisierung oder dass damit vorher was 
passiert ist auf seine Member zugreifen und erwarten das was sinnvolles 
bei rauskommt.

Man kann ja auch nicht einfach

int i;

 in seinen Code schreiben und in der nächsten Zeile erwarten, dass i = 
2017 ist.

Schau dir nochmal an, wie dieses Struct, was du verwendest, korrekt 
verwendet und befüllt wird. Hinweis: es gibt eine Methode namens 
RCC_GetClocksFreq().