From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bob Furber <bob@steroidmicros.com>
Date: Thu, 13 Aug 2009 14:48:02 -0700
Subject: [U-Boot] ext2fs.c/ext2fs_mount() fails when inode_size = 256
In-Reply-To: <20090812214041.72D16833DBD2@gemini.denx.de>
References: <4A82EB38.6060802@steroidmicros.com>
	<20090812200743.5804E833DBD2@gemini.denx.de>
	<4A832C44.3060503@steroidmicros.com>
	<20090812214041.72D16833DBD2@gemini.denx.de>
Message-ID: <4A848A12.2070001@steroidmicros.com>
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

Wolfgang Denk wrote:

Thanks Wolfgang. You were right. I had used u-boot-2009.08-rc1 which 
still had the 128 byte inode size hard-coded in 
...\u-boot-2009.08-rc1\fs\ext2\ext2fs.c. Upgrading to u-boot-2009.08-rc2 
now allows me to boot from 2GB SD cards formatted with Ubuntu-2.6.27.

Thanks,

Bob Furber

> please keep the mailing list on cc:
>   

My carelessness. I am new to this list and accustomed to simply click 
"Reply".

> In message <4A832C44.3060503@steroidmicros.com> you wrote:
>   
>>> You failed to mention some really important fects, like for example
>>> which version of U-Boot you are using.
>>>       
>> ftp://ftp.denx.de/pub/u-boot/ --> u-boot-2009.08-rc2.tar.bz2, which 
>> appeared to be the most recent relrase.
>>     
>
> Indeed.
>
>   
>>> This commit most probably fixes your issue:
>>>
>>> commit 56fdaadc124a8ef9ec0fd8ff578233ec3b1137be
>>> Author: Weirich, Bernhard <Bernhard.Weirich@riedel.net>
>>> Date:   Wed Jun 10 14:00:37 2009 +0200
>>>
>>>     ext2: fix inode size and calculations
>>>     
>>>     Signed-off-by: unsik Kim <donari75@gmail.com>
>>>     Signed-off-by: Bernhard Weirich <bernhard.weirich@riedel.net>
>>>     Signed-off-by: Wolfgang Denk <wd@denx.de>
>>>     Tested-by: Wolfgang Denk <wd@denx.de>
>>>   
>>>       
>> Forgive me, but I am new at this and I have no idea how to locate 
>> "commit 56fdaadc124a8ef9ec0fd8ff578233ec3b1137be".
>>     
>
> You can use for example "git show" in your U-Boot git repository.
> In worst case, you can search the CHANGELOG.
>
> Anyway - commit 56fdaadc is:
>
> 	-> git-describe 56fdaadc
> 	v2009.08-rc1-1-g56fdaad
>
> so it's the first commit after v2009.08-rc1 and thus included in
> v2009.08-rc2.
>
> If this doesn't work for you, I'd be interested in receiving a test
> case, i. e. a minimal (as small as possible) file system image that
> shows the erroneous behaviour.
>
> Best regards,
>
> Wolfgang Denk
>
>