AVR ATTiny13A Problem mit Servo + LED-Ansteuerung

Bitte beachten, was über jeder Texteingabebox hier steht:

1

Antwort schreiben

2

Wichtige Regeln - erst lesen, dann posten!

3

4

  *  ...

5

  *  Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

Mein Vorschlag: häng den Code an den nächsten Post einfach als ino Datei an und ich lösche den oben raus.

Kai W. schrieb:

Trotzdem habe ich den Verdacht, dass beim Takt/Teiler etc. ggf. etaws nicht stimmt.

Passt die clkdiv8 Fuse?

Kai W. schrieb:

ich hoffe ihr könnt mir weiterhelfen, da ich nach vielen vergeblichen Teststunden nicht mehr weiterweiß.

Variablen, die in einem Interrupthandler und in main() genutzt werden, sollten als volatile deklariert werden.

Grüßle, Volker

90% RAM Auslastung sind u.U. zu viel. Der Compiler kann Heap/Stack nicht korrekt berücksichtigen. Definiere doch die zeitWertesekundaer als Progmem und passe die Zugriffe entsprechend an. Das gibt etwas Luft.

Hallo Kai,

Kai W. schrieb:

const uint8_t zeitWerteSekundaer[50] = {1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2, 1, 2};

Die Daten für die Konstante werden in den RAM geschaufelt, lege sie in den Flash, dann hast du fast den gesamten RAM frei.

Schau mal hier rein:
https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Flash_mit_PROGMEM_und_pgm_read

Etwas weiter unten gibt es auch noch die Beschreibung von "__flash", ist IMO einfacher in der Anwendung, nutze ich häufig für Konstanten.

Reinhard

Vielen Dank für eure zahlreichen Antworten! Der Reihe nach:

Da ich nicht weiß, wie ich mit dem Microchip Studio ino-Dateien erzeuge, habe ich mal die main.c angehängt. Sorry, ich dachte, dass mein MiniProg nicht in die Kategorie "lang" fällt.

Bzgl. des "clkdiv8 Fuse": Ich habe im ersten Post ein Bild der Fuses angehängt - der LOW.CKDIV8 ist gesetzt oder was ist gemeint?

volatile: Alle im Interrupt und main verwendeten Variablen sind jetzt als "volatile" deklariert.

RAM/Flash: Vielen Dank für den Hinweis - ich habe versucht ihn umzusetzen. Wäre nett, wenn jemand meine Umsetzung kontrollieren würde.

Ich habe das in der main01.c angehängte Programm auf dem myC getestet und bin etwas ratlos, da:

• die LEDs flackern nicht mehr, bleiben aber sehr lange an bzw. aus, z.T. gehen sie nur kurz an und direkt wieder aus (was ungefähr der vorgesehenen 1 Sek. entsprechen könnte).
• Immerhin Durchläuft der Servo wie geplant einen Links-Rechts-Zyklus, wenn die LED 3 angeht.

MfG Kai

Kleinvieh macht auch Mist... warum nicht so? Ist kürzer und schneller.

uint8_t zufallsZeitWert(){ //gibt Zufallswert für Zeit zurück laufZeit++; if (laufZeit > 49) {laufZeit = 0;} return pgm_read_byte (zeitWerteSekundaer[laufZeit]);

Ansonsten scheint die Funktion des Programms ja deutlich näher am Soll zu liegen. Das deutet darauf hin, dass der RAM Verbrauch tatsächlich zu hoch war.

Kai W. schrieb:

Bzgl. des "clkdiv8 Fuse": Ich habe im ersten Post ein Bild der Fuses angehängt - der LOW.CKDIV8 ist gesetzt oder was ist gemeint?

Tja, wenn ich jetzt wüsste, was diese Oberfläche meint? Ich würde das so interpretieren, dass CKDIV8 aktiv ist, also auf 0 gesetzt wird. Damit wird der Takt der MCU, also auch der Takt, der deren Timer speist, durch 8 dividiert.

Grüßle, Volker

Steve van de Grens schrieb:

Volker B. schrieb:

Tja, wenn ich jetzt wüsste, was diese Oberfläche meint?

Sie meint die Fuses im default Zustand, also 1,2 MHz Taktfrequenz.

Die hexadezimale Darstellung unten drunter ist unmissverständlich.

...wenn man die ausgewählte MCU kennen würde...

Ah, ok, steht im Titel. Korrigiere:

...wenn man nicht zu faul wäre, das Datenblatt zu suchen... :-)

Grüßle, Volker

Hallo,

geh nochmal einen großen Schritt zurück. Laut deiner Konfig nutzt du den internen 9,6MHz RC. Zusätzlich ist CKDIV8 Fuse aktiv. Das heißt dein µC taktet mit 1,2MHz.

Teste einmal ob die 2 Pins sauber takten? Ungetestet, kompiliert jedoch.

1

#define F_CPU 1200000UL  // 9,6MHz / 8

2

#include <avr/io.h>

3

#include <stdlib.h>

4

#include <avr/interrupt.h>

5

#include <avr/pgmspace.h>

6

7

ISR(TIM0_OVF_vect)      // 292 Hz ???

8

{

9

    PINB = _BV(PB1);    // Toggle

10

}

11

12

ISR(TIM0_COMPA_vect)

13

{

14

    PINB = _BV(PB2); 

15

}

16

17

int main(void)

18

{

19

    DDRB    = _BV(PB1); 

20

    DDRB   |= _BV(PB2); 

21

    TCCR0A  = _BV(WGM01) | _BV(WGM00);  // 8Bit Fast-PWM

22

    TIMSK0  = _BV(TOIE0);   // OVF

23

    TIMSK0 |= _BV(OCIE0A);  // CompareA

24

    OCR0A   = 127;

25

    TCCR0B  = _BV(CS01);    // Timer0 starten, Prescaler 8

26

    sei();   

27

28

    while (1) 

29

    { }

30

}

31

32

/*

33

  1 Takt = 833ns

34

  8*256 = 2048 Takte * 833ns = 1,706ms // ein OVF Aufruf

35

*/

Danach überlegst du dir nochmal wie du den Timer konfigurieren musst um vernünftig ein Servosignal zwischen 1-2ms zu erzeugen und dabei eine Periodendauer von 20ms einzuhalten. Kannst auch auf 10ms Periodendauer runtergehen. Das kommt der Auflösung zu Gute bei der Rechnerei.

Vielen Dank für Eure Antworten!

Da nun wieder genug RAM zur Verfügung steht, habe ich den LED-Zustand mit vier bool-Variablen erfasst, anstatt ihn jedes mal abzufragen. Für meinen einfachen Fall genügt aber, denke ich, die einfache if-else-Konstruktion oder würdet ihr dringend zu der switch-case-Variante raten? Aber danke für den Hinweis auf den Zustandsautomaten, werde ich mir merken.

Die Optimierung von Georg G. an der zufallsZeitWert()-Funktion habe ich übernommen.

Danke für die Hinweise bzgl. CKDIV8 - das hatte ich nicht berücksichtigt. Ich habe Dein Programm, Veit D., am MCU getestet. Heraus kam eine Frequenz von 275 Hz (Signallänge an PB1 bzw. PB2: 1,820ms). Zugegebener Maßen, verstehe ich das Programm mit den "_BV()"-Ausdrücken nicht so ganz und hätte eigentlich eher unterschiedliche Frequenzen an den beiden Pins erwartet... Wie auf den Bildern zu sehen ist, trat auch eine wiederkehrende Störung auf... diese ist mir aber beim aktuellen Programm nicht mehr begegnet.

Nach ein wenig herumrechnen, habe ich mich dann für die folgende Variante entschieden: CKDIV8 deaktiviert (also 9,6 MHz) und den Timer-Prescaler auf 256. Das ergibt dann 1ms: 37 Takte (1,072ms gemessen), 2ms: 70 Takte (nachgemessen & korrigiert => 2,0ms) und einen Zyklus von 21,6ms(gemessen)

Servo und LEDs werden nun korrekt angesteuert und funktionieren wie erwartet... zumindest wenn ich nicht die zufallsZeitWert()-Funktion verwende. Wenn ich feste Werte vorgebe, werden diese korrekt umgesetzt. Die LEDs, die ihre Zeitwerte über die Funktion beziehen ändern ihren Zustand gar nicht oder nur sehr selten. Hat jemand eine Idee, voran das liegen könnte? Ich habe den aktualisierten Code angehängt.

MfG Kai

bei "pgm_read_byte" fehlt ein "&". Ist zwar kein Arduino, aber siehe auch hier https://stackoverflow.com/questions/31100771/accessing-individual-bytes-in-progmem-on-arduino-avr oder hier steht das auch: https://teslabs.com/openplayer/docs/docs/prognotes/Progmem%20Tutorial.pdf seite 5 Du willst ja nicht die Adresse wissen, sondern den Wert, der sich dort verbirgt.

Hallo,

_BV() ist ein Makro. avr/sfr_defs.h> void _BV(bit_number); Andere Schreibweise statt Bit << schubsen.

Das beide Pins den gleichen Takt haben stimmt. Da hatte ich einen Denkfehler. Den OCRA Pin müßte der Timer selbst schalten damit die Pulsweite wie Anfangs gedacht verwendet wird. Dann hätte er den doppelten Takt. Ist jetzt auch egal.

Übrigens, das Oszi hat links eine Tastenreihe. Damit kann man im Bildschirm unten Meßwerte einblenden lassen. Zwecks deiner Frequenz und Duty Messung.

Mit Störungen meinst du sicherlich das "Rauschen" auf dem Low/High-Pegel? Das werden Störungen durch schlechte Kontakte/Masse sein. Steckbrett usw.

Hallo,

habe mich mal rangemacht erstmal den Code aufzuräumen. Damit steigen die Chancen das jemand durchblickt. Und du auch um den Fehler zu finden, weil die Code Dopplung entfällt.

Dafür müßtest du das aber als Cpp Projekt neu anlegen und wenigstens unter Projekt > Properties >

etwas oberhalb > Configuration: > All Configuration

links Tooolchain:

AVR/GNU C++ Compiler >

Optimization: Level auf (-Os) setzen

C++ Miscellaneous: "other flags" auf -std=c++11 einstellen, Haken bei verbose kannste auch setzen

C kann leider nicht mit der Struktur "struct Auge" umgehen. Du kannst weiterhin C programmieren, abgesehen von der Struktur. Wenn das erstmal wie bisher funktioniert kann man weiter schauen ...

Vielen Dank für Eure Antworten und Hinweise! Und vielen Dank Veit D., dass Du Dir die Mühe gemacht hast, das Programm umzuschreiben. Dadurch hast Du für andere User sicherlich ein besseres Beispielprogramm erstellt, als mein Anfängerproggi. Zugegeben, mit struct kenne ich mich noch nicht aus, aber ich sehe ein, dass es in diesem Anwendungsfall wohl Sinn macht. Ich werde Eure zahlreichen Hinweise bei den nächsten Projekten versuchen zu berücksichtigen - vielen Dank dafür!

Nach der Ergänzung des fehlenden Adressoperators beim Zugriff auf das Array im Flash funktioniert das Programm nun wie erwartet, womit meine Frage beantwortet wäre. Ich hänge das endgültige Prog. noch an, falls noch anderen Anfängern die struct-Variante zu kompliziert ist.

Abschließend noch der Sinn des Programms: Wer mag, kann mit seinen Kindern zu Halloween Tischtennisbälle als Augen bemalen (Edding, Alufolie als Maske, Klarsichtfolie als Feuchteschutz), mit LED versehen und diese dann in einen kugeligen Busch im Vorgarten platzieren. Je ein Augenpaar wird dann zufällig an-/ausgeschaltet. Der Servo soll dann einen Ast schütteln.

Vielen Dank und viele Grüße Kai

Hallo,

Hauptsache ist das es funktioniert. :-) Je mehr du programmierst und je mehr Codedopplungen man hat, umso eher kommt man dann ins Themengebiet Arrays, Strukturen usw. Das ergibt sich dann fließend mit der Zeit. Aber schön ist das du eine Lösung hast.