comlet rework

main.c

@@ -1,7 +1,13 @@
-#define F_CPU 1000000UL
+//#pragma GCC push_options
+//#pragma GCC optimize ("O0")
+#define F_CPU 8000000UL
 #include <util/delay.h>
 #include <avr/io.h>
 #include <avr/interrupt.h>
+#include <avr/power.h>
+
+
+
 
 #define PATTERN_COUNT 4
 #define attiny45
@@ -13,16 +19,18 @@ typedef struct
 	int time_hold;
 }rgbval;
 volatile uint16_t ms;
-volatile int  counts;
+volatile uint16_t  counts;
-
-
 
+void InitTimer0(void);
+void InitTimer1(void);
+void setrgb(unsigned int r,unsigned int g, unsigned int b);
+void InitInterrupts(void);
 
 void count(void)
 {
 	//counts=0;//0..65535 
 	counts++;
-	if(counts>=128)	//1m	//1ms
+	if(counts>3)	//1m	//1ms
 	{
 			counts=0;
 			if(ms<60000)//1 minute
@@ -34,25 +42,21 @@ void count(void)
 					ms=0;
 			}
 	}
-
 }
 ISR(TIMER0_OVF_vect)
 {
 		count();
 }
 
-
-
 int main (void) {            // (2)
 
-	rgbval pattern[6][PATTERN_COUNT]={	{{0,0,0,10},		{0,0,0,10},		{0,0,0,10},		{0,0,0,10}},
+	rgbval pattern[6][PATTERN_COUNT]={	{{0,0,0,100},		{0,0,0,100},		{0,0,0,100},		{0,0,0,100}},
 										{{1,1,1,10},		{1,1,153,20}, 	{200,50,39,10}, 	{204,255,153,20}},
 										{{1,1,1,10},		{10,255,0,10},	{1,1,255,20},	{1,200,1,20}},
 										{{1,1,5,10},		{1,1,255,10},	{1,1,5,10},		{1,1,255,10}},
 										{{252,102,3,10},	{161,67,47,10},	{118,120,74,10},	{83,120,74,20}},
 										{{87,119,122,10}, {95,87,122,10},	{52, 69, 56,10},	{83, 84, 72,10}}};
-	//rgbval *pattern=pattern1[0];
+	int i=0,i2=0,i3=0;
-	int i=0,i2=0,i3;
 	int r,g,b = 0;
 	int set=0;
 	int input=0;
@@ -61,102 +65,123 @@ int main (void) {            // (2)
 	counts = 0;
 	 
 //disable prescaler	
-	CLKPR = (1<<CLKPR);
+//	CLKPR = (1<<CLKPR); maybe optimization removes this
-	 CLKPR = 0x00;
+//	CLKPR = 0x00;
+	clock_prescale_set(clock_div_1);
 //#######OC0A auf ausgang
-	DDRB = (1 << PB0 )|(1<<PB1)|(1<<PB4);
+	DDRB = (1 << PB0 )|(1<<PB1)|(1<<PB4);//data directions - output
+	PORTB |= (1<<PB2);//pullups
 	cli();
-//#######Timer0-setup#####			|Phase correct pwm
-	TCCR0A = (1<<COM0A1)|(1<<COM0B1)|(1<<WGM00)|(1<<WGM01);	//mode fast pwm (1/Fcpu)*prescaler*(max+1) = 
-	TCCR0B = (1<<CS01)|(1<<CS01);				//prescaler /64 gives 0...255 0.002048s = 2.0ms 1mhz-> /8 
 
-//	PLLCSR |= (1 << PLLE) | (1 << PCKE);
+	InitTimer0();
-//######Timer1-setup
+	InitTimer1();
-
+	//setrgb(100,100,100);
-	TCCR1 = (1<<COM1A0)|(1<<COM1A1)|(1<<CTC1)|(1<<CS12)|(1<<CS10);//|(1<<CS11)|(1<<CS13);	//frequencyi //iinverting PWM because 0 dtc not possible
+ 	InitInterrupts();
-	GTCCR = (1<<PWM1B)|(1<<COM1B0);				//mode + overrun clear, Set on compare match
+	sei();
-	OCR1C = 255;						// clear timer at 255	
-//######PWM_VALUE$
-	//timer 0
-	OCR0A = 1;
-	OCR0B = 1;
-	//timer 1
-	OCR1B = 1;
-//######Configure Pullups
-	PORTB |= (1<<PB2);
-
-	//enable global interrupts
-		  //Interupts
-	TIMSK |= (1<<TOIE0);//interrupt on overflow = one timer cycle
-	TIFR |= (1<<TOV0);
-	TCNT0 = 0;
-sei();
    while(1)
    {
-
+	   for(i=0;i<PATTERN_COUNT;i++)
-	   for(i=1;i<PATTERN_COUNT;i++)
 	   {
 		r = pattern[i2][i].r;
 		g = pattern[i2][i].g;
 		b = pattern[i2][i].b;
-		if(i==(PATTERN_COUNT-1)) {i= -1;}
+		if(i==(PATTERN_COUNT-1)) {i= 0;}
 		//reset timer counter	
 		while(1)
 		{
-				//input handling
+			input=PINB;
-				input=PINB;
 			   if(((input & (1<<PB2)) == 0 && set==0))
-				{
+			   {
-						set=1;
+					   set=1;
 						i=0;
 						i2++;
 						if(i2==6)
 						{
 								i2=0;
 						}
-						//cycle through and stop pwm at value 0
+//Visual feedback "mode change"
-
-						//Visual feedback "mode change"
 						for(i3=0;i3<i2;i3++)
-						{	
+						{
-									OCR0A=1;OCR0B=1;OCR1B=1;	
+								setrgb(0,0,0);			
-									_delay_ms(30);
+								pre_delay = ms;		
-									OCR0A=255;OCR0B=255;OCR1B=255;
+								while((ms-pre_delay) < 200);
-									_delay_ms(30);
+								setrgb(200,0,0);
+								pre_delay = ms;		
+								while((ms-pre_delay) < 200){};
 						}
-				}
+				}//end input handling
 			   else if(((input & (1<<PB2)) != 0))
-			   {
+			   	{
 					set=0;
-			   
 				}
-
+//iterate throught pattern to the next ones
-			   //Pattern handling
-			  
-
-								
-						
 				if(r < pattern[i2][i+1].r) {r++;}
 				if(r > pattern[i2][i+1].r) {r--;}	
 				if(g < pattern[i2][i+1].g) {g++;}
 				if(g > pattern[i2][i+1].g) {g--;}	
 				if(b < pattern[i2][i+1].b) {b++;}
 				if(b > pattern[i2][i+1].b) {b--;}	
-						
+				setrgb(r,g,b);		
-				OCR0A = g;//green
+					//delay_ms(10);	
-				OCR0B = r;//red
+//read timer evaluate time since read in 1ms steps
-				OCR1B = b;//blue
+			//	pre_delay = ms;		
-			//	:wq
+			//	while((ms-pre_delay) <1000);
-			//	_delay_ms(10);	
+				_delay_ms(30);
-				pre_delay = ms;		
+	  			if(r == pattern[i2][i+1].r && g == pattern[i2][i+1].g && b == pattern[i2][i+1].b) break;
-			//	:wq
-			while((ms-pre_delay) < pattern[i2][i+1].time_hold){};
-	  	if(r == pattern[i2][i+1].r && g == pattern[i2][i+1].g && b == pattern[i2][i+1].b) break;
 		}
+
+			//	pre_delay = ms;		
+			//	while((ms-pre_delay) < 10);
 	   }
 
+			/*	pre_delay = ms;		
+					setrgb(255,0,0);
+					while((ms-pre_delay) < 100);
+
+				pre_delay = ms;		
+					setrgb(0,255,0);
+					
+					while((ms-pre_delay) < 100);
+				pre_delay = ms;		
+					setrgb(0,0,0);
+					while((ms-pre_delay) < 100);*/
+
+
    } 
    
+
+
+
    return 0;                 
 }
 
+void InitTimer0(void)
+{
+//#######Timer0-setup#####			|Phase correct pwm
+	TCCR0A = (2<<COM0A0)|(2<<COM0B0)|(1<<WGM00)|(1<<WGM01);	//mode fast pwm (1/Fcpu)*prescaler*(max+1) = 
+			//prescaler /64 gives 0...255 0.002048s = 2.0ms 1mhz-> /8 
+	TCCR0B = (1<<CS01);//|(1<<CS01)/256;
+}	
+void InitTimer1(void)
+{
+//######Timer1-setup
+	PLLCSR &= ~(1<<PCKE);
+	TCCR1 = (1<<COM1A0)|(1<<COM1A1)|(1<<CS12)|(1<<CS10)|(1<<CS11);//(1<<CS10)|(1<<CS12);//|(1<<CS13);	//frequencyi //iinverting PWM because 0 dtc not possible
+	GTCCR = (1<<PWM1B)|(1<<COM1B1);	//mode + overrun clear, Set on compare match
+	OCR1C = 255; // clear timer at 255	
+}
+void setrgb(unsigned int r,unsigned int g,unsigned int b)
+{
+				//depends on timer0
+		OCR0A = g;//green
+		OCR0B = r;//red
+				  //depends on Timer1
+		OCR1B = b;//blue
+}
+void InitInterrupts(void)
+{
+		TIMSK |= (1<<TOIE0);//interrupt on overflow = one timer cycle
+		TIFR |= (1<<TOV0);
+		TCNT0 = 0;
+}
+//#pragma GCC pop_options

makefile

@@ -10,7 +10,7 @@ FORMAT = ihex
 TARGET = main
 SRC = $(TARGET).c
 ASRC = 
-OPT = s
+OPT = 0
 
 # Name of this Makefile (used for "make depend").
 MAKEFILE = Makefile