From mboxrd@z Thu Jan  1 00:00:00 1970
From: Chin Liang See <clsee@altera.com>
Date: Wed, 9 Dec 2015 21:48:49 +0800
Subject: [U-Boot] [PATCH] arm: socfpga: Fix cache configuration
In-Reply-To: <201512081354.51327.marex@denx.de>
References: <1442503829-28882-1-git-send-email-sr@denx.de>
 <20151208111323.GA5801@amd> <5666C74D.5080008@denx.de>
 <201512081354.51327.marex@denx.de>
Message-ID: <1449668929.2573.3.camel@altera.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

On Tue, 2015-12-08 at 13:54 +0100, Marek Vasut wrote:
> On Tuesday, December 08, 2015 at 01:04:29 PM, Stefan Roese wrote:
> > On 08.12.2015 12:13, Pavel Machek wrote:
> > > > > > > Usage:
> > > > > > > ubifsmount <volume-name>
> > > > > > > 
> > > > > > >      - mount 'volume-name' volume
> > > > > > > 
> > > > > > > In the mean time, I was not able to get ubifsmount works.
> > > > > > > Appreciate
> > > > > > > for any quick advise? Else will look into the code
> > > > > > > tomorrow as my
> > > > > > > bed
> > > > > > > is calling me :)
> > > > > > 
> > > > > > I usually write ubinized image into the "rootfs" partition
> > > > > > (sf erase
> > > > > > and
> > > > > > then sf write) and then do 'ubi part rootfs' , which fails
> > > > > > with error
> > > > > > 22
> > > > > > unless I revert this patch. If I dump the SPI NOR area
> > > > > > after writing
> > > > > > the
> > > > > > data, I see that the last 2 bytes of some pages are
> > > > > > corrupted.
> > > > > > 
> > > > > > I am using these parameters to generate my ~11MiB large
> > > > > > ubinized
> > > > > > image:
> > > > > > MKFS_UBIFS_OPTS="-m 1 -e 65408 -c 200"
> > > > > > UBINIZE_OPTS="-m 1 -p 64KiB -s 1"
> > > > > > 
> > > > > > Here is the content of my ubinize.cfg:
> > > > > > [rootfs]
> > > > > > mode=ubi
> > > > > > image=root.ubifs
> > > > > > vol_id=0
> > > > > > vol_type=dynamic
> > > > > > vol_name=rootfs
> > > > > > vol_flags=autoresize
> > > > > 
> > > > > Thanks for the pointers.
> > > > > 
> > > > > I checked the source and enabled the debug message. Noticed
> > > > > my failure
> > > > > is due to small LEB and PEB size. It was set to 4k which is
> > > > > the sub
> > > > > -sector erase size of NOR flash. I suspect you didn't hit
> > > > > this as you
> > > > > generate ubinized image which is 64kB erase size.
> > > > > 
> > > > > I will continue to dig more. Need to ensure it works when
> > > > > user create
> > > > > UBI part in U-Boot on top of serial NOR flash (which is
> > > > > commonly 4kB
> > > > > erase size). Hopefully existing U-Boot already have source
> > > > > taking care
> > > > > this :)
> > > > 
> > > > I am tempted to revert this patch, since it breaks USB and UBI
> > > > for me
> > > > on two different boards though.
> > > 
> > > It caused regressions it was not supposed to change. That means
> > > revert...
> > 
> > Yes, please revert and hopefully someone will find the time
> > to find and fix the problem with this dcache at some time.
> 
> Me, already done, see the other email ;-)
> 
> > Sorry for the inconvenience. But I didn't notice any problems
> > with it until now.
> 
> You were just lucky ;-)


With the mentioned bug, the value that will be programmed to ttbr0 is
the tlb_address itself. The value I have (through bdinfo command) is
0x3fff0000. Wonder what is the value when the issue happen? Just try to
understand more.

Thanks
Chin Liang



> 
> Best regards,
> Marek Vasut