XC8 inline Assembler

[Siro]

Ich lach mich kaputt.....
Du hast ja sowas von recht Witkatz

Oh Mann, das tut ja weh , vielen Dank für die Info
--------------------------------------------------

Ich habe jetzt einen laufenden Code, fast ausschließlich in "C" , basierend auf der Funktion von Klebwax(dessen Code ich geklaut habe).

// so viele LEDs sollen angesteuert werden:
#define LED_COUNT 3

/* Jede LED hat 3 Bytes insgesamt also 24 Bits */
typedef  struct  // __pack weil wir keinen Speicher verschwenden wollen ????
{ 
    U8 green;     /* 8 Bit fuer die Helligkeit */
    U8 red;       /* 8 Bit fuer die Helligkeit */
    U8 blue;      /* 8 Bit fuer die Helligkeit */
} TLed;           /* Type Bezeichner ist TLed */

TLed LedArray[LED_COUNT];

void LedShiftOut(U8* leds, U8 count) 
{
    U8 one_byte;
    U8 bit_count;

    count *=3;             // 3 Bytes pro Led RGB
    while (count) {
        one_byte = *leds++; // next Byte
        for (bit_count = 0; bit_count < 8; bit_count++) 
        {
            if (one_byte & 0x01) // lowest Bit
            {
                LATA5 = 1;
                asm("NOP");
                asm("NOP");
                asm("NOP");
                LATA5 = 0;
            } else
            {
                LATA5 = 1;
                asm("NOP");
                LATA5 = 0;
            }    
            one_byte >>= 1;
        }
        count--;
    }
       
    Delay_ms(1);
}

Das Timing stimmt eigentlich nicht, aber das scheint wesentlich unkritischer zu sein als das Datenblatt vorgibt.
Die Low Phasen dürfen nämlich viel länger sein, das scheint nicht zu stören. Die sind bei mir alle so ca. 2us.
Die High-Phasen müssen lediglich stimmen, dann ist die Welt in Ordnung.
Ich habe zwischen den Bytes sogar 5us Pause, das stellt kein Problem dar.



Nochmal einen DANK an alle Die mir hilfreiche Informationen gegeben haben (beonders Witkatz + Klebwax)

[Klebwax]

@Siro

Jetzt könnte man noch die Lbraryfunktion

NOP();

statt

asm("NOP");

verwenden und alles wäre plain C.

MfG Klebwax

[witkatz]

Jetzt könnte man noch die Lbraryfunktion
NOP();

Oder etwas lesbarer _delay(1); statt NOP(); bzw. _delay(3); statt NOP();NOP();NOP();

[Siro]

Hallo nochmal, kurzer Endbericht:

die Assembler nops habe ich durch "NOP()" ersetzt und das funktioniert erwartungsgemäß einwandfrei.

Mit den Delay wäre etwas problematisch, da ein NOP in meinem Falle bei 32MHz Clock 125ns entspricht.
die delay funktion ist in Millisekunden.
es soll wohl delay_ms und delay_us geben, die meckert er mich an, da fehlt wohl eine Bibliothek.

Aber nichts desto trotz, es laufen grad 50 LEDs einwandfrei, aber das hatte auch noch etwas gedauert, denn

man sollte während des Ausschiebens der Daten alle Interrrupts sperren, sonst hat man Randomize Rainbow

Zudem waren ab LED 21 die Farben falsch. Zuerst dachte ich an ein Speicherproblem, aber das war es nicht.
Da war doch tatsächlich die 21te LED auf dem Band defekt. Zum Glück hatte ich noch einzelne und habe sie ausgetauscht.
Nun leuchten alle 50 wie sie sollen.
Bei voller Ansteuerung in weiss kommt schon so einiges an Strom zusammen.
Das sind dann 20mA * 3 Leds = 60mA * 50 Leds = 3,05 Ampere.

Achja, noch was: Bei kurzzeitiger Unterbrechung der Versorgungsspannung, gehen die LED zwar aus, es bleiben aber die Daten in den 50 LEDs erhalten.
So 3 bis 5 Sekunden habe ich ausprobiert, ohne dass ich erenut Daten reinschieben muss.

[witkatz]

die delay funktion ist in Millisekunden.

Nö. In Millisekunden ist die _delay_ms(), die _delay() ist in Prozessortakten, _delay(1); ersetzt also ein NOP(); und _delay(3); ersetzt NOP();NOP();NOP();
Die Unterstriche sind dabei wichtig. delay() kennt der XC8 nicht, _delay() schon.
Ich finde halt eine Zahl lesbarer als eine NOP-Kette. Ist aber letztendlich wurscht.

- - - Aktualisiert - - -

Nun leuchten alle 50 wie sie sollen.

Nun warten wir auf ein Foto oder Video
Glückwunsch

[Siro]

Das mit den delay muss ich mir nochmal anschauen.
Danke für die Info Witkatz:

Das mit dem Video macht sich garnicht gut, man erkennt nicht annähernd die schönen Farben, das sieht immer weiss aus...
Ich hab es aber mal im Dunkeln gefilmt, nicht schön, besser wie nix.

https://www.dropbox.com/s/mr6gkga5mxkh98k/RGB.MOV?dl=0


hier noch die komplexe1 Schaltung:

rot Plus
blau Minus
grün Datenleitung zu den Leds2


Siro