ADC mit Schieberegister und LEDs

[Max S. (masch_bau)]

Hallo zusammen,

ich bräuchte dringend bitte Hilfe.

Ich soll 8 LEDs mit einem Schieberegister (74HC595) verwenden. 
Anschließend sollen die LEDs die Stellung des Potentiometer angeben.

Programm: MPLAB X IDE; C-Programmiersprache; PIC 16F18446; Compiler: XC8

Header Datei:

#ifndef Schieberegister
#define  Schieberegister

#define serPin LATAbits.LATA1
#define srclkPin LATAbits.LATA4
#define rclkPin LATAbits.LATA5

# define LED_1 0b00000001
# define LED_2 0b00000010
# define LED_3 0b00000100
# define LED_4 0b00001000
# define LED_5 0b00010000
# define LED_6 0b00100000
# define LED_7 0b01000000
# define LED_8 0b10000000


#include <xc.h> // include processor files - each processor file is guarded.  

 void digDisplay();

 char dispCode[8]= {0b00000001,0b00000010,0b00000100,0b00001000,0b00010000,0b00100000,0b01000000,0b10000000};


void serTakt (); // SChieberegister

void schreibTakt (); // SChieberegister

#endif

Schieberegister.c:
#include "Schieberegister.h"
#include "16F18446_Init.h"
#include "16F18446_Konfiguration.h"

//Variable zur Speicherung der Segmentcodierung für die übergene Ziffer

int displayZiffer;

void digDisplay (int dezZiffer){
    displayZiffer=dispCode[dezZiffer];
        
        for (int i=0; i<8;i++) {
            serPin=(displayZiffer>> i) & (0x01);
            serTakt();
        }
        schreibTakt ();
        return;
    }
    
 // Erzeugung einer positiven Flanke am Pin SRCLK

void serTakt (void) {
    srclkPin=0;
    __delay_us(1);
    srclkPin=1;
    __delay_us(1);
    srclkPin=0;
    __delay_us(1);
    return;
}

// Erzeugung einer positiven Flanke am Pin RCLK
void schreibTakt (void) {
    rclkPin=0;
    __delay_us(1);
    rclkPin=1;
    __delay_us(1);
    rclkPin=0;
    __delay_us(1);
    
    return;
}
main.c:
#include "16F18446_Init.h"
#include "Schieberegister.h"

void initADC(void);
int resultADC(void);

int displayWert;

void main(void) {
    
    TRISA= 0x0;
    TRISB=0x00;
    initADC();
    
    
    while (1) {
        
        displayWert=resultADC() / 512;
        
        for (int i=0; i<8;i++){
            digDisplay(i);
            __delay_ms(1000);
            }
            
         }
    return;
}

// AD Wandler:
void initADC () {
    TRISBbits.TRISB4=1;
    ANSELBbits.ANSB4=1;
    ADPCH=0x4;
    
    ADREFbits.PREF=0b00;
    ADREFbits.NREF=0;
    
    ADCON0bits.ON=1;
    return;
}

int resultADC() {
    uint8_t resultLOWBit;
    uint8_t resultHIGHBit;
    
    int dezResult=0;
    
    ADCON0bits.GO=1;
    while (ADCON0bits.GO);
    
    resultLOWBit=ADRESL;
    resultHIGHBit=ADRESH;
    
    for (int i=0;i<=8;i++){
         pow(2.0, (float) i);
        dezResult=dezResult + ((resultLOWBit >> i) & 0x01) * (1<<i);
    }
    
   
    
    return dezResult;
}

[Sebastian R. (sebastian_r569)]

Max S. schrieb:
> ich bräuchte dringend bitte Hilfe.

Und wobei jetzt genau?
Was ist deine konkrete Frage bei deiner Hausaufgabe?

[Max S. (masch_bau)]

Es funktioniert nicht und ich finde meinen Fehler nicht

das ist die direkte Aufgabenstellung "Bauen und programmieren Sie die 
vorangegangene Schaltung so um, dass eine Reihe von 8
LEDs die Stellung des Potentiometers angibt (linker Anschlag ! keine LED 
leuchtet, rechter
Anschlag ! alle LEDs leuchten). Nutzen Sie zur Umsetzung dieser 
Schaltung das Schieberegister
74HC595

[Steve van de Grens (roehrmond)]

Trenne das ganze auf.

Steuer zuerst mal die LED leiste mit einem festen Wert an, dann mit 
einem anderen, um zu testen, ob dieser Teil funktioniert.

Frage den ADC und prüfe das Ergebnis mit einem Debugger oder gebe es 
seriell aus.

Wenn beide Teile einzeln funktionieren, bringe sie wieder zusammen und 
schau dir ganz genau die Schnittstellen dazwischen an.

Viele serielle Ausgaben zwischendurch können dabei sehr hilfreich sein, 
falls du keinen Debugger hast.

[Obelix X. (obelix)]

Was geht denn nicht?
-Analogwert einlesen
-Schieberegister ansteuern
-LED leuchten?

[Wastl (hartundweichware)]

Max S. schrieb:
> Bauen und programmieren Sie die vorangegangene Schaltung so um

Welche Schaltung? Sollen wir hellsehen?

Was an diesem einfachen Satz

>> Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

verstehst du nicht?

[Michael (Firma: HW Entwicklung) (mkn)]

Max S. schrieb:
> Es funktioniert nicht
Das Einfügen des Code mit

 C-Code

?

Das liefern einer verständlichen Beschreibung was nicht geht?
Oder hapert es schon dabei die Aufgabe in einzelne Schritte 
runterzubrechen und die einzeln zu testen?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Max S. schrieb:
> ich bräuchte dringend bitte Hilfe.
Bitte Bedienungsanleitung beachten...

[Obelix X. (obelix)]

char dispCode[8]= {0b00000001,0b00000010,0b00000100,0b00001000,0b00010000,0b00100000,0b01000000,0b10000000};

x = dispCode[i]

vs

x = 1<<i

[Obelix X. (obelix)]

Max S. schrieb:
> void serTakt (); // SChieberegister
> void schreibTakt (); // SChieberegister

Könnte man nicht auch die SPI Schnittstelle verwenden?

[Steve van de Grens (roehrmond)]

Obelix X. schrieb:
> Könnte man nicht auch die SPI Schnittstelle verwenden?

Sicher, aber das ist wohl nicht Teil der Aufgabe.

[Stefan S. (chiefeinherjar)]

Steve van de Grens schrieb:
> Obelix X. schrieb:
>> Könnte man nicht auch die SPI Schnittstelle verwenden?
>
> Sicher, aber das ist wohl nicht Teil der Aufgabe.

Woher liest du das? Ich tippe eher darauf, dass der TO nicht weiß, wie 
er die SPI-Schnittstelle konfigurieren soll.

[Steve van de Grens (roehrmond)]

Stefan S. schrieb:
> Woher liest du das?

In der angehängten Datei ist die Kommunikation mit dem Schieberegister 
bereits vorgegeben.

[Ron-Hardy G. (ron-hardy)]

Max S. schrieb:
> while (1) {
>
>         displayWert=resultADC() / 512;
>
>         for (int i=0; i<8;i++){
>             digDisplay(i);
>             __delay_ms(1000);
>             }
>
>          }

Sieht für mich nach einem Lauflicht aus. Der displayWert wird doch 
garnicht benutzt?

[Udo S. (urschmitt)]

Ron-Hardy G. schrieb:
> Sieht für mich nach einem Lauflicht aus. Der displayWert wird doch
> garnicht benutzt?

Deshalb die Aufgabe:
Beitrag "Re: ADC mit Schieberegister und LEDs"

[Wastl (hartundweichware)]

Obelix X. schrieb:
> die SPI Schnittstelle

Also die Serial Peripheral Interface Schnittstelle .....

weisser Schimmel, tote Leiche, LCD Display, ADC Converter ....

[Obelix X. (obelix)]

Max S. schrieb:
> for (int i=0;i<=8;i++){
>    pow(2.0, (float) i);
>    dezResult=dezResult + ((resultLOWBit >> i) & 0x01) * (1<<i);
> }

Echt Lustig der Code ;-D

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Max S. schrieb:
> ich bräuchte dringend bitte Hilfe.
Denk erst mal nach, vor du vogelwild drauflosprogrammierst.

Max S. schrieb:
> Bauen und programmieren Sie die vorangegangene Schaltung so um, dass
> eine Reihe von 8 LEDs die Stellung des Potentiometers angibt
> linker Anschlag = keine LED leuchtet
>
> rechter Anschlag = alle LEDs leuchten
Damit hast du also 9 Zustände:

0  ........
1  *.......
2  **......
3  ***.....
4  ****....
5  *****...
6  ******..
7  *******.
8  ********

Und jetzt musst du zuallererst mal den ADC-Wert von z.B. 0..1023 (bei 
einem 10-Bit-Wandler ) auf 0..8 abbilden. Dazu brauchst du Mathe, 5. 
oder 6. Klasse, solltest du hinbekommen (ein Tipp: ADC-Wert/114 kommt in 
etwa hin).

Und wenn du dann die Zahl von 0..8 hast, dann dürfte es einfach sein, 
die passende Anzahl Einsen aus dem Ausgang hinauszutakten.

War ja gar nicht so schwierig.

[Gustl B. (gustl_b)]

Das Schieberegister braucht einen Schreibtakt je Bit. Und danach einen 
Latch-Takt damit die Bits ausgegeben werden an den Pins.

Also brauchst du eine Schleife:

For i in range(8):
   Datenpin = bits(i)
   Srclk = 1
   Srclk = 0

Lclk = 1
Lclk = 0

[Obelix X. (obelix)]

Lothar M. schrieb:
> Denk erst mal nach, vor du vogelwild drauflosprogrammierst.

Ich denke, da wurde nicht programmiert, sondern nur kopiert.

[Michi S. (mista_s)]

Max S. schrieb:
> Ich soll 8 LEDs mit einem Schieberegister (74HC595) verwenden.

Warum? Hausübung?


Max S. schrieb:
> dass eine Reihe von 8
> LEDs die Stellung des Potentiometers angibt

Wenn Dich die Stellung des Potis interessiert, dann solltest Du 
eventuell den Rückgabewert vom ADC auch irgendwie weiterverarbeiten, 
meinst Du nicht.
Ich muß mich doch glatt korrigieren, Du dividierst ihn ja durch 512 (hat 
der PIC - ich kenne ihn nicht - einen 12 Bit ADC? Dann wär diese 
Rechnung wenigstens näher dran an richtig, als alles andere) und erst 
das Ergebnis läßt Du links liegen, aber das ist sowieso immer 0. (siehe 
unten)


Max S. schrieb:
> rechter Anschlag ! alle LEDs leuchten

Dazu ist Dein Code schon mal nicht in der Lage; da wird nie mehr als 
eine LED leuchten können. (Umwandlungs-Tabelle mal scharf anschauen)


Max S. schrieb:
> Es funktioniert nicht und ich finde meinen Fehler nicht

Es ist, wie man bereits erkennen kann, weit mehr als nur einer.

Wie gesagt, ich kenne den PIC nicht, hab also keine Ahnung, wie man z.B. 
seinen ADC ansprechen sollte, aber dafür finde ich in Deiner 
ADC-Lese-Funktion ein paar Ungereimtheiten:

*) resultLOWBit und resultHIGHBit - ich hoffe doch, der ADC hat ein paar 
mehr als nur 2 Bit? Sind zwar nur Namen, aber man sollte sich das 
Programmieren doch nicht noch komplizierter machen, als es eh schon ist, 
drum mein Tip: Ein Bit sollte ein Bit sein und ein Byte jedenfalls mehr 
als ein Bit haben (i.A. acht Stück, aber wenn ein Wertebereich kein 
vielfaches von 8 Bit verbraucht, dann kann das höchstwertige Byte auch 
mal weniger Bits beinhalten).

*) Nachdem Du Deine beiden Bits, die vermutlich eher Bytes sind 
(hoffentlich korrekt) von der HW abgeholt hast, wirds wieder etwas 
skuril: das LOWBit verschiebst Du bitweise in Deine Ergebnisvariable 
(außerdem schiebst Du gleich 9 Bits rüber, obwohl hier vermutlich nur 
acht vorhanden sind, klassischer off-by-one Fehler, hier aber egal), das 
könnte man durchaus mit einer einfachen Zuweisung etwas abkürzen, aber 
die Verschiebe-Schleife hast Du immerhin korrekt von irgendwo kopiert; 
trotzdem: such Dir doch qualitativ bessere Kopiervorlagen, das Internet 
ist voll davon.

*) Tja, und vor lauter Freude übers richtig kopiert haben, vergißt Du 
dann komplett, Dich auch Deines HIGHBits des Ergebnisses anzunehmen. 
Damit bleibt der höchstmögliche Rückgabewert halt leider auf das 
beschränkt, was sich in 8 Bit unterbringen läßt, also max. 255 - die Du 
dann in Deiner Hauptschleife gleich mal forsch in 512 Teile dividieren 
willst. (siehe oben)


Das werden jetzt sicher nicht alle Fehler gewesen sein, die Du in Deinem 
Code versteckt hast, aber damit hast Du zumindest mal ein paar Ansätze, 
wo noch etwas zu tun ist.


Was mir sonst noch aufgefallen ist: eine Flanke ist an sich immer ein 
Wechsel von LOW (0) zu HIGH (1) bzw. umgekehrt. Folglich könntest Du in 
Deinen beiden Takt-Routinen jeweils eine Pinwackelei einsparen, welche 
kannst Dir aussuchen, solange es nicht die mittlere ist; aber klar tut 
die eine µs dort auch nicht wirklich weh.

[Peter D. (peda)]

Was erstmal zu klären ist, soll ein Binärwert angezeigt werden oder ein 
Balken.
Für den Binärwert nimmst Du die obersten 8 Bit des ADC, für den Balken 
die obersten 3 Bits.

Und wie schon gesagt wurde, teste den Code funktionsweise und nicht als 
ganzen Haufen auf einmal.

Max S. schrieb:
> ich bräuchte dringend bitte Hilfe.

Das ist natürlich eine super Fehlerbeschreibung, mit der jeder was 
anfangen kann.