Display Ulrich Radig Software erweitern [C]

[Andreas A. (elw-2)]

Hallo!

Ich wollte ganz gerne das Anzeigemodul von Ulrich Radigs Labornetzteil 
für meines Verwenden. Das funktioniert auch im Test richtig gut!

Nur habe ich jetzt das Problem, dass ich auch ganz gerne die 
Wicklungsumschaltung von dem µC steuern lassen wollte. Hier für ist auch 
schon PD7 vorgesehen, aber noch nicht Programmiert.


Nun dachte mich mir, dass kann ja nicht so schwere sein!
Da ich gerade anfange mich mit µC Programmierung in C zu beschäftigen, 
musste ich mich erstmal einlesen, das hat auch so ganz gut geklappt.

Aber wenn ich nun die if/else Abfrage mit einbau, fängt die Anzeige an 
zu flackern, nur warum weiß ich leider nicht :(

Ich hoffe ihr könnt mir helfen, im Anhang ist meine Code und die 
Original teil:

Meiner mit PD7

#include <avr/interrupt.h>
#include <avr/io.h>

#include "multiplex.h"
#include "adc.h"

//---------------------------------------------------------------------------
int main (void) 
{

  DDRD |= (1<<PD0)|(1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
  DDRB |= (1<<PB0)|(1<<PB1)|(1<<PB2)|(1<<PB3)|(1<<PB4)|(1<<PB5);
  
  adc_init();
  
  sei();
  
  init_multiplex();
  
  while(1)
  {    
    if (adc_count > 400)
    {
      //ADC_OFF;
      display1 = (display_value_1 / adc_count);
      display2 = (display_value_2 / adc_count);
      adc_count = 0;
      display_value_1 = 0;
      display_value_2 = 0;
      //ADC_ON;
    }      
    {
      if (display1 > 200)
        PORTD |= (1 << PD7);
      else
        PORTD &= ~(1 << PD7);
    }
  }
}

Original von Radig:

#include <avr/interrupt.h>
#include <avr/io.h>

#include "multiplex.h"
#include "adc.h"

//---------------------------------------------------------------------------
int main (void) 
{

  DDRD |= (1<<PD0)|(1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6);
  DDRB |= (1<<PB0)|(1<<PB1)|(1<<PB2)|(1<<PB3)|(1<<PB4)|(1<<PB5);
  
  adc_init();
  
  sei();
  
  init_multiplex();
  
  while(1)
  {    
    if (adc_count > 400)
    {
      //ADC_OFF;
      display1 = (display_value_1 / adc_count);
      display2 = (display_value_2 / adc_count);
      adc_count = 0;
      display_value_1 = 0;
      display_value_2 = 0;
      //ADC_ON;
    }
  }
}

VG
Andreas

PS:
Der Link im Forum zum Projekt
Beitrag "Netzteil, LM317, Strombegrenzung und Anzeige!"

[Andreas A. (elw-2)]

Push it :(

[Fabian (Gast)]

Ich glaube nicht, dass es an deiner zusätzlichen IF-Abfrage liegt. Das 
muss andere Gründe haben. Benutzt Du ein Relais um die Wicklung 
umzuschalten? Freilaufdiode?! ....
Außerdem musst Du noch eine Hysterese einbauen. Ungefähr so:

      if (display1 > 200)
        PORTD |= (1 << PD7);
      else if (display 1 < 180)
        PORTD &= ~(1 << PD7);

[Andreas A. (elw-2)]

Stimmt, die Hysterese fehlt, danke die habe ich noch ganz vergessen :)

Aber das ändert nicht am Phänomen.

Es noch auf einem Steckbrett aufgebaut. An dem Anschluss für das Relais 
(mit Treibertransistor später) ist aktuell nur eine LED mit 
Vorwiderstand, also Simulation.

Mit dem Original Code ist alles super (aus dem AVR Studio 5 heraus 
Programmiert, mit selbst erzeugter hex), aber sobald ich die Änderung 
mit eine Programmiere fängt alles an zu flackern, also ob dir 
Multiplexabfolge nicht mehr richtig ausgeführt werden würde.

=> Also kann es auch nicht am Steckbrettaufbau liegen, den mit dem 
Originalcode flackert es nicht, nur mit dem angepassten :(

[Andreas A. (elw-2)]

Gerade ist mir was ganz kurioses aufgefallen. Sobald ich mich in der if 
Funktion befinde, hört das Flimmern auf, was komisch er verhaften...

[Sascha W. (sascha-w)]

Hallo,

soweit ich das sehe, hängt an PortD auch die Anzeige. Der Multiplex 
läuft in einem TimerInt. Wenn du in Main PortD auch veränderst, dann 
muss das atomar sein, sonst gibts diesen lustigen Effekt.
Wenn du die richtige Optimierung hat, sollte zwar aus der 
1-Bit-Portoperation in asm ein sbi/cbi werden - das währe dann atomar.

an sonsten so...

cli();
PORTD |= (1 << PD7);
sei();

aber mach das so, das der Portzustand nur dann neu ausgegeben wird, wenn 
er sich ändern soll.

Sascha

[Andreas A. (elw-2)]

>aber mach das so, das der Portzustand nur dann neu ausgegeben wird, wenn
>er sich ändern soll.

ÖÖÖÖ, jetzt noch mal in LOGO! oder S7 FUP bitte!

Wie soll ich das jetzt machen???

Sollte evtl. ne Logo oder S7 in meien NT einbauen, dass kann ich besser 
Programmieren :D :D

[Andreas A. (elw-2)]

Jetzt mal Spaß bei Seite :)

Ich bekomme es nicht hin, habe des jetzt mal in die multiplx.c gepackt.
Das funktioniert jetzt auch so weite, das Flackern ist weg, nur jetzt 
schaltet der Pin natürlich im Takt ein und aus => was somit eine 
Mischspannung ist und eine DC Relais mag das net so.
Gut mit einer Diode und einem Kondensator funktioniert das auch so, aber 
das ich wohl nicht Sinn und Zweck des ganzen.

Hoffe es kann mir eine Helfen!

[Sascha W. (sascha-w)]

also z.B. so ...

main.c

...
volatile char supply=0;

...
  while(1)
  {    
    if (adc_count > 400)
    {
      //ADC_OFF;
      display1 = (display_value_1 / adc_count);
      display2 = (display_value_2 / adc_count);
      adc_count = 0;
      display_value_1 = 0;
      display_value_2 = 0;
      //ADC_ON;
    }      
    if (display1 > 200) {
       supply=1;
    }
    if (display1 < 180) {
       supply=0;
    }

  }
...

multiplex.c

ISR (TIMER2_OVF_vect)
//############################################################################
{  
  PORTD = 0;
  PORTB = (1<<segcounter);
  
  switch (segcounter)
  {  
    case 0:
      PORTD = SEGMENTE[(display1 % 1000 / 100)];
      break;  
    case 1:
      PORTD = SEGMENTE[(display1 % 100 / 10)];
      break;    
    case 2:
      PORTD = SEGMENTE[(display1 % 10)];
      break;
    case 3:
      PORTD = SEGMENTE[(display2 % 1000 / 100)];
      break;
    case 4:
      PORTD = SEGMENTE[(display2 % 100 / 10)];
      break;  
    case 5:
      PORTD = SEGMENTE[(display2 % 10)];
      break;
  }
  if ((segcounter++)>5) segcounter = 0;  

        if (supply=1) {
           PORTD |= (1 << PD7);
        } else {
           PORTD &= ~(1 << PD7);
        }
}

oder so ...

main.c

...
char supply=0;
  while(1)
  {    
    if (adc_count > 400)
    {
      //ADC_OFF;
      display1 = (display_value_1 / adc_count);
      display2 = (display_value_2 / adc_count);
      adc_count = 0;
      display_value_1 = 0;
      display_value_2 = 0;
      //ADC_ON;
    }      
    if (display1 > 200 && supply=0) {
        cli();
        PORTD |= (1 << PD7);
        sei();
        supply=1;
    }
    if (display1 < 180 && supply=1) {
        cli();
        PORTD &= ~(1 << PD7);
        sei();
        supply=0;
    }
  }

Sascha

[Karl H. (kbuchegg)]

Sascha Weber schrieb:

> oder so ...

(2. Variante)

Das hilft letzten Endes auch nichts. Denn in der Multiplex Routine wird 
der PortD als Ganzes neu gesetzt. D.h. in dem Fall würde dann eben das 
Relais flackern.

Das könnte man zwar auch abstellen, aber deine erste Variante ist 
wharscheinlich die Einfachere.

[Sascha W. (sascha-w)]

Karl Heinz Buchegger schrieb:
> Sascha Weber schrieb:
>
>> oder so ...
>
> (2. Variante)
>
> Das hilft letzten Endes auch nichts. Denn in der Multiplex Routine wird
> der PortD als Ganzes neu gesetzt. D.h. in dem Fall würde dann eben das
> Relais flackern.
>
> Das könnte man zwar auch abstellen, aber deine erste Variante ist
> wharscheinlich die Einfachere.

hmm... stimmt - hab ich übersehen, müsste man noch maskieren, damit 
wirklich nur die Segmentbits geändert werden.
Ist sowiso mist, da die ISR mit PORTD=0 anfängt.

also so für Var. 1

ISR (TIMER2_OVF_vect)
//############################################################################
{  
uint8 portdata;
  PORTB = (1<<segcounter);
  
  switch (segcounter)
  {  
    case 0:
      portdata= SEGMENTE[(display1 % 1000 / 100)];
      break;  
    case 1:
      portdata= SEGMENTE[(display1 % 100 / 10)];
      break;    
    case 2:
      portdata= SEGMENTE[(display1 % 10)];
      break;
    case 3:
      portdata= SEGMENTE[(display2 % 1000 / 100)];
      break;
    case 4:
      portdata= SEGMENTE[(display2 % 100 / 10)];
      break;  
    case 5:
      portdata= SEGMENTE[(display2 % 10)];
      break;
  }
  if ((segcounter++)>5) segcounter = 0;  

        if (supply=1) {
           portdata |= (1 << PD7);
        }
  PORTD = portdata;
}

oder so für Var. 2

ISR (TIMER2_OVF_vect)
//############################################################################
{  
uint8 portdata;
  PORTB = (1<<segcounter);
  
  switch (segcounter)
  {  
    case 0:
      portdata= SEGMENTE[(display1 % 1000 / 100)];
      break;  
    case 1:
      portdata= SEGMENTE[(display1 % 100 / 10)];
      break;    
    case 2:
      portdata= SEGMENTE[(display1 % 10)];
      break;
    case 3:
      portdata= SEGMENTE[(display2 % 1000 / 100)];
      break;
    case 4:
      portdata= SEGMENTE[(display2 % 100 / 10)];
      break;  
    case 5:
      portdata= SEGMENTE[(display2 % 10)];
      break;
  }
  if ((segcounter++)>5) segcounter = 0; 
 
  portdata |= (PORTD & PD7);
  PORTD = portdata;
}

Sascha

[Andreas A. (elw-2)]

Hallo!

Erstmal dank für die Antworten!!

@Sascha
Bei deiner Variante 2 bekomme ich in der Main diesen Fehler für deine 
beiden if Abfragen:

"lvalue required as left operand of assignment"

...
#include <avr/interrupt.h>
#include <avr/io.h>

#include "multiplex.h"
#include "adc.h"

volatile char supply=0;

//---------------------------------------------------------------------------
int main (void) 
{

  DDRD |= (1<<PD0)|(1<<PD1)|(1<<PD2)|(1<<PD3)|(1<<PD4)|(1<<PD5)|(1<<PD6)|(1<<PD7);
  DDRB |= (1<<PB0)|(1<<PB1)|(1<<PB2)|(1<<PB3)|(1<<PB4)|(1<<PB5);
  
  adc_init();
  
  init_multiplex();
  
  while(1)
  {    
    if (adc_count > 200)
    {
      //ADC_OFF;
      display1 = (display_value_1 / adc_count);
      display2 = (display_value_2 / adc_count);
      adc_count = 0;
      display_value_1 = 0;
      display_value_2 = 0;
      //ADC_ON;
    }  
        
      if (display1 > 200 && supply=0) 
      {
       cli();
       PORTD |= (1 << PD7);
       sei();
        supply=1;
      }
      
      if (display1 < 180 && supply=1)
      {
      cli();
      PORTD &= ~(1 << PD7);
      sei();
        supply=0;
      }
      
  }
}

Bei Variante 1 sind es 6 Errors in der multiplex

Kann das sein, dass es am Compiler liegen, ich verwende das AVR Studio 
5.

@ Karl Heinz Buchegger
>Das könnte man zwar auch abstellen, aber deine erste Variante ist
>wahrscheinlich die Einfachere.

Vermute mal du meinst meine Gleichrichtung, das ist wahrscheinlich 
wirklich nicht die schönste, aber wirkungsvollste Methode.

[Sascha W. (sascha-w)]

Andreas A. schrieb:

> Kann das sein, dass es am Compiler liegen, ich verwende das AVR Studio
> 5.
nein wohl eher weil du in C programmierst und ich eigentlich in ASM

das erste Problem sollte sich wohl so lösen lassen

statt:

if (display1 < 180 && supply=1)

eher so:

if ((display1 < 180) && (supply=1))

zu Var. 1
wenn es 6 Fehler sind dann ja warscheinlich in switch(), da wird wohl 
der Typ von SEGMENTE nicht zum Typ von portdata [uint8] passen - einfach 
mal entsprechend anpassen!

Sascha

[holger (Gast)]

>eher so:
>
>if ((display1 < 180) && (supply=1))

eher so:

if ((display1 < 180) && (supply==1))