<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/fs/fat/fat_write.c, branch v2014.01</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/fs/fat/fat_write.c?h=v2014.01</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/fs/fat/fat_write.c?h=v2014.01'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2013-11-08T20:25:13Z</updated>
<entry>
<title>fs:fat: fix set file name function</title>
<updated>2013-11-08T20:25:13Z</updated>
<author>
<name>Piotr Wilczek</name>
<email>p.wilczek@samsung.com</email>
</author>
<published>2013-10-11T13:43:33Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=73dc8328c3d842c9093d358dad61d4fd8b3fa2c5'/>
<id>urn:sha1:73dc8328c3d842c9093d358dad61d4fd8b3fa2c5</id>
<content type='text'>
Curently memcpy copies string without null terminating char because
function strlen returns only number of characters excluding
null terminating character. Replace memcpy with strcpy.

Signed-off-by: Piotr Wilczek &lt;p.wilczek@samsung.com&gt;
Signed-off-by: Kyungmin Park &lt;kyungmin.park@samsung.com&gt;
CC: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>fs: fat: don't call disk_write with zero sector num</title>
<updated>2013-09-06T17:09:07Z</updated>
<author>
<name>Wu, Josh</name>
<email>Josh.wu@atmel.com</email>
</author>
<published>2013-07-24T09:55:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6b8f185faf29db302a20f584cc54253667535f08'/>
<id>urn:sha1:6b8f185faf29db302a20f584cc54253667535f08</id>
<content type='text'>
In the set_cluster() function, it will convert the buffer size to sector
numbers. Then call disk_write() to write by sector.
For remaining buffer, the size is less than a sector, call disk_write()
again to write them in one sector.

But if the total buffer size is less then one sector, the original code
will call disk_write() with zero sector number. It is unnecessary.
So this patch fix this. Now it will not call disk_write() if total buffer size
is less than one sector.

Signed-off-by: Josh Wu &lt;josh.wu@atmel.com&gt;
</content>
</entry>
<entry>
<title>Add GPL-2.0+ SPDX-License-Identifier to source files</title>
<updated>2013-07-24T13:44:38Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2013-07-08T07:37:19Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1a4596601fd395f3afb8f82f3f840c5e00bdd57a'/>
<id>urn:sha1:1a4596601fd395f3afb8f82f3f840c5e00bdd57a</id>
<content type='text'>
Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
[trini: Fixup common/cmd_io.c]
Signed-off-by: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>fs/fat: Don't multiply fatsize with sector size</title>
<updated>2013-05-01T20:24:02Z</updated>
<author>
<name>Egbert Eich</name>
<email>eich@suse.com</email>
</author>
<published>2013-04-09T21:11:55Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=bc8d98713f10582f4e35b9208f1b967c6a9f9953'/>
<id>urn:sha1:bc8d98713f10582f4e35b9208f1b967c6a9f9953</id>
<content type='text'>
Bugfix:
Here at this place we need the fat size in sectors not bytes.
This was found during code review when adding support for storage
devices with blocksizes != 512.

Signed-off-by: Egbert Eich &lt;eich@suse.com&gt;
</content>
</entry>
<entry>
<title>FAT: remove ifdefs to make the code more readable</title>
<updated>2013-02-04T14:05:47Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@gmail.com</email>
</author>
<published>2012-12-13T03:30:10Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cb940c7ede85994847c80f4300f2f7eac6e94c48'/>
<id>urn:sha1:cb940c7ede85994847c80f4300f2f7eac6e94c48</id>
<content type='text'>
ifdefs in the code are making it harder to read.
The use of simple if(vfat_enabled) makes no more code and is cleaner.
(the code is discarded by the compiler instead of the preprocessor.)
NB: if -O0 is used, the code won't be discarded

and bonus, now the code compiles even if CONFIG_SUPPORT_VFAT is not
defined.

Signed-off-by: Richard Genoud &lt;richard.genoud@gmail.com&gt;
</content>
</entry>
<entry>
<title>FAT: use toupper/tolower instead of recoding them</title>
<updated>2013-02-04T14:05:46Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@gmail.com</email>
</author>
<published>2012-12-13T00:47:36Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fb7e16cc1c1f5d5bb212118b89218b2e706226b2'/>
<id>urn:sha1:fb7e16cc1c1f5d5bb212118b89218b2e706226b2</id>
<content type='text'>
toupper/tolower function are already declared, so use them.

Signed-off-by: Richard Genoud &lt;richard.genoud@gmail.com&gt;
Acked-by: Marek Vasut &lt;marex@denx.de&gt;
Acked-by: Stefano Babic &lt;sbabic@denx.de&gt;
</content>
</entry>
<entry>
<title>fs: fat: Fix mkcksum() function parameters</title>
<updated>2012-10-17T14:59:11Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2012-10-09T07:20:22Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ff04f6d1224d8952b566b8671222151495883073'/>
<id>urn:sha1:ff04f6d1224d8952b566b8671222151495883073</id>
<content type='text'>
The mkcksum() function now takes one parameter, the pointer to
11-byte wide character array, which it then operates on.

Currently, the function is wrongly passed (dir_entry)-&gt;name, which
is only 8-byte wide character array. Though by further inspecting
the dir_entry structure, it can be noticed that the name[8] entry
is immediatelly followed by ext[3] entry. Thus, name[8] and ext[3]
in the dir_entry structure actually work as this 11-byte wide array
since they're placed right next to each other by current compiler
behavior.

Depending on this is obviously wrong, thus fix this by correctly
passing both (dir_entry)-&gt;name and (dir_entry)-&gt;ext to the mkcksum()
function and adjust the function appropriately.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Cc: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>FAT: Make it possible to read from any file position</title>
<updated>2012-09-26T18:11:32Z</updated>
<author>
<name>Benoît Thébaudeau</name>
<email>benoit.thebaudeau@advansee.com</email>
</author>
<published>2012-09-18T08:14:56Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1170e634dd2a6fdd541ae2153fd3fd73919da8fc'/>
<id>urn:sha1:1170e634dd2a6fdd541ae2153fd3fd73919da8fc</id>
<content type='text'>
When storage devices contain files larger than the embedded RAM, it is
useful to be able to read these files by chunks, e.g. for a software
update to the embedded NAND Flash from an external storage device (USB
stick, SD card, etc.).

Hence, this patch makes it possible by adding a new FAT API to read
files from a given position. This patch also adds this feature to the
fatload command.

Signed-off-by: Benoît Thébaudeau &lt;benoit.thebaudeau@advansee.com&gt;
Cc: Wolfgang Denk &lt;wd@denx.de&gt;
Signed-off-by: Tom Rini &lt;trini@ti.com&gt;
</content>
</entry>
<entry>
<title>fs/fat/fat_write.c: Fix GCC 4.6 warnings</title>
<updated>2012-03-26T08:58:54Z</updated>
<author>
<name>Anatolij Gustschin</name>
<email>agust@denx.de</email>
</author>
<published>2012-03-24T22:40:56Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=bf6b6af74655506ce23ca04d6e5c6b3ad3fcf490'/>
<id>urn:sha1:bf6b6af74655506ce23ca04d6e5c6b3ad3fcf490</id>
<content type='text'>
Fix:
fat_write.c: In function 'find_directory_entry':
fat_write.c:826:8: warning: variable 'prevcksum' set but not used
[-Wunused-but-set-variable]
fat_write.c: In function 'do_fat_write':
fat_write.c:933:6: warning: variable 'root_cluster' set but not used
[-Wunused-but-set-variable]
fat_write.c:925:12: warning: variable 'slotptr' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Anatolij Gustschin &lt;agust@denx.de&gt;
Cc: Donggeun Kim &lt;dg77.kim@samsung.com&gt;
Acked-by: Maximilian Schwerin &lt;mvs@tigris.de&gt;
Acked-by: Kyungmin Park &lt;kyungmin.park@samsung.com&gt;
</content>
</entry>
<entry>
<title>FAT write: Fix compile errors</title>
<updated>2012-03-24T22:27:32Z</updated>
<author>
<name>Donggeun Kim</name>
<email>dg77.kim@samsung.com</email>
</author>
<published>2012-03-22T04:38:55Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=079df7223c05518687cbd2eb9ad1b3d1712ee6d7'/>
<id>urn:sha1:079df7223c05518687cbd2eb9ad1b3d1712ee6d7</id>
<content type='text'>
This patch removes compile errors introduced by
commit 9813b750f32c0056f0a35813b9a9ec0f68b664af
'fs/fat: Fix FAT detection to support non-DOS partition tables'

fat_write.c: In function 'disk_write':
fat_write.c:54: error: 'part_offset' undeclared (first use in this function)
fat_write.c:54: error: (Each undeclared identifier is reported only once
fat_write.c:54: error: for each function it appears in.)
fat_write.c: In function 'do_fat_write':
fat_write.c:950: error: 'part_size' undeclared (first use in this function)

These errors only appear when this code is enabled by
defining CONFIG_FAT_WRITE option.

This patch was originally part of

http://article.gmane.org/gmane.comp.boot-loaders.u-boot/121847

Signed-off-by: Donggeun Kim &lt;dg77.kim@samsung.com&gt;
Signed-off-by: Kyungmin Park &lt;kyungmin.park@samsung.com&gt;
Signed-off-by: Maximilian Schwerin &lt;mvs@tigris.de&gt;

Fixed patch author and added all needed SoB from the original patch
and also submitter's SoB. Extended commit log.
Signed-off-by: Anatolij Gustschin &lt;agust@denx.de&gt;
</content>
</entry>
</feed>
