Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-09 Thread Anton Staaf
On Tue, Dec 6, 2011 at 11:35 AM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message > you > wrote: >> >> Whimper whimper.  This is silly, what happened was that two different >> patches collided when committed.  Mine had the correct change to this >> file.  Doug Andersons also touched this

Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-06 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > Whimper whimper. This is silly, what happened was that two different > patches collided when committed. Mine had the correct change to this > file. Doug Andersons also touched this file but reverted the change. > So when both changes went in the res

Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-06 Thread Anatolij Gustschin
Hi Anton, On Tue, 6 Dec 2011 10:50:14 -0800 Anton Staaf wrote: ... > Whimper whimper. This is silly, what happened was that two different > patches collided when committed. Mine had the correct change to this > file. Doug Andersons also touched this file but reverted the change. > So when both

Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-06 Thread Anton Staaf
On Fri, Dec 2, 2011 at 1:59 PM, Simon Glass wrote: > Hi Graeme, > > On Fri, Dec 2, 2011 at 1:28 PM, Graeme Russ wrote: >> Hi Anton, >> >> On 30/11/11 10:39, Gabe Black wrote: >>> The ALLOC_CACHE_ALIGN_BUFFER macro allocates a pointer to a buffer, and the >>> address of the pointer and not the buf

Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-02 Thread Simon Glass
Hi Graeme, On Fri, Dec 2, 2011 at 1:28 PM, Graeme Russ wrote: > Hi Anton, > > On 30/11/11 10:39, Gabe Black wrote: >> The ALLOC_CACHE_ALIGN_BUFFER macro allocates a pointer to a buffer, and the >> address of the pointer and not the buffer was being passed to a function >> that expected the other.

Re: [U-Boot] [PATCH] Fix some bugs in the EFI support

2011-12-02 Thread Graeme Russ
Hi Anton, On 30/11/11 10:39, Gabe Black wrote: > The ALLOC_CACHE_ALIGN_BUFFER macro allocates a pointer to a buffer, and the > address of the pointer and not the buffer was being passed to a function > that expected the other. This change fixes a few places that bug crops up. > Also, it provides a

[U-Boot] [PATCH] Fix some bugs in the EFI support

2011-11-29 Thread Gabe Black
The ALLOC_CACHE_ALIGN_BUFFER macro allocates a pointer to a buffer, and the address of the pointer and not the buffer was being passed to a function that expected the other. This change fixes a few places that bug crops up. Also, it provides a default definition for the CONFIG_SYS_CACHELINE_SIZE ma