[U-Boot] [U-Boot-Users] PATCH: Update U-Boot EXT2 Filesystem to support dynamic inode size and optimize ext2 read function to fasten read speed!

[Wolfgang Denk]

Dear Ryan CHEN,

In message <008501c8f1e8$042046f0$30065e0a at SHZ.ST.COM> you wrote:
> Hi Wolfgang,
> I have modified my patch according to your suggestion. The only question is
> that use debug() because I found all debug print msg lines are written as :
> #ifdef DEBUG
> printf(......);
> #endif
> Are you sure I need change my debug print to debug()?

Since all the other code is like that I agree to leave it unchanged.

But:

> @@ -66,6 +67,17 @@ extern int ext2fs_devread (int sector, int byte_offset,
> int byte_len,
^^^^^^^^^^^^^^

> @@ -226,10 +242,17 @@ static int ext2fs_read_inode
>  	printf ("ext2fs read inode blkno %d blkoff %d\n", blkno, blkoff);
>  #endif
>  	/* Read the inode.  */
> +#ifdef CFG_EXT2_SUPPORT_DYNAMIC_REV
> +	status = ext2fs_devread (((__le32_to_cpu (blkgrp.inode_table_id) +
> +				   blkno) << LOG2_EXT2_BLOCK_SIZE (data)),
> +				 ext2_inode_size * blkoff,
> +				 sizeof (struct ext2_inode), (char *)
> inode);
^^^^^^^^^
> +#else
>  	status = ext2fs_devread (((__le32_to_cpu (blkgrp.inode_table_id) +
>  				   blkno) << LOG2_EXT2_BLOCK_SIZE (data)),
>  				 sizeof (struct ext2_inode) * blkoff,
>  				 sizeof (struct ext2_inode), (char *)
> inode);
^^^^^^^^^
> +#endif
>  	if (status == 0) {
>  		return (0);
>  	}
> @@ -243,8 +266,13 @@ void ext2fs_free_node (ext2fs_node_t node,
> ext2fs_node_t currroot) {
^^^^^^^^^^^^^^^^^^^^^^^^^^^^

etc.

Your patch is corrupted because your mailer wrapped long lines.

Please fix your mailer and repost (with a nice commit message,
please).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
In a survey taken several years ago, all  incoming  freshmen  at  MIT
were  asked  if  they  expected  to graduate in the top half of their
class. Ninety-seven percent responded that they did.