Re: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-18 Thread Michal Simek
Hi, On 03/18/2014 10:14 PM, Eli Billauer wrote: > Hello, > > We're completely out of sync with our git repositories, but I changed the relevant parts in my zynqpl.c (with the patch of this thread not applied), and it worked well. The alignment went downwards as expected. That's interesting. An

Re: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-18 Thread Eli Billauer
Hello, We're completely out of sync with our git repositories, but I changed the relevant parts in my zynqpl.c (with the patch of this thread not applied), and it worked well. The alignment went downwards as expected. Regards, Eli On 18/03/14 14:17, Michal Simek wrote: Does this work for

Re: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-18 Thread Michal Simek
On 03/18/2014 11:20 AM, Eli Billauer wrote: > 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

Re: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-18 Thread Eli Billauer
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

Re: [U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-17 Thread S Durga Prasad Paladugu
Hi Eli, On Sun, Mar 16, 2014 at 2:10 AM, Eli Billauer wrote: > The aligned buffer is always with a higher address, so copying should run > from the end of the buffer to the beginning, and not the other way around. > > Signed-off-by: Eli Billauer > --- > drivers/fpga/zynqpl.c |5 +++-- > 1

[U-Boot] [PATCH] fpga: zynqpl: Fixed bug in alignment routine

2014-03-16 Thread Eli Billauer
The aligned buffer is always with a higher address, so copying should run from the end of the buffer to the beginning, and not the other way around. Signed-off-by: Eli Billauer --- drivers/fpga/zynqpl.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/fpga/zyn