Probleme mit AT90CAN64 SPI Übertragung

>Mein Problem ist, dass ich mit dem Scope keinerlei Signale am SPI sehe.

Der wichtigste Teil des Programmes fehlt mal wieder :(
Nutzt du auch den SS Pin für das Slaveselect?

Lies dir doch meinen Code durch!
Da kommen Kommandos vor die heißen

1

    Spi_init_ss();                              

2

    Spi_disable_ss();

3

Spi_enable_ss();

Auch ohne die "CAN Software Libraries" zu kennen, könnte man doch 
annehmen, dass das es sich um den SlaveSelect handelt, oder etwa nicht?!
(Sorry wenns ned eindeutig war)

wo ist deine spi_lib.c ? Wo sind deine SPI-Makros definiert? Gibts nen 
Schaltplan? Glaskugel ist leider kaputt sorry ;-)

>Lies dir doch meinen Code durch!

Habe ich getan.

>Da kommen Kommandos vor die heißen
>
>    Spi_init_ss();
>    Spi_disable_ss();
>Spi_enable_ss();

Benutzt du nun den SS Pin vom ATMega64 oder nicht?
Das kann ich dort nicht sehen.

>Auch ohne die "CAN Software Libraries" zu kennen, könnte man doch
>annehmen, dass das es sich um den SlaveSelect handelt, oder etwa nicht?!

Annehmen heisst nicht wissen.

Sorry für meine Faulheit. Die spi_lib.c ist in den atmel libraries. der 
link ist oben...

>Benutzt du nun den SS Pin vom ATMega64 oder nicht?

Sorry, vom AT90CAN..

>Benutzt du nun den SS Pin vom ATMega64 oder nicht?
>Das kann ich dort nicht sehen.

es ist der at90can64, nicht der atmega....aber die o.g. Macros verwenden 
den SlaveSelect Pin...

So. Nun hab ich auch mal die Files aus den Atmel Libs mit angehangen

Auf die Frage nach dem Schaltplan, ist angehängt.
(Das Value des Crystals ist allerdings falsch. Der hat 8 und nicht 16 
MHz)
Mal ne Frage, kann man auch mehr als eine Datei an nen Post anhängen, 
oder muss man die dazu in nen Archiv parken?!

Ich seh nix, außer das

Bool spi_init (U8 config)

so aussehen müsste wegen der Clock Macros

Bool spi_init (U16 config)

>Ich seh nix, außer das
>
>Bool spi_init (U8 config)
>
>so aussehen müsste wegen der Clock Macros
>
>Bool spi_init (U16 config)

Ne! Wenn man mal durchzählt:
SPI_MASTER      1 Bit
SPI_MSB_FIRST   1 Bit
SPI_DATA_MODE_0 2 Bit
SPI_CLKIO_BY_64 3 Bit
SPI_enable      1 Bit
Macht insgesamt 8 Bit...
demnach ist U8 richtig

Ups! Hab mich vertan! spi_enable gehört ja gar ned zur variable 
config...
Aber dann passt U8 immernoch

>Aber dann passt U8 immernoch

1

    // ---------- Pre-definitions for relationship between SCK and CLKIO (16-bit)

2

    // ---------- (The 9th is the Double SPI Speed Bit: SPI2X of SPSR)

3

#define SPI_CLKIO_BY_2            ((1<<(SPI2X+8))|(0<<SPR1)|(0<<SPR0)) // -- 0x100

Und was sagt dir das?
Aber hast recht, bei seiner Clockeinstellung reicht U8.

>>Aber dann passt U8 immernoch

1

    // ---------- Pre-definitions for relationship between SCK and CLKIO (16-bit)

2

    // ---------- (The 9th is the Double SPI Speed Bit: SPI2X of SPSR)

3

#define SPI_CLKIO_BY_2            ((1<<(SPI2X+8))|(0<<SPR1)|(0<<SPR0)) // -- 0x100

>Und was sagt dir das?
>Aber hast recht, bei seiner Clockeinstellung reicht U8.

Mkay! Vom MC und seinen Registern her sind es 16 Bit. Das stimmt.

Aber ich denke mal, dass Atmel das schon richtig macht, da die diese 
Libs selber zum Programmieren verwenden...

>Aber ich denke mal, dass Atmel das schon richtig macht, da die diese
>Libs selber zum Programmieren verwenden...

Dann hat von denen aber noch niemand versucht
SPI_CLKIO_BY_2 einzustellen ;)

Hier ist auch noch ein Tippfehler (macht aber nix)

1

#define Spi_enable_ss()           (PORTB &= ~(1<<DDB0))  // -- Can be user re-defined

Und da sind die Kommentare falsch

1

#define SPI_CLKIO_BY_64           ((0<<(SPI2X+8))|(1<<SPR1)|(0<<SPR0)) // -- 0x102

2

#define SPI_CLKIO_BY_128          ((0<<(SPI2X+8))|(1<<SPR1)|(1<<SPR0)) // -- 0x103

>Hier ist auch noch ein Tippfehler (macht aber nix)

1

#define Spi_enable_ss()           (PORTB &= ~(1<<DDB0))  // -- Can be user re-defined

Was denn?! Habe diese Variante beim Googlen schon des öfteren gesehen, 
nicht nur von Atmel...

>Was denn?! Habe diese Variante beim Googlen schon des öfteren gesehen,
>nicht nur von Atmel...

Ist ja auch nur ein Schönheitsfehler ;)

Hier ist das Problem

1

  buffer[1] = spi_getchar();

2

  buffer[2] = spi_getchar();

Vor jedem spi_getchar(); muß ein Dummybyte gesendet werden.
spi_getchar(); macht das nicht.

>Hier ist das Problem

>  buffer[1] = spi_getchar();
>  buffer[2] = spi_getchar();

>Vor jedem spi_getchar(); muß ein Dummybyte gesendet werden.
>spi_getchar(); macht das nicht.

Ok! Hab ich eingebaut!
Und auch den Fehler mit dem U16 bei der Funktion spi_init hab ich 
behoben!
(Hab das nochmal nachgeguckt, und du hattest recht mit dem 16 bit...)

Allerdings funktioniert es immernoch ned...So wie es aussieht (habe zum 
debuggen nach jedem Schritt den Wert von Port c erhöht) kommt er nicht 
mal über das

1

  // --- Send control byte

2

  spi_transmit_master(buffer[0]);

von oben hinaus...

Jemand ne Idee wo der Fehler liegen könnte?!

Gruss
Florian

Danke Holger!!!
Es funktioniert!!