Variable in _delay_ms() einfügen

[Michael H. (h_m)]

Guten Abend,

ich habe nachfolgenden Code geschrieben. In dem ich einen Analogenwert 
einlesen will, und den Wert des ADCH Registers dann praktisch in 
delay_ms() einfügen damit sich dann je nach Stellung des potis die Zeit 
ändert. Es kommt immer eine Fehlermeldung, was mache ich hier falsch. 
könnte mir das BITTE jemand zeigen ?

/*
 * ATTINY_85.c
 *
 * Created: 03.03.2018 15:59:47
 * Author : USER
 */ 
#define F_CPU 1000000UL
#include <avr/io.h>
#include <util/delay.h>

uint8_t WERT = 0;
uint8_t t = 0;


int main(void)
{
    DDRB &= ~ (1<<PB2); // auf Eingang Bewegungsmelder
  DDRB =    (1<<PB3); // auf Ausgang Magnetventil
  
  // ADC INIT
  ADMUX = (1<<REFS0); // Referenzspannung Vcc wählen +5V
  ADMUX = (1<<MUX1);  // Kanal 1 PB1 wählen
  ADCSRA = (1<<ADEN); // ADC freigeben / aktivieren
  ADCSRA = (1<<ADSC); // ADC starten
  ADCSRA = (1<<ADPS2);// Prescaler /16 
  ADMUX = (1<<ADLAR); // Ausgabe linksbündig ADCH Register
  
  
    while (1) 
    {
    WERT = ADCH;
    t = WERT*3;
    
    if (PINB & (1<<PB2))
    {
      PORTB |= (1<<PB3);
      _delay_ms(t);
    } 
    else
    {
      PORTB &= ~(1<<PB3);
    }
    }
}

und hier wäre die Fehlermeldung.:

Severity  Code  Description  Project  File  Line
Error    recipe for target 'main.o' failed  Attiny85  C:\Users\USER\Documents\Atmel Studio\7.0\Attiny85\Attiny85\Debug\Makefile  79

Severity  Code  Description  Project  File  Line
Error    __builtin_avr_delay_cycles expects a compile time integer constant  Attiny85  c:\program files (x86)\atmel\studio\7.0\toolchain\avr8\avr8-gnu-toolchain\avr\include\util\delay.h  187

[Uhrmacher (Gast)]

Hier steht er doch, der Fehler:

Michael H. schrieb:
> Error    __builtin_avr_delay_cycles expects a compile time integer
> constant

Will heissen du dafst _delay_ms(..) keine Variable übergeben
sondern nur eine Konstatnte.

[sumo (Gast)]

Der Compiler will eine Integer-Konstante als Argument für delay() haben. 
Du bietest ihm eine Variable an.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Steht doch da, _delay_ms() erwartet eben keine Variable, sondern eine 
Konstante.

Kannst du durch sowas lösen:

void MyDelay(uint16_t t) {
 while(t > 0) {
   _delay_ms(1);
    t--;
 }
return;
}

oder ähnlich.

[Michael H. (h_m)]

kann ich das dann gar nicht so bewerkstelligen, das ich mit dem Poti die 
verzögerungszeit einstelle. Das heißt ich kann nur eine ganze 
unveränderliche Zahl eingeben.

[sumo (Gast)]

Michael H. schrieb:
> kann ich das dann gar nicht so bewerkstelligen, das ich mit dem
> Poti die
> verzögerungszeit einstelle. Das heißt ich kann nur eine ganze
> unveränderliche Zahl eingeben.

Doch, so wie Matthias es vorgeschlagen hat.
Da rufst du delay(1) so oft auf, wie du Millisekunden brauchst.

[Dominik (Gast)]

Michael H. schrieb:
> kann ich das dann gar nicht so bewerkstelligen, das ich mit dem
> Poti die verzögerungszeit einstelle. Das heißt ich kann nur eine ganze
> unveränderliche Zahl eingeben.

Doch, indem Du wie Matthias schrieb ein fixes Delay von z.B. einer ms 
öfter abarbeitest.

Matthias S. schrieb:
> Steht doch da, _delay_ms() erwartet eben keine Variable, sondern
> eine Konstante.
>
> Kannst du durch sowas lösen:void MyDelay(uint16_t t) {
>  while(t > 0) {
>    _delay_ms(1);
>     t--;
>  }
> return;
> }
>
> oder ähnlich.

Braucht tatsächlich etwas länger als die x mal 1 ms, kann hier aber 
sicher vernachlässigt werden.
Der Königsweg wäre ein Hardwaretimer.

Gruß Dominik

[Einer K. (Gast)]

Dominik schrieb:
> Der Königsweg wäre ein Hardwaretimer.

Denn irgendwann wird die Anforderung kommen:
"Ich möchte die Wartezeit für andere sinnvolle Aufgaben nutzen!"
Spätestens dann gehts nicht mehr ohne.

[Curby23523 N. (Gast)]

Grundlagen, Grundlagen, Grundlagen.

_delay_ms(x); ist gut zum testen, wenn man schnell eine fixe Delayzeit 
braucht. Ansonsten IMMER einen Hardware-Timer verwenden. Wie schon 
gesagt, funktioniert diese Funktion nur mit Konstanten.

Z.B. ein Hardware-Timer macht einen Interrupt von 1ms und zählt eine 
uint32_t  Variable hoch. Dann ist das dein Software-Zeitstempel.

Datenblatt runterladen und Kapitel über Timer durchlesen und 
ausprobieren!

[Gordonk (Gast)]

Letztlich kann ein Mikrocontroller ja nur zwei Dinge: etwas berechnen 
und sich an etwas erinnern. Nur kann er das irre schnell. Diesen Vorteil 
der Geschwindigkeit nimmst du ihm mit solchen Warte-Funktionen.

Darum empfehle ich dass du dir Artikel zum Thema Multitasking 
durchliesst. Es gibt sowohl hier auf mikrocontroller.net aber auch auf 
embeddedrelated.com sehr gute Artikel dazu. Timer bringen auch mehr 
Verständnis zu anderen Applikationen.

[Walter S. (avatar)]

Michael H. schrieb:
> DDRB &= ~ (1<<PB2); // auf Eingang Bewegungsmelder
>   DDRB =    (1<<PB3); // auf Ausgang Magnetventil

da könntest du die erste Zeile streichen, es sei denn du wolltest 
eigentlich schreiben
DDRB |=    (1<<PB3); // auf Ausgang Magnetventil

desweiteren musst du nach dem Start der AD-Wandlung warten bis die 
Wandlung erfolgt ist, das machst du nicht!

[Walter S. (avatar)]

da ist noch viel mehr seltsam:

die Reihenfolge Intialisierung/Starten der Wandlung
Überlauf bei der Multiplikation mit 3

Ansonsten sind die Vorschläge mit Multitasking prinzipiell zwar richtig,
aber vergiss sie erst Mal für einige Zeit bis du etwas mehr Überblick 
hast