<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/cmd, branch v2016.03-rc3</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<entry>
<title>trace: Fix compiler warnings in trace</title>
<updated>2016-02-26T15:53:10+00:00</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-02-24T16:14:45+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=5eca977935bf58f4227b2681e612a452a0da0286'/>
<id>5eca977935bf58f4227b2681e612a452a0da0286</id>
<content type='text'>
With min() we must use the same type for each parameter. Fix two problems
in trace.c which produce compiler warnings.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
With min() we must use the same type for each parameter. Fix two problems
in trace.c which produce compiler warnings.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'master' of git://git.denx.de/u-boot-usb</title>
<updated>2016-02-24T23:44:17+00:00</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2016-02-24T19:26:20+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fec26e7270ad31e4179d1bd189e5cd8ab93ccba3'/>
<id>fec26e7270ad31e4179d1bd189e5cd8ab93ccba3</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>cmd: mem: Show 64bit addresses which are tested</title>
<updated>2016-02-24T23:44:14+00:00</updated>
<author>
<name>Michal Simek</name>
<email>michal.simek@xilinx.com</email>
</author>
<published>2016-02-24T07:36:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=dfe461d6b5a196c6f13ab0b4ccbeb1b16aa22cf5'/>
<id>dfe461d6b5a196c6f13ab0b4ccbeb1b16aa22cf5</id>
<content type='text'>
Fix print message to show full 64bit addresses.

Signed-off-by: Michal Simek &lt;michal.simek@xilinx.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix print message to show full 64bit addresses.

Signed-off-by: Michal Simek &lt;michal.simek@xilinx.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>booti: Help text rework.</title>
<updated>2016-02-24T23:44:12+00:00</updated>
<author>
<name>Karsten Merker</name>
<email>merker@debian.org</email>
</author>
<published>2016-02-22T19:59:08+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6f6051fac360cf6ca88cfcfa3cb377778ef5742c'/>
<id>6f6051fac360cf6ca88cfcfa3cb377778ef5742c</id>
<content type='text'>
Fix spelling errors in the "booti" help text and bring it more
in line with the bootm/bootz help texts.

Signed-off-by: Karsten Merker &lt;merker@debian.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix spelling errors in the "booti" help text and bring it more
in line with the bootm/bootz help texts.

Signed-off-by: Karsten Merker &lt;merker@debian.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>dfu: usb: f_dfu: Set deferred call for dfu_flush() function</title>
<updated>2016-02-24T18:12:32+00:00</updated>
<author>
<name>Lukasz Majewski</name>
<email>l.majewski@samsung.com</email>
</author>
<published>2016-01-28T15:14:49+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fc18f8d170ecc7e15269ad5312ec643addb42491'/>
<id>fc18f8d170ecc7e15269ad5312ec643addb42491</id>
<content type='text'>
This patch fixes situation when one would like to write large file into
medium with the file system (fat, ext4, etc).
This change sets file size limitation to the DFU internal buffer size.

Since u-boot is not supporting interrupts and seek on file systems, it
becomes challenging to store large file appropriately.

To reproduce this error - create large file (around 26 MiB) and sent it
to the target board.

Lets examine the flow of USB transactions:

0. DFU uses EP0 with 64B MPS [Max Packet Size]

1. Send file - OUT (PC-&gt;target) - dat_26MiB.img is sent with 4096 B transactions

2. Get status - OUT (PC-&gt;target) - wait for DFU_STATE_dfuDNLOAD_IDLE (0x05) sent
				   from target board - IN transaction
				   (target-&gt;PC)

3. The whole file content is sent to target - OUT (PC-&gt;target) with ZLP [Zero
					      Length Packet]

Now the interesting part starts:

4. OUT (PC-&gt;target) Setup transaction (request to share DFU state)

5. IN (target-&gt;PC) - reply the current DFU state
	- In the UDC driver the req-&gt;completion (with dfu_flush) is called
	  after successful IN transfer.
	- The dfu_flush() (called from req-&gt;completion callback) saves the
	  whole file at once (u-boot doesn't support seek on fs).
	  Such operation takes considerable time. When the file
	  is large - e.g. 26MiB - this time may be more than 5 seconds.

6. OUT (PC-&gt;target) - ZLP, is send in the same time when dfu_flush()
 writes data to eMMC memory.
 The dfu-util application has hard coded timeout on USB transaction
 completion set to 5 seconds (it uses libusb calls).

When the file to store is large (e.g. 26 MiB) the time needed to write it
may excess the dfu-util timeout and following error message will be displayed:
"unable to read DFU status" on the HOST PC console.

This change is supposed to leverage DFU's part responsible for storing files
on file systems. Other DFU operations - i.e. raw/partition write to NAND and
eMMC should work as before.

The only functional change is the error reporting. When dfu_flush() fails
the u-boot prompt will exit with error information and dfu-util application
exits afterwards as well.

Test HW:
- Odroid XU3 (Exynos5433) - test with large file
- Trats (Exynos4210) - test for regression - eMMC, raw,

Signed-off-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
Reported-by: Alex Gdalevich &lt;agdalevich@axion-biosystems.com&gt;
Tested-by: Stephen Warren &lt;swarren@nvidia.com&gt;
Tested-by: Heiko Schocher &lt;hs@denx.de&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes situation when one would like to write large file into
medium with the file system (fat, ext4, etc).
This change sets file size limitation to the DFU internal buffer size.

Since u-boot is not supporting interrupts and seek on file systems, it
becomes challenging to store large file appropriately.

To reproduce this error - create large file (around 26 MiB) and sent it
to the target board.

Lets examine the flow of USB transactions:

0. DFU uses EP0 with 64B MPS [Max Packet Size]

1. Send file - OUT (PC-&gt;target) - dat_26MiB.img is sent with 4096 B transactions

2. Get status - OUT (PC-&gt;target) - wait for DFU_STATE_dfuDNLOAD_IDLE (0x05) sent
				   from target board - IN transaction
				   (target-&gt;PC)

3. The whole file content is sent to target - OUT (PC-&gt;target) with ZLP [Zero
					      Length Packet]

Now the interesting part starts:

4. OUT (PC-&gt;target) Setup transaction (request to share DFU state)

5. IN (target-&gt;PC) - reply the current DFU state
	- In the UDC driver the req-&gt;completion (with dfu_flush) is called
	  after successful IN transfer.
	- The dfu_flush() (called from req-&gt;completion callback) saves the
	  whole file at once (u-boot doesn't support seek on fs).
	  Such operation takes considerable time. When the file
	  is large - e.g. 26MiB - this time may be more than 5 seconds.

6. OUT (PC-&gt;target) - ZLP, is send in the same time when dfu_flush()
 writes data to eMMC memory.
 The dfu-util application has hard coded timeout on USB transaction
 completion set to 5 seconds (it uses libusb calls).

When the file to store is large (e.g. 26 MiB) the time needed to write it
may excess the dfu-util timeout and following error message will be displayed:
"unable to read DFU status" on the HOST PC console.

This change is supposed to leverage DFU's part responsible for storing files
on file systems. Other DFU operations - i.e. raw/partition write to NAND and
eMMC should work as before.

The only functional change is the error reporting. When dfu_flush() fails
the u-boot prompt will exit with error information and dfu-util application
exits afterwards as well.

Test HW:
- Odroid XU3 (Exynos5433) - test with large file
- Trats (Exynos4210) - test for regression - eMMC, raw,

Signed-off-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
Reported-by: Alex Gdalevich &lt;agdalevich@axion-biosystems.com&gt;
Tested-by: Stephen Warren &lt;swarren@nvidia.com&gt;
Tested-by: Heiko Schocher &lt;hs@denx.de&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>i2c: spd: Extend SPD memory types</title>
<updated>2016-02-22T15:21:12+00:00</updated>
<author>
<name>Michal Simek</name>
<email>michal.simek@xilinx.com</email>
</author>
<published>2016-02-15T10:58:37+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=18c4e7f71c3ab7298a5453c492e0a9c46cd38c23'/>
<id>18c4e7f71c3ab7298a5453c492e0a9c46cd38c23</id>
<content type='text'>
Decode DDR, DDR3 and DDR4 memories.

Signed-off-by: Michal Simek &lt;michal.simek@xilinx.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Decode DDR, DDR3 and DDR4 memories.

Signed-off-by: Michal Simek &lt;michal.simek@xilinx.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>gpio: Correct handling of 'gpio status'</title>
<updated>2016-02-15T20:58:25+00:00</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-02-14T23:28:59+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=4c80c53c00ac6583f22938fc98e4df688acdf4ce'/>
<id>4c80c53c00ac6583f22938fc98e4df688acdf4ce</id>
<content type='text'>
This is broken - we need to look at the first two characters to distinguish
'gpio status' from 'gpio set'.

Fixes: 0ffe6ab5 (gpio: Allow 's' as an abbreviation for 'status')
Reported-by: Soeren Moch &lt;smoch@web.de&gt;
Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Tested-by: Soeren Moch &lt;smoch@web.de&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Tested-by: Hannes Schmelzer &lt;oe5hpm@oevsv.at&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is broken - we need to look at the first two characters to distinguish
'gpio status' from 'gpio set'.

Fixes: 0ffe6ab5 (gpio: Allow 's' as an abbreviation for 'status')
Reported-by: Soeren Moch &lt;smoch@web.de&gt;
Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Tested-by: Soeren Moch &lt;smoch@web.de&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Tested-by: Hannes Schmelzer &lt;oe5hpm@oevsv.at&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>itest: allow map_physmem to return 0 in limited cases</title>
<updated>2016-02-09T22:41:19+00:00</updated>
<author>
<name>Stephen Warren</name>
<email>swarren@nvidia.com</email>
</author>
<published>2016-01-29T23:10:04+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=986fe378172fa4bd5be1432ab2f8f7c2b1c43bae'/>
<id>986fe378172fa4bd5be1432ab2f8f7c2b1c43bae</id>
<content type='text'>
On some systems, RAM starts at address 0. If the user executes itest
against address 0 on such a system, it will call map_physmem(0, ...)
which will return 0 back; mapping only changes the address on sandbox.
This causes itest to believe map_physmem() has failed, and hence fails
the comparison.

Fix itest so that it allows map_physmem() to return 0 /if/ the orignal
address passed to it was also 0.

This fixes "tegra-uboot-flasher flash" on Tegra20.

This has the disadvantage that on sandbox, failed mapping attempts for
address 0 are not detected. Instead, should the code only call
map_physmem() on sandbox? Or, should map_physmem() return its error status
some other way. Or, should the special case only be allowed on systems
where the base of RAM is 0 somehow?

Fixes: 7861204c9af7 ("itest: make memory access work under sandbox")
Signed-off-by: Stephen Warren &lt;swarren@nvidia.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
On some systems, RAM starts at address 0. If the user executes itest
against address 0 on such a system, it will call map_physmem(0, ...)
which will return 0 back; mapping only changes the address on sandbox.
This causes itest to believe map_physmem() has failed, and hence fails
the comparison.

Fix itest so that it allows map_physmem() to return 0 /if/ the orignal
address passed to it was also 0.

This fixes "tegra-uboot-flasher flash" on Tegra20.

This has the disadvantage that on sandbox, failed mapping attempts for
address 0 are not detected. Instead, should the code only call
map_physmem() on sandbox? Or, should map_physmem() return its error status
some other way. Or, should the special case only be allowed on systems
where the base of RAM is 0 somehow?

Fixes: 7861204c9af7 ("itest: make memory access work under sandbox")
Signed-off-by: Stephen Warren &lt;swarren@nvidia.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove unused CONFIG_CMD_HWFLOW option and associated dead code.</title>
<updated>2016-02-08T15:22:44+00:00</updated>
<author>
<name>David Müller (ELSOFT AG)</name>
<email>d.mueller@elsoft.ch</email>
</author>
<published>2016-02-06T06:57:26+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=b7fbdc60abaf3224e9fca62d0c4759072bdc60ad'/>
<id>b7fbdc60abaf3224e9fca62d0c4759072bdc60ad</id>
<content type='text'>
Signed-off-by: David Müller &lt;d.mueller@elsoft.ch&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: David Müller &lt;d.mueller@elsoft.ch&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cmd: Fix control bmp_display()</title>
<updated>2016-02-06T12:53:09+00:00</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-01-30T22:45:15+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=e517db73a61376b940a9b352eb717f703f6b25df'/>
<id>e517db73a61376b940a9b352eb717f703f6b25df</id>
<content type='text'>
All paths should share the same return.

Reported-by: Coverity (CID:134903)

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
All paths should share the same return.

Reported-by: Coverity (CID:134903)

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Tom Rini &lt;trini@konsulko.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
