Le 19/08/2010 07:58, Mike Frysinger a écrit :
On Wednesday, August 18, 2010 16:36:39 Albert ARIBAUD wrote:
Le 18/08/2010 19:54, Mike Frysinger a écrit :
On Wed, Aug 18, 2010 at 1:46 PM, Albert ARIBAUD wrote:
Le 18/08/2010 18:46, Mike Frysinger a écrit :
you need to include linux/compiler.h first ... but i would have thought this be a header already included globally. maybe that's a new topic to start.
I don't understand why I should introduce a dependency on linux. What is the benefit?
u-boot already includes the header in the source tree, not to mention the fact that this file is already including linux/ctype.h
But absolutely no .h or .c file uses __aligned whereas __attribute__((__aligned__())) is used in several places, right?
that would seem to be the case currently -mike
In that case, I'll stick with the currently used syntax for this patch.
If someone feels the need to switch over to __aligned, I think issuing a separate global patch which changes all occurrences makes more sense than using both forms in the source code.
Amicalement,