Problem mit CLKPR im AtmelStudio 7

[P.Z. (Gast)]

Hallo, ich versuche einen Clockprescaler bei einem ATMEGA48P, mit 
AtmelStudio 7[neuste Version], zu setzten. Leider läuft der Build nicht 
durch.

#include <stdint.h>      // Einbinden von stdint.h
#include <avr/io.h>      // Einbinden von io.h
#include <avr/interrupt.h>  // Einbinden von interrupt.h
#define F_CPU 1000000UL    // Frequenz:1 MHz
#include <util/delay.h>    // Einbinden von delay.h

....

CLKPR = (1<<CLKPCE); // Enable change of CLKPS bits
CLKPR = (1<<CLKPS1); // Set prescaler to 4 and system clock to 1 MHz

.....

Mit auskommentieren der zwei Zeilen läuft der Build sauber durch. 
(WinAVR[neuste Version] reagiert ebenfalls so.)

Beim Verwenden der power.h passiert ebenfalls dasselbe.

Im Build-Prozess bekomme ich folgenden Fehler, den ich leider nicht ganz 
nachvollziehen kann.

Build started.
Project "Kettenoeler.cproj" (default targets):
Target "PreBuildEvent" skipped, due to false condition; ('$(PreBuildEvent)'!='') was evaluated as (''!='').
Target "CoreBuild" in file "C:\Program Files (x86)\Atmel\Studio\7.0\Vs\Compiler.targets" from project "...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\Kettenoeler.cproj" (target "Build" depends on it):
  Task "RunCompilerTask"
    Shell Utils Path C:\Program Files (x86)\Atmel\Studio\7.0\shellUtils
    C:\Program Files (x86)\Atmel\Studio\7.0\shellUtils\make.exe all --jobs 8 --output-sync 
    Building file: .././main.c
    Invoking: AVR/GNU C Compiler : 5.4.0
    "C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-gcc.exe"  -x c -funsigned-char -funsigned-bitfields -DDEBUG  -I"C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.150\include"  -O1 -ffunction-sections -fdata-sections -fpack-struct -fshort-enums -g2 -Wall -mmcu=atmega48p -B "C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.150\gcc\dev\atmega48p" -c -std=gnu99 -MD -MP -MF "main.d" -MT"main.d" -MT"main.o"   -o "main.o" ".././main.c" 
...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\Debug\Makefile(79,1): error: recipe for target 'main.o' failed
    In file included from c:\program files (x86)\atmel\studio\7.0\toolchain\avr8\avr8-gnu-toolchain\avr\include\avr\io.h:99:0,
                     from .././main.c:14:
...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\main.c(22,1): error: expected identifier or '(' before 'volatile'
     CLKPR = (1<<CLKPCE); // Enable change of CLKPS bits
     ^
...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\main.c(22,1): error: expected ')' before '(' token
     CLKPR = (1<<CLKPCE); // Enable change of CLKPS bits
     ^
...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\main.c(23,1): error: expected identifier or '(' before 'volatile'
     CLKPR = (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz
     ^
...\Atmel Studio\7.0\Kettenoeler\Kettenoeler\main.c(23,1): error: expected ')' before '(' token
     CLKPR = (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz
     ^
    make: *** [main.o] Error 1
  Done executing task "RunCompilerTask" -- FAILED.
Done building target "CoreBuild" in project "Kettenoeler.cproj" -- FAILED.
Done building project "Kettenoeler.cproj" -- FAILED.

[M. K. (sylaina)]

Hast du schon mal versucht den Code so zu reduzieren, dass du nur den 
CLKPR ändern willst und hat das einen Effekt?
Was steht vor und nach dem Code? Ich denke das eigentliche Problem liegt 
im nicht gezeigten Code.

[Beo Bachta (Gast)]

P.Z. schrieb:
> CLKPR = (1<<CLKPCE); // Enable change of CLKPS bits
> CLKPR = (1<<CLKPS1); // Set prescaler to 4 and system clock to 1 MHz

Ich weiss nicht was du sonst noch alles falsch nachst aber diese
beiden Zeilen tun vermutlich nicht was du willst.

In der ersten Zeile setzt du im Register genau ein Bit, alle
anderen werden gelöscht.
In der zweiten Zeile setzt du im Register genau ein Bit, alle
anderen werden gelöscht.

Das heisst in der zweiten Zeile machst du alles kaputt was du
in der ersten Zeile wolltest.

[P.Z. (Gast)]

Beo Bachta schrieb:
> Das heisst in der zweiten Zeile machst du alles kaputt was du
> in der ersten Zeile wolltest.

Das stimmt, dass sollte natürlich so aussehen.

CLKPR |= (1<<CLKPCE);
CLKPR |= (1<<CLKPS1);

Ändert aber leider nichts am Compiler

[M. K. (sylaina)]

Beo Bachta schrieb:
> Das heisst in der zweiten Zeile machst du alles kaputt was du
> in der ersten Zeile wolltest.

Beschäftige dich mal mit dem Ändern geschützter Register/Bits. Diese 
beiden Zeilen waren schlicht fehlerfrei. Mit der ersten Zeile hat der TE 
den Schreibschutz der CLKPS-Bits aufgehoben, mit dem zweiten hat er 
innerhalb von 4 Zyklen den neuen, gültigen Wert, in die CLKPS-Bits 
geschrieben.
Dieses Vorgehen war völlig korrekt.

[Beo Bachta (Gast)]

P.Z. schrieb:
> Ändert aber leider nichts am Compiler

Da wirst du wohl doch etwas mehr von deinem wertvollen
Code preisgeben müssen.

[Dieter F. (Gast)]

Wenn ich es auf den angegebenen Code reduziere ist alles O.K.
Der Fehler resultiert aus "...".

expected identifier or '(' before 'volatile'

[P.Z. (Gast)]

Beo Bachta schrieb:
> Da wirst du wohl doch etwas mehr von deinem wertvollen
> Code preisgeben müssen.

Das ist quasi der gesammt Code.

#include <stdint.h>      // Einbinden von stdint.h
#include <avr/io.h>      // Einbinden von io.h
#include <avr/interrupt.h>  // Einbinden von interrupt.h
#define F_CPU 1000000UL    // Frequenz:1 MHz
#include <util/delay.h>    // Einbinden von delay.h

CLKPR = (1<<CLKPCE); // Enable change of CLKPR bits
CLKPR = (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz

int main(void)
{
  DDRD |= (1<<DDD4);

    while (1) 
    {
          PORTD |= (1 << PD4); 
          _delay_ms(100);
          PORTD &= ~(1 << PD4);
          _delay_ms(100);
    }
}

[P.Z. (Gast)]

Dieter F. schrieb:
> Wenn ich es auf den angegebenen Code reduziere ist alles O.K.
> Der Fehler resultiert aus "...".
>
> expected identifier or '(' before 'volatile'
Sorry,
das "..." habe ich da hin gemacht, weil ich den Pfad gekürzt habe. 
Dadurch wurde alles etwas leserlicher.

[Dieter F. (Gast)]

P.Z. schrieb:
> CLKPR = (1<<CLKPCE); // Enable change of CLKPR bits
> CLKPR = (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz

Dann pack das mal als erstes in Deine Main oder bilde eine Init-Funktion 
(in die Du den Code packst), welche Du aus der Main heraus aufrufst.

[Oliver S. (oliverso)]

P.Z. schrieb:
> Das ist quasi der gesammt Code.

Quasi oder tatsächlich?

Wenn tatsächlich, such dir ein C-Buch. Dann wird dir der Fehler schnell 
klar. Wenn quasi, dann poste den echten Code.

Oliver

[Dieter F. (Gast)]

Und vergiss das "|=" nicht :-)

[P.Z. (Gast)]

Oliver S. schrieb:
>
> Quasi oder tatsächlich?
>
> Wenn tatsächlich, such dir ein C-Buch. Dann wird dir der Fehler schnell
> klar. Wenn quasi, dann poste den echten Code.
>
> Oliver

Es ist der gesammte Code.

Dieter F. schrieb:
> P.Z. schrieb:
>> CLKPR = (1<<CLKPCE); // Enable change of CLKPR bits
>> CLKPR = (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz
>
> Dann pack das mal als erstes in Deine Main oder bilde eine Init-Funktion
> (in die Du den Code packst), welche Du aus der Main heraus aufrufst.

Danke. Nun scheint es zu funktionieren. Macht auch definitiv Sinn, im 
Hinblick auf den Compiler.

#include <stdint.h>      // Einbinden von stdint.h
#include <avr/io.h>      // Einbinden von io.h
#include <avr/interrupt.h>  // Einbinden von interrupt.h
#define F_CPU 1000000UL    // Frequenz:1 MHz
#include <util/delay.h>    // Einbinden von delay.h

int setClock(void)
{
  CLKPR |= (1<<CLKPCE); // Enable change of CLKPR bits
  CLKPR |= (1<<CLKPS1); // Set prescaler to 4, and system clock to 1 MHz
}


int main(void)
{
  DDRD |= (1<<DDD4);
  setClock();
    while (1) 
    {
          PORTD |= (1 << PD4); 
          _delay_ms(100);
          PORTD &= ~(1 << PD4);
          _delay_ms(100);
    }
}

[M. K. (sylaina)]

Das CLKPR muss auch innerhalb einer Funktion stehen, außerhalb wäre es 
nicht zulässig.

[spess53 (Gast)]

Hi

>Das CLKPR muss auch innerhalb einer Funktion stehen, außerhalb wäre es
>nicht zulässig.

In verschiedenen Funktionen wären die vier Takte wohl kaum einzuhalten.

MfG Spess

[M. K. (sylaina)]

spess53 schrieb:
> In verschiedenen Funktionen wären die vier Takte wohl kaum einzuhalten.

Und wenns regnet wird man nicht zwangsläufig nass. Man kann's auch jetzt 
kompliziert aufziehen, man kann's aber auch lassen. ;)