enc28j60 spi Byteverschiebung

[grundschüler (Gast)]

frohe weihnachten!

ich versuche den enc28j60 mittels U-Radig_SW an einen M328 
anzuschließen. Problem:

statt
FF.FF.FF.FF.FF.FF.00.1C.BF.A5.DD.91.
sieht der Anfang des EthernetHeaders so aus:
FF.FF.FF.FF.FF.00.1C.BF.A5.DD.91.08

Ich habe das Problem auf den spi-Aufruf eingegrenzt. Im EncRegister 
stehen die korrekten Werte. bei Aufruf von spi_get() wird der erste Wert 
übersprungen:

static inline void xspi_put( unsigned char value )
{
  //ENC_DEBUG("spi_put(%2x)\n", (unsigned) value );
  SPDR = value;
  while( !(SPSR & (1<<SPIF)) ) ;
}

static inline unsigned char xspi_get(void)
{
  unsigned char value = SPDR;
  //ENC_DEBUG("spi_get()=%2x\n", (unsigned) value );
  return value;
}


static void enc_read_buf( unsigned char *buf, unsigned int len )
{
  enc_select();
  spi_put( ENC_SPI_OP_RBM );
//if(len > 0){len--; buf++ ;*(buf) = xspi_get();}
  for(; len > 0; len--, buf++ ) {
    xspi_put( 0x00 );
    *(buf) = xspi_get();
  }
  enc_deselect();
}



unsigned int enc_receive_packet( unsigned int bufsize, unsigned char *buf )
{
  unsigned char rxheader[18];
  unsigned int len, status;
  unsigned char u;

  // check rx packet counter
  u = enc_read_reg( ENC_REG_EPKTCNT );
  ENC_DEBUG("enc_receive: EPKTCNT=%i\n", (int) u);
  if( u == 0 ) {
    // packetcounter is 0, there is nothing to receive, go back
    return 0;
  }

  //set read pointer to next packet
  enc_write_reg( ENC_REG_ERDPTL, LO8(enc_next_packet_ptr) );
  enc_write_reg( ENC_REG_ERDPTH, HI8(enc_next_packet_ptr) );

  // read enc rx packet header
  enc_read_buf( rxheader, 6 );
  enc_next_packet_ptr  =             rxheader[0];
  enc_next_packet_ptr |= ((unsigned)(rxheader[1])) << 8;
  len                  =             rxheader[2];
  len                 |= ((unsigned)(rxheader[3])) << 8;
  status               =             rxheader[4];
  status              |= ((unsigned)(rxheader[5])) << 8;

//kk
ePtr=enc_next_packet_ptr;
eLen=len;
eStatus=status;

/*
//kk
  //kein Fehler:
  for(i=6;i<12;i++){
  tmp_macd[i-6]=rxheader[i];
  }

  for(i=12;i<18;i++){
  tmp_macs[i-12]=rxheader[i];
  }

  
  //Fehler:
  enc_read_buf( tmp_macd, 6 );//1.byte wird ausgelassen
  enc_read_buf( tmp_macs, 6 );//1.byte wird ausgelassen
*/
  // added by Sjors: reset the ENC when needed
  // If the receive OK bit is not 1 or the zero bit is not zero, or the packet is larger then the buffer, reset the enc chip and SPI
  if ((!(status & (1<<7))) || (status & 0x8000) || (len > bufsize))
  { 
    enc_init();
  }
  //ENC_DEBUG("enc_receive: status=%4x\n", (unsigned) status);

  // skip the checksum (4 bytes) at the end of the buffer
  len -= 4;

  // if the application buffer is to small, we just truncate
  if( len > bufsize ) len = bufsize;

  // now read the packet data into buffer
  enc_read_buf( buf, len );//1.byte wird ausgelassen



  // adjust the ERXRDPT pointer (= free this packet in rx buffer)
  if(    enc_next_packet_ptr-1 > ENC_RX_BUFFER_END
      || enc_next_packet_ptr-1 < ENC_RX_BUFFER_START ) {
    enc_write_reg( ENC_REG_ERXRDPTL, LO8(ENC_RX_BUFFER_END) );
    enc_write_reg( ENC_REG_ERXRDPTH, HI8(ENC_RX_BUFFER_END) );
  } else {
    enc_write_reg( ENC_REG_ERXRDPTL, LO8(enc_next_packet_ptr-1) );
    enc_write_reg( ENC_REG_ERXRDPTH, HI8(enc_next_packet_ptr-1) );
  }

  // trigger a decrement of the rx packet counter
  // this will clear PKTIF if EPKTCNT reaches 0
  enc_setbits_reg( ENC_REG_ECON2, (1<<ENC_BIT_PKTDEC) );

  // return number of bytes written to the buffer
  ENC_DEBUG("enc_receive: %i bytes\n", len);
  return len;
}

Ich habe versucht, das erste dem spi_get() vorausgehende spi_put() zu 
unterdrücken. dann kommt nur Unsinn.


Da es sich um ein bewährtes Standardprogramm mit einem Standardchip 
handelt, stehe ich vor einem Rätsel. Bin für jede Hilfe dankbar.

[dunno.. (Gast)]

Wie sehen denn spi_get bzw put aus?

Die geposteten Funktionen sind xspi.?

Wird bei SPI put das spdr geflusht..?

[grundschüler (Gast)]

dunno.. schrieb:
> spi_get bzw put

xspi ist nur eine 1:1 Kopie von spi um die Funktion beim Aufruf durch 
enc_read_buf separat bearbeiten zu können. an enc_read_reg brauch ich 
dann nichts zu ändern, denn dort funktioniert der spi-put-get-Aufruf 
einwandfrei.

[grundschüler (Gast)]

heureca
ich habe geändert:

static void enc_read_buf( unsigned char *buf, unsigned int len )
{
  enc_select();
  spi_put( ENC_SPI_OP_RBM );
//if(len > 0){len--; buf++ ;*(buf) = xspi_get();}
  for(; len > 0; len--, buf++ ) {
    spi_put( 0x00 );
    *(buf) = spi_get();
  }
  enc_deselect();
}
static void xenc_read_buf( unsigned char *buf, unsigned int len )
{
  enc_select();
//  spi_put( ENC_SPI_OP_RBM );
//if(len > 0){len--; buf++ ;*(buf) = xspi_get();}
  for(; len > 0; len--, buf++ ) {
    spi_put( ENC_SPI_OP_RBM );
    *(buf) = spi_get();
  }
  enc_deselect();
}


unsigned int enc_receive_packet( unsigned int bufsize, unsigned char *buf )
{
...

  // read enc rx packet header
  enc_read_buf( rxheader, 6 );
...
  // now read the packet data into buffer
  xenc_read_buf( buf, len );//1.byte wird ausgelassen
...

beim 2. Aufruf von enc_read_buf() ist spi_put( ENC_SPI_OP_RBM ); ein 
spi_put zuviel, da der buffer hierdurch um ein byte weitergelesen wird. 
Eigentlich logisch. Mit der geänderten xenc_read_buf() kommt jetzt das 
Ping. Alles funktioniert wie es soll.

Was ich jetzt nicht verstehe ist, wie die Original-U-Radig Funktion 
trotz diesem einmal zuviel spi_put funktionieren kann. Ich selbst habe 
das Original schon zigmal verwendet. Jetzt auf einmal geht es nicht 
mehr.





Was ich