<feed xmlns='http://www.w3.org/2005/Atom'>
<title>bcm63xx/u-boot/fs/jffs2, branch master</title>
<subtitle>Broadcom-s U-Boot</subtitle>
<id>https://git.openwrt.org/project/bcm63xx/u-boot/atom?h=master</id>
<link rel='self' href='https://git.openwrt.org/project/bcm63xx/u-boot/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/'/>
<updated>2018-05-07T13:34:12Z</updated>
<entry>
<title>SPDX: Convert all of our single license tags to Linux Kernel style</title>
<updated>2018-05-07T13:34:12Z</updated>
<author>
<name>Tom Rini</name>
</author>
<published>2018-05-06T21:58:06Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=83d290c56fab2d38cd1ab4c4cc7099559c1d5046'/>
<id>urn:sha1:83d290c56fab2d38cd1ab4c4cc7099559c1d5046</id>
<content type='text'>
When U-Boot started using SPDX tags we were among the early adopters and
there weren't a lot of other examples to borrow from.  So we picked the
area of the file that usually had a full license text and replaced it
with an appropriate SPDX-License-Identifier: entry.  Since then, the
Linux Kernel has adopted SPDX tags and they place it as the very first
line in a file (except where shebangs are used, then it's second line)
and with slightly different comment styles than us.

In part due to community overlap, in part due to better tag visibility
and in part for other minor reasons, switch over to that style.

This commit changes all instances where we have a single declared
license in the tag as both the before and after are identical in tag
contents.  There's also a few places where I found we did not have a tag
and have introduced one.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>fs/jffs2/mini_inflate.c: use "static inline" instead of "inline"</title>
<updated>2017-10-16T13:42:51Z</updated>
<author>
<name>Vagrant Cascadian</name>
</author>
<published>2017-10-10T16:42:39Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=c005f7edbbc41b33de2c5f8d31a8c12d86ffdb59'/>
<id>urn:sha1:c005f7edbbc41b33de2c5f8d31a8c12d86ffdb59</id>
<content type='text'>
This makes gcc no longer expect an out-of-line version of the
functions being present elsewhere.

This fixes a failure to build on several marvell targets with gcc-7 on
Debian:

  https://bugs.debian.org/877963

Signed-off-by: Adrian Bunk &lt;bunk@debian.org&gt;
Signed-off-by: Vagrant Cascadian &lt;vagrant@debian.org&gt;
</content>
</entry>
<entry>
<title>jffs2: ECC corrected nand data is ignored</title>
<updated>2017-10-16T13:42:51Z</updated>
<author>
<name>Engling, Uwe</name>
</author>
<published>2017-10-10T14:20:55Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=41ba7f525fa21941dee062b87adbbd7dde72f32f'/>
<id>urn:sha1:41ba7f525fa21941dee062b87adbbd7dde72f32f</id>
<content type='text'>
Hello,

I ran into a problem with the JFFS2 filesystem driver implemented in U-Boot.

I've got a NAND device that has correctable ECC errors (corrected somewhere in mtd/nand/nand_base.c).
The NAND driver tells the filesystem layer (jffs2_1pass.c) above that there occurred correctable ECC errors and returns with a "value &gt; 0".
The JFFS2 driver recognizes the corrected ECC errors as real error and skips this block because the only accepts a "return value == 0" as correct.

This problem exists for over 8 years (I checked version 2010.09) so I'm a little bit worried that I interpreted something wrong or didn't get the whole context.

Can someone confirm this bug (and the bugfix) in the u-boot jffs2 driver?

There was a mail in 2012 that mentioned the same problem, but there was no patch:
http://u-boot.10912.n7.nabble.com/JFFS2-seems-to-drop-nand-data-with-ECC-corrections-td142008.html
Sometime after this discussion the return value of nand_read() changed from -EUCLEAN as correctable ECC error to a positive value with the count of ECC corrected errors.

With kind reguards,
Uwe Engling
</content>
</entry>
<entry>
<title>treewide: replace with error() with pr_err()</title>
<updated>2017-10-04T15:59:44Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2017-09-16T05:10:41Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=9b643e312d528f291966c1f30b0d90bf3b1d43dc'/>
<id>urn:sha1:9b643e312d528f291966c1f30b0d90bf3b1d43dc</id>
<content type='text'>
U-Boot widely uses error() as a bit noisier variant of printf().

This macro causes name conflict with the following line in
include/linux/compiler-gcc.h:

  # define __compiletime_error(message) __attribute__((error(message)))

This prevents us from using __compiletime_error(), and makes it
difficult to fully sync BUILD_BUG macros with Linux.  (Notice
Linux's BUILD_BUG_ON_MSG is implemented by using compiletime_assert().)

Let's convert error() into now treewide-available pr_err().

Done with the help of Coccinelle, excluing tools/ directory.

The semantic patch I used is as follows:

// &lt;smpl&gt;
@@@@
-error
+pr_err
 (...)
// &lt;/smpl&gt;

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
[trini: Re-run Coccinelle]
Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>fs: use get_nand_dev_by_index()</title>
<updated>2017-07-12T02:41:47Z</updated>
<author>
<name>Grygorii Strashko</name>
</author>
<published>2017-06-27T00:12:57Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=eb6a5c3c020099bbbc41cf3509718828ec1733cc'/>
<id>urn:sha1:eb6a5c3c020099bbbc41cf3509718828ec1733cc</id>
<content type='text'>
As part of preparation for nand DM conversion the new API has been
introduced to remove direct access to nand_info array. So, use it here
instead of accessing to nand_info array directly.

Signed-off-by: Grygorii Strashko &lt;grygorii.strashko@ti.com&gt;
</content>
</entry>
<entry>
<title>fs: Kconfig: Add a separate option for FS_JFFS2</title>
<updated>2017-05-22T16:45:31Z</updated>
<author>
<name>Simon Glass</name>
</author>
<published>2017-05-17T09:25:38Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=59e12a4a8c219c4a334a2c2023edd1aa20f82a9e'/>
<id>urn:sha1:59e12a4a8c219c4a334a2c2023edd1aa20f82a9e</id>
<content type='text'>
Rather than using CMD_JFFS2 for both the filesystem and its command, we
should have a separate option for each. This allows us to enable JFFS2
support without the command, if desired, which reduces U-Boot's size
slightly.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>treewide: replace #include &lt;asm/errno.h&gt; with &lt;linux/errno.h&gt;</title>
<updated>2016-09-23T21:55:42Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2016-09-21T02:28:55Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=1221ce459d04a428f8880f58581f671b736c3c27'/>
<id>urn:sha1:1221ce459d04a428f8880f58581f671b736c3c27</id>
<content type='text'>
Now, arch/${ARCH}/include/asm/errno.h and include/linux/errno.h have
the same content.  (both just wrap &lt;asm-generic/errno.h&gt;)

Replace all include directives for &lt;asm/errno.h&gt; with &lt;linux/errno.h&gt;.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
[trini: Fixup include/clk.]
Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>nand: Embed mtd_info in struct nand_chip</title>
<updated>2016-06-04T01:27:48Z</updated>
<author>
<name>Scott Wood</name>
</author>
<published>2016-05-30T18:57:55Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=b616d9b0a708eb90eb474e1b6ec6dfe4c48a1678'/>
<id>urn:sha1:b616d9b0a708eb90eb474e1b6ec6dfe4c48a1678</id>
<content type='text'>
nand_info[] is now an array of pointers, with the actual mtd_info
instance embedded in struct nand_chip.

This is in preparation for syncing the NAND code with Linux 4.6,
which makes the same change to struct nand_chip.  It's in a separate
commit due to the large amount of changes required to accommodate the
change to nand_info[].

Signed-off-by: Scott Wood &lt;oss@buserror.net&gt;
</content>
</entry>
<entry>
<title>mtd: nand: Remove nand_info_t typedef</title>
<updated>2016-06-04T01:27:48Z</updated>
<author>
<name>Scott Wood</name>
</author>
<published>2016-05-30T18:57:54Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=151c06ec61d74b77cf27d6d622bab6370c949c66'/>
<id>urn:sha1:151c06ec61d74b77cf27d6d622bab6370c949c66</id>
<content type='text'>
This typedef serves no purpose other than causing confusion with
struct nand_chip.

Signed-off-by: Scott Wood &lt;oss@buserror.net&gt;
</content>
</entry>
<entry>
<title>jffs2: Fix set but not used warning</title>
<updated>2016-04-01T21:17:43Z</updated>
<author>
<name>Tom Rini</name>
</author>
<published>2016-03-27T18:48:36Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/project/bcm63xx/u-boot/commit/?id=1c8fdf8779bed79ea8c0cdbcb2559c2961afaa1d'/>
<id>urn:sha1:1c8fdf8779bed79ea8c0cdbcb2559c2961afaa1d</id>
<content type='text'>
We only use 'ofs' in jffs2_sum_scan_sumnode when debugging as it's part
of a dbg_summary call.  Mark this as __maybe_unused.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
</feed>
