From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eli Billauer <eli.billauer@gmail.com>
Date: Tue, 18 Mar 2014 12:20:29 +0200
Subject: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine
In-Reply-To: <CAAhU36hjUdmJ3sNbqGBN1E0D3uN3kEGOgcKpsQG03QMtDysW-w@mail.gmail.com>
References: <1394916029-12079-1-git-send-email-eli.billauer@gmail.com>
	<CAAhU36hjUdmJ3sNbqGBN1E0D3uN3kEGOgcKpsQG03QMtDysW-w@mail.gmail.com>
Message-ID: <53281DED.8030903@gmail.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 18/03/14 08:11, S Durga Prasad Paladugu wrote:
> This looks like not correct because if you look at the code above 
> this, it always ensuring that the new aligned buffer start is in front 
> of the actual buffer

Maybe it should, but it doesn't. In the boot log it says:

zynq_load: Align buffer at 10006f to 100080(swap 1)

In fact, it makes sense to align upwards. Copying the buffer downwards 
would destroy the data in the beginning of the buffer. Not that it 
probably matters either way.

Regards,

    Eli