Frage zu Attiny-Programm

[Tom (Gast)]

Hallo, ich habe keinen Peil von Attinys, deshalb wende ich mich an euch.
Folgendes Programm soll je nach Potistellung einen Ultraschallton 
ausgeben.
Was ich wissen möchte, welcher Tastgrad hat der Ton? 0,5?

//Attiny25 , running @ 1MHZ
// Using timer 1
//
//                           +-\/-+
//  Ain0       (D  5)  PB5  1|    |8   VCC
//  Ain3       (D  3)  PB3  2|    |7   PB2  (D  2)  INT0  Ain1 <- Potmeter input
//  Ain2       (D  4)  PB4  3|    |6   PB1  (D  1)        pwm1 <- Speaker output
//                     GND  4|    |5   PB0  (D  0)        pwm0 <- Led output
//                           +----+

//  calculate output frequency
//  clockspeed / prescaler / OCR1C = frequency(Hz)
//  clockspeed / prescaler / frequency = OCR1C
//  OCR1C = OCR1A

#include <avr/sleep.h>
#include <avr/interrupt.h>
#include <avr/wdt.h>

#define potmeter A1

#define adcDisable() (ADCSRA &= ~(1<<ADEN)) // disable ADC
#define adcEnable()  (ADCSRA |=  (1<<ADEN)) // re-enable ADC
byte compareValue = 0;

void setup() {
  disableWatchdog();
  adcEnable();
  compareValue = map(analogRead(potmeter), 0, 1023, 25, 10);
  startTimer1(); //20 - 50 kHz -> OCR0A: 25 - 10
  adcDisable();
  pinMode(0, OUTPUT);
  digitalWrite(0, HIGH);
  delay(1000);
  digitalWrite(0, LOW);
  stopTimer1();
  enableWatchdog();
  enterSleep();
}

void loop() {
}

void startTimer1(void) {
  pinMode(1, OUTPUT);
  TCNT1 = 0;
  TCCR1 = 0;
  GTCCR |= (1 << PSR1); //section 13.3.2 reset the prescaler
  TCCR1 |= (1 << CTC1); // section 12.3.1 CTC mode
  TCCR1 |= (1 << COM1A0); //togle pin PB1 table 12-4
  TCCR1 |= (1 << CS10); //prescaler 1 table 12-5
  //TCCR1 |= (1 << CS11);
  //TCCR1 |= (1 << CS12);
  //TCCR1 |= (1 << CS13);
  OCR1C = compareValue;
  OCR1A = compareValue;
}

void stopTimer1(void) {
  TCNT1 = 0;
  TCCR1 = 0;
}

void enableWatchdog(void) {
  wdt_enable(WDTO_8S); //enable watchdog 
}

void disableWatchdog(void) {
  MCUSR &= ~(1<<WDRF); // reset status flag
  wdt_disable(); //reset watchdog
}

void enterSleep(void) {
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  sleep_mode();
}

[Stefan F. (Gast)]

c-hater, deine persönlichen Angriffe sind unter aller Kanone. Dagegen 
hilft kein Fachwissen. Dein Auftreten hier ist fast genau so peinlich, 
wie das des Bademeisters, ist dir das nicht bewusst?

[c-hater (Gast)]

c-hater schrieb im Beitrag #6475530:

> Aber du hast es ja nicht
> einmal für nötig gehalten, zu erwähnen, welcher es denn konkret ist.

Verdammt, in der Schaltungs-Skizze taucht doch eine konkreter Tiny auf, 
das hatte ich glatt übersehen. Ein Tiny25.

Also für den ist das Programm nicht nur wahrscheinlich, sondern sogar 
ganz sicher falsch.

Wie's richtig wäre, steht im Datenblatt.

[Stefan F. (Gast)]

c-hater schrieb:
> Verdammt,

Ja genau.

> Also für den ist das Programm nicht nur wahrscheinlich,
> sondern sogar ganz sicher falsch.

Kläre uns auf, anstatt nur zu meckern.

[Stefan F. (Gast)]

Tom schrieb:
> Was ich wissen möchte, welcher Tastgrad hat der Ton? 0,5?

Ja genau. Und das Programm sieht gar nicht schlecht aus. Lass den 
c-hater mal über c haten, den kannst du nicht mehr für voll nehmen. 
Irgendwie hat er sich in seiner Rolle verloren und kommt da nicht mehr 
heraus.

[c-hater hater (Gast)]

Das Programm ist von ELEKTOR
https://issuu.com/meta727/docs/elektor_electronic__usa__09-10_2017
Seite 102/103

Tom hat einfach nur dazu etwas wissen wollen.

Warum C-Hater deswegen hohl dreht?

[batman (Gast)]

Nee, dafür brauchen wir beim Arduino den c++-hater.

[Stefan F. (Gast)]

c-hater schrieb im Beitrag #6475561:
> Wenn du Flachwichser auch irgendwas kompetent zur Sache beigetragen
> hättest, könnte ich den Angriff akzeptieren. Hast du aber nicht.

Du hast schon wieder ein paar Sekunden zu schnell geschossen. Ich habe 
sein Programm geprüft, Zeile für Zeile mit den DB verglichen. Das dauert 
eine Weile. Dabei bin ich zu dem oben genannten Schluss gekommen.

[Stefan F. (Gast)]

c-hater schrieb im Beitrag #6475561:
> Mein Beitrag hingegen könnte ihm u.U. weiterhelfen

Inwiefern?

[pnp (Gast)]

c-hater schrieb im Beitrag #6475561:
> Flachwichser

Der "Hasser" ist mal wieder in Hochform... ;-(

[Schlaumaier (Gast)]

8 Beiträge und nicht eine Antwort die den TO hilft.

Sauber.

@TO

Lerne mal den Unterschied zwischen SETUP und LOOP.  Kleiner Tipp. Setup 
wird EINMAL ausgeführt und nie wieder.

LOOP wird immer ausgeführt es sei den der Code ist in einer Unterroutine 
beschäftigt.

Du schaltet in dein Setup den Timer EIN und danach wieder aus. Ergo, hat 
das Programm 0 Funktion, außer das es mal für genau 1 Sek. zuckt.

Ich würde den Bereich.
  digitalWrite(0, HIGH);
  delay(1000);
  digitalWrite(0, LOW);

in das LOOP stecken.

Dann wird je Sekunden umgeschaltet.

Aber in einen gebe ich den Leuten hier recht. Dein Code ist Mist.

[Thomas E. (thomase)]

pnp schrieb:
> Der "Hasser" ist mal wieder in Hochform...

Ach was. Seine Domina ist im Corona-Lockdown.

[c-hater (Gast)]

Stefan ⛄ F. schrieb:

> Du hast schon wieder ein paar Sekunden zu schnell geschossen. Ich sein
> Programm geprüft, Zeile für Zeile mit den DB verglichen. Das dauert eine
> Weile. Dabei bin ich zu dem oben genannten Schluss gekommen.

Aha, es war dir also bei weitem wichtiger, gegen mich zu feuern, als dem 
TO zu helfen. Das läßt tief blicken...

Im Übrigen hast du fachlich allerdings wirklich Recht. Das Programm 
sieht zwar arduinomäßig Scheiße aus, ist aber tatsächlich korrekt.

[c-hater (Gast)]

Stefan ⛄ F. schrieb:
> c-hater schrieb im Beitrag #6475561:
>> Mein Beitrag hingegen könnte ihm u.U. weiterhelfen
>
> Inwiefern?

Darstellung der Eigenschaften des CTC-Mode...

[c-hater hater (Gast)]

>Dein Code ist Mist.
@Schlaumaier

Mann ... der Code ist von Elektor. Nicht von Tom.

[Schlaumaier (Gast)]

c-hater hater schrieb:
>>Dein Code ist Mist.
> @Schlaumaier
>
> Mann ... der Code ist von Elektor. Nicht von Tom.

Woher soll ich das denn wissen. Trotzdem. Ich würde dann doch gerne mal 
den Original-Code sehen.

Ich würde fast wette das die Befehle :
  digitalWrite(0, HIGH);
  delay(1000);
  digitalWrite(0, LOW);

In die LOOP-Routine gehören.


Aber verrate mir mal woher du das weißt. Habe ich da was übersehen ??!?

[c-hater hater (Gast)]

>Aber verrate mir mal woher du das weißt. Habe ich da was übersehen ??!?
Ja. Scroll mal nach oben.

[Apollo M. (Firma: @home) (majortom)]

c-hater schrieb im Beitrag #6475530:
> Sprich: auf ganzer Linie wegen völliger Unfähigkeit verloren. Sogar die
> Frage im Forum verhauen, weil nichtmal dafür die Kompetenz genügt. Such'
> dir ein anderes Hobby. Briefmarkensammeln z.B...

... könnte von mir sein, alles wird nur noch zugemüllt bzw. das Niveau 
sinkt hier nur!

[Schlaumaier (Gast)]

c-hater hater schrieb:
>>Aber verrate mir mal woher du das weißt. Habe ich da was übersehen ??!?
> Ja. Scroll mal nach oben.

Ok. Hab ich bei der Zankerei wirklich.

Habe mir gerade mal den Original-code angesehen. Die wollen wirklich das 
das Teil wirklich nur einmal Krach macht. Gut für die Ohren. ;)

[Schlaumaier (Gast)]

Aber mir hat das was gebracht. Ich kannte die Seite noch nicht. Danke 
C-Hater dafür.

Ach ich schreibe selbst nicht direkt in Arduino-C weil ich C nicht mag. 
Hab mich mit der Syntax nur nebenbei beschäftigt. Was aber nix an meinen 
Aussagen ändert.

[Stefan F. (Gast)]

Schlaumaier schrieb:
> 8 Beiträge und nicht eine Antwort die den TO hilft.

Wieso? Seine Frage wurde von zwei Leuten klar beantwortet.

[Stefan F. (Gast)]

Schlaumaier schrieb:
> Habe mir gerade mal den Original-code angesehen. Die wollen wirklich das
> das Teil wirklich nur einmal Krach macht.

Schlaumaier schrieb:
> Ich würde fast wette das die Befehle :
>   digitalWrite(0, HIGH);
>   delay(1000);
>   digitalWrite(0, LOW);
>
> In die LOOP-Routine gehören.

> Habe ich da was übersehen ??!?

Das Ding soll alle 10 Sekunden "Piep" machen.

Der Watchdog Timer löst alle 10 Sekunden einen Reset aus. Dann soll das 
Ding jeweils eine Sekunde lang einen Ton von sich geben. Dafür ist das 
delay(). Danach legt er sich schlafen, um Strom zu sparen.... bis der 
Watchdog ihn in 9s wieder weckt.

Der Quelltext ist einwandfrei. Die haben von Arduino verwendet, was da 
ist, und die fehlenden Features selbst "zu Fuß" implementiert. Das ist 
völlig Ok. Man muss nicht für jeden Furz eine Library verwenden.

[Schlaumaier (Gast)]

Stefan ⛄ F. schrieb:
> Man muss nicht für jeden Furz eine Library verwenden.

Aber die die drin ist habe ich übersehen.
#include <avr/sleep.h>  <- die da.

Ich muss wirklich mich mal mehr mit C-Codes rumärgern. ;) Ist echt nicht 
gut wenn man ein immer ein anders Programm die schreiben lässt. ;(

[Stefan F. (Gast)]

c-hater schrieb:
>>> Mein Beitrag hingegen könnte ihm u.U. weiterhelfen
>> Inwiefern?
> Darstellung der Eigenschaften des CTC-Mode...

Bei mir ist nur aggressives Kläffen angekommen.

[Stefan F. (Gast)]

Schlaumaier schrieb:
> Aber mir hat das was gebracht. Ich kannte die Seite noch nicht. Danke
> C-Hater dafür.

Danke nicht c-hater, sondern c-hater hater.

[Schlaumaier (Gast)]

Stefan ⛄ F. schrieb:
> Danke nicht c-hater, sondern c-hater hater.

och nö.

Ich glaub zwar nicht das er mich mag, aber was Recht ist muss Recht 
bleiben. Und bei mir gilt halt. Wen dir einer etwas guter Tut, sag 
DANKE.

[pnp (Gast)]

Schlaumaier schrieb:
> Stefan ⛄ F. schrieb:
>> Danke nicht c-hater, sondern c-hater hater.
>
> och nö.
>
> Ich glaub zwar nicht das er mich mag, aber was Recht ist muss Recht
> bleiben. Und bei mir gilt halt. Wen dir einer etwas guter Tut, sag
> DANKE.

Warum dankst du nicht dem, der den Link schrieb?
Das war "c-hater hater" und nicht der andere ohne Manieren...

[Dieter D. (Firma: Hobbytheoretiker) (dieter_1234)]

Stefan ⛄ F. schrieb:
> Das Ding soll alle 10 Sekunden "Piep" machen.

Well Done. Die Details von Dir danach hätte eigentlich der TO liefern 
muessen.

c-hater schrieb im Beitrag #6475530:
> Du hast nicht nur keine Ahnung von ATtinys,
...
> Such' dir ein anderes Hobby. Briefmarkensammeln z.B...

Einen solchen Beitrag als erstes Feedback sieht sehr cholerische aus.
Jeder backt erst einmal kleine Brötchen und stolpert über Grundlagen, 
wenn nicht im engeren Kreis von Familie, Freunden oder Bekannten jemand 
dabei ist, von dem erstmal vieles abzuschauen war.

[Schlaumaier (Gast)]

pnp schrieb:
> Warum dankst du nicht dem, der den Link schrieb?
> Das war "c-hater hater" und nicht der andere ohne Manieren...

Weil ich beide Verwechselt habe und deinen Text Fehlinterpretiert habe.

Ihr macht mich mit den Mist ganz kirre.

Aber du hast Recht.

DANKE den "c-hater hater" und nicht den anderen.

[Stefan F. (Gast)]

Dieter D. schrieb:
> Die Details von Dir danach hätte eigentlich der TO liefern
> muessen.

Warum, die waren doch für seine Frage irrelevant.

[HildeK (Gast)]

Stefan ⛄ F. schrieb:
> Der Watchdog Timer löst alle 10 Sekunden einen Reset aus. Dann soll das
> Ding jeweils eine Sekunde lang einen Ton von sich geben. Dafür ist das
> delay(). Danach legt er sich schlafen, um Strom zu sparen.... bis der
> Watchdog ihn in 9s wieder weckt.

Fast. WD ist auf 8s eingestellt = 8s Schlaf, 1s macht er den Ton, also 
Wiederholung alle 9s. Ist aber marginal ...
Ansonsten richtig analysiert.