Dear Ryan CHEN,

In message <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED]
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.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to