From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Thu, 27 Jun 2013 12:17:25 +0200
Subject: [U-Boot] [PATCH] fs/ext4: caculate log2blksz from blksz,
 other its value = 0
In-Reply-To: <1372326151-22868-1-git-send-email-dennis.yxun@gmail.com>
References: <1372326151-22868-1-git-send-email-dennis.yxun@gmail.com>
Message-ID: <20130627121725.7541eb25@lilith>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Lan,

On Thu, 27 Jun 2013 17:42:31 +0800, "Lan Yixun (dlan)"
<dennis.yxun@gmail.com> wrote:

> From: "Lan Yixun (dlan)" <dennis.yxun@gmail.com>
> 
> err:
> lt703a # ext4ls nand 3:0
> Failed to mount ext2 filesystem...
> ** Unrecognized filesystem type **
> 
> bug may introduced by commit:
> 
> commit 50ce4c07df1c98aabf4630b35152ed95a87242f7
> Author: Egbert Eich <eich@suse.com>
> Date:   Wed May 1 01:13:19 2013 +0000
> 
>     fs/ext4: Support device block sizes != 512 bytes

Please make the commit summary a description of what is wrong and how
it is fixed, and move the context (how the bug was found, console
output, etc) after the commit message separator '---' below.

Also, fix probably typo in the commit summary (first commit message
line) ("other" for "otherwise").

> Signed-off-by: Lan Yixun (dlan) <dennis.yxun@gmail.com>
> ---
>  disk/part.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/disk/part.c b/disk/part.c
> index fc8404d..dbc94c4 100644
> --- a/disk/part.c
> +++ b/disk/part.c
> @@ -551,6 +551,8 @@ int get_device_and_partition(const char *ifname, const char *dev_part_str,
>  			goto cleanup;
>  		}
>  
> +		(*dev_desc)->log2blksz = LOG2((*dev_desc)->blksz);
> +
>  		info->start = 0;
>  		info->size = (*dev_desc)->lba;
>  		info->blksz = (*dev_desc)->blksz;
> @@ -634,6 +636,8 @@ int get_device_and_partition(const char *ifname, const char *dev_part_str,
>  		goto cleanup;
>  	}
>  
> +	(*dev_desc)->log2blksz = LOG2((*dev_desc)->blksz);
> +
>  	ret = part;
>  	goto cleanup;
>  


Amicalement,
-- 
Albert.