Speicherschreiber in ATTiny HID Code?

[Satz K. (satz_k)]

Hi,

ich habe folgenden eientlich recht simplen Code, mit dem ich auf meinem 
ATTiny Daten vom USB empfange:

static uchar keyData[DATA_LENGTH+4];

uchar   usbFunctionWrite(uchar *data, uchar len)
{
   uchar i;

   if (bytesRemainingW == 0) return 1;               /* end of transfer */
   if (len > bytesRemainingW) len = bytesRemainingW;
//   for (i=0; i<len; i++) keyData[currentAddressW+i]=data[i];
    currentAddressW += len;
    bytesRemainingW -= len;
   if (bytesRemainingW==0)
   {
      return 1;
   }
   return 0;
}

Sobald ich die auskommentierte Zeile reinnehme, wird mein USB-Gerät 
nicht mehr erkannt. Scheinbar schreibe ich da über den gültigen Bereich 
von keyData hinaus. Allerdings: wieso? Ich sehe das Problem echt 
nicht...

[holger (Gast)]

>Allerdings: wieso? Ich sehe das Problem echt
>nicht...

Jeder andere hier auch nicht.
Niemand weiss wie gross keydata[DATA_LENGTH+4] ist, was du mit
currentAddressW sonst noch so machst...

[Satz K. (satz_k)]

Mehr als das ist da nicht:

#define DATA_LENGTH    128

usbMsgLen_t usbFunctionSetup(uchar data[8])
{
usbRequest_t    *rq = (void *)data;

    if((rq->bmRequestType & USBRQ_TYPE_MASK) == USBRQ_TYPE_CLASS){
        if(rq->bRequest == USBRQ_HID_GET_REPORT){
            bytesRemainingR = DATA_LENGTH;
            currentAddressR = 0;
            return USB_NO_MSG;
        }else if(rq->bRequest == USBRQ_HID_SET_REPORT){
            bytesRemainingW = DATA_LENGTH;
            currentAddressW = 0;
            return USB_NO_MSG;
        }
    }else{
    }
    return 0;
}

[Karl H. (kbuchegg)]

Satz K. schrieb:

>    if (len > bytesRemainingW) len = bytesRemainingW;
> //   for (i=0; i<len; i++) keyData[currentAddressW+i]=data[i];


dein
  currentAddressW+i
wird halt irgendwann mal größer oder gleich
  DATA_LENGTH + 4
werden.

[holger (Gast)]

Ich sehe keinen einzigen Aufruf von

uchar   usbFunctionWrite(uchar *data, uchar len)

>Mehr als das ist da nicht:

Da muss mehr sein!

Machs doch mal so:

   for (i=0; i<len; i++) 
      if((currentAddressW+i) < sizeof(keyData))
         keyData[currentAddressW+i]=data[i];

Das wird dein Problem aber nicht lösen, nur die Überschreiber.

[Satz K. (satz_k)]

Also: der Code basier auf VUSB, usbFunctionWrite() müsste von usbPoll() 
getriggert werden, welches wiederum aus einer Schleife in main() 
aufgerufen wird.

Was ich überhaupt nicht verstehe: ich sende/lese noch gar keine Daten, 
wenn die schleife drin ist, wird das Gerät gar nicht mehr erkannt, also 
keine Chance, überhaupt damit zu kommunizieren und in einen möglichen 
Speicherschreiber reinzulaufen.

[Karl H. (kbuchegg)]

Wie gross ist DATA_LENGTH?

Überlauf des SRAM durch zuviel verbrauchten Speicher?

[holger (Gast)]

>Was ich überhaupt nicht verstehe: ich sende/lese noch gar keine Daten,
>wenn die schleife drin ist, wird das Gerät gar nicht mehr erkannt, also
>keine Chance, überhaupt damit zu kommunizieren und in einen möglichen
>Speicherschreiber reinzulaufen.

So ein Tiny ist ja nicht gerade gesegnet mit RAM.
Wenn der Compiler keine Verwendung von keydata[] sieht
legt er es vieleicht nicht an oder der Linker schmeisst es raus
und dein RAM reicht noch.

Wird keydata[] verwendet knallts dann eventuell halt
weil nicht genügend RAM da ist.

[Amateur (Gast)]

Da ist was faul im Staate...

Funktionen, die nie aufgerufen werden, zeichnen sich durch eine kaum zu 
überbietende Stabilität aus.

Mag ja sein, dass Dein Programm nicht funktioniert, aber nicht wegen 
eines Aufrufes von usbFunctionWrite.

[Satz K. (satz_k)]

holger schrieb:
>>Was ich überhaupt nicht verstehe: ich sende/lese noch gar keine Daten,
>>wenn die schleife drin ist, wird das Gerät gar nicht mehr erkannt, also
>>keine Chance, überhaupt damit zu kommunizieren und in einen möglichen
>>Speicherschreiber reinzulaufen.
>
> So ein Tiny ist ja nicht gerade gesegnet mit RAM.
> Wenn der Compiler keine Verwendung von keydata[] sieht
> legt er es vieleicht nicht an oder der Linker schmeisst es raus
> und dein RAM reicht noch.
>
> Wird keydata[] verwendet knallts dann eventuell halt
> weil nicht genügend RAM da ist.

Hm, und wie zwinge ich den Compiler, keyData[] tatsächlich anzulegen?

[holger (Gast)]

>Hm, und wie zwinge ich den Compiler, keyData[] tatsächlich anzulegen?

Diese Zeile aktivieren:

//   for (i=0; i<len; i++) keyData[currentAddressW+i]=data[i];

Aber dann macht es bumm weil dein RAM scheinbar nicht ausreicht.

[Amateur (Gast)]

@Satz K.

Ich bin ja mal gespannt, wann Du das Rätsel, über den verwendeten Typen, 
auflöst.
Praktisch in jedem Post wird erwähnt, dass es bei den Tiny's nicht so 
Doll mit Speicher aussieht und dass der Fehler eventuell damit 
zusammenhängen könnte.
Musst Du aber nicht, vielleicht errät es ja der eine oder andere.