<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/drivers/dfu, branch v2015.10-rc2</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/drivers/dfu?h=v2015.10-rc2</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/drivers/dfu?h=v2015.10-rc2'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2015-07-27T22:53:52Z</updated>
<entry>
<title>dfu: fix 64-bit compile warnings</title>
<updated>2015-07-27T22:53:52Z</updated>
<author>
<name>Stephen Warren</name>
<email>swarren@nvidia.com</email>
</author>
<published>2015-07-22T20:54:04Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=e621c7ab9ace9349787881dea801739860fedcd9'/>
<id>urn:sha1:e621c7ab9ace9349787881dea801739860fedcd9</id>
<content type='text'>
Use %p to print pointers.

The max value of (i_buf - i_buf_start) should be dfu_buf_size, which is
an unsigned long, so cast the pointer difference to that type to print.

Change-Id: Iee242df9f8eb091aecfe0cea4c282b28b547acfe
Signed-off-by: Stephen Warren &lt;swarren@nvidia.com&gt;
Signed-off-by: Tom Warren &lt;twarren@nvidia.com&gt;
</content>
</entry>
<entry>
<title>usb: USB download gadget and functions config options coherent naming</title>
<updated>2015-07-22T06:57:53Z</updated>
<author>
<name>Paul Kocialkowski</name>
<email>contact@paulk.fr</email>
</author>
<published>2015-06-12T17:56:58Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=01acd6abbdd5a5951f68d08c245550c720ea6ad8'/>
<id>urn:sha1:01acd6abbdd5a5951f68d08c245550c720ea6ad8</id>
<content type='text'>
This introduces a coherent scheme for naming USB download gadget and functions
config options. The download USB gadget config option is moved to
CONFIG_USB_GADGET_DOWNLOAD for better consistency with other gadgets and each
function's config option is moved to a CONFIG_USB_FUNCTION_ prefix.

Signed-off-by: Paul Kocialkowski &lt;contact@paulk.fr&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;

Test HW: Odroid_XU3 (Exynos5422), trats (Exynos4210)
</content>
</entry>
<entry>
<title>dfu: nand: Verify writes</title>
<updated>2015-03-31T04:24:38Z</updated>
<author>
<name>Peter Tyser</name>
<email>ptyser@xes-inc.com</email>
</author>
<published>2015-02-03T17:58:14Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=9ac71f112eb3cffc42e012fcdf5009e5b3b01a1d'/>
<id>urn:sha1:9ac71f112eb3cffc42e012fcdf5009e5b3b01a1d</id>
<content type='text'>
Previously NAND writes were not verified and could fail silently.  Add
a verification step after all writes to NAND.

Signed-off-by: Peter Tyser &lt;ptyser@xes-inc.com&gt;
Reviewed-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
Tested-by: Heiko Schocher &lt;hs@denx.de&gt;
Acked-by: Heiko Schocher &lt;hs@denx.de&gt;
</content>
</entry>
<entry>
<title>dfu: mmc: file buffer: remove static allocation</title>
<updated>2015-03-09T15:13:28Z</updated>
<author>
<name>Przemyslaw Marczak</name>
<email>p.marczak@samsung.com</email>
</author>
<published>2015-03-04T13:01:23Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=41ac233c61af654e86eb980ba02e7e38d05cfcd0'/>
<id>urn:sha1:41ac233c61af654e86eb980ba02e7e38d05cfcd0</id>
<content type='text'>
For writing files, DFU implementation requires the file buffer
with the len at least of file size. For big files it requires
the same big buffer.

Previously the file buffer was allocated as a static variable,
so it was a part of U-Boot .bss section. For 32MiB len of buffer
we have 32MiB of additional space, required for this section.

The .bss needs to be cleared after the relocation.
This introduces an additional boot delay at every start, but usually
the dfu feature is not required at the standard boot, so the buffer
should be allocated only if required.

This patch removes the static allocation of this buffer,
and alloc it with memalign after first call of function:
- dfu_fill_entity_mmc()
and the buffer is freed on dfu_free_entity() call.

This was tested on Trats2.
A quick test with trace. Boot time from start to main_loop() entry:
- ~888ms - before this change (arch memset enabled for .bss clear)
- ~464ms - after this change

Signed-off-by: Przemyslaw Marczak &lt;p.marczak@samsung.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Cc: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
Cc: Stephen Warren &lt;swarren@nvidia.com&gt;
Cc: Pantelis Antoniou &lt;panto@antoniou-consulting.com&gt;
Cc: Tom Rini &lt;trini@konsulko.com&gt;
Cc: Marek Vasut &lt;marek.vasut@gmail.com&gt;
</content>
</entry>
<entry>
<title>dfu: samsung: move call to set_dfu_alt_info() to dfu common code</title>
<updated>2015-02-25T16:47:02Z</updated>
<author>
<name>Przemyslaw Marczak</name>
<email>p.marczak@samsung.com</email>
</author>
<published>2015-02-17T11:24:11Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=899a52821515d79f9e4e525fd3098bdb574062ed'/>
<id>urn:sha1:899a52821515d79f9e4e525fd3098bdb574062ed</id>
<content type='text'>
This common call can be used for setting proper entities based
on dfu command arguments.
The config: CONFIG_SET_DFU_ALT_INFO, was used only for few configs,
and now it is common.

The board file should implement:
- set_dfu_alt_info() function

Signed-off-by: Przemyslaw Marczak &lt;p.marczak@samsung.com&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
[Test HW: Odroid U3 (Exynos 4412)]
</content>
</entry>
<entry>
<title>dfu: dfu_get_buf: check the value of env dfu_bufsiz before use</title>
<updated>2014-12-18T11:26:06Z</updated>
<author>
<name>Przemyslaw Marczak</name>
<email>p.marczak@samsung.com</email>
</author>
<published>2014-12-15T09:34:11Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f597fc3d4c4f73d45670649d6f3e678934139c25'/>
<id>urn:sha1:f597fc3d4c4f73d45670649d6f3e678934139c25</id>
<content type='text'>
In function dfu_get_buf(), the size of allocated buffer could
be defined by the env variable. The size from this variable
was passed for memalign() without checking its value.
And the the memalign will return non null pointer for size 0.

This could possibly cause data abort, so now the value of var
is checked before use. And if this variable is set to 0 then
the default size will be used.

This commit also changes the base passed to simple_strtoul()
to 0. Now decimal and hex values can be used for the variable
dfu_bufsiz.

Signed-off-by: Przemyslaw Marczak &lt;p.marczak@samsung.com&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
[TestHW: Exynos4412-Trats2]
</content>
</entry>
<entry>
<title>dfu: mmc: check if mmc device exists in mmc_block_op()</title>
<updated>2014-12-18T11:26:06Z</updated>
<author>
<name>Przemyslaw Marczak</name>
<email>p.marczak@samsung.com</email>
</author>
<published>2014-12-15T09:34:09Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=7da6fa27168c468e58814d54fe3f66109036ec2e'/>
<id>urn:sha1:7da6fa27168c468e58814d54fe3f66109036ec2e</id>
<content type='text'>
The function mmc_block_op() is the last function before
the physicall data write, but the mmc device pointer is not
checked. If mmc device not exists, then data abort will occur.
To avoid this, first the mmc device pointer is checked.

Signed-off-by: Przemyslaw Marczak &lt;p.marczak@samsung.com&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
[TestHW: Exynos4412-Trats2]
</content>
</entry>
<entry>
<title>usb, g_dnl: generalize DFU detach functions</title>
<updated>2014-12-18T11:26:05Z</updated>
<author>
<name>Rob Herring</name>
<email>robh@kernel.org</email>
</author>
<published>2014-12-10T20:43:03Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fe1b28c9f0954047f20b20253596b5ca9aef4a32'/>
<id>urn:sha1:fe1b28c9f0954047f20b20253596b5ca9aef4a32</id>
<content type='text'>
In order to add detach functions for fastboot, make the DFU detach related
functions common so they can be shared.

Signed-off-by: Rob Herring &lt;robh@kernel.org&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
[TestHW: Exynos4412-Trats2]
</content>
</entry>
<entry>
<title>Merge branch 'master' of git://git.denx.de/u-boot-usb</title>
<updated>2014-12-11T23:40:49Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@ti.com</email>
</author>
<published>2014-12-11T23:40:49Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fc9b0b80435cda721fbdbe507c9e4f388b0ea62b'/>
<id>urn:sha1:fc9b0b80435cda721fbdbe507c9e4f388b0ea62b</id>
<content type='text'>
Conflicts:
	board/freescale/mx6sxsabresd/mx6sxsabresd.c

Signed-off-by: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>linux/kernel.h: sync min, max, min3, max3 macros with Linux</title>
<updated>2014-11-23T11:48:30Z</updated>
<author>
<name>Masahiro Yamada</name>
<email>yamada.m@jp.panasonic.com</email>
</author>
<published>2014-11-06T18:03:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=b41411954d4ccf6ddaa581178462017557b82b5d'/>
<id>urn:sha1:b41411954d4ccf6ddaa581178462017557b82b5d</id>
<content type='text'>
U-Boot has never cared about the type when we get max/min of two
values, but Linux Kernel does.  This commit gets min, max, min3, max3
macros synced with the kernel introducing type checks.

Many of references of those macros must be fixed to suppress warnings.
We have two options:
 - Use min, max, min3, max3 only when the arguments have the same type
   (or add casts to the arguments)
 - Use min_t/max_t instead with the appropriate type for the first
   argument

Signed-off-by: Masahiro Yamada &lt;yamada.m@jp.panasonic.com&gt;
Acked-by: Pavel Machek &lt;pavel@denx.de&gt;
Acked-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
Tested-by: Lukasz Majewski &lt;l.majewski@samsung.com&gt;
[trini: Fixup arch/blackfin/lib/string.c]
Signed-off-by: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
</feed>
