Hey Flo,

dein top und bottom sind simple Zählvariablen oder Werte zwischen 0 und 255. Bei jedem halbwegs ausgewachsenen AVR mit mehr als 256 Byte SRAM (oder sogar schon bei weniger) sind Pointer aber 16 Bit breit (u.U. sogar noch breiter?).
Deine Zugriffe machen in gar keinem Fall Sinn, du müsstest top entweder auch als Zeiger definieren und dann top mit Basisadresse + Fifogröße initialisieren, oder komplett offsetbasiert arbeiten und zur Basisadresse immer einen Zählerwert hinzuaddieren.
Bitte sieh dir endlich Mal funktionierenden Code an und lies nochmal in einem Tutorial über C und Pointer genau durch wie das ganz funktioniert und was du falsch gemacht hast!

mfG
Markus