annotate MEMIF-wait-states @ 17:3d65bdaf00da

MEMIF-wait-states article written
author Mychaela Falconia <falcon@freecalypso.org>
date Sun, 16 Jun 2019 23:30:33 +0000
parents
children c01155dec65b
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
17
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
1 The Calypso chip's MEMIF (ARM memory interface) block has a few configuration
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
2 registers; most settings in these registers are quite straightforward, but the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
3 WS setting (number of wait states to be inserted for external memory access)
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
4 requires some non-trivial analysis.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
5
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
6 Calypso MEMIF timings are described on pages 7 through 11 of this TI document:
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
7
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
8 ftp://ftp.freecalypso.org/pub/GSM/Calypso/cal000_a.pdf
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
9
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
10 When running on a Calypso C035 target, our TCS211 reference fw as well as most
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
11 vendor firmwares we've examined run the ARM7 core at its maximum clock frequency
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
12 of 52 MHz. These same firmwares typically configure WS=3 for both flash and
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
13 XRAM. Most Calypso-based phones and modems have flash and RAM chips with 70 ns
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
14 access time, and for a long time it seemed that this combination of ARM7 at
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
15 52 MHz and WS=3 was OK for 70 ns memories: one ARM7 clock cycle at 52 MHz is
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
16 19.23 ns, WS=3 means 4 cycles total per access (it's an N+1 arrangement),
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
17 19.23 ns * 4 = 76.92 ns, thus it should be OK for 70 ns memories, right? Not
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
18 so fast: as shown in the formula on cal000_a.pdf page 11 and can be seen from
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
19 the timing diagrams, two other timing parameters (tda and tsu) also need to be
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
20 factored in. The sum of tda+tsu for 2.8V MEMIF as given in the only document
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
21 we have available is 10.5 ns, thus if we run the ARM7 core at 52 MHz and set
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
22 WS=3, the available safe window for memory access time is only about 66 ns,
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
23 which is 4 ns short of the 70 ns flash and RAM access time specs.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
24
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
25 TI's reference fw setting of WS=3 in conjuction with ARM7 running at 52 MHz has
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
26 made its way into the official firmwares of Openmoko devices and several Compal
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
27 phones, including Mot C11x/12x, Mot C139/140 and Sony Ericsson J100. At least
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
28 in the case of Openmoko we know that the hardware features a flash chip with
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
29 70 ns access time (the combined flash+RAM chip is K5A3281CTM-D755, with the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
30 suffix meaning 70 ns access time for flash and 55 ns for RAM), and in the case
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
31 of Compal phones it is highly unlikely that they used flash chips faster than
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
32 70 ns, thus we have strong evidence that the access time spec is being violated
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
33 by about 4 ns. It works in practice because the official specs are guaranteed
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
34 worst-case numbers, but it is still wrong in the strict sense.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
35
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
36 We have strong evidence that this WS=3 setting comes from TI's mainline
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
37 reference fw, as opposed to being customized by or for Openmoko or Compal.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
38 The evidence is in the following instruction sequence which appears verbatim-
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
39 identical across Openmoko's, Mot C11x/12x and C139/140 firmware versions:
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
40
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
41 ldr r1, =0xFFFFFB00
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
42 mov r0, #0xA3
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
43 strh r0, [r1, #0]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
44 strh r0, [r1, #2]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
45 mov r2, #0xA5
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
46 strh r2, [r1, #4]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
47 strh r0, [r1, #6]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
48 mov r0, #0x80
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
49 strh r0, [r1, #0xA]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
50 mov r0, #0xC0
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
51 strh r0, [r1, #0xC]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
52 mov r0, #0x40
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
53 strh r0, [r1, #8]
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
54
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
55 (The SE J100 version differs only in the nCS2 configuration; apparently this
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
56 SE J100 phone has its ringtone melody generator chip hooked up to nCS2, whereas
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
57 on both OM's modem and Mot C11x/12x/139/140 this chip select is unused and
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
58 unconnected, meaning that its setting is a dummy just like nCS3 and nCS4.)
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
59
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
60 The above instruction sequence has been reconstructed into the following
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
61 sequence of C macro calls:
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
62
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
63 MEM_INIT_CS0(3, MEM_DVS_16, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
64 MEM_INIT_CS1(3, MEM_DVS_16, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
65 MEM_INIT_CS2(5, MEM_DVS_16, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
66 MEM_INIT_CS3(3, MEM_DVS_16, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
67 MEM_INIT_CS4(0, MEM_DVS_8, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
68
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
69 MEM_INIT_CS6(0, MEM_DVS_32, MEM_WRITE_EN, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
70 MEM_INIT_CS7(0, MEM_DVS_32, MEM_WRITE_DIS, 0);
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
71
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
72 (The last two lines setting nCS6 and nCS7 don't need to be considered, as those
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
73 are internal to the Calypso chip itself.)
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
74
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
75 Thus we see that what appears to be TI's mainline code sets WS=3 for both nCS0
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
76 and nCS1 (flash and XRAM, respectively), and then sets what appears to be a
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
77 dummy config for the unused nCS2, nCS3 and nCS4. I say "appears to be" because
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
78 we have no original source with comments, only a COFF binary object which our
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
79 reconstructed recompilable C code has been made to match.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
80
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
81 We may never know the truth unless we miraculously find a surviving copy of the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
82 original (not reconstructed from disassembly) init.c source from TCS211, but my
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
83 (Mother Mychaela's) current working hypothesis is that the above MEMIF settings
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
84 were originally made for the D-Sample board and never changed for Leonardo.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
85 The D-Sample board has flash on nCS0, main XRAM bank on nCS1, an additional
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
86 XRAM bank (typically unused) on nCS2 and peripherals (principally the LCD) on
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
87 nCS3. Furthermore, the original D-Sample boards had Calypso C05 chips populated
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
88 on them, and that chip version has no nCS4, only CS4 which is muxed with ADD22
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
89 and used for the latter on the D-Sample.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
90
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
91 I further hypothetize that the above MEMIF settings were likely cast into code
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
92 in the days of Calypso C05, and that the WS=3 setting was computed when the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
93 ARM7 core ran at 39 MHz. The combination of ARM7 at 39 MHz, WS=3 and the same
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
94 tda+tsu = 10.5 ns adjustment from the available cal000_a.pdf document
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
95 (officially corresponding to Calypso C035 F751774) gives an access time of
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
96 92 ns, which is very sensible. The hypothesis further goes that later TI moved
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
97 to Calypso C035 silicon and started running the ARM7 core at 52 MHz, but the WS
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
98 setting was never changed (overlooked), and the 92 ns access time turned into a
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
99 mere 66 ns. The latter works with 70 ns memories in practice despite being
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
100 strictly incorrect (negative margin), and so the error escaped notice.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
101
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
102 Solution adopted for FreeCalypso
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
103 ================================
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
104
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
105 Pirelli's firmware on the DP-L10 sets WS=4 for both flash and XRAM, and we have
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
106 always used the same setting in FreeCalypso when running on this target. When
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
107 we made our FCDEV3B hardware using the same Spansion flash+RAM chip copied from
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
108 the Pirelli DP-L10, we adopted the same WS=4 setting for our own FreeCalypso
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
109 hardware family on the reasoning that it is needed for this chip. But now we
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
110 have a better theoretical foundation: the flash+RAM chip in question has 70 ns
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
111 access time for both flash and pSRAM parts, same as most other flash and RAM
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
112 chips used in most Calypso devices, and the WS=4 setting should really be used
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
113 for all Calypso C035 targets (ARM7 at 52 MHz) with 70 ns memories. Thus the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
114 new FreeCalypso strategy is to treat WS=4 as the generic default for Calypso
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
115 C035 platforms unless explicitly overridden for specific targets, and to stop
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
116 treating TI's reconstructed setup with WS=3 as canonical.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
117
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
118 When running on Openmoko GTA01/02, Mot C11x/12x, Mot C139/140 and SE J100
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
119 targets (this specific list), we are going to keep WS=3 for nCS0 and nCS1 and
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
120 the dummies for nCS2, nCS3 and nCS4 unchanged for now, i.e., run with exactly
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
121 the same MEMIF settings as each manufacturer's respective original official fw.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
122 The reason is political: we are not the product manufacturer of record, and the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
123 error of negative design margin in the memory access timings is the liability
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
124 of FIC/Openmoko and Compal/Motorola/SE, not us. If we change from WS=3 to WS=4
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
125 on these targets, our firmware will necessarily run a little slower, and given
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
126 that the original official fw "works just fine", we may be accused of needlessly
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
127 or artificially slowing down our aftermarket fw. But when we market our own
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
128 handset or modem products under the FreeCalypso trademark, then the full
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
129 responsibility for the entire product (hw+fw) falls on us, hence we use the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
130 correct WS=4 setting.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
131
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
132 Interim WS setting during boot
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
133 ==============================
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
134
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
135 There is one more complication to this picture. The MEMIF settings discussed
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
136 above for the operational phase with Calypso DPLL producing fast clocks are
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
137 made in the Init_Target() function, but there is another interim setting
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
138 established early on in assembly code, used prior to DPLL enabling, when the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
139 ARM7 core runs at unmultiplied 13 MHz or 26 MHz as fed to the Calypso by the
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
140 board. This interim setting is first set in bootloader.s, then again in int.s
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
141 (with the definition residing in the included init.asm file), and the registers
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
142 are set to 0x2A1, meaning WS=1 and 1 dummy cycle.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
143
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
144 Unlike the situation with the censored init.c source file, we have the original
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
145 source for the assembly modules in question, and the only preprocessor
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
146 conditionals found therein are based on BOARD and CHIPSET symbols. Remember
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
147 that TI's Leonardo board never got its own BOARD number, instead it shares
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
148 BOARD=41 with D-Sample, yet the two boards have different Calypso clock inputs:
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
149 13 MHz on the DS, 26 MHz on the Leonardo. The C code in init.c (this part
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
150 survived in the LoCosto source) uses a preprocessor conditional on the RF_FAM
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
151 symbol to differentiate between 13 MHz and 26 MHz input clock arrangements, but
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
152 there is no conditional of any such sort in the assembly code. Thus it is my
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
153 (Mother Mychaela's) educated guess that the WS=1 setting was chosen assuming a
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
154 13 MHz clock, and when Leonardo came along with its 26 MHz clock, the problem
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
155 spot was once again overlooked.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
156
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
157 WS=1 at 13 MHz is equivalent to WS=7 at 52 MHz, thus there is plenty of margin.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
158 But WS=1 at 26 MHz is equivalent to WS=3 at 52 MHz, once again putting us in
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
159 the troubled territory of negative margin with 70 ns flash and RAM chips.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
160 Except that this case is even more difficult for firmware engineers to spot:
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
161 Pirelli's fw still has the same 0x2A1 setting in its early boot path, i.e.,
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
162 their fw engineers have changed WS=3 to WS=4 for the main body of the fw, but
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
163 missed the early boot code.
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
164
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
165 The solution adopted for FreeCalypso is to change the early MEMIF setting from
3d65bdaf00da MEMIF-wait-states article written
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
166 0x2A1 to 0x2A2, i.e., set WS=2 for the interim boot phase.