Forum: Mikrocontroller und Digitale Elektronik AT91RM9200 - Obere Hälfte SDRAM (D16..D31) ist immer 0xFF!


von Oliver (Gast)


Lesenswert?

Hallo Leute,

ich habe hier ein riesen Problem. Mein SDRAM am AT91RM9200 funktioniert 
nicht. Egal was ich schreibe, die oberen 16 bit sind immer gesetzt. Die 
unteren funktionieren aber.

Konfiguration: 2 x MT48LC16M16, 32bit breit angebunden. Die 
Anschlußbelegung entspricht dem Datenblatt und mehreren anderen Boards, 
die die gleiche Konfiguration verwenden. Alle Pins wurden auf Durchgang 
und Kurzschlüsse zu Nachbarpins durchgemessen. Die Stromversorgung ist 
sauber, an jedem Pin ist direkt ein 100nF Kondensator.

Initialisierung des SDRAM-Controllers ebenfalls nach Datenblatt:
1
  // Init SDRAM
2
3
  int *reg;
4
5
  // Assign CS1 line to SDRAM controller
6
   reg = (int *)0xFFFFFF60;
7
  *reg = 0x00000002;
8
9
  // Assign D16..D31 to SDRAM controller
10
   reg = (int *)0xFFFFF800;
11
        *reg = 0xFFFF0000;
12
  
13
  /* Configure for:
14
    2 x MT48LC16M16A2 on a 32 bit data bus
15
    TXSR  8
16
    TRAS  5
17
    TRCD  2
18
    TRP  2
19
    TRC  7
20
    TWR  2
21
    CAS  2
22
    9  Columns
23
    13 Rows
24
    4  Banks
25
  */
26
   reg = (int *)0xFFFFFF98;
27
  *reg = 0x42913959; 
28
29
  // Issue an "all banks precharge" command
30
   reg = (int *)0xFFFFFF90;
31
  *reg = 0x00000002; 
32
   reg = (int *)0x20000000;
33
  *reg = 0; 
34
35
  // Perform 8 "auto refresh" cycles
36
   reg = (int *)0xFFFFFF90;
37
  *reg = 0x00000004; 
38
   reg = (int *)0x20000000;
39
  *reg = 0; 
40
  *reg = 0; 
41
  *reg = 0; 
42
  *reg = 0; 
43
  *reg = 0; 
44
  *reg = 0; 
45
  *reg = 0; 
46
  *reg = 0; 
47
48
  // Perform "mode register set" cycle
49
   reg = (int *)0xFFFFFF90;
50
  *reg = 0x00000003; 
51
   reg = (int *)0x20000000;
52
  *reg = 0; 
53
54
  // Set refresh rate
55
   reg = (int *)0xFFFFFF94;
56
  *reg = 700; // decimal value in us
57
   reg = (int *)0x20000000;
58
  *reg = 0; 
59
60
  // Switch to normal mode
61
   reg = (int *)0xFFFFFF90;
62
  *reg = 0x00000000; 
63
   reg = (int *)0x20000000;
64
  *reg = 0;

So sieht mein Speichertest aus (Auszug):
1
  int base = 0x20000000;
2
  int len  = 64 * 1024 * 1024;

Test-Routine:
1
  int *ptr;
2
  int i;
3
4
  puts("Writing Zeros...\n");
5
  ptr = (int *) base;
6
  for (i = 0; i < len; i++)
7
      *(ptr++) = 0;
8
9
  puts("Verifying Zeros...\n");
10
  ptr = (int *) base;
11
  for (i = 0; i < len; i++)
12
  {
13
    if (*ptr != 0)
14
    {
15
      puts("At location "); uintprint(ptr); puts(" RAM reads ");
16
      uintprint(*ptr); puts(" !\n");
17
      return 1; /* Error */
18
    }
19
    ptr++;
20
  }

Ich bekomme nun folgerichtig an der Stelle 0x20000000 (SDRAM-Anfang) den 
Wert 0xFFFF0000 und die entsprechende Fehlermeldung.

Ich hoffe, es kann jemand helfen!

von Oliver (Gast)


Lesenswert?

Tja, manchmal braucht es ein Posting mit schöner Code-Formatierung, bis 
man selbst erkennt, was das Problem ist:
1
   // Assign D16..D31 to SDRAM controller
2
    reg = (int *)0xFFFFF800;
3
   *reg = 0xFFFF0000;

Das kommt davon, wenn man aus Faulheit die schönen vordefinierten 
Registerbezeichnungen nicht raussucht.

Möge jemand mit dem gleichen Problem diesen Thread finden.

von Marcus O. (marcus6100)


Lesenswert?

Hallo Oliver,

du solltest die Zeiger alle als "volatile int *"
definieren. Ansonsten wird der Compiler ggf.
Speicherzugriffe wegoptimieren.

Z.B. würde in "Perform 8 "auto refresh" cycles"
nur noch ein einziger Schreibzugriff mit 0
durchgeführt.

Anstatt int wäre außerdem die Verwendung von
uint32_t besser.

Gruß
Marcus

von Oliver (Gast)


Lesenswert?

Ja, danke, in der endgültigen Version nehme ich die vordefinierten 
AT91C...-Funktionen und Datentypen aus der AT91RM9200.h

Bitte melde dich an um einen Beitrag zu schreiben. Anmeldung ist kostenlos und dauert nur eine Minute.
Bestehender Account
Schon ein Account bei Google/GoogleMail? Keine Anmeldung erforderlich!
Mit Google-Account einloggen
Noch kein Account? Hier anmelden.