Brain-Lag die 30ste

[Xilence (Gast)]

Hi,
ich bastel grad für nen Freund den Code für nen RC-Switch zusammen
Basiert auf nem Tiny85 (hatte ich halt gerade da)
Pinbelegung ist vorgegeben auf Impuls -> PB1 aka. PCINT1
Ausgang ist PB4
war ursprünglich ein ON/OFF switch mit nem Tiny13 (deswegen auch PCINT1, 
da ist beim Tiny13 ja INT0)
wie auch immer

komm aber auf Teufel komm raus mit dem Code nicht weiter
mein hirn sagt mir geht - mein Testaufbau 
(funke+empfänger+testverbraucher in form einer led) sagt geht nicht
(hardware ist ok - wenn ich den pin auf high setze ist die led auch an)

/*
 * _1Ch_RC_Switch.c
 *
 * Created: 24.04.2012 17:14:30
 *  Author: Xilence
 */ 

#include <stdlib.h>
#include <avr/io.h>
#include <inttypes.h>
#include <avr/interrupt.h>
#ifndef F_CPU
#define F_CPU 8000000UL
#endif

#ifndef cbi
#define cbi(sfr, bit) (_SFR_BYTE(sfr) &= ~_BV(bit))
#endif
#ifndef sbi
#define sbi(sfr, bit) (_SFR_BYTE(sfr) |= _BV(bit))
#endif

// PB4 Output for PWM
// PB1 Input from Receiver 


// Represents the Timer1
volatile uint8_t valueFromTimer;
// Represents the PWM-Value
volatile uint8_t valueForPWM;

int main(void){
  
  valueForPWM = 0;
  valueFromTimer = 0;
  
  // set OUT_PIN as Output
  sbi(DDRB,PB4);
  PORTB = 0;
  
  // Timer0 - with 1024 Prescaler
  sbi(TCCR0B,CS02); sbi(TCCR0B,CS00);
  
  // Overflow Interrupt enable
  sbi(TIMSK,TOIE0); sbi(TIMSK,TOIE1);
  
  // Set both Timers to 0
  TCNT1 = 0;  TCNT0 = 0;
   
   // activate PinChange-Interrupt
  sbi(GIMSK,PCIE);    
  // Interrupt at PCINT1
  sbi(PCMSK,PCINT1);
   
//Interrupts on
   sei();      

  while (1){
  }
}

// for high and low pwm-cutoff
void getPWM(void){
  if (valueFromTimer > 250) {
    valueForPWM = 255;
  } else if (valueFromTimer < 130) {
    valueForPWM = 0;
  } else {
    valueForPWM = (valueFromTimer-125)*2;
  }  
}

// pinchange ISR
ISR(PCINT0_vect){
  if (PORTB & (1<<PCINT1)){
    TCCR1 |= ((1<<CS12) | (1<<CS11) | (1<<CS10));
  } else {
    valueFromTimer = TCNT1;
    TCCR1 &= ~((1<<CS10) | (1<<CS11)| (1<<CS12));
    TCNT1 = 0;
  }
}

// Overflow when signal to long
ISR(TIMER1_OVF_vect){
  TCCR1 &= ~((1<<CS10) | (1<<CS11)| (1<<CS12));
  TCNT1 = 0;
}

// softPWM 
ISR(TIMER0_OVF_vect){
  getPWM();
  if (PORTB & (1<<PB4)) {
    cbi(PORTB,PB4);
    TCNT0 = valueForPWM;
  } else {
    if (valueForPWM>0) sbi(PORTB,PB4);
    TCNT0 = 255-valueForPWM;
  }
};

ist wahrscheinlich wieder irgendwas was jeden außenstehenden ins auge 
springt und ich seh's natürlich wieder nicht...

MFG Xilence

[flex (Gast)]

PCINT0

ISR(PCINT0_vect){

oder PCINT1?

sbi(PCMSK,PCINT1);

[flex (Gast)]

Ne, Fehlalarm, das ist richtig.

[Xilence (Gast)]

der InputPin ist PCINT1
aber die ISR ist doch für alle PinChange-Interrupts PCINT0_vect
mir wird auch im AVR-Studio kein anderer Vektor angezeigt
im DB steht auch nur PCINT_0 in der vector-list

:confused:

[Xilence (Gast)]

hab - grad auch noch mal mit meinem avr-servotester n durchgang gemacht 
(der geht eigentlich ganz gut - Fahrtregler als auch servos akzeptieren 
den als Input)
geht auch nicht
hab auch mal spaßeshalber die PWM-auskommentiert (on/off statement)
liegt anscheinend aber eher an der Input Analyse
find aber den Fehler nicht...
AVR läuft auf 8MHz (spricht auch mein Multimeter - sry hab kein oszi)
das und der 64er PreScaler im Timer geben mir ein Messfenster von 2,04ms
gut des is vtl. weng knapp aber immerhin bei kurzen Impulsen (1-1,8 ms) 
sollte das ja gehen - machts ja aber auch net

[Simon (Gast)]

[...]
    if (PORTB & (1<<PCINT1)){
[...]

PINB für das Auslesen verwenden, damit sollte es gehen. Dann würde ich 
die getPWM in die Main-Schleife verschieben (der hat ja zZ sowieso 
nichts zu tun) und wer weiß, vielleicht willst du ja mal ein wneig mehr 
Logik als drei Verzweigungen reinbauen. Ansonsten schaut es auf den 
ersten Blick OK aus, auch wenn sich mir wegen cbi und sbi die Haare 
aufstellen. Wenn du kein gcc 3.4 verwendest kannst du sie dir sparen und 
die _BV() Makros und AND/OR-Masken verwenden, die können das Gleiche wie 
cbi mit dem Bonus, dass der gcc auch noch Opitmieren kann.

[Xilence (Gast)]

Jo das mit PIN vs. PORT wars
die getPWM hab ich noch mit in die PinChange-ISR gesteckt
geht auch wunderbar

kannst du für die Makros ein Beispiel geben bzw nen Link wo sie 
beschrieben werden?

Achja - woran kann es eigentlich liegen das die PWM bei niedrigen Werten 
anfängt merklich zu flackern. Ist da der AVR so unpräzise das er immer 
in den Cutoff springt?