Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-19 Thread Thierry Reding
On Wed, Sep 19, 2012 at 12:44:27AM +0200, Marek Vasut wrote: > Dear Simon Glass, > > > Hi, > > > > On Tue, Sep 18, 2012 at 2:21 PM, Marek Vasut wrote: > > > Dear Simon Glass, > > > > > >> Hi, > > >> > > >> On Tue, Sep 18, 2012 at 1:04 PM, Thierry Reding > > >> > > >> wrote: > > >> > On Tue,

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-19 Thread Thierry Reding
On Tue, Sep 18, 2012 at 11:20:30PM +0200, Marek Vasut wrote: > Dear Thierry Reding, > > > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: > > > Dear Thierry Reding, > > > > > > [...] > > > > > > > > Sure, but after you apply the bounce buffer, you can safely > > > > > invalidate the

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Simon Glass, > Hi, > > On Tue, Sep 18, 2012 at 2:21 PM, Marek Vasut wrote: > > Dear Simon Glass, > > > >> Hi, > >> > >> On Tue, Sep 18, 2012 at 1:04 PM, Thierry Reding > >> > >> wrote: > >> > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: > >> >> Dear Thierry Reding, > >>

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Simon Glass
Hi, On Tue, Sep 18, 2012 at 2:21 PM, Marek Vasut wrote: > Dear Simon Glass, > >> Hi, >> >> On Tue, Sep 18, 2012 at 1:04 PM, Thierry Reding >> >> wrote: >> > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: >> >> Dear Thierry Reding, >> >> >> >> [...] >> >> >> >> > > Sure, but after y

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Simon Glass, > Hi, > > On Tue, Sep 18, 2012 at 1:04 PM, Thierry Reding > > wrote: > > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: > >> Dear Thierry Reding, > >> > >> [...] > >> > >> > > Sure, but after you apply the bounce buffer, you can safely > >> > > invalidate the w

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Thierry Reding, > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: > > Dear Thierry Reding, > > > > [...] > > > > > > Sure, but after you apply the bounce buffer, you can safely > > > > invalidate the whole cacheline, so align it up and be done with it. > > > > > > That's what

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Simon Glass
Hi, On Tue, Sep 18, 2012 at 1:04 PM, Thierry Reding wrote: > On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: >> Dear Thierry Reding, >> >> [...] >> >> > > Sure, but after you apply the bounce buffer, you can safely invalidate >> > > the whole cacheline, so align it up and be done wit

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Thierry Reding
On Tue, Sep 18, 2012 at 09:36:18PM +0200, Marek Vasut wrote: > Dear Thierry Reding, > > [...] > > > > Sure, but after you apply the bounce buffer, you can safely invalidate > > > the whole cacheline, so align it up and be done with it. > > > > That's what I proposed to do last time around but it

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Thierry Reding, [...] > > Sure, but after you apply the bounce buffer, you can safely invalidate > > the whole cacheline, so align it up and be done with it. > > That's what I proposed to do last time around but it was NAK'ed. By who? > At the > time I didn't ensure that the buffer was ac

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Thierry Reding
On Tue, Sep 18, 2012 at 09:21:14PM +0200, Marek Vasut wrote: > Dear Thierry Reding, > > > On Tue, Sep 18, 2012 at 08:37:44PM +0200, Marek Vasut wrote: > > > Dear Simon Glass, > > > > > > > Hi Thierry, > > > > > > > > On Tue, Sep 18, 2012 at 7:54 AM, Thierry Reding > > > > > > > > wrote: > > >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Thierry Reding, > On Tue, Sep 18, 2012 at 08:37:44PM +0200, Marek Vasut wrote: > > Dear Simon Glass, > > > > > Hi Thierry, > > > > > > On Tue, Sep 18, 2012 at 7:54 AM, Thierry Reding > > > > > > wrote: > > > > On Mon, Sep 17, 2012 at 02:39:01PM -0700, Simon Glass wrote: > > > >> Hi Thierr

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Thierry Reding
On Tue, Sep 18, 2012 at 08:37:44PM +0200, Marek Vasut wrote: > Dear Simon Glass, > > > Hi Thierry, > > > > On Tue, Sep 18, 2012 at 7:54 AM, Thierry Reding > > > > wrote: > > > On Mon, Sep 17, 2012 at 02:39:01PM -0700, Simon Glass wrote: > > >> Hi Thierry, > > >> > > >> On Sat, Sep 15, 2012 at

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Marek Vasut
Dear Simon Glass, > Hi Thierry, > > On Tue, Sep 18, 2012 at 7:54 AM, Thierry Reding > > wrote: > > On Mon, Sep 17, 2012 at 02:39:01PM -0700, Simon Glass wrote: > >> Hi Thierry, > >> > >> On Sat, Sep 15, 2012 at 11:49 PM, Thierry Reding > >> > >> wrote: > >> > On Sat, Sep 15, 2012 at 07:45:30

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Simon Glass
Hi Thierry, On Tue, Sep 18, 2012 at 7:54 AM, Thierry Reding wrote: > On Mon, Sep 17, 2012 at 02:39:01PM -0700, Simon Glass wrote: >> Hi Thierry, >> >> On Sat, Sep 15, 2012 at 11:49 PM, Thierry Reding >> wrote: >> > On Sat, Sep 15, 2012 at 07:45:30PM -0700, Simon Glass wrote: >> >> Hi, >> >> >> >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-18 Thread Thierry Reding
On Mon, Sep 17, 2012 at 02:39:01PM -0700, Simon Glass wrote: > Hi Thierry, > > On Sat, Sep 15, 2012 at 11:49 PM, Thierry Reding > wrote: > > On Sat, Sep 15, 2012 at 07:45:30PM -0700, Simon Glass wrote: > >> Hi, > >> > >> On Sat, Sep 15, 2012 at 1:41 PM, Thierry Reding > >> wrote: > >> > On Sat,

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-17 Thread Simon Glass
Hi Thierry, On Sat, Sep 15, 2012 at 11:49 PM, Thierry Reding wrote: > On Sat, Sep 15, 2012 at 07:45:30PM -0700, Simon Glass wrote: >> Hi, >> >> On Sat, Sep 15, 2012 at 1:41 PM, Thierry Reding >> wrote: >> > On Sat, Sep 15, 2012 at 10:11:54PM +0200, Marek Vasut wrote: >> >> Dear Thierry Reding, >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Thierry Reding
On Sat, Sep 15, 2012 at 07:45:30PM -0700, Simon Glass wrote: > Hi, > > On Sat, Sep 15, 2012 at 1:41 PM, Thierry Reding > wrote: > > On Sat, Sep 15, 2012 at 10:11:54PM +0200, Marek Vasut wrote: > >> Dear Thierry Reding, > >> > >> > On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: > >>

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Simon Glass
Hi, On Sat, Sep 15, 2012 at 1:41 PM, Thierry Reding wrote: > On Sat, Sep 15, 2012 at 10:11:54PM +0200, Marek Vasut wrote: >> Dear Thierry Reding, >> >> > On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: >> > > Hi, >> > > >> > > On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: >> >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Marek Vasut
Dear Thierry Reding, > On Sat, Sep 15, 2012 at 10:11:54PM +0200, Marek Vasut wrote: > > Dear Thierry Reding, > > > > > On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: > > > > Hi, > > > > > > > > On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: > > > > > Dear Stephen Warren, > >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Thierry Reding
On Sat, Sep 15, 2012 at 10:11:54PM +0200, Marek Vasut wrote: > Dear Thierry Reding, > > > On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: > > > Hi, > > > > > > On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: > > > > Dear Stephen Warren, > > > > > > > >> On 09/12/2012 04:38 PM,

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Thierry Reding
On Sat, Sep 15, 2012 at 10:01:47PM +0200, Thierry Reding wrote: > I think I traced this to the copying of CSD a while back. The problem is > that the transferred buffer is 8 bytes, so there's no way to make it > aligned properly. Unfortunately the entailing discussion did not yield a > solution at

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Thierry Reding
On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: > Hi, > > On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: > > Dear Stephen Warren, > > > >> On 09/12/2012 04:38 PM, Marek Vasut wrote: > >> > Dear Stephen Warren, > >> > > >> >> On 09/12/2012 10:19 AM, Tom Warren wrote: > >> >>> Fol

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-15 Thread Marek Vasut
Dear Thierry Reding, > On Fri, Sep 14, 2012 at 08:53:32AM -0700, Simon Glass wrote: > > Hi, > > > > On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: > > > Dear Stephen Warren, > > > > > >> On 09/12/2012 04:38 PM, Marek Vasut wrote: > > >> > Dear Stephen Warren, > > >> > > > >> >> On 09/12/2

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-14 Thread Simon Glass
Hi, On Wed, Sep 12, 2012 at 4:42 PM, Marek Vasut wrote: > Dear Stephen Warren, > >> On 09/12/2012 04:38 PM, Marek Vasut wrote: >> > Dear Stephen Warren, >> > >> >> On 09/12/2012 10:19 AM, Tom Warren wrote: >> >>> Folks, >> >>> >> >>> Stephen Warren has posted an internal bug regarding the cache >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-12 Thread Marek Vasut
Dear Stephen Warren, > On 09/12/2012 04:38 PM, Marek Vasut wrote: > > Dear Stephen Warren, > > > >> On 09/12/2012 10:19 AM, Tom Warren wrote: > >>> Folks, > >>> > >>> Stephen Warren has posted an internal bug regarding the cache > >>> alignment 'warnings' seen on Tegra20 boards when accessing MM

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-12 Thread Stephen Warren
On 09/12/2012 04:38 PM, Marek Vasut wrote: > Dear Stephen Warren, > >> On 09/12/2012 10:19 AM, Tom Warren wrote: >>> Folks, >>> >>> Stephen Warren has posted an internal bug regarding the cache >>> alignment 'warnings' seen on Tegra20 boards when accessing MMC. Here's >>> the gist: >>> >>> Executi

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-12 Thread Marek Vasut
Dear Stephen Warren, > On 09/12/2012 10:19 AM, Tom Warren wrote: > > Folks, > > > > Stephen Warren has posted an internal bug regarding the cache > > alignment 'warnings' seen on Tegra20 boards when accessing MMC. Here's > > the gist: > > > > Executing "mmc dev 0" still yields cache warnings: >

Re: [U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-12 Thread Stephen Warren
On 09/12/2012 10:19 AM, Tom Warren wrote: > Folks, > > Stephen Warren has posted an internal bug regarding the cache > alignment 'warnings' seen on Tegra20 boards when accessing MMC. Here's > the gist: > > Executing "mmc dev 0" still yields cache warnings: > > Tegra20 (Harmony) # mmc dev 0 > ERR

[U-Boot] Cache alignment warnings on Tegra (ARM)

2012-09-12 Thread Tom Warren
Folks, Stephen Warren has posted an internal bug regarding the cache alignment 'warnings' seen on Tegra20 boards when accessing MMC. Here's the gist: Executing "mmc dev 0" still yields cache warnings: Tegra20 (Harmony) # mmc dev 0 ERROR: v7_dcache_inval_range- stop address is not aligned- 0x3fb6