Hello Mark,

Am 29.06.2015 um 07:02 schrieb Mark Tomlinson:
If a destination is not provided, jffs2_1pass_read_inode() only
returns the length of the file. In this case, avoid reading all
the data nodes, and return as soon as the length of the file is
known.

Signed-off-by: Mark Tomlinson <mark.tomlin...@alliedtelesis.co.nz>
---

  fs/jffs2/jffs2_1pass.c | 6 ++++++
  1 file changed, 6 insertions(+)

diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c
index b1d6470..2335db1 100644
--- a/fs/jffs2/jffs2_1pass.c
+++ b/fs/jffs2/jffs2_1pass.c
@@ -719,6 +719,12 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32 inode, char 
*dest)
                }
                put_fl_mem(jNode, pL->readbuf);
        }
+       /* If no destination is provided, we are done.
+        * Just return the total size.
+        */

please change this into

        /*
         * If no destination is provided, we are done.
         * Just return the total size.
         */

to fit with Coding style.


+       if (!dest) {
+               return totalSize;
+       }

no {} needed

Beside of this:

Acked-by: Heiko Schocher <h...@denx.de>

bye,
Heiko
  #endif

        for (b = pL->frag.listHead; b != NULL; b = b->next) {


--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to