I have one last detail / complication to understand before my project is finished. The servo Part works when it is a stand alone / all by itself when it is on the UNO, but red lines at line 137. Since it is a servo signal it has to be PWM. Ideas on a way to overcome this challenge will be very welcome. It is the very last piece of my puzzle, Thanks in advance. // ATtiny85 torch detector // Author: Nick Gammon // Date: 25 February 2015 // ATMEL ATTINY 25/45/85 / ARDUINO // Pin 1 is /RESET // // +-\/-+ // Ain0 (D 5) PB5 1| |8 Vcc // Ain3 (D 3) PB3 2| |7 PB2 (D 2) Ain1 // Ain2 (D 4) PB4 3| |6 PB1 (D 1) pwm1 // GND 4| |5 PB0 (D 0) pwm0 // +----+ /* Pin 2 (PB3) Pin 7 (PB2) 56 k Gnd Pin 5 (PB0) 100 R Gnd */ #include // Sleep Modes #include // Power management #include // Watchdog timer #include Servo servo1; int servoPin = 9; const byte POWER_PIR = 7; const byte PIR_SIGNAL = 12; int val = 0; const byte LED = 13; const byte LDR_ENABLE = 8; const byte LDR_READ = A0; const int LIGHT_THRESHOLD = 200; // Flash LED when darker than this // when ADC completed, take an interrupt EMPTY_INTERRUPT (ADC_vect); float getReading (byte port) // Take an ADC reading in sleep mode (ADC) { power_adc_enable() ; ADCSRA = bit (ADEN) | bit (ADIF); // enable ADC, turn off any pending interrupt // set a2d prescale factor to 128 // 8 MHz / 128 = 62.5 KHz, inside the desired 50-200 KHz range. ADCSRA |= bit (ADPS0) | bit (ADPS1) | bit (ADPS2); if (port >= A0) port -= A0; #if defined(__AVR_ATmega328P__) ADMUX = bit (REFS0) | (port & 0x07); // AVcc #endif noInterrupts (); set_sleep_mode (SLEEP_MODE_ADC); // sleep during sample sleep_enable(); // start the conversion ADCSRA |= bit (ADSC) | bit (ADIE); interrupts (); sleep_cpu (); sleep_disable (); // reading should be done, but better make sure // maybe the timer interrupt fired // ADSC is cleared when the conversion finishes while (bit_is_set (ADCSRA, ADSC)) { } byte low = ADCL; byte high = ADCH; ADCSRA = 0; // disable ADC power_adc_disable(); return (high << 8) | low; } // end of getReading // watchdog interrupt ISR (WDT_vect) { wdt_disable(); // disable watchdog } // end of WDT_vect #if defined(__AVR_ATmega328P__) #define watchdogRegister WDTCSR //#else // #define watchdogRegister WDTCSR #endif void setup () { wdt_reset(); pinMode (POWER_PIR, OUTPUT); pinMode (LED, OUTPUT); servo1.attach(servoPin); pinMode (LDR_ENABLE, OUTPUT); pinMode (PIR_SIGNAL, INPUT); ADCSRA = 0; // turn off ADC power_all_disable (); // power off ADC, Timer 0 and 1, serial interface } // end of setup void loop () { // power up the LDR, take a reading digitalWrite (LDR_ENABLE, HIGH); int value = getReading (LDR_READ); // power off the LDR digitalWrite (LDR_ENABLE, LOW); if (value < LIGHT_THRESHOLD) // if it's dark, do stuff { power_timer0_enable (); delay (1); // let timer reach a known point digitalWrite (POWER_PIR, HIGH); } else digitalWrite (POWER_PIR, LOW); { val = digitalRead (PIR_SIGNAL); // read the input pin digitalWrite (LED,val); // sets the LED to the button's value val = digitalRead (PIR_SIGNAL); // read the input pin void analogWrite(uint8_t, int) servo1.write(0); //degrees delay(50); // delay between moves servo1.write(10); delay(50); servo1.write(0); delay(50); servo1.write(10); delay(50); } { power_timer0_disable (); } goToSleep (); } // end of loop void goToSleep () { set_sleep_mode (SLEEP_MODE_PWR_DOWN); noInterrupts (); // timed sequence coming up // pat the dog wdt_reset(); // clear various "reset" flags MCUSR = 0; // allow changes, disable reset, clear existing interrupt watchdogRegister = bit (WDCE) | bit (WDE) | bit (WDIF); // set interrupt mode and an interval (WDE must be changed from 1 to 0 here) watchdogRegister = bit (WDIE) | bit (WDP2) | bit (WDP1) | bit (WDP0); // set WDIE, and 2 seconds delay sleep_enable (); // ready to sleep interrupts (); // interrupts are required now sleep_cpu (); // sleep sleep_disable (); // precaution } // end of goToSleep

[image] Fredric58: Perhaps you can explain why it doesn't work? Thanks, Fred Compare your code with #27. You forgot the curly brackets around the servo movement statements.

On Bobs advice...... Starting at the bottom. Please look at this simple code. There's a strange anomaly. if I set val = 1 the servo moves from 0 to 10 at about half speed until you wave your hand in front of the PIR then the servo runs from 0 to 10 at full speed. If I set val = 0 it does the opposite. It shouldn't be doing anything unless you wave your hand in front of the PIR. Note: the PIR has an adjustable timer for the signal, mine is set to roughly 18 seconds and then it is supposed to go low. To clarify it runs all the time at 2 different speeds depending on the state of the PIR....... Kind of odd. #include int POWER_PIR = 7; int PIR_SIGNAL = 12; Servo servo1; int servoPin = 9; int val = 0; void setup() { pinMode (POWER_PIR, OUTPUT); pinMode (PIR_SIGNAL, INPUT); servo1.attach(servoPin); } void loop() { digitalWrite (POWER_PIR, HIGH); val = digitalRead (PIR_SIGNAL); // read the input pin if ( val ) servo1.write(0); delay(50); servo1.write(10); delay(50); servo1.write(0); delay(50); servo1.write(10); delay(50); }

You really haven't got the hang of testing a variable and acting on it state Look at you loop() function after Auto Formatting it in the IDE void loop() { digitalWrite(POWER_PIR, HIGH); val = digitalRead(PIR_SIGNAL); // read the input pin if (val) servo1.write(0); delay(50); servo1.write(10); delay(50); servo1.write(0); delay(50); servo1.write(10); delay(50); } The only line of code that depends on the value of the val variable is the first servo1.write(). Subsequent commands are executed unconditionally. This is not the first time that this has been pointed put to you Put the dependant code block in braces like this void loop() { digitalWrite(POWER_PIR, HIGH); val = digitalRead(PIR_SIGNAL); // read the input pin if (val) { servo1.write(0); delay(50); servo1.write(10); delay(50); servo1.write(0); delay(50); servo1.write(10); delay(50); } }

That fixed the whole damn thing!

Bob, did you look at the power_down code without using these 3 libraries? #include // Sleep Modes #include // Power management #include // Watchdog timer Which would you consider the more advantageous route? And I meant to ask, since this goes to sleep is it really necessary to do and else on the POWER_PIR. I don't think it would be because once it wakes up, everything shuts down??? { // power up the LDR, take a reading digitalWrite (LDR_ENABLE, HIGH); int value = getReading (LDR_READ); // power off the LDR digitalWrite (LDR_ENABLE, LOW); if (value < LIGHT_THRESHOLD) // if it's dark, do stuff { power_timer0_enable (); delay (1); // let timer reach a known point power_timer1_enable(); delay (1); digitalWrite (POWER_PIR, HIGH); } else digitalWrite (POWER_PIR, LOW);

[image] Fredric58: Which would you consider the more advantageous route? Sorry, but I know little or nothing about such things so cannot offer an opinion

just curious, is it possible to fly a helli upside down?

Yes, perfectly possible when flung a collective pitch heli [NOOB RC Heli Pilot Flies Upside Down with AWESOME AutoPilot Mode!!!]

Expected initializer before 'servo1'

Other Hardware Robotics

Fredric58 July 15, 2023, 10:18pm 68

Using millis() for timing. A beginners guide

Just stumbled across this..............Well presented. Thanks for putting it together.

Topic		Replies	Views
Servo "zuckt"/macht Geräusche im Leerlauf Deutsch	48	928	October 18, 2026
Robot esapode con molti servi ; software Software	199	3430	December 15, 2024
Attiny85 sleep, interrupt with servo Programming	83	1451	August 27, 2025
Problem to drive a servo motor with Arduino uno General Guidance	39	884	January 1, 2026
Bei Signal Eingang, eine LED für 2 Sekunden Leuchten lassen Deutsch	33	515	August 8, 2024

Expected initializer before 'servo1'

Using millis() for timing. A beginners guide

Related topics