<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/drivers/ddr/altera, branch v2016.03</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/drivers/ddr/altera?h=v2016.03</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/drivers/ddr/altera?h=v2016.03'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2016-01-16T06:07:22Z</updated>
<entry>
<title>ddr: altera: Init the rule ID in debug code</title>
<updated>2016-01-16T06:07:22Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-12-29T08:38:52Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1720fad0f19827b94a845dc26ac7d390edbae836'/>
<id>urn:sha1:1720fad0f19827b94a845dc26ac7d390edbae836</id>
<content type='text'>
Init the rule ID, otherwise the debug code will always dump the
protection settings entry 0.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Cc: Chin Liang See &lt;clsee@altera.com&gt;
Cc: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
Cc: Dinh Nguyen &lt;dinh.linux@gmail.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: Repair uninited variable</title>
<updated>2015-08-23T09:56:19Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-10T21:01:43Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=35e47b7132fa515e32189077ec7b80090562c709'/>
<id>urn:sha1:35e47b7132fa515e32189077ec7b80090562c709</id>
<content type='text'>
Fix the following problem:
drivers/ddr/altera/sequencer.c: In function 'sdram_calibration_full':
drivers/ddr/altera/sequencer.c:1943:25: warning: 'found_failing_read' may be used uninitialized in this function [-Wmaybe-uninitialized]
  if (found_passing_read &amp;&amp; found_failing_read)
                         ^
drivers/ddr/altera/sequencer.c:1803:26: note: 'found_failing_read' was declared here
  u32 found_passing_read, found_failing_read, initial_failing_dtap;
                          ^

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
</content>
</entry>
<entry>
<title>ddr: altera: Replace float multiplication with integer one</title>
<updated>2015-08-23T09:56:19Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-10T20:50:11Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6d7a33301a05472bc788415c1d1575041e478931'/>
<id>urn:sha1:6d7a33301a05472bc788415c1d1575041e478931</id>
<content type='text'>
This gem is really really rare, there was an actual float used in
the Altera DDR init code, which pulled in floating point ops from
the libgcc, just wow.

Since we don't support floating point operations the same way Linux
does not support them, replace this with an integer multiplication
and division combo. This removes some 2kiB of size from the SPL as
the floating point ops are no longer pulled in from libgcc.

This was detected by enabling CONFIG_USE_PRIVATE_LIBGCC=y , which
does not contain the floating point bits.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Clean checkpatch issues</title>
<updated>2015-08-08T12:14:30Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:47:01Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=139823ecb2472de2e8e5a785cfa608ec00618375'/>
<id>urn:sha1:139823ecb2472de2e8e5a785cfa608ec00618375</id>
<content type='text'>
Fix most of the dangling checkpatch issues, no functional change.
There are still 7 warnings, 1 checks , but those are left in place
for the sake of readability of the code.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Clean data types</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:42:26Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=5ded7320c8ba99fc413cb5dc3591f26e052986c3'/>
<id>urn:sha1:5ded7320c8ba99fc413cb5dc3591f26e052986c3</id>
<content type='text'>
Replace uintNN_t with uNN. No functional change.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Pluck out misc macros from code</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:26:55Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=96fd4362cf8431be4601b723d68a5093cc8fa460'/>
<id>urn:sha1:96fd4362cf8431be4601b723d68a5093cc8fa460</id>
<content type='text'>
Actually convert the sequencer code to use socfpga_sdram_misc_config
instead of the various macros. This is just an sed exercise here, no
manual coding needed.

This patch actually removes the need to include any board-specific
files in sequencer.c , so sequencer.c namespace is now no longer
poluted by QTS-generated macros.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Zap SEQ_T(INIT|RESET)_CNTR._VAL</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:24:12Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3cd0906cc21bc88bfe318b32a08c5fd8d8915b7f'/>
<id>urn:sha1:3cd0906cc21bc88bfe318b32a08c5fd8d8915b7f</id>
<content type='text'>
This is another macro used to obfuscate the real code. The
T(INIT|RESET)_CNTR._VAL is always defined, so this indirection
is unnecessary. Get rid of this.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Zap VFIFO_SIZE</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:21:56Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=98cfc9058bf4b4019bac34765d235d2dcb0192bb'/>
<id>urn:sha1:98cfc9058bf4b4019bac34765d235d2dcb0192bb</id>
<content type='text'>
Just use READ_VALID_FIFO_SIZE directly, no need for this macro obfuscation.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Wrap misc remaining macros</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:18:47Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=042ff2d0facaa520f1bc02b0695eb6a4e1d30b7a'/>
<id>urn:sha1:042ff2d0facaa520f1bc02b0695eb6a4e1d30b7a</id>
<content type='text'>
Introduce structure socfpga_sdram_misc_config to wrap the remaining
misc configuration values in board file. Again, introduce a function,
socfpga_get_sdram_misc_config(), which returns this the structure. This
is almost the final step toward wrapping the nasty QTS generated macros
in board files and reducing the pollution of the namespace.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
<entry>
<title>ddr: altera: sequencer: Pluck out IO_* macros from code</title>
<updated>2015-08-08T12:14:29Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2015-08-02T17:10:58Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=160695d88612d1e1c094fef33d279c297796b881'/>
<id>urn:sha1:160695d88612d1e1c094fef33d279c297796b881</id>
<content type='text'>
Actually convert the sequencer code to use socfpga_sdram_io_config
instead of the IO_* macros. This is just an sed excercise here, no
manual coding needed.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Dinh Nguyen &lt;dinguyen@opensource.altera.com&gt;
</content>
</entry>
</feed>
